On Thu, Sep 21, 2017 at 08:53:00AM +0000, Fabrizio Castro wrote: > Hello Simon, > > > -----Original Message----- > > From: Simon Horman [mailto:horms@xxxxxxxxxxxx] > > Sent: 21 September 2017 09:42 > > To: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > > Cc: Chris Paterson <Chris.Paterson2@xxxxxxxxxxx>; Rob Herring <robh+dt@xxxxxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>; > > Magnus Damm <magnus.damm@xxxxxxxxx>; Russell King <linux@xxxxxxxxxxxxxxx>; Fabrizio Castro > > <fabrizio.castro@xxxxxxxxxxxxxx>; devicetree@xxxxxxxxxxxxxxx; Linux-Renesas <linux-renesas-soc@xxxxxxxxxxxxxxx>; linux-arm- > > kernel@xxxxxxxxxxxxxxxxxxx > > 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@xxxxxxxxxxx> wrote: > > > > > From: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > > > > > > > > > Enable the SDHI0 controller on iWave RZ/G1E carrier board. > > > > > > > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > > > > > Signed-off-by: Chris Paterson <chris.paterson2@xxxxxxxxxxx> > > > > > > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > > > > > > 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 for catching this, UHS can be quite troublesome in my experience.
