bnxt_en: Enhance log message in bnxt_get_module_status()

Rturn early with -EOPNOTSUPP and an extack message if the PHY type is
BaseT since module status is not available for BaseT.

Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
Signed-off-by: Gautam R A <gautam-r.a@broadcom.com>
Signed-off-by: Michael Chan <michael.chan@broadcom.com>
Link: https://patch.msgid.link/20251126215648.1885936-3-michael.chan@broadcom.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>

diff --git a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c b/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
index baac639f9c9418404b8556867d43d6f4cb90f951..efb9bf20e66b023400004f2a15d5b05af52f824c 100644 (file)
--- a/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt_ethtool.c
@@ -4623,6 +4623,11 @@ static int bnxt_get_module_status(struct bnxt *bp, struct netlink_ext_ack *extac
            PORT_PHY_QCFG_RESP_MODULE_STATUS_WARNINGMSG)
                return 0;
 
+       if (bp->link_info.phy_type == PORT_PHY_QCFG_RESP_PHY_TYPE_BASET ||
+           bp->link_info.phy_type == PORT_PHY_QCFG_RESP_PHY_TYPE_BASETE){
+               NL_SET_ERR_MSG_MOD(extack, "Operation not supported as PHY type is Base-T");
+               return -EOPNOTSUPP;
+       }
        switch (bp->link_info.module_status) {
        case PORT_PHY_QCFG_RESP_MODULE_STATUS_PWRDOWN:
                NL_SET_ERR_MSG_MOD(extack, "Transceiver module is powering down");