OSDN Git Service

(root) / uclinux-h8 / linux.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: f440d90) | patch
staging: bcm2835-audio: use | instead of || otherwise result is just boolean 1
authorColin Ian King <colin.king@canonical.com>
Wed, 1 Feb 2017 11:57:11 +0000 (11:57 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 2 Feb 2017 12:05:06 +0000 (13:05 +0100)
commit7dd551e20ea2e27fc0579ad0be9e3c660e1147af
treef33cee256f9ae39738839484122424baf3e2a6cftree | snapshot
parentf440d909085e497540788fecc056dc39ea0471edcommit | diff
staging: bcm2835-audio: use | instead of || otherwise result is just boolean 1

The incorrect or operator is being used resulting in cookies being
set to 1 (true) rather than the intended hex value.  Fix this by
using | instead of ||.

Issue found with static analysis with cppcheck

Fixes: 0cec463e391e05b807a ("staging: bcm2835-audio: Simplify callback structure for write data")

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/bcm2835-audio/bcm2835-vchiq.c diff | blob | history
About OSDN
Find Software
Develop Software
Help