Hi Uli, On Wed, Nov 15, 2017 at 4:25 PM, Ulrich Hecht <[email protected]> wrote: > No devices to add, I2C1 has an external connector only. > > Signed-off-by: Ulrich Hecht <[email protected]>
Thanks for your patch! Reviewed-by: Geert Uytterhoeven <[email protected]> but please see below. > --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts > @@ -84,6 +89,12 @@ > }; > }; > > +&i2c1 { > + pinctrl-0 = <&i2c1_pins>; > + pinctrl-names = "default"; > + status = "okay"; If no devices are connected, perhaps it's wise to defer the status update to e.g. an overlay that describes what's connected to CN23? Or do you want it enabled to allow adding devices manually using /sys/bus/i2c/devices/i2c-1/new_device? > +}; Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected] In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
