[PATCH net-next] ARM: dts: imx6qdl: Remove unnecessary mdio #address-cells/#size-cells
Florian Fainelli
f.fainelli at gmail.com
Sat Jul 24 09:37:35 PDT 2021
On 7/23/2021 6:08 AM, Vladimir Oltean wrote:
> Hi Fabio,
>
> On Fri, Jul 23, 2021 at 08:28:35AM -0300, Fabio Estevam wrote:
>> Since commit dabb5db17c06 ("ARM: dts: imx6qdl: move phy properties into
>> phy device node") the following W=1 dtc warnings are seen:
>>
>> arch/arm/boot/dts/imx6qdl-aristainetos2.dtsi:323.7-334.4: Warning (avoid_unnecessary_addr_size): /soc/bus at 2100000/ethernet at 2188000/mdio: unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
>>
>> Remove the unnecessary mdio #address-cells/#size-cells to fix it.
>>
>> Fixes: dabb5db17c06 ("ARM: dts: imx6qdl: move phy properties into phy device node")
>> Signed-off-by: Fabio Estevam <festevam at gmail.com>
>> ---
>
> Are you actually sure this is the correct fix? If I look at mdio.yaml, I
> think it is pretty clear that the "ethernet-phy" subnode of the MDIO
> controller must have an "@[0-9a-f]+$" pattern, and a "reg" property. If
It is valid to omit the "reg" property of an Ethernet PHY which the
kernel will then dynamically scan for. If you know the Ethernet PHY
address it's obviously better to set it so you avoid scanning and the
time spent in doing that. The boot loader could (should?) also provide
that information to the kernel for the same reasons.
--
Florian
More information about the linux-arm-kernel
mailing list