Re: [PATCH net-next v2 12/14] net: ethernet: qualcomm: Initialize PPE L2 bridge settings

From: Lei Wei
Date: Wed Jan 15 2025 - 01:58:10 EST




On 1/14/2025 9:02 PM, Andrew Lunn wrote:
I would like to clarify that representing the bridge and its slave ports
inside PPE (using a VSI - virtual switch instance) is a pre-requisite before
learning can take place on a port. At this point, since switchdev
is not enabled, VSI is not created for port/bridge and hence FDB learning
does not take place. Later when we enable switchdev and represent the
bridge/slave-ports in PPE, FDB learning will automatically occur on top of
this initial configuration. I will add this note in the comments and commit
message to make it clear.

So it seems like the comment is not the best. You don't actually
enable learning...


Yes, I will update the comment to make it more clear.

Andrew