On Tue, Aug 03, 2021 at 04:14:35PM +0800, DENG Qingfang wrote: > On Tue, Aug 03, 2021 at 08:54:24AM +0200, Oleksij Rempel wrote: > > + if (dsa_is_cpu_port(ds, port)) { > > + /* CPU port should be allowed to communicate with all user > > + * ports. > > + */ > > + port_mask = dsa_user_ports(ds); > > + /* Enable Atheros header on CPU port. This will allow us > > + * communicate with each port separately > > + */ > > + port_ctrl |= AR9331_SW_PORT_CTRL_HEAD_EN; > > + } else if (dsa_is_user_port(ds, port)) { > > + /* User ports should communicate only with the CPU port. > > + */ > > + port_mask = BIT(dsa_to_port(ds, port)->cpu_dp->index); > > + port_ctrl |= AR9331_SW_PORT_CTRL_LEARN_EN; > > All user ports should start with address learning disabled. > To toggle it, implement .port_pre_bridge_flags and .port_bridge_flags. Ok, thx! Will move it to the separate patch -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |