On Mon, Sep 24, 2018 at 03:09:49PM +0000, Fabrizio Castro wrote: > Hello Geert, > > Thank you for your feedback. > > > Subject: Re: [PATCH 4/4] ARM: dts: iwg23s-sbc: Add uSD card support > > > > Hi Fabrizio, > > > > On Mon, Sep 24, 2018 at 11:30 AM Fabrizio Castro > > <fabrizio.castro@xxxxxxxxxxxxxx> wrote: > > > > -----Original Message----- > > > > From: Simon Horman <horms@xxxxxxxxxxxx> > > > > Sent: 24 September 2018 10:14 > > > > To: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > > > Cc: Ulf Hansson <ulf.hansson@xxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>; Laurent > > > > Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>; Geert Uytterhoeven <geert+renesas@xxxxxxxxx>; Linus Walleij > > > > <linus.walleij@xxxxxxxxxx>; Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx>; Magnus Damm <magnus.damm@xxxxxxxxx>; > > > > linux-mmc@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-renesas-soc@xxxxxxxxxxxxxxx; linux-gpio@xxxxxxxxxxxxxxx; Chris > > > > Paterson <Chris.Paterson2@xxxxxxxxxxx>; Biju Das <biju.das@xxxxxxxxxxxxxx> > > > > Subject: Re: [PATCH 4/4] ARM: dts: iwg23s-sbc: Add uSD card support > > > > > > > > On Fri, Sep 21, 2018 at 12:55:11PM +0100, Fabrizio Castro wrote: > > > > > Add uSD card support to the iwg23s single board computer powered > > > > > by the RZ/G1C SoC (a.k.a. r8a77470). > > > > > > > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > > > > Reviewed-by: Biju Das <biju.das@xxxxxxxxxxxxxx> > > > > > --- > > > > > Hello Simon, > > > > > > > > > > this patch can only be taken after patch "pinctrl: sh-pfc: r8a77470: > > > > > Add SDHI2 voltage switch" from this series appears on a release candidate > > > > > or a release. > > > > > > > > What is the nature of that dependency. Does adding this patch without > > > > its dependency cause a regression? > > > > > > Since the SDHI2 pins definition contain "power-source" property, adding this > > > patch without its dependency will cause an error at boot up as the kernel > > > would be looking for flag SH_PFC_PIN_CFG_IO_VOLTAGE for each pin contained > > > in sdhi2_pins and sdhi2_pins_uhs, and since that particular flag would be missing > > > (as such a definition comes from patch "pinctrl: sh-pfc: r8a77470: Add SDHI2 voltage > > > switch") the SD card would not be functional, but this won't have any impact on > > > the rest of the system. > > > > But that won't be a regression, as currently there's no support for > > SDHI2 anyway, > > right? All pieces will start working when both the pinctrl and DTS support will > > be merged together. > > Exactly, it won't be a regression, you'd just get weird messages from the kernel, that's all. That is fine. But lets finalise the discussion of the bindings before I apply this patch.
