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

[Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>]

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...

MBR, Sergei