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

Subject: media: s5p-mfc: Fix encoder menu controls initialization
Author:  Sylwester Nawrocki <[email protected]>
Date:    Tue Nov 21 11:43:44 2017 -0500

This patch fixes the menu_skip_mask field initialization and
addresses a following issue found by the SVACE static analysis:

* NO_EFFECT.SELF: assignment to self in expression 'cfg.menu_skip_mask = 
cfg.menu_skip_mask'
  No effect at drivers/media/platform/s5p-mfc/s5p_mfc_enc.c:2083

Signed-off-by: Sylwester Nawrocki <[email protected]>
Signed-off-by: Mauro Carvalho Chehab <[email protected]>

 drivers/media/platform/s5p-mfc/s5p_mfc_enc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c 
b/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c
index 2a5fd7c42cd5..0d5d465561be 100644
--- a/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c
+++ b/drivers/media/platform/s5p-mfc/s5p_mfc_enc.c
@@ -2080,7 +2080,7 @@ int s5p_mfc_enc_ctrls_setup(struct s5p_mfc_ctx *ctx)
 
                        if (cfg.type == V4L2_CTRL_TYPE_MENU) {
                                cfg.step = 0;
-                               cfg.menu_skip_mask = cfg.menu_skip_mask;
+                               cfg.menu_skip_mask = controls[i].menu_skip_mask;
                                cfg.qmenu = mfc51_get_menu(cfg.id);
                        } else {
                                cfg.step = controls[i].step;

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to