Re: [PATCH] arm64: dts: renesas: disable SD functions for plain eMMC

From: Adam Ford <aford173@gmail.com>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Wolfram Sang <wsa+renesas@sang-engineering.com>,
	Linux-Renesas <linux-renesas-soc@vger.kernel.org>
Subject: Re: [PATCH] arm64: dts: renesas: disable SD functions for plain eMMC
Date: Tue, 19 Jan 2021 14:46:24 -0600	[thread overview]
Message-ID: <CAHCN7x+_h00tG4jw_NQ+kx7eW=ZOcrDJ1PAVgMWK-rq53TyoiQ@mail.gmail.com> (raw)
In-Reply-To: <CAMuHMdVLwcQ6WpEhybxwCjJ2S-tuOoWqSiQCmB2-=Bunw4fX4A@mail.gmail.com>

On Tue, Jan 19, 2021 at 2:35 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
>
> Hi Adam,
>
> On Tue, Jan 19, 2021 at 7:43 PM Adam Ford <aford173@gmail.com> wrote:
> > On Tue, Jan 19, 2021 at 8:48 AM Wolfram Sang
> > <wsa+renesas@sang-engineering.com> wrote:
> > >
> > > Some SDHI instances are solely used for eMMC. Disable SD and SDIO
> > > for faster initialization.
> > >
> > > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
>
> > > --- a/arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi
> > > +++ b/arch/arm64/boot/dts/renesas/beacon-renesom-som.dtsi
> > > @@ -312,6 +312,8 @@ &sdhi3 {
> > >         vqmmc-supply = <&reg_1p8v>;
> > >         bus-width = <8>;
> > >         mmc-hs200-1_8v;
> > > +       no-sd;
> > > +       no-sdio;
> >
> > No objections on the Beacon board.   I cannot speak for the rest.
>
> Does this mean I can add your
>
>     Acked-by: Adam Ford <aford173@gmail.com> [beacon]
>

Yes.  I wasn't sure what the appropriate response sould be. (Review-by
or Acked-by)

adam
> ?
>
> Thanks!
>
> Gr{oetje,eeting}s,
>
>                         Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
>
> 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