mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/2] Series short description
@ 2009-01-02 16:18 Alan Cox
  2009-01-02 16:19 ` [PATCH 1/2] i8k: Enable i8k on Dell Precision Systems Alan Cox
  2009-01-02 16:19 ` [PATCH 2/2] i8k: Add Dell Vostro systems Alan Cox
  0 siblings, 2 replies; 4+ messages in thread
From: Alan Cox @ 2009-01-02 16:18 UTC (permalink / raw)
  To: torvalds, linux-kernel

Various i8k updates got lost but have now been chased up, signed off and
this adds them to the tree.
---

Andy Spencer (1):
      i8k: Enable i8k on Dell Precision Systems

Federico Heinz (1):
      i8k: Add Dell Vostro systems


 drivers/char/i8k.c |   16 +++++++++++++++-
 1 files changed, 15 insertions(+), 1 deletions(-)

-- 

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] i8k: Enable i8k on Dell Precision Systems
  2009-01-02 16:18 [PATCH 0/2] Series short description Alan Cox
@ 2009-01-02 16:19 ` Alan Cox
  2009-01-02 16:19 ` [PATCH 2/2] i8k: Add Dell Vostro systems Alan Cox
  1 sibling, 0 replies; 4+ messages in thread
From: Alan Cox @ 2009-01-02 16:19 UTC (permalink / raw)
  To: torvalds, linux-kernel

From: Andy Spencer <spenceal@rose-hulman.edu>

Patch to enable i8k on Dell Precisions.

Signed-off-by: Andy Spencer <spenceal@rose-hulman.edu>
Signed-off-by: Alan Cox <alan@redhat.com>
---

 drivers/char/i8k.c |    7 +++++++
 1 files changed, 7 insertions(+), 0 deletions(-)


diff --git a/drivers/char/i8k.c b/drivers/char/i8k.c
index b60d425..099fc89 100644
--- a/drivers/char/i8k.c
+++ b/drivers/char/i8k.c
@@ -485,6 +485,13 @@ static struct dmi_system_id __initdata i8k_dmi_table[] = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "MP061"),
 		},
 	},
+	{
+		.ident = "Dell Precision",
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+			DMI_MATCH(DMI_PRODUCT_NAME, "Precision"),
+		},
+	},
 	{ }
 };
 


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 2/2] i8k: Add Dell Vostro systems
  2009-01-02 16:18 [PATCH 0/2] Series short description Alan Cox
  2009-01-02 16:19 ` [PATCH 1/2] i8k: Enable i8k on Dell Precision Systems Alan Cox
@ 2009-01-02 16:19 ` Alan Cox
  2009-01-02 16:45   ` Sergei Trofimovich
  1 sibling, 1 reply; 4+ messages in thread
From: Alan Cox @ 2009-01-02 16:19 UTC (permalink / raw)
  To: torvalds, linux-kernel

From: Federico Heinz <fheinz@vialibre.org.ar>

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@vialibre.org.ar>
Signed-off-by: Alan Cox <alan@redhat.com>
---

 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
@@ -492,7 +492,14 @@ static struct dmi_system_id __initdata i8k_dmi_table[] = {
 			DMI_MATCH(DMI_PRODUCT_NAME, "Precision"),
 		},
 	},
-	{ }
+	{
+		.ident = "Dell Vostro",
+		.matches = {
+			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
+			DMI_MATCH(DMI_PRODUCT_NAME, "Vostro"),
+		},
+	},
+        { }
 };
 
 /*


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH 2/2] i8k: Add Dell Vostro systems
  2009-01-02 16:19 ` [PATCH 2/2] i8k: Add Dell Vostro systems Alan Cox
@ 2009-01-02 16:45   ` Sergei Trofimovich
  0 siblings, 0 replies; 4+ messages in thread
From: Sergei Trofimovich @ 2009-01-02 16:45 UTC (permalink / raw)
  To: Alan Cox, torvalds, linux-kernel

On Fri, 02 Jan 2009 16:19:23 +0000
Alan Cox <alan@lxorguk.ukuu.org.uk> wrote:

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

> From: Federico Heinz <fheinz@vialibre.org.ar>
> 
> 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@vialibre.org.ar>
> Signed-off-by: Alan Cox <alan@redhat.com>
> ---
> 
>  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

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2009-01-02 17:25 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-01-02 16:18 [PATCH 0/2] Series short description Alan Cox
2009-01-02 16:19 ` [PATCH 1/2] i8k: Enable i8k on Dell Precision Systems Alan Cox
2009-01-02 16:19 ` [PATCH 2/2] i8k: Add Dell Vostro systems Alan Cox
2009-01-02 16:45   ` Sergei Trofimovich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome