<div dir="ltr">Hi Philipp,<br><div class="gmail_extra"><br><div class="gmail_quote">2017-10-02 9:27 GMT-03:00 Philipp Zabel <span dir="ltr"><<a href="mailto:p.zabel@pengutronix.de" target="_blank">p.zabel@pengutronix.de</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Marco,<br>
<span class=""><br>
On Fri, 2017-09-22 at 15:05 -0300, Marco Franchi wrote:<br>
> Improve the binding example by removing the '@di0' notation, which<br>
> fixes the following build warning:<br>
><br>
> Warning (unit_address_vs_reg): Node /display@di0 has a unit name, but <br>
> no reg property<br>
><br>
> Signed-off-by: Marco Franchi <<a href="mailto:marco.franchi@nxp.com">marco.franchi@nxp.com</a>><br>
> ---<br>
>  Documentation/devicetree/<wbr>bindings/display/imx/fsl-imx-<wbr>drm.txt | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/Documentation/devicetree/<wbr>bindings/display/imx/fsl-imx-<br>
> drm.txt b/Documentation/devicetree/<wbr>bindings/display/imx/fsl-imx-<br>
> drm.txt<br>
> index f798547..44814f0 100644<br>
> --- a/Documentation/devicetree/<wbr>bindings/display/imx/fsl-imx-<wbr>drm.txt<br>
> +++ b/Documentation/devicetree/<wbr>bindings/display/imx/fsl-imx-<wbr>drm.txt<br>
> @@ -129,7 +129,7 @@ Optional properties:<br>
>  <br>
>  example:<br>
>  <br>
> -display@di0 {<br>
> +display-di0 {<br>
>       compatible = "fsl,imx-parallel-display";<br>
>       edid = [edid-data];<br>
>       interface-pix-fmt = "rgb24";<br>
<br>
</span>Thank you for the patch. We certainly should get rid of the @di0<br>
notation. Since these nodes mostly describe the DISP0 and DISP1 pin<br>
groups controlled by the IOMUXC, I'd prefer to rename this to<br>
<br>
-display@di0 {<br>
+disp0 {<br>
<br>
instead, if that is fine with you.<br></blockquote><div>Its sounds good to me. Can you change this or do you prefer I send a v2?<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'd also like to move the disp0/disp1 nodes into the SoC dtsi files to<br>
avoid having to duplicate port@0 all over the place.<br></blockquote><div>I agree and I can help with this too.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
regards<br>
<span class="HOEnZb"><font color="#888888">Philipp<br>
</font></span></blockquote></div>Best</div><div class="gmail_extra"><br></div><div class="gmail_extra">Marco</div></div>