Re: [PATCH v2 3/4] arm64: dts: renesas: r8a77970: add LVDS support

From: Sergei Shtylyov
Date: Tue Apr 10 2018 - 13:12:45 EST


On 04/10/2018 03:13 PM, jacopo mondi wrote:

>>> From: Niklas SÃderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
>>>
>>> Add the LVDS device to r8a77970.dtsi in a disabled state. Also connect
>>> the it to the LVDS output of the DU.
>>>
>>> Signed-off-by: Niklas SÃderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
>>> Signed-off-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx>
>>> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
>>>
>>> ---
>>> v1 -> v2:
>>> - Rebased on the modified endpoint name and changed patch subject to
>>> comply with other patches in the series from Sergei.
>>
>> Stop, what was wrong with my R8A77970 LVDS patch, reposted on the popular
>> demand? :-)
>>
>
> Nothing wrong, I found out about your right now.
> I see a very small difference as
> lvds@feb90000 vs lvds-encoder@feb90000

Yes, I thought the latter was closer to a "generic" name that DT spec requires...

> You can add your signed-off-by ofc if you think that's the case.

Didn't understand what you mean here...

> Thanks
> j

MBR, Sergei