Link VF representors to parent PCI device to benefit from
systemd defined naming scheme.
Without this change the representor is visible as ethN.
Signed-off-by: Ivan Vecera <[email protected]>
---
drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c
index 2f1a1f2d2157..1467b94a6427 100644
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c
@@ -468,6 +468,7 @@ static void bnxt_vf_rep_netdev_init(struct bnxt *bp, struct bnxt_vf_rep *vf_rep,
struct net_device *pf_dev = bp->dev;
u16 max_mtu;
+ SET_NETDEV_DEV(dev, &bp->pdev->dev);
dev->netdev_ops = &bnxt_vf_rep_netdev_ops;
dev->ethtool_ops = &bnxt_vf_rep_ethtool_ops;
/* Just inherit all the featues of the parent PF as the VF-R
--
2.39.3
On Tue, Jun 20, 2023 at 04:48:55PM +0200, Ivan Vecera wrote:
> Link VF representors to parent PCI device to benefit from
> systemd defined naming scheme.
>
> Without this change the representor is visible as ethN.
>
> Signed-off-by: Ivan Vecera <[email protected]>
Reviewed-by: Simon Horman <[email protected]>
On Tue, 20 Jun 2023 22:19:32 +0530 Sriharsha Basavapatna wrote:
> This electronic communication and the information and any files transmitted
> with it, or attached to it, are confidential and are intended solely for
> the use of the individual or entity to whom it is addressed and may contain
> information that is confidential, legally privileged, protected by privacy
> laws, or otherwise restricted from disclosure to anyone else. If you are
> not the intended recipient or the person responsible for delivering the
> e-mail to the intended recipient, you are hereby notified that any use,
> copying, distributing, dissemination, forwarding, printing, or copying of
> this e-mail is strictly prohibited. If you received this e-mail in error,
> please return the e-mail to the sender, delete it from your computer, and
> destroy any printed copy of it.
Your emails to the list must not include any legal note of this nature.
I'll have to manually delete your review tag from the system.
On Tue, Jun 20, 2023 at 10:00 PM Ivan Vecera <[email protected]> wrote:
>
> Link VF representors to parent PCI device to benefit from
> systemd defined naming scheme.
>
> Without this change the representor is visible as ethN.
>
> Signed-off-by: Ivan Vecera <[email protected]>
Acked-by: Sriharsha Basavapatna <[email protected]>
> ---
> drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c
> index 2f1a1f2d2157..1467b94a6427 100644
> --- a/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c
> +++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_vfr.c
> @@ -468,6 +468,7 @@ static void bnxt_vf_rep_netdev_init(struct bnxt *bp, struct bnxt_vf_rep *vf_rep,
> struct net_device *pf_dev = bp->dev;
> u16 max_mtu;
>
> + SET_NETDEV_DEV(dev, &bp->pdev->dev);
> dev->netdev_ops = &bnxt_vf_rep_netdev_ops;
> dev->ethtool_ops = &bnxt_vf_rep_ethtool_ops;
> /* Just inherit all the featues of the parent PF as the VF-R
> --
> 2.39.3
>
>
--
This electronic communication and the information and any files transmitted
with it, or attached to it, are confidential and are intended solely for
the use of the individual or entity to whom it is addressed and may contain
information that is confidential, legally privileged, protected by privacy
laws, or otherwise restricted from disclosure to anyone else. If you are
not the intended recipient or the person responsible for delivering the
e-mail to the intended recipient, you are hereby notified that any use,
copying, distributing, dissemination, forwarding, printing, or copying of
this e-mail is strictly prohibited. If you received this e-mail in error,
please return the e-mail to the sender, delete it from your computer, and
destroy any printed copy of it.
On Tue, Jun 20, 2023 at 7:52 AM Ivan Vecera <[email protected]> wrote:
>
> Link VF representors to parent PCI device to benefit from
> systemd defined naming scheme.
>
> Without this change the representor is visible as ethN.
>
> Signed-off-by: Ivan Vecera <[email protected]>
Thanks.
Reviewed-by: Michael Chan <[email protected]>
On Tue, Jun 20, 2023 at 10:26:21AM -0700, Michael Chan wrote:
> On Tue, Jun 20, 2023 at 7:52 AM Ivan Vecera <[email protected]> wrote:
> >
> > Link VF representors to parent PCI device to benefit from
> > systemd defined naming scheme.
> >
> > Without this change the representor is visible as ethN.
> >
> > Signed-off-by: Ivan Vecera <[email protected]>
>
> Thanks.
>
> Reviewed-by: Michael Chan <[email protected]>
nit:) broadcon -> broadcom
On Tue, Jun 20, 2023 at 12:41 PM Simon Horman <[email protected]> wrote:
>
> On Tue, Jun 20, 2023 at 10:26:21AM -0700, Michael Chan wrote:
> > On Tue, Jun 20, 2023 at 7:52 AM Ivan Vecera <[email protected]> wrote:
> > >
> > > Link VF representors to parent PCI device to benefit from
> > > systemd defined naming scheme.
> > >
> > > Without this change the representor is visible as ethN.
> > >
> > > Signed-off-by: Ivan Vecera <[email protected]>
> >
> > Thanks.
> >
> > Reviewed-by: Michael Chan <[email protected]>
>
> nit:) broadcon -> broadcom
>
>
Typo fixed.
Reviewed-by: Michael Chan <[email protected]>
Hello:
This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <[email protected]>:
On Tue, 20 Jun 2023 16:48:55 +0200 you wrote:
> Link VF representors to parent PCI device to benefit from
> systemd defined naming scheme.
>
> Without this change the representor is visible as ethN.
>
> Signed-off-by: Ivan Vecera <[email protected]>
>
> [...]
Here is the summary with links:
- [net-next] bnxt_en: Link representors to PCI device
https://git.kernel.org/netdev/net-next/c/7ad7b7023fcb
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html