Re: [PATCH 16/18] arm64: dts: h3ulcb: rename SDHI0 pins

[Vladimir Barinov <vladimir.barinov@xxxxxxxxxxxxxxxxxx>]

Hi Sergei,

On 21.11.2016 16:13, Sergei Shtylyov wrote:
> Hello.
>
> On 11/21/2016 03:05 PM, Simon Horman wrote:
>
> > From: Vladimir Barinov <vladimir.barinov@xxxxxxxxxxxxxxxxxx>
> >
> > This changes SDHI0 pin names for H3ULCB board
> >
> > Signed-off-by: Vladimir Barinov <vladimir.barinov@xxxxxxxxxxxxxxxxxx>
> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>
> > ---
> >  arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts 
> > b/arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts
> > index 8d0ac076d8e2..6ffb0517421a 100644
> > --- a/arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts
> > +++ b/arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dts
> > @@ -163,13 +163,13 @@
> >          function = "avb";
> >      };
> >
> > -    sdhi0_pins_3v3: sd0_3v3 {
> > +    sdhi0_pins: sd0 {
> >          groups = "sdhi0_data4", "sdhi0_ctrl";
> >          function = "sdhi0";
> >          power-source = <3300>;
> >      };
> >
> > -    sdhi0_pins_1v8: sd0_1v8 {
> > +    sdhi0_pins_uhs: sd0 {
>
>    I'm afraid the following will just override the props of the node 
> above which is not what we ant.
Thank you for pointing to this.

This is my typo.
I will rework the patch to have different node names.

Regards,
Vladimir

> >          groups = "sdhi0_data4", "sdhi0_ctrl";
> >          function = "sdhi0";
> >          power-source = <1800>;
> [...]
>
> MBR, Sergei