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

From: Simon Horman [mailto:horms@verge.net.au]
Sent: 21 September 2017 10:04

On Thu, Sep 21, 2017 at 08:53:00AM +0000, Fabrizio Castro wrote:

quoted

Hello Simon,

quoted

-----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
[off-list ref]; Mark Rutland [off-list ref];

Magnus

quoted

quoted

Damm [off-list ref]; Russell King

[off-list ref];

quoted

quoted

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
[off-list ref]
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 for catching this, UHS can be quite troublesome in my experience.

So we've seen!