[PATCH 4/8] ARM: dts: iwg22d: Enable SDHI0 controller

Fabrizio Castro fabrizio.castro at bp.renesas.com
Thu Sep 21 01:53:00 PDT 2017


Hello Simon,

> -----Original Message-----
> From: Simon Horman [mailto:horms at verge.net.au]
> Sent: 21 September 2017 09:42
> To: Geert Uytterhoeven <geert at linux-m68k.org>
> Cc: Chris Paterson <Chris.Paterson2 at renesas.com>; Rob Herring <robh+dt at kernel.org>; Mark Rutland <mark.rutland at arm.com>;
> Magnus Damm <magnus.damm at gmail.com>; Russell King <linux at armlinux.org.uk>; Fabrizio Castro
> <fabrizio.castro at bp.renesas.com>; devicetree at vger.kernel.org; Linux-Renesas <linux-renesas-soc at vger.kernel.org>; linux-arm-
> kernel at lists.infradead.org
> Subject: Re: [PATCH 4/8] ARM: dts: iwg22d: Enable SDHI0 controller
>
> On Thu, Sep 21, 2017 at 10:38:40AM +0200, Simon Horman wrote:
> > On Wed, Sep 20, 2017 at 01:40:20PM +0200, Geert Uytterhoeven wrote:
> > > On Wed, Sep 13, 2017 at 7:05 PM, Chris Paterson
> > > <chris.paterson2 at renesas.com> wrote:
> > > > From: Fabrizio Castro <fabrizio.castro at bp.renesas.com>
> > > >
> > > > Enable the SDHI0 controller on iWave RZ/G1E carrier board.
> > > >
> > > > Signed-off-by: Fabrizio Castro <fabrizio.castro at bp.renesas.com>
> > > > Signed-off-by: Chris Paterson <chris.paterson2 at renesas.com>
> > >
> > > Reviewed-by: Geert Uytterhoeven <geert+renesas at glider.be>
> >
> > Thanks, applied.
>
> Sorry, I somehow overlooked Fabrizio's comments.
> I have dropped this patch for now.

Thank you for doing this.
We are working on a new patch now, I'll submit a V2 as soon as it is ready.
Just to recap (all in one place), we found an HW issue and a SW one preventing SDHI0 from working properly at UHS SDR50 and SDR104, therefore we need a new patch to limit the speed of this interface.

Thanks,
Fabrizio





Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.



More information about the linux-arm-kernel mailing list