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