On Thu, May 09, 2024 at 01:13:55PM +0200, Jiri Pirko wrote: > Tue, May 07, 2024 at 01:45:07PM CEST, [email protected] > wrote: > >From: Piotr Raczynski <[email protected]> > > > >Implement subfunction driver. It is probe when subfunction port is > >activated. > > > >VSI is already created. During the probe VSI is being configured. > >MAC unicast and broadcast filter is added to allow traffic to pass. > > > >Store subfunction pointer in VSI struct. The same is done for VF > >pointer. Make union of subfunction and VF pointer as only one of them > >can be set with one VSI. > > > >Reviewed-by: Jiri Pirko <[email protected]> > >Signed-off-by: Piotr Raczynski <[email protected]> > >Signed-off-by: Michal Swiatkowski <[email protected]> > > Perhaps it would make things clearer for reviewer to have all patches > related to sf auxdev/devlink/netdev at the end of the patchset, after > activation patch. Not sure why you want to mix it here.
I need this code to use it in port representor implementation. You suggested in previous review to move activation at the end [1]. [1] https://lore.kernel.org/netdev/Zhje0mQgQTMXwICb@nanopsycho/
