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@xxxxxxxxxx> > Reviewed-by: Hannes Reinecke <hare@xxxxxxx> > --- > 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@xxxxxxxxxx>