All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] fixing dmi match for hp t5745 and hp st5747 thin client
@ 2012-05-01 17:37 Marc Gariepy
  2012-05-02 17:44 ` Adam Jackson
  0 siblings, 1 reply; 3+ messages in thread
From: Marc Gariepy @ 2012-05-01 17:37 UTC (permalink / raw)
  Cc: dri-devel, Marc Gariepy

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

Thanks

Signed-off-by: Marc Gariepy <mgariepy@ubuntu.com>
---
 drivers/gpu/drm/i915/intel_lvds.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/i915/intel_lvds.c b/drivers/gpu/drm/i915/intel_lvds.c
index 30e2c82..9c71183 100644
--- a/drivers/gpu/drm/i915/intel_lvds.c
+++ b/drivers/gpu/drm/i915/intel_lvds.c
@@ -750,7 +750,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"),
 		},
 	},
 	{
@@ -758,7 +758,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"),
 		},
 	},
 	{
-- 
1.7.5.4

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

* Re: [PATCH] fixing dmi match for hp t5745 and hp st5747 thin client
  2012-05-01 17:37 [PATCH] fixing dmi match for hp t5745 and hp st5747 thin client Marc Gariepy
@ 2012-05-02 17:44 ` Adam Jackson
  2012-05-02 19:42   ` Daniel Vetter
  0 siblings, 1 reply; 3+ messages in thread
From: Adam Jackson @ 2012-05-02 17:44 UTC (permalink / raw)
  To: Marc Gariepy; +Cc: dri-devel

On 5/1/12 1:37 PM, Marc Gariepy wrote:
> 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

Reviewed-by: Adam Jackson <ajax@redhat.com>

- ajax

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

* Re: [PATCH] fixing dmi match for hp t5745 and hp st5747 thin client
  2012-05-02 17:44 ` Adam Jackson
@ 2012-05-02 19:42   ` Daniel Vetter
  0 siblings, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2012-05-02 19:42 UTC (permalink / raw)
  To: Adam Jackson; +Cc: Marc Gariepy, dri-devel

On Wed, May 02, 2012 at 01:44:15PM -0400, Adam Jackson wrote:
> On 5/1/12 1:37 PM, Marc Gariepy wrote:
> >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
> 
> Reviewed-by: Adam Jackson <ajax@redhat.com>
Picked up for -fixes, thanks for the patch. It's a bit late in the -rc
series, but I've figure we better fix up the broken quirk entry.
-Daniel
-- 
Daniel Vetter
Mail: daniel@ffwll.ch
Mobile: +41 (0)79 365 57 48

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

end of thread, other threads:[~2012-05-02 19:41 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-01 17:37 [PATCH] fixing dmi match for hp t5745 and hp st5747 thin client Marc Gariepy
2012-05-02 17:44 ` Adam Jackson
2012-05-02 19:42   ` Daniel Vetter

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.