[U-Boot] [PATCH v1 04/14] DTS: imx: Remove not needed '#address-cells' and '#size-cells' properties

Marek Vasut marex at denx.de
Wed Jan 2 14:10:40 UTC 2019


On 1/2/19 10:43 AM, Lukasz Majewski wrote:
> Hi Marek,
> 
>> On 1/2/19 12:37 AM, Lukasz Majewski wrote:
>>> This commit fixes warnings produced by newest in u-boot DTC
>>> compiler: 'unnecessary #address-cells/#size-cells without "ranges"
>>> or child "reg" property'
>>>
>>> Signed-off-by: Lukasz Majewski <lukma at denx.de>  
>>
>> I presume the DTSIs are taken from Linux, is this a backport from
>> Linux then ?
> 
> Regarding Linux - with Version: DTC 1.4.7-gc86da84d and make W=1 I do
> not see those warnings as in  scripts/Makefile.lib :
> 
> # Disable noisy checks by default
> ifeq ($(findstring 1,$(KBUILD_ENABLE_EXTRA_GCC_CHECKS)),)
> DTC_FLAGS += -Wno-unit_address_vs_reg \
>         -Wno-unit_address_format \
>         -Wno-avoid_unnecessary_addr_size \
> 
> those warnings are disabled (-Wno-avoid_unnecessary_addr_size).
> 
> On the other hand - in u-boot those warnings are checked (maybe are
> necessary for OF_PLATDATA correct operation?).

No, keep the DTs synced and fix things upstream (in Linux) if needed,
then backport the fixes.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list