diff mbox series

drm: panel-orientation-quirks: Make Lenovo Yoga Tab 3 X90F DMI match less strict

Message ID 20240825132131.6643-1-hdegoede@redhat.com (mailing list archive)
State New, archived
Headers show
Series drm: panel-orientation-quirks: Make Lenovo Yoga Tab 3 X90F DMI match less strict | expand

Commit Message

Hans de Goede Aug. 25, 2024, 1:21 p.m. UTC
There are 2G and 4G RAM versions of the Lenovo Yoga Tab 3 X90F and it
turns out that the 2G version has a DMI product name of
"CHERRYVIEW D1 PLATFORM" where as the 4G version has
"CHERRYVIEW C0 PLATFORM". The sys-vendor + product-version check are
unique enough that the product-name check is not necessary.

Drop the product-name check so that the existing DMI match for the 4G
RAM version also matches the 2G RAM version.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 drivers/gpu/drm/drm_panel_orientation_quirks.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Jani Nikula Aug. 29, 2024, 9:54 a.m. UTC | #1
On Sun, 25 Aug 2024, Hans de Goede <hdegoede@redhat.com> wrote:
> There are 2G and 4G RAM versions of the Lenovo Yoga Tab 3 X90F and it
> turns out that the 2G version has a DMI product name of
> "CHERRYVIEW D1 PLATFORM" where as the 4G version has
> "CHERRYVIEW C0 PLATFORM". The sys-vendor + product-version check are
> unique enough that the product-name check is not necessary.
>
> Drop the product-name check so that the existing DMI match for the 4G
> RAM version also matches the 2G RAM version.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
>  drivers/gpu/drm/drm_panel_orientation_quirks.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c b/drivers/gpu/drm/drm_panel_orientation_quirks.c
> index c16c7678237e..c48471c09e2f 100644
> --- a/drivers/gpu/drm/drm_panel_orientation_quirks.c
> +++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c
> @@ -391,7 +391,6 @@ static const struct dmi_system_id orientation_data[] = {
>  	}, {	/* Lenovo Yoga Tab 3 X90F */
>  		.matches = {
>  		 DMI_MATCH(DMI_SYS_VENDOR, "Intel Corporation"),
> -		 DMI_MATCH(DMI_PRODUCT_NAME, "CHERRYVIEW D1 PLATFORM"),

DMI_MATCH uses strstr() internally (as opposed to DMI_EXACT_MATCH which
is a full strcmp()) so you could still have 

		DMI_MATCH(DMI_PRODUCT_NAME, "CHERRYVIEW"),

Up to you, and either way,

Acked-by: Jani Nikula <jani.nikula@intel.com>


>  		 DMI_MATCH(DMI_PRODUCT_VERSION, "Blade3-10A-001"),
>  		},
>  		.driver_data = (void *)&lcd1600x2560_rightside_up,
diff mbox series

Patch

diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c b/drivers/gpu/drm/drm_panel_orientation_quirks.c
index c16c7678237e..c48471c09e2f 100644
--- a/drivers/gpu/drm/drm_panel_orientation_quirks.c
+++ b/drivers/gpu/drm/drm_panel_orientation_quirks.c
@@ -391,7 +391,6 @@  static const struct dmi_system_id orientation_data[] = {
 	}, {	/* Lenovo Yoga Tab 3 X90F */
 		.matches = {
 		 DMI_MATCH(DMI_SYS_VENDOR, "Intel Corporation"),
-		 DMI_MATCH(DMI_PRODUCT_NAME, "CHERRYVIEW D1 PLATFORM"),
 		 DMI_MATCH(DMI_PRODUCT_VERSION, "Blade3-10A-001"),
 		},
 		.driver_data = (void *)&lcd1600x2560_rightside_up,