lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

leds: Fix &&/|| confusion in leds-pca9532.c

Browse Source 
This fixes the expression in the driver to do the correct thing,
not that I think anyone would send SND_* without EV_SND.

Thanks to Roel Kluin <roel.kluin@gmail.com> for noticing.

Signed-off-by: Riku Voipio <riku.voipio@iki.fi>
Signed-off-by: Richard Purdie <rpurdie@linux.intel.com>
This commit is contained in:
Riku Voipio
2009-03-03 22:13:06 +02:00
committed by Richard Purdie
parent b0edba7ef8
commit 7fbc3a9b13
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/leds/leds-pca9532.c
View File

@@ -169,7 +169,7 @@ static int pca9532_event(struct input_dev *dev, unsigned int type,
{ {
struct pca9532_data *data = input_get_drvdata(dev); struct pca9532_data *data = input_get_drvdata(dev);
if (type != EV_SND && (code != SND_BELL || code != SND_TONE)) if (!(type == EV_SND && (code == SND_BELL || code == SND_TONE)))
return -1; return -1;
/* XXX: allow different kind of beeps with psc/pwm modifications */ /* XXX: allow different kind of beeps with psc/pwm modifications */