On Mon, Nov 25, 2024 at 5:32 AM Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote: > > Google Juniper platforms have a very old bootloader which populates > /firmware node without proper address/size-cells leading to warnings: > > Missing '#address-cells' in /firmware > WARNING: CPU: 0 PID: 1 at drivers/of/base.c:106 of_bus_n_addr_cells+0x90/0xf0 > Modules linked in: > CPU: 0 UID: 0 PID: 1 Comm: swapper/0 Not tainted 6.12.0 #1 933ab9971ff4d5dc58cb378a96f64c7f72e3454d > Hardware name: Google juniper sku16 board (DT) > ... > Missing '#size-cells' in /firmware > WARNING: CPU: 0 PID: 1 at drivers/of/base.c:133 of_bus_n_size_cells+0x90/0xf0 > Modules linked in: > CPU: 0 UID: 0 PID: 1 Comm: swapper/0 Tainted: G W 6.12.0 #1 933ab9971ff4d5dc58cb378a96f64c7f72e3454d > Tainted: [W]=WARN > Hardware name: Google juniper sku16 board (DT) > > The platform won't receive updated bootloader/firmware so add it to > excluded platform list to silence the warning. I'm interested to know what needs these in /firmware. /firmware is supposed to be for things without an MMIO interface. An alternative solution is to add the properties. That doesn't require CONFIG_OF_DYNAMIC and is often the approach powerpc uses. I'm also wondering if ranges is also missing? Rob
