net: phy: bcm54811: New link mode for BroadR-Reach
authorKamil Horák (2N) <kamilh@axis.com>
Fri, 12 Jul 2024 15:07:06 +0000 (17:07 +0200)
committerJakub Kicinski <kuba@kernel.org>
Mon, 15 Jul 2024 03:38:34 +0000 (20:38 -0700)
Introduce a new link mode necessary for 10 MBit single-pair
connection in BroadR-Reach mode on bcm5481x PHY by Broadcom.
This new link mode, 10baseT1BRR, is known as 1BR10 in the Broadcom
terminology. Another link mode to be used is 1BR100 and it is already
present as 100baseT1, because Broadcom's 1BR100 became 100baseT1
(IEEE 802.3bw).

Signed-off-by: Kamil Horák (2N) <kamilh@axis.com>
Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com>
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
Link: https://patch.msgid.link/20240712150709.3134474-2-kamilh@axis.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/phy/phy-core.c
include/uapi/linux/ethtool.h
net/ethtool/common.c

index a235ea2..1f98b6a 100644 (file)
@@ -13,7 +13,7 @@
  */
 const char *phy_speed_to_str(int speed)
 {
-       BUILD_BUG_ON_MSG(__ETHTOOL_LINK_MODE_MASK_NBITS != 102,
+       BUILD_BUG_ON_MSG(__ETHTOOL_LINK_MODE_MASK_NBITS != 103,
                "Enum ethtool_link_mode_bit_indices and phylib are out of sync. "
                "If a speed or mode has been added please update phy_speed_to_str "
                "and the PHY settings array.\n");
@@ -266,6 +266,7 @@ static const struct phy_setting settings[] = {
        PHY_SETTING(     10, FULL,     10baseT1S_Full           ),
        PHY_SETTING(     10, HALF,     10baseT1S_Half           ),
        PHY_SETTING(     10, HALF,     10baseT1S_P2MP_Half      ),
+       PHY_SETTING(     10, FULL,     10baseT1BRR_Full         ),
 };
 #undef PHY_SETTING
 
index 230110b..4a0a6e7 100644 (file)
@@ -2054,6 +2054,7 @@ enum ethtool_link_mode_bit_indices {
        ETHTOOL_LINK_MODE_10baseT1S_Full_BIT             = 99,
        ETHTOOL_LINK_MODE_10baseT1S_Half_BIT             = 100,
        ETHTOOL_LINK_MODE_10baseT1S_P2MP_Half_BIT        = 101,
+       ETHTOOL_LINK_MODE_10baseT1BRR_Full_BIT           = 102,
 
        /* must be last entry */
        __ETHTOOL_LINK_MODE_MASK_NBITS
index 67d06cd..5f99a32 100644 (file)
@@ -211,6 +211,7 @@ const char link_mode_names[][ETH_GSTRING_LEN] = {
        __DEFINE_LINK_MODE_NAME(10, T1S, Full),
        __DEFINE_LINK_MODE_NAME(10, T1S, Half),
        __DEFINE_LINK_MODE_NAME(10, T1S_P2MP, Half),
+       __DEFINE_LINK_MODE_NAME(10, T1BRR, Full),
 };
 static_assert(ARRAY_SIZE(link_mode_names) == __ETHTOOL_LINK_MODE_MASK_NBITS);
 
@@ -251,6 +252,7 @@ static_assert(ARRAY_SIZE(link_mode_names) == __ETHTOOL_LINK_MODE_MASK_NBITS);
 #define __LINK_MODE_LANES_T1S_P2MP     1
 #define __LINK_MODE_LANES_VR8          8
 #define __LINK_MODE_LANES_DR8_2                8
+#define __LINK_MODE_LANES_T1BRR                1
 
 #define __DEFINE_LINK_MODE_PARAMS(_speed, _type, _duplex)      \
        [ETHTOOL_LINK_MODE(_speed, _type, _duplex)] = {         \
@@ -374,6 +376,7 @@ const struct link_mode_info link_mode_params[] = {
        __DEFINE_LINK_MODE_PARAMS(10, T1S, Full),
        __DEFINE_LINK_MODE_PARAMS(10, T1S, Half),
        __DEFINE_LINK_MODE_PARAMS(10, T1S_P2MP, Half),
+       __DEFINE_LINK_MODE_PARAMS(10, T1BRR, Full),
 };
 static_assert(ARRAY_SIZE(link_mode_params) == __ETHTOOL_LINK_MODE_MASK_NBITS);