Re: [RFC PATCH v2 net-next 01/14] net: dsa: mt7530: fix comments regarding port 5 and 6 for both switches

From: Arınç ÜNAL
Date: Mon Apr 10 2023 - 10:54:16 EST


On 10.04.2023 17:30, Daniel Golle wrote:
On Fri, Apr 07, 2023 at 04:46:13PM +0300, arinc9.unal@xxxxxxxxx wrote:
From: Arınç ÜNAL <arinc.unal@xxxxxxxxxx>

There's no logic to numerically order the CPU ports. State the port number
and its being a CPU port instead.

Port 5 is often used as a user port as well, eg. on the BPi-R3 where
it serves to provide SerDes for the 2nd SFP cage.
On other boards (e.g. Netgear WAX-206) it is used to connect a 2.5G
PHY used as WAN port.

Hence just stating that port 5 "a CPU port" could be a bit misleading
as it is not always used as a CPU port.

Makes sense. I was not using the DSA term, so the "a CPU port" here rather meant that the port connects to the CPU. I'll change it to "which can be used as a CPU port" on both ports, that should explain that it can be used as other than a CPU port, and it does not necessarily connect to a CPU MAC.

Thanks.
Arınç