Re: [PATCH v2 5/6] ata: ahci: Clarify mobile_lpm_policy description

[Niklas Cassel <cassel@kernel.org>]

On Fri, Jun 27, 2025 at 04:52:06PM +0900, Damien Le Moal wrote:
> Despite its name, the mobile_lpm_policy module parameter defines the
> default LPM policy to use for an AHCI adapter for all chipsets,
> including desktop and server chipsets. Clarify this point in the
> parameter description.
> 
> No functional changes.
> 
> Signed-off-by: Damien Le Moal <dlemoal@kernel.org>
> Reviewed-by: Hannes Reinecke <hare@suse.de>
> ---
>  drivers/ata/ahci.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
> index e5e5c2e81d09..9347d0ec8793 100644
> --- a/drivers/ata/ahci.c
> +++ b/drivers/ata/ahci.c
> @@ -674,7 +674,9 @@ MODULE_PARM_DESC(marvell_enable, "Marvell SATA via AHCI (1 = enabled)");
>  
>  static int mobile_lpm_policy = -1;
>  module_param(mobile_lpm_policy, int, 0644);
> -MODULE_PARM_DESC(mobile_lpm_policy, "Default LPM policy for mobile chipsets");
> +MODULE_PARM_DESC(mobile_lpm_policy,
> +		 "Default LPM policy. Despite its name, this parameter applies "
> +		 "to all chipsets, including desktop and servers chipsets");

Nit: desktop is singular, servers is plural.
Make both singular or both plural.

(I think both in singular sounds slightly better.)


Reviewed-by: Niklas Cassel <cassel@kernel.org>