diff options
| author | Even Xu <even.xu@intel.com> | 2025-02-13 05:40:21 +0300 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.com> | 2025-02-19 00:09:49 +0300 |
| commit | d05c76fceb096b4423745856325d72c048d681e4 (patch) | |
| tree | 194c6b0f1d4b8eb04f001513b9cd4ba18f190908 | |
| parent | 03dcede2199a649c1190a135aaf6c70af4ed5a28 (diff) | |
| download | linux-d05c76fceb096b4423745856325d72c048d681e4.tar.xz | |
Hid: Intel-thc-hid: Intel-thc: Fix "dubious: !x | !y" issue
Change to use "||" to make it more readable and avoid miss
understanding.
Signed-off-by: Even Xu <even.xu@intel.com>
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202501292144.eFDq4ovr-lkp@intel.com
Signed-off-by: Jiri Kosina <jkosina@suse.com>
| -rw-r--r-- | drivers/hid/intel-thc-hid/intel-thc/intel-thc-dma.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hid/intel-thc-hid/intel-thc/intel-thc-dma.c b/drivers/hid/intel-thc-hid/intel-thc/intel-thc-dma.c index eb23bea77686..8f97e71df7f4 100644 --- a/drivers/hid/intel-thc-hid/intel-thc/intel-thc-dma.c +++ b/drivers/hid/intel-thc-hid/intel-thc/intel-thc-dma.c @@ -295,7 +295,7 @@ static void release_dma_buffers(struct thc_device *dev, return; for (i = 0; i < config->prd_tbl_num; i++) { - if (!config->sgls[i] | !config->sgls_nent[i]) + if (!config->sgls[i] || !config->sgls_nent[i]) continue; dma_unmap_sg(dev->dev, config->sgls[i], |