Re: [PATCH] platform/x86: dell-laptop: Add quirk entry for Latitude 7520

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 4/26/22 14:08, gabriele.mzt@xxxxxxxxx wrote:
> From: Gabriele Mazzotta <gabriele.mzt@xxxxxxxxx>
> 
> The Latitude 7520 supports AC timeouts, but it has no KBD_LED_AC_TOKEN
> and so changes to stop_timeout appear to have no effect if the laptop
> is plugged in.
> 
> Signed-off-by: Gabriele Mazzotta <gabriele.mzt@xxxxxxxxx>

Thank you for your patch, I've applied this patch to my review-hans 
branch:
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans

Note it will show up in my review-hans branch once I've pushed my
local branch there, which might take a while.

Once I've run some tests on this branch the patches there will be
added to the platform-drivers-x86/for-next branch and eventually
will be included in the pdx86 pull-request to Linus for the next
merge-window.

Regards,

Hans



> ---
> As per the discussion here https://github.com/dell/libsmbios/issues/48,
> this is really a BIOS bug. My Latitude 7520 has the latest BIOS
> update installed, which was released just few months ago, but the
> issue is still there.
> ---
>  drivers/platform/x86/dell/dell-laptop.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/drivers/platform/x86/dell/dell-laptop.c b/drivers/platform/x86/dell/dell-laptop.c
> index 8230e7a68a5e..1321687d923e 100644
> --- a/drivers/platform/x86/dell/dell-laptop.c
> +++ b/drivers/platform/x86/dell/dell-laptop.c
> @@ -80,6 +80,10 @@ static struct quirk_entry quirk_dell_inspiron_1012 = {
>  	.kbd_led_not_present = true,
>  };
>  
> +static struct quirk_entry quirk_dell_latitude_7520 = {
> +	.kbd_missing_ac_tag = true,
> +};
> +
>  static struct platform_driver platform_driver = {
>  	.driver = {
>  		.name = "dell-laptop",
> @@ -336,6 +340,15 @@ static const struct dmi_system_id dell_quirks[] __initconst = {
>  		},
>  		.driver_data = &quirk_dell_inspiron_1012,
>  	},
> +	{
> +		.callback = dmi_matched,
> +		.ident = "Dell Latitude 7520",
> +		.matches = {
> +			DMI_MATCH(DMI_SYS_VENDOR, "Dell Inc."),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "Latitude 7520"),
> +		},
> +		.driver_data = &quirk_dell_latitude_7520,
> +	},
>  	{ }
>  };
>  




[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux