drm/msm/dsi: Don't get DSI index from DT

The DSI host and PHY driver currently expects the DT bindings to provide
custom properties "qcom,dsi-host-index" and "qcom,dsi-phy-index" so that
the driver can identify which DSI instance it is.

The binding isn't acceptable, but the driver still needs to figure out
what its instance id. This is now done by storing the mmio starting
addresses for each DSI instance in every SoC version in the driver. The
driver then identifies the index number by trying to match the stored
address with comparing the resource start address we get from DT.

We don't have compatible strings for DSI PHY on each SoC, but only the
DSI PHY type. We only support one SoC version for each PHY type, so we
get away doing the same thing above for the PHY driver. We can revisit
this when we support two SoCs with the same DSI PHY.

Signed-off-by: Archit Taneja <architt@codeaurora.org>
Signed-off-by: Rob Clark <robdclark@gmail.com>

5 files changed, 39 insertions, 5 deletions

diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
index e2f42d8ea294..f39386ed75e4 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.c
@@ -271,6 +271,30 @@ static const struct of_device_id dsi_phy_dt_match[] = {
         {}
 };
 
+/*
+ * Currently, we only support one SoC for each PHY type. When we have multiple
+ * SoCs for the same PHY, we can try to make the index searching a bit more
+ * clever.
+ */
+static int dsi_phy_get_id(struct msm_dsi_phy *phy)
+{
+        struct platform_device *pdev = phy->pdev;
+        const struct msm_dsi_phy_cfg *cfg = phy->cfg;
+        struct resource *res;
+        int i;
+
+        res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "dsi_phy");
+        if (!res)
+                return -EINVAL;
+
+        for (i = 0; i < cfg->num_dsi_phy; i++) {
+                if (cfg->io_start[i] == res->start)
+                        return i;
+        }
+
+        return -EINVAL;
+}
+
 static int dsi_phy_driver_probe(struct platform_device *pdev)
 {
         struct msm_dsi_phy *phy;
@@ -289,10 +313,10 @@ static int dsi_phy_driver_probe(struct platform_device *pdev)
         phy->cfg = match->data;
         phy->pdev = pdev;
 
-        ret = of_property_read_u32(dev->of_node,
-                                "qcom,dsi-phy-index", &phy->id);
-        if (ret) {
-                dev_err(dev, "%s: PHY index not specified, %d\n",
+        phy->id = dsi_phy_get_id(phy);
+        if (phy->id < 0) {
+                ret = phy->id;
+                dev_err(dev, "%s: couldn't identify PHY index, %d\n",
                         __func__, ret);
                 goto fail;
         }
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
index 0d54ed00386d..f24a85439b94 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy.h
@@ -38,6 +38,8 @@ struct msm_dsi_phy_cfg {
          * Fill default H/W values in illegal cells, eg. cell {0, 1}.
          */
         bool src_pll_truthtable[DSI_MAX][DSI_MAX];
+        const resource_size_t io_start[DSI_MAX];
+        const int num_dsi_phy;
 };
 
 extern const struct msm_dsi_phy_cfg dsi_phy_28nm_hpm_cfgs;
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_20nm.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_20nm.c
index f4bc11af849a..c757e2070cac 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_20nm.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_20nm.c
@@ -145,6 +145,8 @@ const struct msm_dsi_phy_cfg dsi_phy_20nm_cfgs = {
         .ops = {
                 .enable = dsi_20nm_phy_enable,
                 .disable = dsi_20nm_phy_disable,
-        }
+        },
+        .io_start = { 0xfd998300, 0xfd9a0300 },
+        .num_dsi_phy = 2,
 };
 
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c
index 96d1852af418..63d7fba31380 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm.c
@@ -145,6 +145,8 @@ const struct msm_dsi_phy_cfg dsi_phy_28nm_hpm_cfgs = {
                 .enable = dsi_28nm_phy_enable,
                 .disable = dsi_28nm_phy_disable,
         },
+        .io_start = { 0xfd922b00, 0xfd923100 },
+        .num_dsi_phy = 2,
 };
 
 const struct msm_dsi_phy_cfg dsi_phy_28nm_lp_cfgs = {
@@ -160,5 +162,7 @@ const struct msm_dsi_phy_cfg dsi_phy_28nm_lp_cfgs = {
                 .enable = dsi_28nm_phy_enable,
                 .disable = dsi_28nm_phy_disable,
         },
+        .io_start = { 0x1a98500 },
+        .num_dsi_phy = 1,
 };
 
diff --git a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm_8960.c b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm_8960.c
index 213355a3e767..7bdb9de54968 100644
--- a/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm_8960.c
+++ b/drivers/gpu/drm/msm/dsi/phy/dsi_phy_28nm_8960.c
@@ -192,4 +192,6 @@ const struct msm_dsi_phy_cfg dsi_phy_28nm_8960_cfgs = {
                 .enable = dsi_28nm_phy_enable,
                 .disable = dsi_28nm_phy_disable,
         },
+        .io_start = { 0x4700300, 0x5800300 },
+        .num_dsi_phy = 2,
 };