Re: [PATCH v2 1/4] ACPI / AC: Remove initializer for unused ident dmi_system_id

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

 



On Thursday, April 12, 2018 12:01:57 PM CEST Hans de Goede wrote:
> The ac.c code does not use the dmi_system_id ident member, so there is
> no need to initialize it. This saves us storing the unused "thinkpad e530"
> string as const data.
> 
> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
> ---
>  drivers/acpi/ac.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/acpi/ac.c b/drivers/acpi/ac.c
> index 47a7ed557bd6..1b60a6db8e53 100644
> --- a/drivers/acpi/ac.c
> +++ b/drivers/acpi/ac.c
> @@ -318,8 +318,8 @@ static int thinkpad_e530_quirk(const struct dmi_system_id *d)
>  
>  static const struct dmi_system_id ac_dmi_table[] = {
>  	{
> +	/* Thinkpad e530 */
>  	.callback = thinkpad_e530_quirk,
> -	.ident = "thinkpad e530",
>  	.matches = {
>  		DMI_MATCH(DMI_SYS_VENDOR, "LENOVO"),
>  		DMI_MATCH(DMI_PRODUCT_NAME, "32597CG"),
> 

Whole series applied, thanks!


--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux