Hi Simon, On Mon, Aug 22, 2016 at 9:41 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: > On Fri, Aug 19, 2016 at 11:42:54PM +0300, Sergei Shtylyov wrote: >> Index: renesas/arch/arm/boot/dts/r8a7792-blanche.dts >> =================================================================== >> --- renesas.orig/arch/arm/boot/dts/r8a7792-blanche.dts >> +++ renesas/arch/arm/boot/dts/r8a7792-blanche.dts > > ... > >> @@ -186,3 +251,63 @@ >> cd-gpios = <&gpio11 11 GPIO_ACTIVE_LOW>; >> status = "okay"; >> }; >> + >> +&i2c1 { >> + status = "okay"; >> + clock-frequency = <400000>; > > Is it intentional that there are no pinctrl properties here? $ git grep -w I2C arch/arm/boot/dts/r8a7792.dtsi arch/arm/boot/dts/r8a7792.dtsi: /* I2C doesn't need pinmux */ $ Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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