diff options
| author | Colin Ian King <colin.king@canonical.com> | 2017-10-31 11:19:42 +0100 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-12-08 16:11:22 +0100 |
| commit | 60a881862c69d211701d9bba81203a2f44d38367 (patch) | |
| tree | 34cec26131814ed60e49c29a5d9d7a699d966d83 /drivers/media/pci/pt3 | |
| parent | media: use ARRAY_SIZE (diff) | |
| download | linux-60a881862c69d211701d9bba81203a2f44d38367.tar.xz linux-60a881862c69d211701d9bba81203a2f44d38367.zip | |
media: pt3: remove redundant assignment to mask
Variable mask is being set to 0x80 and then set to this value again
in the following for-loop. Remove the extraneous first setting of mask.
Cleans up clang warning:
drivers/media/pci/pt3/pt3_i2c.c:88:2: warning: Value stored to 'mask'
is never read
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/media/pci/pt3')
| -rw-r--r-- | drivers/media/pci/pt3/pt3_i2c.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/media/pci/pt3/pt3_i2c.c b/drivers/media/pci/pt3/pt3_i2c.c index ec6a8a2e4744..b66138c7b364 100644 --- a/drivers/media/pci/pt3/pt3_i2c.c +++ b/drivers/media/pci/pt3/pt3_i2c.c @@ -85,7 +85,6 @@ static void put_byte_write(struct pt3_i2cbuf *cbuf, u8 val) { u8 mask; - mask = 0x80; for (mask = 0x80; mask > 0; mask >>= 1) cmdbuf_add(cbuf, (val & mask) ? I_DATA_H_NOP : I_DATA_L_NOP); cmdbuf_add(cbuf, I_DATA_H_ACK0); |