Re: [PATCH] scsi: ata: make ahci_sdev_attrs static

[Damien Le Moal <damien.lemoal@opensource.wdc.com>]

On 10/19/21 19:24, Jiapeng Chong wrote:
> From: chongjiapeng <jiapeng.chong@linux.alibaba.com>
> 
> This symbol is not used outside of libahci.c, so marks it static.
> 
> Fix the following sparse warning:
> 
> drivers/ata/libahci.c:134:18: warning: symbol 'ahci_sdev_attrs' was not
> declared. Should it be static?
> 
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Fixes: c3f69c7f629f ("scsi: ata: Switch to attribute groups")
> Signed-off-by: chongjiapeng <jiapeng.chong@linux.alibaba.com>
> ---
>  drivers/ata/libahci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/ata/libahci.c b/drivers/ata/libahci.c
> index 28430c093a7f..8a6835bfd18a 100644
> --- a/drivers/ata/libahci.c
> +++ b/drivers/ata/libahci.c
> @@ -131,7 +131,7 @@ const struct attribute_group *ahci_shost_groups[] = {
>  };
>  EXPORT_SYMBOL_GPL(ahci_shost_groups);
>  
> -struct attribute *ahci_sdev_attrs[] = {
> +static struct attribute *ahci_sdev_attrs[] = {
>  	&dev_attr_sw_activity.attr,
>  	&dev_attr_unload_heads.attr,
>  	&dev_attr_ncq_prio_supported.attr,
> 

Jiapeng,

This patch does not apply to the current libata tree since it is against
the latest changes queued in the scsi tree. After fixing the patch
title, can you resend this patch to the linux-scsi list and to Martin ?



-- 
Damien Le Moal
Western Digital Research