Message ID | 20240425082138.374445-5-thomas.bonnefille@bootlin.com (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | Add I2C support on TH1520 | expand |
Thomas Bonnefille wrote: > This commit enables the I2C0 controller of the TH1520, together with > the FT24C32A EEPROM that is connected to it. > In addition, this commit also enables the I2C controllers I2C2, I2C4 > and I2C5 as they are all three exposed on headers (P9 19 and 20 for I2C2, > P9 17 and 18 for I2C5 and MikroBus 7 and 5 for I2C4). > > Signed-off-by: Thomas Bonnefille <thomas.bonnefille@bootlin.com> > --- > .../boot/dts/thead/th1520-beaglev-ahead.dts | 22 +++++++++++++++++++ > 1 file changed, 22 insertions(+) > > diff --git a/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts b/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts > index d9b4de9e4757..22a6935e7204 100644 > --- a/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts > +++ b/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts > @@ -79,3 +79,25 @@ &sdio0 { > &uart0 { > status = "okay"; > }; > + > +&i2c0 { > + status = "okay"; > + clock-frequency = <100000>; > + > + eeprom: eeprom@50 { Nothing seems to reference this. Are you planning on adding adding some code that needs to read this? Otherwise the label is not really needed. > + compatible = "atmel,24c32"; > + reg = <0x50>; > + }; > +}; > + > +&i2c2 { > + status = "okay"; > +}; > + > +&i2c4 { > + status = "okay"; > +}; > + > +&i2c5 { > + status = "okay"; > +}; Does u-boot or some other firmware set up pinctrl for these 3 I2Cs? Otherwise enabling them doesn't really make sense before we have the pinctrl settings for them. /Emil > -- > 2.44.0 > > > _______________________________________________ > linux-riscv mailing list > linux-riscv@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-riscv
diff --git a/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts b/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts index d9b4de9e4757..22a6935e7204 100644 --- a/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts +++ b/arch/riscv/boot/dts/thead/th1520-beaglev-ahead.dts @@ -79,3 +79,25 @@ &sdio0 { &uart0 { status = "okay"; }; + +&i2c0 { + status = "okay"; + clock-frequency = <100000>; + + eeprom: eeprom@50 { + compatible = "atmel,24c32"; + reg = <0x50>; + }; +}; + +&i2c2 { + status = "okay"; +}; + +&i2c4 { + status = "okay"; +}; + +&i2c5 { + status = "okay"; +};
This commit enables the I2C0 controller of the TH1520, together with the FT24C32A EEPROM that is connected to it. In addition, this commit also enables the I2C controllers I2C2, I2C4 and I2C5 as they are all three exposed on headers (P9 19 and 20 for I2C2, P9 17 and 18 for I2C5 and MikroBus 7 and 5 for I2C4). Signed-off-by: Thomas Bonnefille <thomas.bonnefille@bootlin.com> --- .../boot/dts/thead/th1520-beaglev-ahead.dts | 22 +++++++++++++++++++ 1 file changed, 22 insertions(+)