]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit
MIPS: mscc: ocelot: disable all switch ports by default
authorVladimir Oltean <vladimir.oltean@nxp.com>
Thu, 19 Aug 2021 17:04:15 +0000 (20:04 +0300)
committerThomas Bogendoerfer <tsbogend@alpha.franken.de>
Sat, 21 Aug 2021 08:38:29 +0000 (10:38 +0200)
commit0181f6f19c6c35b24f1516d8db22f3bbce762633
treeffbf43f06eac60d243fbb866519627c6e5cff950
parentcd92dbaf5d0444c403ca818ec37d945f05e9d240
MIPS: mscc: ocelot: disable all switch ports by default

The ocelot switch driver used to ignore ports which do not have a
phy-handle property and not probe those, but this is not quite ok since
it is valid to not have a phy-handle property if there is a fixed-link.

It seems that checking for a phy-handle was a proxy for the proper check
which is for the status, but that doesn't make a lot of sense, since the
ocelot driver already iterates using for_each_available_child_of_node
which skips the disabled ports, so I have no idea.

Anyway, a widespread pattern in device trees is for a SoC dtsi to
disable by default all hardware, and let board dts files enable what is
used. So let's do that and enable only the ports with a phy-handle in
the pcb120 and pcb123 device tree files.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Thomas Bogendoerfer <tsbogend@alpha.franken.de>
arch/mips/boot/dts/mscc/ocelot.dtsi
arch/mips/boot/dts/mscc/ocelot_pcb120.dts
arch/mips/boot/dts/mscc/ocelot_pcb123.dts