From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 23 Oct 2017 16:34:00 -0400 Subject: [U-Boot] [PATCH] ARM: SPL: FIT: fix DTC warnings on FIT generation In-Reply-To: <56b176e7-5bd1-b674-a06d-377bd97888c0@ti.com> References: <20171004232439.20817-1-andre.przywara@arm.com> <20171016211349.GU12015@bill-the-cat> <56b176e7-5bd1-b674-a06d-377bd97888c0@ti.com> Message-ID: <20171023203400.GA22919@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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? -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: