Skip to content

Commit

Permalink
media: atomisp: gmin_platform: Add Lenovo Ideapad Miix 310 gmin_vars
Browse files Browse the repository at this point in the history
The _DSM used to get sensor variables like CsiPort returns the wrong
csi-port for the front OV2680 sensor on the Lenovo Ideapad Miix 310
add a gmin_vars DMI quirk / override setting the right CsiPort.

Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
  • Loading branch information
Hans de Goede authored and Mauro Carvalho Chehab committed Apr 15, 2023
1 parent bd8856c commit 671397d
Showing 1 changed file with 19 additions and 0 deletions.
19 changes: 19 additions & 0 deletions drivers/staging/media/atomisp/pci/atomisp_gmin_platform.c
Original file line number Diff line number Diff line change
Expand Up @@ -304,7 +304,17 @@ static struct gmin_cfg_var surface3_vars[] = {
{},
};

static struct gmin_cfg_var lenovo_ideapad_miix_310_vars[] = {
/* _DSM contains the wrong CsiPort! */
{ "OVTI2680:01_CsiPort", "0" },
{}
};

static const struct dmi_system_id gmin_vars[] = {
/*
* These DMI IDs were present when the atomisp driver was merged into
* drivers/staging and it is unclear if they are really necessary.
*/
{
.ident = "BYT-T FFD8",
.matches = {
Expand Down Expand Up @@ -341,13 +351,22 @@ static const struct dmi_system_id gmin_vars[] = {
},
.driver_data = i8880_vars,
},
/* Later added DMI ids, these are confirmed to really be necessary! */
{
.ident = "Surface 3",
.matches = {
DMI_MATCH(DMI_BOARD_NAME, "Surface 3"),
},
.driver_data = surface3_vars,
},
{
.ident = "Lenovo Ideapad Miix 310",
.matches = {
DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
DMI_MATCH(DMI_PRODUCT_VERSION, "MIIX 310-10"),
},
.driver_data = lenovo_ideapad_miix_310_vars,
},
{}
};

Expand Down

0 comments on commit 671397d

Please sign in to comment.