> From: Simon Horman [mailto:horms@xxxxxxxxxxxx] > Sent: 21 September 2017 10:04 > > 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. So we've seen!
