diff options
| author | Julia Lawall <julia@diku.dk> | 2008-03-04 18:00:13 -0500 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2008-03-18 02:31:34 -0400 |
| commit | abc5a87f4e627e058813a95b112e9f5f21668f9d (patch) | |
| tree | e0ecdbe2a99e7071fd0d47fdaab36d27e8a7321d /drivers | |
| parent | 93d74463d018ddf05c169ad399e62e90e0f82fc0 (diff) | |
asus_acpi: remove misleading mask
led_out is boolean, so there is no functional change here,
but apparently an extra mask with 1 caused some style checkers
to flag this as logic bug.
Signed-off-by: Julia Lawall <julia@diku.dk>
Acked-by: Luca Tettamanti <kronos.it@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/acpi/asus_acpi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/acpi/asus_acpi.c b/drivers/acpi/asus_acpi.c index d25ef961415c..44ad90c03c2e 100644 --- a/drivers/acpi/asus_acpi.c +++ b/drivers/acpi/asus_acpi.c | |||
| @@ -610,7 +610,7 @@ write_led(const char __user * buffer, unsigned long count, | |||
| 610 | (led_out) ? (hotk->status | ledmask) : (hotk->status & ~ledmask); | 610 | (led_out) ? (hotk->status | ledmask) : (hotk->status & ~ledmask); |
| 611 | 611 | ||
| 612 | if (invert) /* invert target value */ | 612 | if (invert) /* invert target value */ |
| 613 | led_out = !led_out & 0x1; | 613 | led_out = !led_out; |
| 614 | 614 | ||
| 615 | if (!write_acpi_int(hotk->handle, ledname, led_out, NULL)) | 615 | if (!write_acpi_int(hotk->handle, ledname, led_out, NULL)) |
| 616 | printk(KERN_WARNING "Asus ACPI: LED (%s) write failed\n", | 616 | printk(KERN_WARNING "Asus ACPI: LED (%s) write failed\n", |