Commit 599482c5 authored by Kelly Anderson's avatar Kelly Anderson Committed by Hans de Goede
Browse files

platform/x86: ideapad-laptop: Add platform support for Ideapad 5 Pro 16ACH6-82L5 



Adding support specifically for Ideapad 5 Pro 16ACH6-82L5 by adding a
allow list that can validate notebooks for which dytc_version
is less than 5, and seem to work fine at dytc_version 4. This code has
been tested to work properly on the specified system.

Signed-off-by: default avatarKelly Anderson <kelly@xilka.com>
Link: https://lore.kernel.org/r/11840239.O9o76ZdvQC@comer.internal


Reviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent 6a684635

drivers/platform/x86/ideapad-laptop.c

+24 −3
Original line number Diff line number Diff line
@@ -868,6 +868,18 @@ static void dytc_profile_refresh(struct ideapad_private *priv) 
	}

}



static const struct dmi_system_id ideapad_dytc_v4_allow_table[] = {

	{

		/* Ideapad 5 Pro 16ACH6 */

		.ident = "LENOVO 82L5",

		.matches = {

			DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),

			DMI_MATCH(DMI_PRODUCT_NAME, "82L5")

		}

	},

	{}

};



static int ideapad_dytc_profile_init(struct ideapad_private *priv)

{

	int err, dytc_version;
@@ -882,12 +894,21 @@ static int ideapad_dytc_profile_init(struct ideapad_private *priv) 
		return err;



	/* Check DYTC is enabled and supports mode setting */

	if (!test_bit(DYTC_QUERY_ENABLE_BIT, &output))

	if (!test_bit(DYTC_QUERY_ENABLE_BIT, &output)) {

		dev_info(&priv->platform_device->dev, "DYTC_QUERY_ENABLE_BIT returned false\n");

		return -ENODEV;

	}



	dytc_version = (output >> DYTC_QUERY_REV_BIT) & 0xF;

	if (dytc_version < 5)



	if (dytc_version < 5) {

		if (dytc_version < 4 || !dmi_check_system(ideapad_dytc_v4_allow_table)) {

			dev_info(&priv->platform_device->dev,

				 "DYTC_VERSION is less than 4 or is not allowed: %d\n",

				 dytc_version);

			return -ENODEV;

		}

	}



	priv->dytc = kzalloc(sizeof(*priv->dytc), GFP_KERNEL);

	if (!priv->dytc)