On Mon Jan 13 10:01:31 2025 +0530, Depeng Shao wrote:
> Add a PHY configuration sequence and PHY resource for the sm8550 which
> uses a Qualcomm Gen 2 version 2.1.2 CSI-2 PHY.
> 
> The PHY can be configured as two phase or three phase in C-PHY or D-PHY
> mode. This configuration supports two-phase D-PHY mode.
> 
> Reviewed-by: Bryan O'Donoghue <bryan.odonog...@linaro.org>
> Signed-off-by: Depeng Shao <quic_depe...@quicinc.com>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 .../platform/qcom/camss/camss-csiphy-3ph-1-0.c     | 111 +++++++++++++++++++++
 drivers/media/platform/qcom/camss/camss.c          | 109 ++++++++++++++++++++
 2 files changed, 220 insertions(+)

---

diff --git a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c 
b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
index b5a116ced6a7..a6cc957b986e 100644
--- a/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
+++ b/drivers/media/platform/qcom/camss/camss-csiphy-3ph-1-0.c
@@ -318,6 +318,111 @@ csiphy_lane_regs lane_regs_sm8250[] = {
        {0x0884, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
 };
 
+/* GEN2 2.1.2 2PH DPHY mode */
+static const struct
+csiphy_lane_regs lane_regs_sm8550[] = {
+       {0x0E90, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E98, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E94, 0x07, 0x01, CSIPHY_DEFAULT_PARAMS},
+       {0x00A0, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0090, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0098, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0094, 0x07, 0x01, CSIPHY_DEFAULT_PARAMS},
+       {0x0494, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x04A0, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0490, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0498, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0494, 0x07, 0x01, CSIPHY_DEFAULT_PARAMS},
+       {0x0894, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x08A0, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0890, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0898, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0894, 0x07, 0x01, CSIPHY_DEFAULT_PARAMS},
+       {0x0C94, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0CA0, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C90, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C98, 0x08, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C94, 0x07, 0x01, CSIPHY_DEFAULT_PARAMS},
+       {0x0E30, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E28, 0x04, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E00, 0x80, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E0C, 0xFF, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E38, 0x1F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E2C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E34, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E1C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E14, 0x60, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E3C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E04, 0x0C, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E20, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0E08, 0x10, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+       {0x0E10, 0x52, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0030, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0000, 0x8E, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0038, 0xFE, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x002C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0034, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x001C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0014, 0x60, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x003C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0004, 0x0C, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0020, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0008, 0x10, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+       {0x0010, 0x52, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0430, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0400, 0x8E, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0438, 0xFE, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x042C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0434, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x041C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0414, 0x60, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x043C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0404, 0x0C, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0420, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0408, 0x10, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+       {0x0410, 0x52, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0830, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0800, 0x8E, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0838, 0xFE, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x082C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0834, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x081C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0814, 0x60, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x083C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0804, 0x0C, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0820, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0808, 0x10, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+       {0x0810, 0x52, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C30, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C00, 0x8E, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C38, 0xFE, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C2C, 0x01, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C34, 0x0F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C1C, 0x0A, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C14, 0x60, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C3C, 0xB8, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C04, 0x0C, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C20, 0x00, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C08, 0x10, 0x00, CSIPHY_SETTLE_CNT_LOWER_BYTE},
+       {0x0C10, 0x52, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0094, 0xD7, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x005C, 0x04, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0060, 0xBD, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0064, 0x7F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0494, 0xD7, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x045C, 0x04, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0460, 0xBD, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0464, 0x7F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0894, 0xD7, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x085C, 0x04, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0860, 0xBD, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0864, 0x7F, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C94, 0xD7, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C5C, 0x04, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C60, 0xBD, 0x00, CSIPHY_DEFAULT_PARAMS},
+       {0x0C64, 0x7F, 0x00, CSIPHY_DEFAULT_PARAMS},
+};
+
 static void csiphy_hw_version_read(struct csiphy_device *csiphy,
                                   struct device *dev)
 {
@@ -520,6 +625,7 @@ static bool csiphy_is_gen2(u32 version)
        case CAMSS_8250:
        case CAMSS_8280XP:
        case CAMSS_845:
+       case CAMSS_8550:
                ret = true;
                break;
        }
@@ -608,6 +714,11 @@ static int csiphy_init(struct csiphy_device *csiphy)
                regs->lane_regs = &lane_regs_sc8280xp[0];
                regs->lane_array_size = ARRAY_SIZE(lane_regs_sc8280xp);
                break;
+       case CAMSS_8550:
+               regs->lane_regs = &lane_regs_sm8550[0];
+               regs->lane_array_size = ARRAY_SIZE(lane_regs_sm8550);
+               regs->offset = 0x1000;
+               break;
        default:
                WARN(1, "unknown csiphy version\n");
                return -ENODEV;
diff --git a/drivers/media/platform/qcom/camss/camss.c 
b/drivers/media/platform/qcom/camss/camss.c
index cbcbcfe46419..0c40b8dbc786 100644
--- a/drivers/media/platform/qcom/camss/camss.c
+++ b/drivers/media/platform/qcom/camss/camss.c
@@ -1943,6 +1943,113 @@ static const struct resources_icc icc_res_sc8280xp[] = {
        },
 };
 
+static const struct camss_subdev_resources csiphy_res_8550[] = {
+       /* CSIPHY0 */
+       {
+               .regulators = { "vdda-phy", "vdda-pll" },
+               .clock = { "csiphy0", "csiphy0_timer" },
+               .clock_rate = { { 400000000, 480000000 },
+                               { 400000000 } },
+               .reg = { "csiphy0" },
+               .interrupt = { "csiphy0" },
+               .csiphy = {
+                       .hw_ops = &csiphy_ops_3ph_1_0,
+                       .formats = &csiphy_formats_sdm845
+               }
+       },
+       /* CSIPHY1 */
+       {
+               .regulators = { "vdda-phy", "vdda-pll" },
+               .clock = { "csiphy1", "csiphy1_timer" },
+               .clock_rate = { { 400000000, 480000000 },
+                               { 400000000 } },
+               .reg = { "csiphy1" },
+               .interrupt = { "csiphy1" },
+               .csiphy = {
+                       .hw_ops = &csiphy_ops_3ph_1_0,
+                       .formats = &csiphy_formats_sdm845
+               }
+       },
+       /* CSIPHY2 */
+       {
+               .regulators = { "vdda-phy", "vdda-pll" },
+               .clock = { "csiphy2", "csiphy2_timer" },
+               .clock_rate = { { 400000000, 480000000 },
+                               { 400000000 } },
+               .reg = { "csiphy2" },
+               .interrupt = { "csiphy2" },
+               .csiphy = {
+                       .hw_ops = &csiphy_ops_3ph_1_0,
+                       .formats = &csiphy_formats_sdm845
+               }
+       },
+       /* CSIPHY3 */
+       {
+               .regulators = { "vdda-phy", "vdda-pll" },
+               .clock = { "csiphy3", "csiphy3_timer" },
+               .clock_rate = { { 400000000, 480000000 },
+                               { 400000000 } },
+               .reg = { "csiphy3" },
+               .interrupt = { "csiphy3" },
+               .csiphy = {
+                       .hw_ops = &csiphy_ops_3ph_1_0,
+                       .formats = &csiphy_formats_sdm845
+               }
+       },
+       /* CSIPHY4 */
+       {
+               .regulators = { "vdda-phy", "vdda-pll" },
+               .clock = { "csiphy4", "csiphy4_timer" },
+               .clock_rate = { { 400000000, 480000000 },
+                               { 400000000 } },
+               .reg = { "csiphy4" },
+               .interrupt = { "csiphy4" },
+               .csiphy = {
+                       .hw_ops = &csiphy_ops_3ph_1_0,
+                       .formats = &csiphy_formats_sdm845
+               }
+       },
+       /* CSIPHY5 */
+       {
+               .regulators = { "vdda-phy", "vdda-pll" },
+               .clock = { "csiphy5", "csiphy5_timer" },
+               .clock_rate = { { 400000000, 480000000 },
+                               { 400000000 } },
+               .reg = { "csiphy5" },
+               .interrupt = { "csiphy5" },
+               .csiphy = {
+                       .hw_ops = &csiphy_ops_3ph_1_0,
+                       .formats = &csiphy_formats_sdm845
+               }
+       },
+       /* CSIPHY6 */
+       {
+               .regulators = { "vdda-phy", "vdda-pll" },
+               .clock = { "csiphy6", "csiphy6_timer" },
+               .clock_rate = { { 400000000, 480000000 },
+                               { 400000000 } },
+               .reg = { "csiphy6" },
+               .interrupt = { "csiphy6" },
+               .csiphy = {
+                       .hw_ops = &csiphy_ops_3ph_1_0,
+                       .formats = &csiphy_formats_sdm845
+               }
+       },
+       /* CSIPHY7 */
+       {
+               .regulators = { "vdda-phy", "vdda-pll" },
+               .clock = { "csiphy7", "csiphy7_timer" },
+               .clock_rate = { { 400000000, 480000000 },
+                               { 400000000 } },
+               .reg = { "csiphy7" },
+               .interrupt = { "csiphy7" },
+               .csiphy = {
+                       .hw_ops = &csiphy_ops_3ph_1_0,
+                       .formats = &csiphy_formats_sdm845
+               }
+       }
+};
+
 static const struct resources_icc icc_res_sm8550[] = {
        {
                .name = "ahb",
@@ -2995,8 +3102,10 @@ static const struct camss_resources sc7280_resources = {
 static const struct camss_resources sm8550_resources = {
        .version = CAMSS_8550,
        .pd_name = "top",
+       .csiphy_res = csiphy_res_8550,
        .icc_res = icc_res_sm8550,
        .icc_path_num = ARRAY_SIZE(icc_res_sm8550),
+       .csiphy_num = ARRAY_SIZE(csiphy_res_8550),
        .link_entities = camss_link_entities
 };
 

Reply via email to