diff options
| author | Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2020-09-02 08:37:12 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2020-09-03 11:16:13 +0200 |
| commit | 15a36aae1ec1c1f17149b6113b92631791830740 (patch) | |
| tree | ecda15386a77a783b44675e1f70d7b34f511fdfc /drivers/media/pci | |
| parent | 883c84aca1e34d80a9b7b337225b0ef2f34b4cd6 (diff) | |
| download | linux-15a36aae1ec1c1f17149b6113b92631791830740.tar.gz linux-15a36aae1ec1c1f17149b6113b92631791830740.tar.bz2 linux-15a36aae1ec1c1f17149b6113b92631791830740.zip | |
media: saa7134: avoid a shift overflow
As reported by smatch:
drivers/media/pci/saa7134//saa7134-tvaudio.c:686 saa_dsp_writel() warn: should 'reg << 2' be a 64 bit type?
On a 64-bits Kernel, the shift might be bigger than 32 bits.
In real, this should never happen, but let's shut up the warning.
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Diffstat (limited to 'drivers/media/pci')
| -rw-r--r-- | drivers/media/pci/saa7134/saa7134-tvaudio.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/media/pci/saa7134/saa7134-tvaudio.c b/drivers/media/pci/saa7134/saa7134-tvaudio.c index 79e1afb71075..5cc4ef21f9d3 100644 --- a/drivers/media/pci/saa7134/saa7134-tvaudio.c +++ b/drivers/media/pci/saa7134/saa7134-tvaudio.c @@ -683,7 +683,8 @@ int saa_dsp_writel(struct saa7134_dev *dev, int reg, u32 value) { int err; - audio_dbg(2, "dsp write reg 0x%x = 0x%06x\n", reg << 2, value); + audio_dbg(2, "dsp write reg 0x%x = 0x%06x\n", + (reg << 2) & 0xffffffff, value); err = saa_dsp_wait_bit(dev,SAA7135_DSP_RWSTATE_WRR); if (err < 0) return err; |