From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Andr=c3=a9_Przywara?= Date: Tue, 24 Oct 2017 23:04:14 +0100 Subject: [U-Boot] [PATCH] ARM: SPL: FIT: fix DTC warnings on FIT generation In-Reply-To: References: <20171004232439.20817-1-andre.przywara@arm.com> <20171016211349.GU12015@bill-the-cat> <56b176e7-5bd1-b674-a06d-377bd97888c0@ti.com> <20171023203400.GA22919@bill-the-cat> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 24/10/17 03:31, Masahiro Yamada wrote: Hi, > 2017-10-24 11:03 GMT+09:00 Lokesh Vutla : >> >> >> On Tuesday 24 October 2017 02:04 AM, Tom Rini wrote: >>> On Mon, Oct 23, 2017 at 11:08:57AM +0530, Lokesh Vutla wrote: >>>> >>>> >>>> On Tuesday 17 October 2017 02:43 AM, Tom Rini wrote: >>>>> On Tue, Oct 17, 2017 at 02:29:14AM +0530, Jagan Teki wrote: >>>>>> On Mon, Oct 9, 2017 at 10:15 AM, Simon Glass wrote: >>>>>>> Hi Andre, >>>>>>> >>>>>>> On 4 October 2017 at 17:24, Andre Przywara wrote: >>>>>>>> Newer versions of the device tree compiler (rightfully) complain about >>>>>>>> mismatches between attributed node names (name@) and a missing >>>>>>>> reg property in that node. >>>>>>>> Adjust the FIT build script for 64-bit Allwinner boards to remove the >>>>>>>> bogus addresses from the node names and avoid the warnings. >>>>>>>> >>>>>>>> Signed-off-by: Andre Przywara >>>>>>>> --- >>>>>>>> board/sunxi/mksunxi_fit_atf.sh | 16 ++++++++-------- >>>>>>>> 1 file changed, 8 insertions(+), 8 deletions(-) >>>>>>> >>>>>>> It looks like we have this problem all over the place. The >>>>>>> documentation in doc/uImage now seems to have this problem too. >>>>>>> >>>>>>> I wonder if instead we should add reg / #address-cells / #size-cells properties? >>>>>> >>>>>> If the update on dts, might be an another-overhead to maintain u-boot >>>>>> dts wrt Linux dts sync. >>>>> >>>>> Anything that DTC is warning about in a dts that we get from the kernel, >>>>> should be fixed in the kernel. The kernel dtc is what we're using, and >>>>> is/will/can also complain about it. >>>> >>>> Kernel suppress these warning by default[1] and enables these warnings >>>> with W= compiler option. May be this should be included in u-boot as well? >>>> >>>> [1] >>>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/scripts/Makefile.lib#n272 >>> >>> We have slightly different logic for that today in >>> scripts/Makefile.extrawarn, perhaps we need to re-sync? >>> >> >> ahh you are right. I do not see these warning with the latest U-Boot. >> But I still agree with the $patch as these warnings are meant to be >> fixed at some point. > > > Please use "-" instead of "@". > > Update all the FIT examples. > You may need to update some C files. Yeah! "find ./ -type f | xargs -1 sed -i -e s/@/-/g" didn't give the expected results, though :-D But I bit the bullet and fixed every @1 sucker I could find, just need some more time to make proper patches. Cheers, Andre. > > > This is the conclusion of Device Tree community. > See the following commit of Linux. > > > > commit b21569cf1de925e0a42c9964bd7f520cb4a4d875 > Author: Viresh Kumar > Date: Thu Jun 22 09:15:11 2017 +0530 > > PM / OPP: Use - instead of @ for DT entries > > Compiling the DT file with W=1, DTC warns like follows: > > Warning (unit_address_vs_reg): Node /opp_table0/opp at 1000000000 has a > unit name, but no reg property > > Fix this by replacing '@' with '-' as the OPP nodes will never have a > "reg" property. > > Reported-by: Krzysztof Kozlowski > Reported-by: Masahiro Yamada > Suggested-by: Mark Rutland > Signed-off-by: Viresh Kumar > Acked-by: Rob Herring > Reviewed-by: Stephen Boyd > Signed-off-by: Rafael J. Wysocki > > >