net: hns3: add ioctl support for imp-controlled PHYs

When the imp-controlled PHYs feature is enabled, driver will not
register mdio bus. In order to support ioctl ops for phy tool to
read or write phy register in this case, the firmware implement
a new command for driver and driver implement ioctl by using this
new command.

Signed-off-by: Guangbin Huang <huangguangbin2@huawei.com>
Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
index f45ceaa8126fdf9563c077f58c369e263ef0c131..abeacc990a9b41f36fa49c5c6d3539fbeb778f74 100644 (file)
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.h
@@ -306,6 +306,7 @@ enum hclge_opcode_type {
 
        /* PHY command */
        HCLGE_OPC_PHY_LINK_KSETTING     = 0x7025,
+       HCLGE_OPC_PHY_REG               = 0x7026,
 };
 
 #define HCLGE_TQP_REG_OFFSET           0x80000
@@ -1166,6 +1167,13 @@ struct hclge_phy_link_ksetting_1_cmd {
        u8 rsv[22];
 };
 
+struct hclge_phy_reg_cmd {
+       __le16 reg_addr;
+       u8 rsv0[2];
+       __le16 reg_val;
+       u8 rsv1[18];
+};
+
 int hclge_cmd_init(struct hclge_dev *hdev);
 static inline void hclge_write_reg(void __iomem *base, u32 reg, u32 value)
 {

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index dbca489445dfe08886e40cc2755c36183d0ef624..adc2ec7265e6711eb87ba2a6eda6823e8ab27e8a 100644 (file)
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
@@ -8904,6 +8904,29 @@ static int hclge_set_mac_addr(struct hnae3_handle *handle, void *p,
        return 0;
 }
 
+static int hclge_mii_ioctl(struct hclge_dev *hdev, struct ifreq *ifr, int cmd)
+{
+       struct mii_ioctl_data *data = if_mii(ifr);
+
+       if (!hnae3_dev_phy_imp_supported(hdev))
+               return -EOPNOTSUPP;
+
+       switch (cmd) {
+       case SIOCGMIIPHY:
+               data->phy_id = hdev->hw.mac.phy_addr;
+               /* this command reads phy id and register at the same time */
+               fallthrough;
+       case SIOCGMIIREG:
+               data->val_out = hclge_read_phy_reg(hdev, data->reg_num);
+               return 0;
+
+       case SIOCSMIIREG:
+               return hclge_write_phy_reg(hdev, data->reg_num, data->val_in);
+       default:
+               return -EOPNOTSUPP;
+       }
+}
+
 static int hclge_do_ioctl(struct hnae3_handle *handle, struct ifreq *ifr,
                          int cmd)
 {
@@ -8911,7 +8934,7 @@ static int hclge_do_ioctl(struct hnae3_handle *handle, struct ifreq *ifr,
        struct hclge_dev *hdev = vport->back;
 
        if (!hdev->hw.mac.phydev)
-               return -EOPNOTSUPP;
+               return hclge_mii_ioctl(hdev, ifr, cmd);
 
        return phy_mii_ioctl(hdev->hw.mac.phydev, ifr, cmd);
 }

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mdio.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mdio.c
index e898207025406099b206b3a8e9cceac14dce2421..08e88d9422cde3dfe145fdd9813a8b513e730b20 100644 (file)
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mdio.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mdio.c
@@ -268,3 +268,42 @@ void hclge_mac_stop_phy(struct hclge_dev *hdev)
 
        phy_stop(phydev);
 }
+
+u16 hclge_read_phy_reg(struct hclge_dev *hdev, u16 reg_addr)
+{
+       struct hclge_phy_reg_cmd *req;
+       struct hclge_desc desc;
+       int ret;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_PHY_REG, true);
+
+       req = (struct hclge_phy_reg_cmd *)desc.data;
+       req->reg_addr = cpu_to_le16(reg_addr);
+
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret)
+               dev_err(&hdev->pdev->dev,
+                       "failed to read phy reg, ret = %d.\n", ret);
+
+       return le16_to_cpu(req->reg_val);
+}
+
+int hclge_write_phy_reg(struct hclge_dev *hdev, u16 reg_addr, u16 val)
+{
+       struct hclge_phy_reg_cmd *req;
+       struct hclge_desc desc;
+       int ret;
+
+       hclge_cmd_setup_basic_desc(&desc, HCLGE_OPC_PHY_REG, false);
+
+       req = (struct hclge_phy_reg_cmd *)desc.data;
+       req->reg_addr = cpu_to_le16(reg_addr);
+       req->reg_val = cpu_to_le16(val);
+
+       ret = hclge_cmd_send(&hdev->hw, &desc, 1);
+       if (ret)
+               dev_err(&hdev->pdev->dev,
+                       "failed to write phy reg, ret = %d.\n", ret);
+
+       return ret;
+}

diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mdio.h b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mdio.h
index dd9a1218a7b03018a30359970218163af4d052b4..fd0e20190b90f1109a14813c49b5d89609d4212d 100644 (file)
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mdio.h
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_mdio.h
@@ -9,5 +9,7 @@ int hclge_mac_connect_phy(struct hnae3_handle *handle);
 void hclge_mac_disconnect_phy(struct hnae3_handle *handle);
 void hclge_mac_start_phy(struct hclge_dev *hdev);
 void hclge_mac_stop_phy(struct hclge_dev *hdev);
+u16 hclge_read_phy_reg(struct hclge_dev *hdev, u16 reg_addr);
+int hclge_write_phy_reg(struct hclge_dev *hdev, u16 reg_addr, u16 val);
 
 #endif