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

Hello Simon,

-----Original Message-----
From: Simon Horman [mailto:horms@verge.net.au]
Sent: 21 September 2017 09:42
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Chris Paterson <redacted>; Rob Herring <robh+dt@kernel.org>; Mark Rutland <mark.rutland@arm.com>;
Magnus Damm [off-list ref]; Russell King [off-list ref]; Fabrizio Castro
[off-list ref]; devicetree@vger.kernel.org; Linux-Renesas [off-list ref]; linux-arm-
kernel@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:

quoted

On Wed, Sep 20, 2017 at 01:40:20PM +0200, Geert Uytterhoeven wrote:

quoted

On Wed, Sep 13, 2017 at 7:05 PM, Chris Paterson
[off-list ref] wrote:

quoted

From: Fabrizio Castro <redacted>

Enable the SDHI0 controller on iWave RZ/G1E carrier board.

Signed-off-by: Fabrizio Castro <redacted>
Signed-off-by: Chris Paterson <redacted>

Reviewed-by: Geert Uytterhoeven <geert+renesas@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.