[PATCH 09/22] scsi: hisi_sas: retrieve SAS address for pci-based controller

From: John Garry
Date: Wed May 17 2017 - 06:20:01 EST


For a pci-based controller, retrieve the SAS address from the
ACPI tables.

The retrieval is based on the ACPI device node name. Sample is
as follows:
Scope(_SB)
{
Device(SAS0) {
Name(_HID, "HISI0163")
Name (_DSD, Package () {
ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"),
Package () {
Package (2) {"sas-addr", Package() {0x50, 0x01, 0x88, 0x20, 0x16, 00, 00, 0x00}},
}
})
}
}

If the ACPI node or node property is not available, then we fall
back on a default address.

Signed-off-by: John Garry <john.garry@xxxxxxxxxx>
---
drivers/scsi/hisi_sas/hisi_sas.h | 2 +
drivers/scsi/hisi_sas/hisi_sas_pci_init.c | 104 ++++++++++++++++++++++++++++++
drivers/scsi/hisi_sas/hisi_sas_v3_hw.c | 2 +-
3 files changed, 107 insertions(+), 1 deletion(-)

diff --git a/drivers/scsi/hisi_sas/hisi_sas.h b/drivers/scsi/hisi_sas/hisi_sas.h
index 9bc9550..9fd874d 100644
--- a/drivers/scsi/hisi_sas/hisi_sas.h
+++ b/drivers/scsi/hisi_sas/hisi_sas.h
@@ -190,6 +190,7 @@ struct hisi_sas_pci_hw {
int n_phy;
int queue_count;
const struct hisi_sas_hw *hw;
+ const char *fw_name;
};

struct hisi_hba {
@@ -250,6 +251,7 @@ struct hisi_hba {
struct hisi_sas_slot *slot_info;
unsigned long flags;
const struct hisi_sas_hw *hw; /* Low level hw interface */
+ const char *fw_name;
unsigned long sata_dev_bitmap[BITS_TO_LONGS(HISI_SAS_MAX_DEVICES)];
struct work_struct rst_work;
};
diff --git a/drivers/scsi/hisi_sas/hisi_sas_pci_init.c b/drivers/scsi/hisi_sas/hisi_sas_pci_init.c
index e00b482..6d92893 100644
--- a/drivers/scsi/hisi_sas/hisi_sas_pci_init.c
+++ b/drivers/scsi/hisi_sas/hisi_sas_pci_init.c
@@ -11,6 +11,107 @@
#include "hisi_sas.h"
#define DRV_NAME "hisi_sas_pci"

+
+static const u8 default_sas_addr[SAS_ADDR_SIZE] = {
+ 0x50, 0x01, 0x88, 0x20, 0x16, 0x0, 0x0, 0x0};
+static void hisi_sas_pci_set_sas_addr(struct hisi_hba *hisi_hba);
+
+static void
+hisi_sas_pci_set_default_sas_addr(struct hisi_hba *hisi_hba)
+{
+ dev_warn(hisi_hba->dev, "using default SAS ADDR\n");
+ memcpy(hisi_hba->sas_addr, default_sas_addr, SAS_ADDR_SIZE);
+}
+
+#if defined(CONFIG_ACPI)
+static int
+acpi_get_sas_address(struct device *dev, struct acpi_device *adev,
+ u8 *dst)
+{
+ u8 addr[SAS_ADDR_SIZE];
+ int rc;
+
+ rc = fwnode_property_read_u8_array(acpi_fwnode_handle(adev),
+ "sas-addr", addr, SAS_ADDR_SIZE);
+ if (rc) {
+ dev_warn(dev, "could not read sas-addr (%d)\n", rc);
+ return rc;
+ }
+
+ dev_info(dev, "SAS address set to: %pM\n", addr);
+
+ memcpy(dst, addr, SAS_ADDR_SIZE);
+
+ return 0;
+}
+
+static acpi_status
+hisi_sas_acpi_match_id(acpi_handle handle, u32 lvl,
+ void *context, void **ret_val)
+{
+ struct acpi_buffer string = { ACPI_ALLOCATE_BUFFER, NULL };
+ struct hisi_hba *hisi_hba = context;
+ struct device *dev = hisi_hba->dev;
+ struct acpi_device *adev;
+ int rc;
+
+ if (ACPI_FAILURE(acpi_get_name(handle, ACPI_SINGLE_NAME, &string))) {
+ dev_warn(dev, "Invalid link device\n");
+ return AE_OK;
+ }
+
+ if (strncmp(string.pointer, hisi_hba->fw_name, 4))
+ return AE_OK;
+
+ rc = acpi_bus_get_device(handle, &adev);
+
+ if (rc == 0)
+ rc = acpi_get_sas_address(dev, adev, hisi_hba->sas_addr);
+ else
+ dev_warn(dev, "could not get acpi bus device (%d)\n", rc);
+
+ if (rc == 0)
+ *(acpi_handle *)ret_val = handle;
+
+ kfree(string.pointer);
+ return AE_CTRL_TERMINATE;
+}
+
+static acpi_status
+hisi_sas_acpi_set_sas_addr(struct hisi_hba *hisi_hba)
+{
+ acpi_status status1;
+ acpi_handle *acpi_handle = NULL;
+
+ status1 = acpi_get_devices(NULL, hisi_sas_acpi_match_id,
+ hisi_hba, (void **)&acpi_handle);
+
+ if (status1 != AE_OK)
+ return status1;
+
+ if (!acpi_handle)
+ return AE_ERROR;
+
+ return AE_OK;
+}
+#else
+static acpi_status
+hisi_sas_acpi_set_sas_addr(struct hisi_hba *hisi_hba)
+{
+ return AE_ERROR;
+}
+#endif
+
+static void hisi_sas_pci_set_sas_addr(struct hisi_hba *hisi_hba)
+{
+ acpi_status status = AE_ERROR;
+ if (!acpi_disabled)
+ status = hisi_sas_acpi_set_sas_addr(hisi_hba);
+ if (status == AE_OK)
+ return;
+ hisi_sas_pci_set_default_sas_addr(hisi_hba);
+}
+
static struct Scsi_Host *
hisi_sas_shost_alloc_pci(struct pci_dev *pdev,
const struct hisi_sas_pci_hw *hw)
@@ -32,6 +133,9 @@
hisi_hba->shost = shost;
SHOST_TO_SAS_HA(shost) = &hisi_hba->sha;

+ hisi_hba->fw_name = hw->fw_name;
+ hisi_sas_pci_set_sas_addr(hisi_hba);
+
init_timer(&hisi_hba->timer);

if (hisi_sas_alloc(hisi_hba, shost)) {
diff --git a/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c b/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c
index 80707f6..68b9b0e 100644
--- a/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c
+++ b/drivers/scsi/hisi_sas/hisi_sas_v3_hw.c
@@ -20,7 +20,7 @@ enum {
};

static const struct hisi_sas_pci_hw hisi_sas_pci_hw_info[] = {
- [hip08] = {8, 16, &hisi_sas_v3_hw},
+ [hip08] = {8, 16, &hisi_sas_v3_hw, "SAS0"},
};

static const struct pci_device_id sas_v3_pci_table[] = {
--
1.9.1