On Tue, May 8, 2018 at 9:19 AM, Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> wrote: > On Tue, 2018-05-08 at 08:59 -0500, Rob Herring wrote: >> OF graph endpoint nodes are supposed to be named 'endpoint' with an >> address if there is more than one. The i.MX IPU binding graph has used >> unique endpoint names instead which now generate dtc warnings: > > This partially reverts commit 416196cd9099 ("ARM: dts: imx6: fix dtc > warnings for ipu endpoints") and also solves the same issue differently > by adding reg properties. Should this get a Fixes: tag for these > changes? Yes, I'd missed that. This has become my concern with the dtc warnings. They are often fixed in the wrong way to silence warnings. This one is trivial, but sometimes they point to deeper issues like poor binding design. This is certainly true for the new unique_unit_address check. Rob -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html