[PATCH 1/2] ARM: socfpga: dts: add eeprom, lcd, and rtc on i2c0
Dinh Nguyen
dinguyen at altera.com
Thu Apr 3 15:33:36 PDT 2014
On Thu, 2014-04-03 at 07:19 +0200, Steffen Trumtrar wrote:
> Hi!
>
> On Wed, Apr 02, 2014 at 09:40:33PM -0500, dinguyen at altera.com wrote:
> > From: Dinh Nguyen <dinguyen at altera.com>
> >
> > The Altera Cyclone5 and Arria5 devkit has an EEPROM, LCD and a RTC on the
> > board. This patch adds support for them.
> >
> > Signed-off-by: Dinh Nguyen <dinguyen at altera.com>
> > ---
> > arch/arm/boot/dts/socfpga_arria5_socdk.dts | 22 ++++++++++++++++++++++
> > arch/arm/boot/dts/socfpga_cyclone5_socdk.dts | 22 ++++++++++++++++++++++
> > 2 files changed, 44 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/socfpga_arria5_socdk.dts b/arch/arm/boot/dts/socfpga_arria5_socdk.dts
> > index 88e4a8e..674dcf8 100644
> > --- a/arch/arm/boot/dts/socfpga_arria5_socdk.dts
> > +++ b/arch/arm/boot/dts/socfpga_arria5_socdk.dts
> > @@ -59,3 +59,25 @@
> > rxdv-skew-ps = <0>;
> > rxc-skew-ps = <2000>;
> > };
> > +
> > +&i2c0 {
> > + status = "okay";
> > + lcd: lcd at 28 {
> > + compatible = "newhaven,nhd-0216k3z-nsw-bbw";
> > + reg = <0x28>;
> > + height = <2>;
> > + width = <16>;
> > + brightness = <8>;
> > + };
>
> I would prefer if we have the display in an extra patch.
> This seems simple and okay, but everybody should get a chance to at least
> discuss this new binding+driver instead of just "hiding" it in this patch.
>
You're right. I forgot that the driver for this LCD display is not even
upstreamed yet.
Thanks,
Dinh
> > +
> > + eeprom at 51 {
> > + compatible = "atmel,24c32";
> > + reg = <0x51>;
> > + pagesize = <32>;
> > + };
> > +
> > + rtc at 68 {
> > + compatible = "dallas,ds1339";
> > + reg = <0x68>;
> > + };
> > +};
> > diff --git a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts
> > index 5e9445a..0d85021 100644
> > --- a/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts
> > +++ b/arch/arm/boot/dts/socfpga_cyclone5_socdk.dts
> > @@ -52,3 +52,25 @@
> > rxdv-skew-ps = <0>;
> > rxc-skew-ps = <2000>;
> > };
> > +
> > +&i2c0 {
> > + status = "okay";
> > + lcd: lcd at 28 {
> > + compatible = "newhaven,nhd-0216k3z-nsw-bbw";
> > + reg = <0x28>;
> > + height = <2>;
> > + width = <16>;
> > + brightness = <8>;
> > + };
> > +
> > + eeprom at 51 {
> > + compatible = "atmel,24c32";
> > + reg = <0x51>;
> > + pagesize = <32>;
> > + };
> > +
> > + rtc at 68 {
> > + compatible = "dallas,ds1339";
> > + reg = <0x68>;
> > + };
> > +};
> > --
> > 1.7.9.5
> >
> >
>
> The rest looks okay.
>
> Regards,
> Steffen
>
More information about the linux-arm-kernel
mailing list