On Fri, Jan 08, 2021 at 12:35:23AM +0100, Andrew Lunn wrote: > > +static struct dsa_port *devlink_to_dsa_port(struct devlink_port *dlp) > > +{ > > + return container_of(dlp, struct dsa_port, devlink_port); > > +} > > I wonder if this should be moved to include/net/dsa.h next to the > other little helpers used to convert between devlink structures and > DSA structures?
I wrote this before your series for devlink regions. The way I'm using devlink_to_dsa_port, I guess I can just replace it with your dsa_devlink_port_to_port function.