On 05/07/2018 05:17 AM, Fabio Estevam wrote: > From: Fabio Estevam <fabio.este...@nxp.com> > > If the example binding is used on a real dts file, the following DTC > warning is seen with W=1: > > arch/arm/boot/dts/imx6q-b450v3.dtb: Warning (avoid_unnecessary_addr_size): > /mdio-gpio/switch@0: unnecessary #address-cells/#size-cells without "ranges" > or child "reg" property > > Remove unnecessary #address-cells/#size-cells to improve the binding > document examples.
In most cases this is unnecessary because the parent node is an MDIO, I2C or SPI controller, and those typically have #address-cells = <1> and #size-cells = <0> because of their specific binding, but this is not necessarily true if using e.g: a MMIO mapped Ethernet switch. With the particular example though, this appears fine: Reviewed-by: Florian Fainelli <f.faine...@gmail.com> -- Florian
