cxgb3: Use VPD API in t3_seeprom_wp()
authorHeiner Kallweit <hkallweit1@gmail.com>
Fri, 10 Sep 2021 06:25:49 +0000 (08:25 +0200)
committerBjorn Helgaas <bhelgaas@google.com>
Tue, 26 Oct 2021 00:20:21 +0000 (19:20 -0500)
Use standard VPD API to replace t3_seeprom_write(), this prepares for
removing this function. Chelsio T3 maps the EEPROM write protect flag
to an arbitrary place in VPD address space, therefore we have to use
pci_write_vpd_any().

Link: https://lore.kernel.org/r/f768fdbe-3a16-d539-57d2-c7c908294336@gmail.com
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/chelsio/cxgb3/t3_hw.c

index 4ecf40b..ec4b49e 100644 (file)
@@ -642,7 +642,14 @@ int t3_seeprom_write(struct adapter *adapter, u32 addr, __le32 data)
  */
 int t3_seeprom_wp(struct adapter *adapter, int enable)
 {
-       return t3_seeprom_write(adapter, EEPROM_STAT_ADDR, enable ? 0xc : 0);
+       u32 data = enable ? 0xc : 0;
+       int ret;
+
+       /* EEPROM_STAT_ADDR is outside VPD area, use pci_write_vpd_any() */
+       ret = pci_write_vpd_any(adapter->pdev, EEPROM_STAT_ADDR, sizeof(u32),
+                               &data);
+
+       return ret < 0 ? ret : 0;
 }
 
 static int vpdstrtouint(char *s, u8 len, unsigned int base, unsigned int *val)