Messages in this thread | | | Date | Sun, 10 Feb 2019 11:56:14 +0100 | From | Neil Armstrong <> | Subject | Re: [PATCH] arm64: dts: meson-gxbb-odroidc2: enable I2C bus B |
| |
Hi,
Le 09/02/2019 16:50, Stijn Tintel a écrit : > The ODROID C2 has 2 I2C buses, so enable both of them.
Thanks for the change, but this I2C bus is on the 40-pin header, until we have a way to add support for HATs/CAPEs/<whatever hooks on the header> upstream, we prefer not enabling any functions on these pins by default.
You can, if you want add support for most functions, but "disabled" with a proper description of pins used by these functions.
You can also maintain a set of overlay files like it's done for the libretech-cc boards https://github.com/libre-computer-project/libretech-overlays/tree/for-4.13.y/overlays and use them with the Armbian mainline based latest image for example.
Neil
> > Signed-off-by: Stijn Tintel <stijn@linux-ipv6.be> > --- > arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts > index 1cc9dc68ef00..ee666471a580 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb-odroidc2.dts > @@ -180,6 +180,12 @@ > pinctrl-names = "default"; > }; > > +&i2c_B { > + status = "okay"; > + pinctrl-0 = <&i2c_b_pins>; > + pinctrl-names = "default"; > +}; > + > &ir { > status = "okay"; > pinctrl-0 = <&remote_input_ao_pins>; >
| |