On Fri Mar 13 18:13:04 2026 +0800, Wenmeng Liu wrote:
> Add missing required clocks (cpas_ahb and camnoc_axi) for VFE lite
> instances on sa8775p platform. These clocks are necessary for proper
> VFE lite operation:
> 
> Reviewed-by: Bryan O'Donoghue <[email protected]>
> Signed-off-by: Wenmeng Liu <[email protected]>
> Fixes: e7b59e1d06fb ("media: qcom: camss: Add support for VFE 690")
> Cc: [email protected]
> Signed-off-by: Bryan O'Donoghue <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/qcom/camss/camss.c | 40 +++++++++++++++++++------------
 1 file changed, 25 insertions(+), 15 deletions(-)

---

diff --git a/drivers/media/platform/qcom/camss/camss.c 
b/drivers/media/platform/qcom/camss/camss.c
index cb0134718985..9335636d7c4d 100644
--- a/drivers/media/platform/qcom/camss/camss.c
+++ b/drivers/media/platform/qcom/camss/camss.c
@@ -3742,15 +3742,17 @@ static const struct camss_subdev_resources 
vfe_res_8775p[] = {
        /* VFE2 (lite) */
        {
                .regulators = {},
-               .clock = { "cpas_vfe_lite", "vfe_lite_ahb",
+               .clock = { "cpas_ahb", "cpas_vfe_lite", "vfe_lite_ahb",
                           "vfe_lite_csid", "vfe_lite_cphy_rx",
-                          "vfe_lite"},
+                          "vfe_lite", "camnoc_axi"},
                .clock_rate = {
-                       { 0, 0, 0, 0  },
+                       { 0 },
+                       { 0 },
                        { 300000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 480000000, 600000000, 600000000, 600000000 },
+                       { 400000000 },
                },
                .reg = { "vfe_lite0" },
                .interrupt = { "vfe_lite0" },
@@ -3765,15 +3767,17 @@ static const struct camss_subdev_resources 
vfe_res_8775p[] = {
        /* VFE3 (lite) */
        {
                .regulators = {},
-               .clock = { "cpas_vfe_lite", "vfe_lite_ahb",
+               .clock = { "cpas_ahb", "cpas_vfe_lite", "vfe_lite_ahb",
                           "vfe_lite_csid", "vfe_lite_cphy_rx",
-                          "vfe_lite"},
+                          "vfe_lite", "camnoc_axi"},
                .clock_rate = {
-                       { 0, 0, 0, 0  },
+                       { 0 },
+                       { 0 },
                        { 300000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 480000000, 600000000, 600000000, 600000000 },
+                       { 400000000 },
                },
                .reg = { "vfe_lite1" },
                .interrupt = { "vfe_lite1" },
@@ -3788,15 +3792,17 @@ static const struct camss_subdev_resources 
vfe_res_8775p[] = {
        /* VFE4 (lite) */
        {
                .regulators = {},
-               .clock = { "cpas_vfe_lite", "vfe_lite_ahb",
+               .clock = { "cpas_ahb", "cpas_vfe_lite", "vfe_lite_ahb",
                           "vfe_lite_csid", "vfe_lite_cphy_rx",
-                          "vfe_lite"},
+                          "vfe_lite", "camnoc_axi"},
                .clock_rate = {
-                       { 0, 0, 0, 0  },
+                       { 0 },
+                       { 0 },
                        { 300000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 480000000, 600000000, 600000000, 600000000 },
+                       { 400000000 },
                },
                .reg = { "vfe_lite2" },
                .interrupt = { "vfe_lite2" },
@@ -3811,15 +3817,17 @@ static const struct camss_subdev_resources 
vfe_res_8775p[] = {
        /* VFE5 (lite) */
        {
                .regulators = {},
-               .clock = { "cpas_vfe_lite", "vfe_lite_ahb",
+               .clock = { "cpas_ahb", "cpas_vfe_lite", "vfe_lite_ahb",
                           "vfe_lite_csid", "vfe_lite_cphy_rx",
-                          "vfe_lite"},
+                          "vfe_lite", "camnoc_axi"},
                .clock_rate = {
-                       { 0, 0, 0, 0  },
+                       { 0 },
+                       { 0 },
                        { 300000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 480000000, 600000000, 600000000, 600000000 },
+                       { 400000000 },
                },
                .reg = { "vfe_lite3" },
                .interrupt = { "vfe_lite3" },
@@ -3834,15 +3842,17 @@ static const struct camss_subdev_resources 
vfe_res_8775p[] = {
        /* VFE6 (lite) */
        {
                .regulators = {},
-               .clock = { "cpas_vfe_lite", "vfe_lite_ahb",
+               .clock = { "cpas_ahb", "cpas_vfe_lite", "vfe_lite_ahb",
                           "vfe_lite_csid", "vfe_lite_cphy_rx",
-                          "vfe_lite"},
+                          "vfe_lite", "camnoc_axi"},
                .clock_rate = {
-                       { 0, 0, 0, 0  },
+                       { 0 },
+                       { 0 },
                        { 300000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 400000000, 400000000, 400000000, 400000000 },
                        { 480000000, 600000000, 600000000, 600000000 },
+                       { 400000000 },
                },
                .reg = { "vfe_lite4" },
                .interrupt = { "vfe_lite4" },
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to