* [PATCH 1/2] ACPI: video: Drop dmi_system_id.ident settings from video_detect_dmi_table[]
@ 2021-11-03 14:26 Hans de Goede
2021-11-03 14:26 ` [PATCH 2/2] ACPI: video: use platform backlight driver on Xiaomi Mi Pad 2 Hans de Goede
2021-11-03 18:33 ` [PATCH 1/2] ACPI: video: Drop dmi_system_id.ident settings from video_detect_dmi_table[] Rafael J. Wysocki
0 siblings, 2 replies; 3+ messages in thread
From: Hans de Goede @ 2021-11-03 14:26 UTC (permalink / raw)
To: Rafael J . Wysocki; +Cc: Hans de Goede, Len Brown, Tsuchiya Yuto, linux-acpi
The .ident field of the dmi_system_id structs in the
video_detect_dmi_table[] is not used by the code.
Change all .ident = "..." assignments to comments, this reduces the
size of video_detect.o / video.ko by about 1500 bytes.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/acpi/video_detect.c | 70 ++++++++++++++++++-------------------
1 file changed, 35 insertions(+), 35 deletions(-)
diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c
index 33474fd96991..7a1f8e823208 100644
--- a/drivers/acpi/video_detect.c
+++ b/drivers/acpi/video_detect.c
@@ -115,7 +115,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
*/
{
.callback = video_detect_force_vendor,
- .ident = "X360",
+ /* X360 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
DMI_MATCH(DMI_PRODUCT_NAME, "X360"),
@@ -124,7 +124,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_vendor,
- .ident = "Asus UL30VT",
+ /* Asus UL30VT */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK Computer Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "UL30VT"),
@@ -132,7 +132,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_vendor,
- .ident = "Asus UL30A",
+ /* Asus UL30A */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK Computer Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "UL30A"),
@@ -140,7 +140,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_vendor,
- .ident = "GIGABYTE GB-BXBT-2807",
+ /* GIGABYTE GB-BXBT-2807 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "GIGABYTE"),
DMI_MATCH(DMI_PRODUCT_NAME, "GB-BXBT-2807"),
@@ -148,7 +148,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_vendor,
- .ident = "Sony VPCEH3U1E",
+ /* Sony VPCEH3U1E */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
DMI_MATCH(DMI_PRODUCT_NAME, "VPCEH3U1E"),
@@ -164,7 +164,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
*/
{
.callback = video_detect_force_video,
- .ident = "ThinkPad T420",
+ /* ThinkPad T420 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T420"),
@@ -172,7 +172,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_video,
- .ident = "ThinkPad T520",
+ /* ThinkPad T520 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T520"),
@@ -180,7 +180,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_video,
- .ident = "ThinkPad X201s",
+ /* ThinkPad X201s */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X201s"),
@@ -188,7 +188,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_video,
- .ident = "ThinkPad X201T",
+ /* ThinkPad X201T */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X201T"),
@@ -199,7 +199,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugs.freedesktop.org/show_bug.cgi?id=81515 */
.callback = video_detect_force_video,
- .ident = "HP ENVY 15 Notebook",
+ /* HP ENVY 15 Notebook */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
DMI_MATCH(DMI_PRODUCT_NAME, "HP ENVY 15 Notebook PC"),
@@ -207,7 +207,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_video,
- .ident = "SAMSUNG 870Z5E/880Z5E/680Z5E",
+ /* SAMSUNG 870Z5E/880Z5E/680Z5E */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
DMI_MATCH(DMI_PRODUCT_NAME, "870Z5E/880Z5E/680Z5E"),
@@ -215,7 +215,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_video,
- .ident = "SAMSUNG 370R4E/370R4V/370R5E/3570RE/370R5V",
+ /* SAMSUNG 370R4E/370R4V/370R5E/3570RE/370R5V */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
DMI_MATCH(DMI_PRODUCT_NAME,
@@ -225,7 +225,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1186097 */
.callback = video_detect_force_video,
- .ident = "SAMSUNG 3570R/370R/470R/450R/510R/4450RV",
+ /* SAMSUNG 3570R/370R/470R/450R/510R/4450RV */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
DMI_MATCH(DMI_PRODUCT_NAME,
@@ -235,7 +235,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1557060 */
.callback = video_detect_force_video,
- .ident = "SAMSUNG 670Z5E",
+ /* SAMSUNG 670Z5E */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
DMI_MATCH(DMI_PRODUCT_NAME, "670Z5E"),
@@ -244,7 +244,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1094948 */
.callback = video_detect_force_video,
- .ident = "SAMSUNG 730U3E/740U3E",
+ /* SAMSUNG 730U3E/740U3E */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
DMI_MATCH(DMI_PRODUCT_NAME, "730U3E/740U3E"),
@@ -253,7 +253,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugs.freedesktop.org/show_bug.cgi?id=87286 */
.callback = video_detect_force_video,
- .ident = "SAMSUNG 900X3C/900X3D/900X3E/900X4C/900X4D",
+ /* SAMSUNG 900X3C/900X3D/900X3E/900X4C/900X4D */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
DMI_MATCH(DMI_PRODUCT_NAME,
@@ -263,7 +263,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1272633 */
.callback = video_detect_force_video,
- .ident = "Dell XPS14 L421X",
+ /* Dell XPS14 L421X */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "XPS L421X"),
@@ -272,7 +272,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1163574 */
.callback = video_detect_force_video,
- .ident = "Dell XPS15 L521X",
+ /* Dell XPS15 L521X */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "XPS L521X"),
@@ -281,7 +281,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.kernel.org/show_bug.cgi?id=108971 */
.callback = video_detect_force_video,
- .ident = "SAMSUNG 530U4E/540U4E",
+ /* SAMSUNG 530U4E/540U4E */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
DMI_MATCH(DMI_PRODUCT_NAME, "530U4E/540U4E"),
@@ -290,7 +290,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
/* https://bugs.launchpad.net/bugs/1894667 */
{
.callback = video_detect_force_video,
- .ident = "HP 635 Notebook",
+ /* HP 635 Notebook */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
DMI_MATCH(DMI_PRODUCT_NAME, "HP 635 Notebook PC"),
@@ -301,7 +301,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1201530 */
.callback = video_detect_force_native,
- .ident = "Lenovo Ideapad S405",
+ /* Lenovo Ideapad S405 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_BOARD_NAME, "Lenovo IdeaPad S405"),
@@ -310,7 +310,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1187004 */
.callback = video_detect_force_native,
- .ident = "Lenovo Ideapad Z570",
+ /* Lenovo Ideapad Z570 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_PRODUCT_NAME, "102434U"),
@@ -318,7 +318,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_native,
- .ident = "Lenovo E41-25",
+ /* Lenovo E41-25 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_PRODUCT_NAME, "81FS"),
@@ -326,7 +326,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_native,
- .ident = "Lenovo E41-45",
+ /* Lenovo E41-45 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_PRODUCT_NAME, "82BK"),
@@ -335,7 +335,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1217249 */
.callback = video_detect_force_native,
- .ident = "Apple MacBook Pro 12,1",
+ /* Apple MacBook Pro 12,1 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Apple Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro12,1"),
@@ -343,7 +343,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_native,
- .ident = "Dell Vostro V131",
+ /* Dell Vostro V131 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "Vostro V131"),
@@ -352,7 +352,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.redhat.com/show_bug.cgi?id=1123661 */
.callback = video_detect_force_native,
- .ident = "Dell XPS 17 L702X",
+ /* Dell XPS 17 L702X */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "Dell System XPS L702X"),
@@ -360,7 +360,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_native,
- .ident = "Dell Precision 7510",
+ /* Dell Precision 7510 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "Precision 7510"),
@@ -368,7 +368,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_native,
- .ident = "Acer Aspire 5738z",
+ /* Acer Aspire 5738z */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5738"),
@@ -378,7 +378,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
{
/* https://bugzilla.kernel.org/show_bug.cgi?id=207835 */
.callback = video_detect_force_native,
- .ident = "Acer TravelMate 5735Z",
+ /* Acer TravelMate 5735Z */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
DMI_MATCH(DMI_PRODUCT_NAME, "TravelMate 5735Z"),
@@ -387,7 +387,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_native,
- .ident = "ASUSTeK COMPUTER INC. GA401",
+ /* ASUSTeK COMPUTER INC. GA401 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
DMI_MATCH(DMI_PRODUCT_NAME, "GA401"),
@@ -395,7 +395,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_native,
- .ident = "ASUSTeK COMPUTER INC. GA502",
+ /* ASUSTeK COMPUTER INC. GA502 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
DMI_MATCH(DMI_PRODUCT_NAME, "GA502"),
@@ -403,7 +403,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_native,
- .ident = "ASUSTeK COMPUTER INC. GA503",
+ /* ASUSTeK COMPUTER INC. GA503 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
DMI_MATCH(DMI_PRODUCT_NAME, "GA503"),
@@ -416,7 +416,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
*/
{
.callback = video_detect_force_none,
- .ident = "Dell OptiPlex 9020M",
+ /* Dell OptiPlex 9020M */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
DMI_MATCH(DMI_PRODUCT_NAME, "OptiPlex 9020M"),
@@ -424,7 +424,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
},
{
.callback = video_detect_force_none,
- .ident = "MSI MS-7721",
+ /* MSI MS-7721 */
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "MSI"),
DMI_MATCH(DMI_PRODUCT_NAME, "MS-7721"),
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 2/2] ACPI: video: use platform backlight driver on Xiaomi Mi Pad 2
2021-11-03 14:26 [PATCH 1/2] ACPI: video: Drop dmi_system_id.ident settings from video_detect_dmi_table[] Hans de Goede
@ 2021-11-03 14:26 ` Hans de Goede
2021-11-03 18:33 ` [PATCH 1/2] ACPI: video: Drop dmi_system_id.ident settings from video_detect_dmi_table[] Rafael J. Wysocki
1 sibling, 0 replies; 3+ messages in thread
From: Hans de Goede @ 2021-11-03 14:26 UTC (permalink / raw)
To: Rafael J . Wysocki; +Cc: Hans de Goede, Len Brown, Tsuchiya Yuto, linux-acpi
The Xiaomi Mi Pad 2 is a Cherry Trail based x86 tablet which does not
use the i915's driver backlight control support:
i915 0000:00:02.0: [drm:intel_panel_setup_backlight [i915]] no backlight present per VBT
Like all Cherry Trail devices the ACPI tables on the Xiaomi Mi Pad 2
contain a broken ACPI-video implementation which causes 6 different
acpi_video backlights to get registered when used.
The lack of the i915 driver registering a BACKLIGHT_RAW (aka native) type
backlight causes acpi_video_get_backlight_type() to pick the broken
acpi_video backlight code as the backlight driver to use.
There actually is a separate lp8556 backlight controller connected
over I2C which gets registered as a BACKLIGHT_PLATFORM (aka vendor).
Add a quirk to force acpi_video_get_backlight_type() to return
acpi_backlight_vendor, so that the broken acpi_video backlight
interfaces do not get registered.
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
drivers/acpi/video_detect.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c
index 7a1f8e823208..068e393ea0c6 100644
--- a/drivers/acpi/video_detect.c
+++ b/drivers/acpi/video_detect.c
@@ -154,6 +154,14 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
DMI_MATCH(DMI_PRODUCT_NAME, "VPCEH3U1E"),
},
},
+ {
+ .callback = video_detect_force_vendor,
+ /* Xiaomi Mi Pad 2 */
+ .matches = {
+ DMI_MATCH(DMI_SYS_VENDOR, "Xiaomi Inc"),
+ DMI_MATCH(DMI_PRODUCT_NAME, "Mipad2"),
+ },
+ },
/*
* These models have a working acpi_video backlight control, and using
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/2] ACPI: video: Drop dmi_system_id.ident settings from video_detect_dmi_table[]
2021-11-03 14:26 [PATCH 1/2] ACPI: video: Drop dmi_system_id.ident settings from video_detect_dmi_table[] Hans de Goede
2021-11-03 14:26 ` [PATCH 2/2] ACPI: video: use platform backlight driver on Xiaomi Mi Pad 2 Hans de Goede
@ 2021-11-03 18:33 ` Rafael J. Wysocki
1 sibling, 0 replies; 3+ messages in thread
From: Rafael J. Wysocki @ 2021-11-03 18:33 UTC (permalink / raw)
To: Hans de Goede
Cc: Rafael J . Wysocki, Len Brown, Tsuchiya Yuto, ACPI Devel Maling List
On Wed, Nov 3, 2021 at 3:26 PM Hans de Goede <hdegoede@redhat.com> wrote:
>
> The .ident field of the dmi_system_id structs in the
> video_detect_dmi_table[] is not used by the code.
>
> Change all .ident = "..." assignments to comments, this reduces the
> size of video_detect.o / video.ko by about 1500 bytes.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
> drivers/acpi/video_detect.c | 70 ++++++++++++++++++-------------------
> 1 file changed, 35 insertions(+), 35 deletions(-)
>
> diff --git a/drivers/acpi/video_detect.c b/drivers/acpi/video_detect.c
> index 33474fd96991..7a1f8e823208 100644
> --- a/drivers/acpi/video_detect.c
> +++ b/drivers/acpi/video_detect.c
> @@ -115,7 +115,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> */
> {
> .callback = video_detect_force_vendor,
> - .ident = "X360",
> + /* X360 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> DMI_MATCH(DMI_PRODUCT_NAME, "X360"),
> @@ -124,7 +124,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_vendor,
> - .ident = "Asus UL30VT",
> + /* Asus UL30VT */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK Computer Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "UL30VT"),
> @@ -132,7 +132,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_vendor,
> - .ident = "Asus UL30A",
> + /* Asus UL30A */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK Computer Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "UL30A"),
> @@ -140,7 +140,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_vendor,
> - .ident = "GIGABYTE GB-BXBT-2807",
> + /* GIGABYTE GB-BXBT-2807 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "GIGABYTE"),
> DMI_MATCH(DMI_PRODUCT_NAME, "GB-BXBT-2807"),
> @@ -148,7 +148,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_vendor,
> - .ident = "Sony VPCEH3U1E",
> + /* Sony VPCEH3U1E */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Sony Corporation"),
> DMI_MATCH(DMI_PRODUCT_NAME, "VPCEH3U1E"),
> @@ -164,7 +164,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> */
> {
> .callback = video_detect_force_video,
> - .ident = "ThinkPad T420",
> + /* ThinkPad T420 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T420"),
> @@ -172,7 +172,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_video,
> - .ident = "ThinkPad T520",
> + /* ThinkPad T520 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad T520"),
> @@ -180,7 +180,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_video,
> - .ident = "ThinkPad X201s",
> + /* ThinkPad X201s */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X201s"),
> @@ -188,7 +188,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_video,
> - .ident = "ThinkPad X201T",
> + /* ThinkPad X201T */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> DMI_MATCH(DMI_PRODUCT_VERSION, "ThinkPad X201T"),
> @@ -199,7 +199,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugs.freedesktop.org/show_bug.cgi?id=81515 */
> .callback = video_detect_force_video,
> - .ident = "HP ENVY 15 Notebook",
> + /* HP ENVY 15 Notebook */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
> DMI_MATCH(DMI_PRODUCT_NAME, "HP ENVY 15 Notebook PC"),
> @@ -207,7 +207,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_video,
> - .ident = "SAMSUNG 870Z5E/880Z5E/680Z5E",
> + /* SAMSUNG 870Z5E/880Z5E/680Z5E */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> DMI_MATCH(DMI_PRODUCT_NAME, "870Z5E/880Z5E/680Z5E"),
> @@ -215,7 +215,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_video,
> - .ident = "SAMSUNG 370R4E/370R4V/370R5E/3570RE/370R5V",
> + /* SAMSUNG 370R4E/370R4V/370R5E/3570RE/370R5V */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> DMI_MATCH(DMI_PRODUCT_NAME,
> @@ -225,7 +225,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1186097 */
> .callback = video_detect_force_video,
> - .ident = "SAMSUNG 3570R/370R/470R/450R/510R/4450RV",
> + /* SAMSUNG 3570R/370R/470R/450R/510R/4450RV */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> DMI_MATCH(DMI_PRODUCT_NAME,
> @@ -235,7 +235,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1557060 */
> .callback = video_detect_force_video,
> - .ident = "SAMSUNG 670Z5E",
> + /* SAMSUNG 670Z5E */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> DMI_MATCH(DMI_PRODUCT_NAME, "670Z5E"),
> @@ -244,7 +244,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1094948 */
> .callback = video_detect_force_video,
> - .ident = "SAMSUNG 730U3E/740U3E",
> + /* SAMSUNG 730U3E/740U3E */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> DMI_MATCH(DMI_PRODUCT_NAME, "730U3E/740U3E"),
> @@ -253,7 +253,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugs.freedesktop.org/show_bug.cgi?id=87286 */
> .callback = video_detect_force_video,
> - .ident = "SAMSUNG 900X3C/900X3D/900X3E/900X4C/900X4D",
> + /* SAMSUNG 900X3C/900X3D/900X3E/900X4C/900X4D */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> DMI_MATCH(DMI_PRODUCT_NAME,
> @@ -263,7 +263,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1272633 */
> .callback = video_detect_force_video,
> - .ident = "Dell XPS14 L421X",
> + /* Dell XPS14 L421X */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "XPS L421X"),
> @@ -272,7 +272,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1163574 */
> .callback = video_detect_force_video,
> - .ident = "Dell XPS15 L521X",
> + /* Dell XPS15 L521X */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "XPS L521X"),
> @@ -281,7 +281,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.kernel.org/show_bug.cgi?id=108971 */
> .callback = video_detect_force_video,
> - .ident = "SAMSUNG 530U4E/540U4E",
> + /* SAMSUNG 530U4E/540U4E */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "SAMSUNG ELECTRONICS CO., LTD."),
> DMI_MATCH(DMI_PRODUCT_NAME, "530U4E/540U4E"),
> @@ -290,7 +290,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> /* https://bugs.launchpad.net/bugs/1894667 */
> {
> .callback = video_detect_force_video,
> - .ident = "HP 635 Notebook",
> + /* HP 635 Notebook */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
> DMI_MATCH(DMI_PRODUCT_NAME, "HP 635 Notebook PC"),
> @@ -301,7 +301,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1201530 */
> .callback = video_detect_force_native,
> - .ident = "Lenovo Ideapad S405",
> + /* Lenovo Ideapad S405 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> DMI_MATCH(DMI_BOARD_NAME, "Lenovo IdeaPad S405"),
> @@ -310,7 +310,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1187004 */
> .callback = video_detect_force_native,
> - .ident = "Lenovo Ideapad Z570",
> + /* Lenovo Ideapad Z570 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> DMI_MATCH(DMI_PRODUCT_NAME, "102434U"),
> @@ -318,7 +318,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_native,
> - .ident = "Lenovo E41-25",
> + /* Lenovo E41-25 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> DMI_MATCH(DMI_PRODUCT_NAME, "81FS"),
> @@ -326,7 +326,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_native,
> - .ident = "Lenovo E41-45",
> + /* Lenovo E41-45 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
> DMI_MATCH(DMI_PRODUCT_NAME, "82BK"),
> @@ -335,7 +335,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1217249 */
> .callback = video_detect_force_native,
> - .ident = "Apple MacBook Pro 12,1",
> + /* Apple MacBook Pro 12,1 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Apple Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "MacBookPro12,1"),
> @@ -343,7 +343,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_native,
> - .ident = "Dell Vostro V131",
> + /* Dell Vostro V131 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "Vostro V131"),
> @@ -352,7 +352,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.redhat.com/show_bug.cgi?id=1123661 */
> .callback = video_detect_force_native,
> - .ident = "Dell XPS 17 L702X",
> + /* Dell XPS 17 L702X */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "Dell System XPS L702X"),
> @@ -360,7 +360,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_native,
> - .ident = "Dell Precision 7510",
> + /* Dell Precision 7510 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "Precision 7510"),
> @@ -368,7 +368,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_native,
> - .ident = "Acer Aspire 5738z",
> + /* Acer Aspire 5738z */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
> DMI_MATCH(DMI_PRODUCT_NAME, "Aspire 5738"),
> @@ -378,7 +378,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> {
> /* https://bugzilla.kernel.org/show_bug.cgi?id=207835 */
> .callback = video_detect_force_native,
> - .ident = "Acer TravelMate 5735Z",
> + /* Acer TravelMate 5735Z */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Acer"),
> DMI_MATCH(DMI_PRODUCT_NAME, "TravelMate 5735Z"),
> @@ -387,7 +387,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_native,
> - .ident = "ASUSTeK COMPUTER INC. GA401",
> + /* ASUSTeK COMPUTER INC. GA401 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
> DMI_MATCH(DMI_PRODUCT_NAME, "GA401"),
> @@ -395,7 +395,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_native,
> - .ident = "ASUSTeK COMPUTER INC. GA502",
> + /* ASUSTeK COMPUTER INC. GA502 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
> DMI_MATCH(DMI_PRODUCT_NAME, "GA502"),
> @@ -403,7 +403,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_native,
> - .ident = "ASUSTeK COMPUTER INC. GA503",
> + /* ASUSTeK COMPUTER INC. GA503 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "ASUSTeK COMPUTER INC."),
> DMI_MATCH(DMI_PRODUCT_NAME, "GA503"),
> @@ -416,7 +416,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> */
> {
> .callback = video_detect_force_none,
> - .ident = "Dell OptiPlex 9020M",
> + /* Dell OptiPlex 9020M */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> DMI_MATCH(DMI_PRODUCT_NAME, "OptiPlex 9020M"),
> @@ -424,7 +424,7 @@ static const struct dmi_system_id video_detect_dmi_table[] = {
> },
> {
> .callback = video_detect_force_none,
> - .ident = "MSI MS-7721",
> + /* MSI MS-7721 */
> .matches = {
> DMI_MATCH(DMI_SYS_VENDOR, "MSI"),
> DMI_MATCH(DMI_PRODUCT_NAME, "MS-7721"),
> --
Applied as 5.16-rc material along with the [2/2], thanks!
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-11-03 18:34 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-03 14:26 [PATCH 1/2] ACPI: video: Drop dmi_system_id.ident settings from video_detect_dmi_table[] Hans de Goede
2021-11-03 14:26 ` [PATCH 2/2] ACPI: video: use platform backlight driver on Xiaomi Mi Pad 2 Hans de Goede
2021-11-03 18:33 ` [PATCH 1/2] ACPI: video: Drop dmi_system_id.ident settings from video_detect_dmi_table[] Rafael J. Wysocki
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.