Re: [PATCH v7 3/4 - variant 1] drm: rcar-du: Fix legacy DT to create LVDS encoder nodes

2018-03-02 Thread Laurent Pinchart
Hi Niklas, On Friday, 2 March 2018 02:42:18 EET Niklas Söderlund wrote: > Hi Laurent, > > Thanks for your patch, > > All comments in this mail also applies to variant 2 of this patch as the > difference between them are so small. > > First I got a question about your usage of __init thru out

Re: [PATCH v7 3/4 - variant 1] drm: rcar-du: Fix legacy DT to create LVDS encoder nodes

2018-03-01 Thread Frank Rowand
On 03/01/18 14:05, Laurent Pinchart wrote: > The internal LVDS encoders now have their own DT bindings. Before > switching the driver infrastructure to those new bindings, implement > backward-compatibility through live DT patching. > > Patching is disabled and will be enabled along with support

Re: [PATCH v7 3/4 - variant 1] drm: rcar-du: Fix legacy DT to create LVDS encoder nodes

2018-03-01 Thread Niklas Söderlund
Hi Laurent, Thanks for your patch, All comments in this mail also applies to variant 2 of this patch as the difference between them are so small. First I got a question about your usage of __init thru out this driver. What would happen if a system is booted without a DU node and then a

[PATCH v7 3/4 - variant 1] drm: rcar-du: Fix legacy DT to create LVDS encoder nodes

2018-03-01 Thread Laurent Pinchart
The internal LVDS encoders now have their own DT bindings. Before switching the driver infrastructure to those new bindings, implement backward-compatibility through live DT patching. Patching is disabled and will be enabled along with support for the new DT bindings in the DU driver.