Hello Alexander, On 25.05.22 10:33, Alexander Shiyan wrote: >>> Most likely this is the result of double inclusion of the file arm/am33xx.dtsi, >>> first time from "am33xx.dtsi" and second from arm/am335x-myirtech-myd.dts. >> >> Hi, you need to include if after the upstream DT. That's why it doesn't have >> a /dts-v1/; > > Yes, if "am33xx.dtsi" is after <arm/am335x-myirtech-myd.dts>, it compiles fine. > But in this case, upstreaming <arm/am33xx.dtsi> (from local > am33xx.dtsi) overwrites > any binding changes from the board file. Argh. Other platforms (e.g. imx6q.dtsi) intentionally skip dts/ includes to avoid just that... One could remedy that at the cost of annoying out-of-tree users, but that's not something that you need to do IMO. Reviewed-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> Cheers, Ahmad > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
