Re: [PATCH 2/2] i8k: Add Dell Vostro systems

From: Sergei Trofimovich
Date: Fri Jan 02 2009 - 12:25:37 EST


On Fri, 02 Jan 2009 16:19:23 +0000
Alan Cox <alan@xxxxxxxxxxxxxxxxxxx> wrote:

Hi! I bet, it does not pass checkpatch (tabs to spaces clash)
Sorry, can't send fixed patch.

> From: Federico Heinz <fheinz@xxxxxxxxxxxxxxx>
>
> This trivial patch adds support for i8k on the new Dell Vostro models.
> I tested it on my Vostro 1400, and it works. It does print a warning
> when loading the module:
>
> i8k: unable to get SMM BIOS version
>
> But I couldn't figure out how to fix that. The module seems to work fine,
> anyway...
>
> Signed-off-by: Federico Heinz <fheinz@xxxxxxxxxxxxxxx>
> Signed-off-by: Alan Cox <alan@xxxxxxxxxx>
> ---
>
> drivers/char/i8k.c | 9 ++++++++-
> 1 files changed, 8 insertions(+), 1 deletions(-)
>
>
> diff --git a/drivers/char/i8k.c b/drivers/char/i8k.c
> index 099fc89..fc8cf7a 100644
> --- a/drivers/char/i8k.c
> +++ b/drivers/char/i8k.c

...
> - { }
^^^^^^
This one
vvvvvvv
> + { }
> };

--

Sergei
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/