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

Subject: media: ccs: Apply module manufacturer hack on non-CCS devices only
Author:  Sakari Ailus <sakari.ai...@linux.intel.com>
Date:    Thu Apr 6 19:41:03 2023 +0300

Some modules don't have any model identification information in the
register address space. The driver as a SMIA++ driver attempted to use
sensor information in this case. This workaround is definitely not for CCS
devices.

Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/i2c/ccs/ccs-core.c | 25 ++++++++++++-------------
 1 file changed, 12 insertions(+), 13 deletions(-)

---

diff --git a/drivers/media/i2c/ccs/ccs-core.c b/drivers/media/i2c/ccs/ccs-core.c
index 4adb2da34066..20c3974f5ac7 100644
--- a/drivers/media/i2c/ccs/ccs-core.c
+++ b/drivers/media/i2c/ccs/ccs-core.c
@@ -2894,19 +2894,18 @@ static int ccs_identify_module(struct ccs_sensor 
*sensor)
                        "smia version %2.2d smiapp version %2.2d\n",
                        minfo->smia_version, minfo->smiapp_version);
                minfo->name = SMIAPP_NAME;
-       }
-
-       /*
-        * Some modules have bad data in the lvalues below. Hope the
-        * rvalues have better stuff. The lvalues are module
-        * parameters whereas the rvalues are sensor parameters.
-        */
-       if (minfo->sensor_smia_manufacturer_id &&
-           !minfo->smia_manufacturer_id && !minfo->model_id) {
-               minfo->smia_manufacturer_id =
-                       minfo->sensor_smia_manufacturer_id;
-               minfo->model_id = minfo->sensor_model_id;
-               minfo->revision_number = minfo->sensor_revision_number;
+               /*
+                * Some modules have bad data in the lvalues below. Hope the
+                * rvalues have better stuff. The lvalues are module
+                * parameters whereas the rvalues are sensor parameters.
+                */
+               if (minfo->sensor_smia_manufacturer_id &&
+                   !minfo->smia_manufacturer_id && !minfo->model_id) {
+                       minfo->smia_manufacturer_id =
+                               minfo->sensor_smia_manufacturer_id;
+                       minfo->model_id = minfo->sensor_model_id;
+                       minfo->revision_number = minfo->sensor_revision_number;
+               }
        }
 
        for (i = 0; i < ARRAY_SIZE(ccs_module_idents); i++) {

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

Reply via email to