On Mon, May 28, 2018 at 11:14:07PM +0300, Sergei Shtylyov wrote:
> Define the Condor board dependent part of the I2C0 device node.
> 
> The I2C0 bus is populated by 2 ON Semiconductor PCA9654 I/O expanders
> and Analog Devices  ADV7511W HDMI transmitter (but we're only describing
> the former chips now).
> 
> Signed-off-by: Sergei Shtylyov <[email protected]>
> 
> ---
>  arch/arm64/boot/dts/renesas/r8a77980-condor.dts |   27 
> ++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
> 
> Index: renesas/arch/arm64/boot/dts/renesas/r8a77980-condor.dts
> ===================================================================
> --- renesas.orig/arch/arm64/boot/dts/renesas/r8a77980-condor.dts
> +++ renesas/arch/arm64/boot/dts/renesas/r8a77980-condor.dts
> @@ -80,6 +80,28 @@
>       clock-frequency = <32768>;
>  };
>  
> +&i2c0 {
> +     pinctrl-0 = <&i2c0_pins>;
> +     pinctrl-names = "default";
> +
> +     status = "okay";
> +     clock-frequency = <400000>;
> +
> +     io_expander0: gpio@20 {

Hi Sergei,

I'm a little confused about where 0x20 and 0x21 are derived from.
Could you explain a little?

> +             compatible = "onnn,pca9654";
> +             reg = <0x20>;
> +             gpio-controller;
> +             #gpio-cells = <2>;
> +     };
> +
> +     io_expander1: gpio@21 {
> +             compatible = "onnn,pca9654";
> +             reg = <0x21>;
> +             gpio-controller;
> +             #gpio-cells = <2>;
> +     };
> +};
> +
>  &mmc0 {
>       pinctrl-0 = <&mmc_pins>;
>       pinctrl-1 = <&mmc_pins_uhs>;
> @@ -104,6 +126,11 @@
>               function = "canfd0";
>       };
>  
> +     i2c0_pins: i2c0 {
> +             groups = "i2c0";
> +             function = "i2c0";
> +     };
> +
>       mmc_pins: mmc {
>               groups = "mmc_data8", "mmc_ctrl", "mmc_ds";
>               function = "mmc";
> 

Reply via email to