On 8/18/09, Marcel Holtmann <marcel@xxxxxxxxxxxx> wrote:
> Hi Mario,
...
>> +static void compal_rfkill_query(struct rfkill *rfkill, void *data)
>> +{
>> + unsigned long radio = (unsigned long) data;
>> + u8 result;
>> + bool blocked;
>> +
>> + ec_read(COMPAL_EC_COMMAND_WIRELESS, &result);
>> +
>> + if ((result & KILLSWITCH_MASK) == 0)
>> + blocked = 1;
>> + else if (radio == WLAN_MASK)
>> + blocked = !(result & WLAN_MASK);
>
> You are using spaces instead of tabs here.
Btw Mario, scripts/checkpatch.pl is a good way to check for whitespace
errors and a bunch of other trivia. So human patch review can
concentrate on the important stuff :-).
Thanks
Alan
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
