On 3/15/26 5:20 PM, Daniel Golle wrote:
(struct dsa_switch *ds, int port,
+ struct dsa_bridge bridge,
+ bool *tx_fwd_offload,
+ struct netlink_ext_ack *extack)
+{
+ struct mxl862xx_priv *priv = ds->priv;
+ struct mxl862xx_bridge *mxlbridge;
+
+ mxlbridge = mxl862xx_find_bridge(ds, bridge);
+ if (!mxlbridge) {
+ mxlbridge = mxl862xx_allocate_bridge(ds, bridge.num);
+ if (IS_ERR(mxlbridge))
+ return PTR_ERR(mxlbridge);
+ }
+
+ __set_bit(port, mxlbridge->portmap);
AFAICS mxl862xx_bridge role is to track the above portmap, which in turn
looks like a quite generic information which could be easily tracked by
the DSA core. I'm wondering if it would be useful to move implement
tracking there, and remove the mxl862xx_bridge entirely.
/P