linux-kernel - Re: [PATCH] platform/x86: alienware-wmi: Fix zone attribute declaration

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <488547a0-3560-4350-38ec-649c36f2861d@linux.intel.com>
Date: Wed, 15 Jan 2025 16:45:31 +0200 (EET)
From: Ilpo Järvinen <ilpo.jarvinen@...ux.intel.com>
To: Kurt Borja <kuurtb@...il.com>
cc: platform-driver-x86@...r.kernel.org, oe-kbuild-all@...ts.linux.dev, 
    Armin Wolf <W_Armin@....de>, Mario Limonciello <mario.limonciello@....com>, 
    Hans de Goede <hdegoede@...hat.com>, Dell.Client.Kernel@...l.com, 
    LKML <linux-kernel@...r.kernel.org>, linux-next@...r.kernel.org, 
    kernel test robot <lkp@...el.com>
Subject: Re: [PATCH] platform/x86: alienware-wmi: Fix zone attribute
 declaration

On Tue, 14 Jan 2025, Kurt Borja wrote:

> Statically declare `zone*` attributes.
> 
> Fixes: 7c605f6460e8 ("platform/x86: alienware-wmi: Improve rgb-zones group creation")
> Reported-by: kernel test robot <lkp@...el.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202501142200.ezULWY9P-lkp@intel.com/
> Signed-off-by: Kurt Borja <kuurtb@...il.com>
> ---
>  drivers/platform/x86/dell/alienware-wmi.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/platform/x86/dell/alienware-wmi.c b/drivers/platform/x86/dell/alienware-wmi.c
> index b4b43f3e3fd9..d7f577e0d146 100644
> --- a/drivers/platform/x86/dell/alienware-wmi.c
> +++ b/drivers/platform/x86/dell/alienware-wmi.c
> @@ -518,7 +518,7 @@ static ssize_t zone00_store(struct device *dev, struct device_attribute *attr,
>  	return zone_store(dev, attr, buf, count, 0);
>  }
>  
> -DEVICE_ATTR_RW(zone00);
> +static DEVICE_ATTR_RW(zone00);
>  
>  static ssize_t zone01_show(struct device *dev, struct device_attribute *attr,
>  			   char *buf)
> @@ -532,7 +532,7 @@ static ssize_t zone01_store(struct device *dev, struct device_attribute *attr,
>  	return zone_store(dev, attr, buf, count, 1);
>  }
>  
> -DEVICE_ATTR_RW(zone01);
> +static DEVICE_ATTR_RW(zone01);
>  
>  static ssize_t zone02_show(struct device *dev, struct device_attribute *attr,
>  			   char *buf)
> @@ -546,7 +546,7 @@ static ssize_t zone02_store(struct device *dev, struct device_attribute *attr,
>  	return zone_store(dev, attr, buf, count, 2);
>  }
>  
> -DEVICE_ATTR_RW(zone02);
> +static DEVICE_ATTR_RW(zone02);
>  
>  static ssize_t zone03_show(struct device *dev, struct device_attribute *attr,
>  			   char *buf)
> @@ -560,7 +560,7 @@ static ssize_t zone03_store(struct device *dev, struct device_attribute *attr,
>  	return zone_store(dev, attr, buf, count, 3);
>  }
>  
> -DEVICE_ATTR_RW(zone03);
> +static DEVICE_ATTR_RW(zone03);
>  
>  /*
>   * Lighting control state device attribute (Global)
> 
> base-commit: 58126788aa7726c0e91de6b25e6e332fa06089ab
> 

Thanks. I've folded this into the original commit in the review-ilpo-next 
branch.

-- 
 i.