Hi Randy, On Mon, Apr 19, 2021 at 09:19:13AM -0700, Randy Dunlap wrote: > > +int ql2xdev_loss_tmo = 60; > > +module_param(ql2xdev_loss_tmo, int, 0444); > > +MODULE_PARM_DESC(ql2xdev_loss_tmo, > > + "Time to wait for device to recover before reporting\n" > > + "an error. Default is 60 seconds\n"); > > No need for the \n in the quoted string. Just change it to a space. I just followed the current style in this file. I guess this style question is up to the maintainers to decide what they want. Thanks, Daniel