On Tue Aug 11 08:29:05 2026 +0300, Sakari Ailus wrote:
> Print information mainly from SSDB: port number, number of lanes, rotation
> and orientation as well as VCM model on debug level. Also print the fwnode
> path of the connected sensor.
> 
> Signed-off-by: Sakari Ailus <[email protected]>
> Reviewed-by: Hans de Goede <[email protected]>

Patch committed.

Thanks,
Sakari Ailus

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

---

diff --git a/drivers/media/pci/intel/ipu-bridge.c 
b/drivers/media/pci/intel/ipu-bridge.c
index 1bb3a3e98d6b..125fe5da7a60 100644
--- a/drivers/media/pci/intel/ipu-bridge.c
+++ b/drivers/media/pci/intel/ipu-bridge.c
@@ -412,6 +412,7 @@ static enum v4l2_fwnode_orientation 
ipu_bridge_parse_orientation(struct acpi_dev
 
 int ipu_bridge_parse_ssdb(struct acpi_device *adev, struct ipu_sensor *sensor)
 {
+       acpi_handle handle = acpi_device_handle(ACPI_PTR(adev));
        struct ipu_sensor_ssdb ssdb = {};
        int ret;
 
@@ -435,8 +436,15 @@ int ipu_bridge_parse_ssdb(struct acpi_device *adev, struct 
ipu_sensor *sensor)
        sensor->rotation = ipu_bridge_parse_rotation(adev, &ssdb);
        sensor->orientation = ipu_bridge_parse_orientation(adev);
 
-       if (ssdb.vcmtype)
+       acpi_handle_debug(handle,
+                         "CSI-2 port %u, lanes %u, mclkspeed %u, rotation %u 
(SSDB %u), orientation %u\n",
+                         sensor->link, sensor->lanes, sensor->mclkspeed,
+                         sensor->rotation, ssdb.degree, sensor->orientation);
+
+       if (ssdb.vcmtype) {
                sensor->vcm_type = ipu_vcm_types[ssdb.vcmtype - 1];
+               acpi_handle_debug(handle, "VCM %s\n", sensor->vcm_type);
+       }
 
        return 0;
 }
@@ -832,8 +840,8 @@ static int ipu_bridge_connect_sensor(const struct 
ipu_sensor_config *cfg,
                if (ret)
                        goto err_free_swnodes;
 
-               dev_info(bridge->dev, "Found supported sensor %s\n",
-                        acpi_dev_name(adev));
+               dev_info(bridge->dev, "Found supported sensor %s (%pfw)\n",
+                        acpi_dev_name(adev), primary);
 
                bridge->n_sensors++;
        }
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to