Re: [PATCH] ARM: shmobile: porter: add SDHI0/2 DT support

[Simon Horman <horms@verge.net.au>]

On Thu, Oct 08, 2015 at 01:54:00PM +0300, Sergei Shtylyov wrote:
> Hello.
> 
> On 10/8/2015 5:12 AM, Simon Horman wrote:
> 
> >>>>Define the Porter board dependent part of the SDHI0/2 device nodes along with
> >>>>the  necessary  voltage regulators (note that the Vcc regulators are dummy --
> >>>>they are required but don't actually exist on the board). Also, GPIOs have to
> >>>>be used for the CD and WP signals due to the SDHI driver constraints...
> >>>>
> >>>>This patch is analogous to the commit 1299df03d719 (ARM: shmobile: henninger:
> >>>>add SDHI0/2 DT support) as there  are no differences between those boards in
> >>>>this respect.
> >>>>
> >>>>Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
> >>>>
> >>>>---
> >>>>This patch is against the 'renesas-devel-20151006-v4.3-rc4' tag of Simon
> >>>>Horman's 'renesas.git' repo.
> >>>
> >>>Thanks I have queued this up after addressing some issues
> >>>in the changelog that checkpatch complained about:
> >>>
> >>>* text > 75 characters wide
> >>>* Missing '"' around name of patch referred to
> >>>
> >>>Its looking like there is already quite a log shared between
> >>
> >>    Lot?
> >>
> >>>r8a7791-porter.dts and r8a7791-henninger.dts. Do you have any
> >>>thoughts on consolidating or sharing common elements between
> >>>them somehow?
> >>
> >>    I'm still planning to remove the Henninger's .dts for good. It was never
> >>sold and so had no users outside Renesas (and Cogent Embedded), I was told.
> >
> >Thanks, that sounds like a good plan to me.
> >Do you have any thoughts on when?
> 
>    Well, I was going to reach the feature parity between these 2 boards,
> i.e. to support all devices on Porter that are supported on Henninger...

THanks, that sounds like an entirely reasonable plan to me.