media: dvb-frontends: drx39xyj: Remove unnecessary ternary operators
authorRuan Jinjie <ruanjinjie@huawei.com>
Fri, 4 Aug 2023 06:55:21 +0000 (14:55 +0800)
committerHans Verkuil <hverkuil-cisco@xs4all.nl>
Wed, 27 Sep 2023 07:39:55 +0000 (09:39 +0200)
There are a few ternary operators, the true or false judgement of which
is unnecessary in C language semantics.

Signed-off-by: Ruan Jinjie <ruanjinjie@huawei.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
drivers/media/dvb-frontends/drx39xyj/drxj.c

index a738573c8cd7a020a6b0e3d680594e7f275b5bd9..19d8de400a68759dc12610ee32c3d6f12020e397 100644 (file)
@@ -4779,8 +4779,8 @@ set_frequency(struct drx_demod_instance *demod,
        bool image_to_select;
        s32 fm_frequency_shift = 0;
 
-       rf_mirror = (ext_attr->mirror == DRX_MIRROR_YES) ? true : false;
-       tuner_mirror = demod->my_common_attr->mirror_freq_spect ? false : true;
+       rf_mirror = ext_attr->mirror == DRX_MIRROR_YES;
+       tuner_mirror = !demod->my_common_attr->mirror_freq_spect;
        /*
           Program frequency shifter
           No need to account for mirroring on RF
@@ -8765,7 +8765,7 @@ static int qam_flip_spec(struct drx_demod_instance *demod, struct drx_channel *c
                goto rw_error;
        }
        ext_attr->iqm_fs_rate_ofs = iqm_fs_rate_ofs;
-       ext_attr->pos_image = (ext_attr->pos_image) ? false : true;
+       ext_attr->pos_image = !ext_attr->pos_image;
 
        /* freeze dq/fq updating */
        rc = drxj_dap_read_reg16(dev_addr, QAM_DQ_MODE__A, &data, 0);