This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: ipu-bridge: Move initialization of node_names.vcm to 
ipu_bridge_init_swnode_names()
Author:  Hans de Goede <hdego...@redhat.com>
Date:    Wed Jul 5 23:29:55 2023 +0200

Move initialization of node_names.vcm to ipu_bridge_init_swnode_names()
where it belongs.

And make the initialization of nodes[SWNODE_VCM] unconditional,
ipu_bridge_init_swnode_group() takes care of not registering it
when there is no VCM.

Reviewed-by: Andy Shevchenko <a...@kernel.org>
Reviewed-by: Daniel Scally <dan.sca...@ideasonboard.com>
Signed-off-by: Hans de Goede <hdego...@redhat.com>
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mche...@kernel.org>

 drivers/media/pci/intel/ipu-bridge.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

---

diff --git a/drivers/media/pci/intel/ipu-bridge.c 
b/drivers/media/pci/intel/ipu-bridge.c
index ad1f78be3aa7..d1bc7035eeea 100644
--- a/drivers/media/pci/intel/ipu-bridge.c
+++ b/drivers/media/pci/intel/ipu-bridge.c
@@ -201,6 +201,12 @@ static void ipu_bridge_init_swnode_names(struct ipu_sensor 
*sensor)
        snprintf(sensor->node_names.endpoint,
                 sizeof(sensor->node_names.endpoint),
                 SWNODE_GRAPH_ENDPOINT_NAME_FMT, 0); /* And endpoint 0 */
+       if (sensor->ssdb.vcmtype) {
+               /* append ssdb.link to distinguish nodes with same model VCM */
+               snprintf(sensor->node_names.vcm, sizeof(sensor->node_names.vcm),
+                        "%s-%u", ipu_vcm_types[sensor->ssdb.vcmtype - 1],
+                        sensor->ssdb.link);
+       }
 }
 
 static void ipu_bridge_init_swnode_group(struct ipu_sensor *sensor)
@@ -237,13 +243,7 @@ static void ipu_bridge_create_connection_swnodes(struct 
ipu_bridge *bridge,
                                                sensor->node_names.endpoint,
                                                &nodes[SWNODE_IPU_PORT],
                                                sensor->ipu_properties);
-       if (sensor->ssdb.vcmtype) {
-               /* append ssdb.link to distinguish VCM nodes with same HID */
-               snprintf(sensor->node_names.vcm, sizeof(sensor->node_names.vcm),
-                        "%s-%u", ipu_vcm_types[sensor->ssdb.vcmtype - 1],
-                        sensor->ssdb.link);
-               nodes[SWNODE_VCM] = NODE_VCM(sensor->node_names.vcm);
-       }
+       nodes[SWNODE_VCM] = NODE_VCM(sensor->node_names.vcm);
 
        ipu_bridge_init_swnode_group(sensor);
 }

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to