Patchwork [Precise,SRU] fixing dmi match for hp t5745 and hp st5747 thin client

login
register
mail settings
Submitter Tim Gardner
Date Aug. 23, 2012, 1:08 p.m.
Message ID <1345727314-8389-1-git-send-email-tim.gardner@canonical.com>
Download mbox | patch
Permalink /patch/179629/
State New
Headers show

Comments

Tim Gardner - Aug. 23, 2012, 1:08 p.m.
From: Marc Gariepy <mgariepy@ubuntu.com>

BugLink: http://bugs.launchpad.net/bugs/995998

Match the correct information which is DMI_PRODUCT_NAME instead of DMI_BOARD_NAME
See dmidecode information on launchpad for both thin client:

https://bugs.launchpad.net/ubuntu/+source/linux/+bug/911920
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/911916

Signed-off-by: Marc Gariepy <mgariepy@ubuntu.com>
Reviewed-by: Adam Jackson <ajax@redhat.com>
Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
(cherry picked from commit 62004978df3898649e152751eb6ac264a323ec36)

Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
---
 drivers/gpu/drm/i915/intel_lvds.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Leann Ogasawara - Aug. 23, 2012, 1:25 p.m.
On 08/23/2012 06:08 AM, Tim Gardner wrote:
> From: Marc Gariepy <mgariepy@ubuntu.com>
>
> BugLink: http://bugs.launchpad.net/bugs/995998
>
> Match the correct information which is DMI_PRODUCT_NAME instead of DMI_BOARD_NAME
> See dmidecode information on launchpad for both thin client:
>
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/911920
> https://bugs.launchpad.net/ubuntu/+source/linux/+bug/911916
>
> Signed-off-by: Marc Gariepy <mgariepy@ubuntu.com>
> Reviewed-by: Adam Jackson <ajax@redhat.com>
> Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
> (cherry picked from commit 62004978df3898649e152751eb6ac264a323ec36)
>
> Signed-off-by: Tim Gardner <tim.gardner@canonical.com>

Acked-by: Leann Ogasawara <leann.ogasawara@canonical.com>

> ---
>  drivers/gpu/drm/i915/intel_lvds.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c
> index 0820e54..80c2503 100644
> --- a/drivers/gpu/drm/i915/intel_lvds.c
> +++ b/drivers/gpu/drm/i915/intel_lvds.c
> @@ -768,7 +768,7 @@ static const struct dmi_system_id intel_no_lvds[] = {
>  		.ident = "Hewlett-Packard t5745",
>  		.matches = {
>  			DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
> -			DMI_MATCH(DMI_BOARD_NAME, "hp t5745"),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "hp t5745"),
>  		},
>  	},
>  	{
> @@ -776,7 +776,7 @@ static const struct dmi_system_id intel_no_lvds[] = {
>  		.ident = "Hewlett-Packard st5747",
>  		.matches = {
>  			DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
> -			DMI_MATCH(DMI_BOARD_NAME, "hp st5747"),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "hp st5747"),
>  		},
>  	},
>  	{
Tim Gardner - Aug. 23, 2012, 1:53 p.m.

Patch

diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c
index 0820e54..80c2503 100644
--- a/drivers/gpu/drm/i915/intel_lvds.c
+++ b/drivers/gpu/drm/i915/intel_lvds.c
@@ -768,7 +768,7 @@  static const struct dmi_system_id intel_no_lvds[] = {
 		.ident = "Hewlett-Packard t5745",
 		.matches = {
 			DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
-			DMI_MATCH(DMI_BOARD_NAME, "hp t5745"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "hp t5745"),
 		},
 	},
 	{
@@ -776,7 +776,7 @@  static const struct dmi_system_id intel_no_lvds[] = {
 		.ident = "Hewlett-Packard st5747",
 		.matches = {
 			DMI_MATCH(DMI_BOARD_VENDOR, "Hewlett-Packard"),
-			DMI_MATCH(DMI_BOARD_NAME, "hp st5747"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "hp st5747"),
 		},
 	},
 	{