|
Thomas Bogendoerfer |
bce3ca |
From: Shannon Nelson <snelson@pensando.io>
|
|
Thomas Bogendoerfer |
bce3ca |
Date: Tue, 12 Oct 2021 16:15:20 -0700
|
|
Thomas Bogendoerfer |
bce3ca |
Subject: ionic: no devlink_unregister if not registered
|
|
Thomas Bogendoerfer |
bce3ca |
Patch-mainline: v5.16-rc1
|
|
Thomas Bogendoerfer |
bce3ca |
Git-commit: d1f24712a86abd04d82cf4b00fb4ab8ff2d23c8a
|
|
Thomas Bogendoerfer |
bce3ca |
References: jsc#SLE-19282
|
|
Thomas Bogendoerfer |
bce3ca |
|
|
Thomas Bogendoerfer |
bce3ca |
Don't try to unregister the devlink if it hasn't been registered
|
|
Thomas Bogendoerfer |
bce3ca |
yet. This bit of error cleanup code got missed in the recent
|
|
Thomas Bogendoerfer |
bce3ca |
devlink registration changes.
|
|
Thomas Bogendoerfer |
bce3ca |
|
|
Thomas Bogendoerfer |
bce3ca |
Fixes: 7911c8bd546f ("ionic: Move devlink registration to be last devlink command")
|
|
Thomas Bogendoerfer |
bce3ca |
Signed-off-by: Shannon Nelson <snelson@pensando.io>
|
|
Thomas Bogendoerfer |
bce3ca |
Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
|
|
Thomas Bogendoerfer |
bce3ca |
Link: https://lore.kernel.org/r/20211012231520.72582-1-snelson@pensando.io
|
|
Thomas Bogendoerfer |
bce3ca |
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
|
|
Thomas Bogendoerfer |
bce3ca |
Acked-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>
|
|
Thomas Bogendoerfer |
bce3ca |
---
|
|
Thomas Bogendoerfer |
bce3ca |
drivers/net/ethernet/pensando/ionic/ionic_devlink.c | 1 -
|
|
Thomas Bogendoerfer |
bce3ca |
1 file changed, 1 deletion(-)
|
|
Thomas Bogendoerfer |
bce3ca |
|
|
Thomas Bogendoerfer |
bce3ca |
--- a/drivers/net/ethernet/pensando/ionic/ionic_devlink.c
|
|
Thomas Bogendoerfer |
bce3ca |
+++ b/drivers/net/ethernet/pensando/ionic/ionic_devlink.c
|
|
Thomas Bogendoerfer |
bce3ca |
@@ -87,7 +87,6 @@ int ionic_devlink_register(struct ionic
|
|
Thomas Bogendoerfer |
bce3ca |
err = devlink_port_register(dl, &ionic->dl_port, 0);
|
|
Thomas Bogendoerfer |
bce3ca |
if (err) {
|
|
Thomas Bogendoerfer |
bce3ca |
dev_err(ionic->dev, "devlink_port_register failed: %d\n", err);
|
|
Thomas Bogendoerfer |
bce3ca |
- devlink_unregister(dl);
|
|
Thomas Bogendoerfer |
bce3ca |
return err;
|
|
Thomas Bogendoerfer |
bce3ca |
}
|
|
Thomas Bogendoerfer |
bce3ca |
|