From: hongbo.w...@nxp.com Date: Thu, 30 Jul 2020 18:25:05 +0800 > + if (vlan->proto == ETH_P_8021AD) { > + ocelot->enable_qinq = true; > + ocelot_port->qinq_mode = true; > + } ... > + if (vlan->proto == ETH_P_8021AD) { > + ocelot->enable_qinq = false; > + ocelot_port->qinq_mode = false; > + } > +
I don't understand how this can work just by using a boolean to track the state. This won't work properly if you are handling multiple QinQ VLAN entries. Also, I need Andrew and Florian to review and ACK the DSA layer changes that add the protocol value to the device notifier block.