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

Subject: media: ivsc: csi: remove privacy status in struct mei_csi
Author:  Wentong Wu <wentong...@intel.com>
Date:    Fri Jun 7 21:25:47 2024 +0800

The privacy status is maintained by privacy_ctrl, on which all
of the privacy status changes will go through, so there is no
point in maintaining one more element any more.

Reported-by: Hao Yao <hao....@intel.com>
Signed-off-by: Wentong Wu <wentong...@intel.com>
Tested-by: Jason Chen <jason.z.c...@intel.com>
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/pci/intel/ivsc/mei_csi.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

---

diff --git a/drivers/media/pci/intel/ivsc/mei_csi.c 
b/drivers/media/pci/intel/ivsc/mei_csi.c
index 16791a7f4f15..2a9c12c975ca 100644
--- a/drivers/media/pci/intel/ivsc/mei_csi.c
+++ b/drivers/media/pci/intel/ivsc/mei_csi.c
@@ -138,9 +138,6 @@ struct mei_csi {
        u32 nr_of_lanes;
        /* frequency of the CSI-2 link */
        u64 link_freq;
-
-       /* privacy status */
-       enum ivsc_privacy_status status;
 };
 
 static const struct v4l2_mbus_framefmt mei_csi_format_mbus_default = {
@@ -271,10 +268,9 @@ static void mei_csi_rx(struct mei_cl_device *cldev)
 
        switch (notif.cmd_id) {
        case CSI_PRIVACY_NOTIF:
-               if (notif.cont.cont < CSI_PRIVACY_MAX) {
-                       csi->status = notif.cont.cont;
-                       v4l2_ctrl_s_ctrl(csi->privacy_ctrl, csi->status);
-               }
+               if (notif.cont.cont < CSI_PRIVACY_MAX)
+                       v4l2_ctrl_s_ctrl(csi->privacy_ctrl,
+                                        notif.cont.cont == CSI_PRIVACY_ON);
                break;
        case CSI_SET_OWNER:
        case CSI_SET_CONF:

Reply via email to