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

Subject: media: dvb-core: dvb_frontend: address some clang warnings
Author:  Mauro Carvalho Chehab <[email protected]>
Date:    Mon Nov 22 08:59:53 2021 +0000

The typecasts at the dvb-core generate clang warnings when W=1
is enabled.

The warns are harmless, but they cause the build to break with
CONFIG_WERROR, so do the cast on a way that it won't produce
warnings anymore.

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

 drivers/media/dvb-core/dvb_frontend.c | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)

---

diff --git a/drivers/media/dvb-core/dvb_frontend.c 
b/drivers/media/dvb-core/dvb_frontend.c
index 8fc59f3cfb6c..48e735cdbe6b 100644
--- a/drivers/media/dvb-core/dvb_frontend.c
+++ b/drivers/media/dvb-core/dvb_frontend.c
@@ -2554,8 +2554,7 @@ static int dvb_frontend_handle_ioctl(struct file *file,
 
        case FE_DISEQC_SEND_BURST:
                if (fe->ops.diseqc_send_burst) {
-                       err = fe->ops.diseqc_send_burst(fe,
-                                               (enum fe_sec_mini_cmd)parg);
+                       err = fe->ops.diseqc_send_burst(fe, (long)parg);
                        fepriv->state = FESTATE_DISEQC;
                        fepriv->status = 0;
                }
@@ -2563,9 +2562,8 @@ static int dvb_frontend_handle_ioctl(struct file *file,
 
        case FE_SET_TONE:
                if (fe->ops.set_tone) {
-                       err = fe->ops.set_tone(fe,
-                                              (enum fe_sec_tone_mode)parg);
-                       fepriv->tone = (enum fe_sec_tone_mode)parg;
+                       fepriv->tone = (long)parg;
+                       err = fe->ops.set_tone(fe, fepriv->tone);
                        fepriv->state = FESTATE_DISEQC;
                        fepriv->status = 0;
                }
@@ -2573,9 +2571,8 @@ static int dvb_frontend_handle_ioctl(struct file *file,
 
        case FE_SET_VOLTAGE:
                if (fe->ops.set_voltage) {
-                       err = fe->ops.set_voltage(fe,
-                                                 (enum fe_sec_voltage)parg);
-                       fepriv->voltage = (enum fe_sec_voltage)parg;
+                       fepriv->voltage = (long)parg;
+                       err = fe->ops.set_voltage(fe, fepriv->voltage);
                        fepriv->state = FESTATE_DISEQC;
                        fepriv->status = 0;
                }

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

Reply via email to