On Wed, Mar 17, 2021 at 02:46:41PM +0100, Marek Behún wrote: > There are two implementations of the .set_egress_port method, and both > of them, if successful, set chip->*gress_dest_port variable. > > To avoid code repetition, wrap this method into > mv88e6xxx_set_egress_port. > > Signed-off-by: Marek Behún <ka...@kernel.org> > Reviewed-by: Pavana Sharma <pavana.sha...@digi.com> > ---
Separately from this series, do you think you can rename the "egress_port" into "monitor_port" across the driver? Seeing an EGRESS_DIR_INGRESS is pretty strange. Reviewed-by: Vladimir Oltean <olte...@gmail.com>