Re: [PATCH V3 3/5] smartpqi: add module param expose ld first

["Michael Kerrisk (man-pages)" <mtk.manpages@xxxxxxxxx>]

Hello Don,

On Mon, 26 Aug 2019 at 23:48, <Don.Brace@xxxxxxxxxxxxx> wrote:
>
> -----Original Message-----
> From: Michael Kerrisk (man-pages) [mailto:mtk.manpages@xxxxxxxxx]
> Sent: Monday, August 26, 2019 4:46 PM
> To: Don Brace <don.brace@xxxxxxxxxxxxx>
> Cc: mtk.manpages@xxxxxxxxx; linux-man@xxxxxxxxxxxxxxx; Kevin Barnett - C33748 <Kevin.Barnett@xxxxxxxxxxxxx>; Scott Teel - C33730 <Scott.Teel@xxxxxxxxxxxxx>; Justin Lindley - C33718 <Justin.Lindley@xxxxxxxxxxxxx>; Scott Benesh - C33703 <Scott.Benesh@xxxxxxxxxxxxx>; bader.alisaleh@xxxxxxxxxxxxx; Gerry Morong - C33720 <Gerry.Morong@xxxxxxxxxxxxx>; Mahesh Rajashekhara - I30583 <Mahesh.Rajashekhara@xxxxxxxxxxxxx>; Gilbert Wu <gilbert.wu@xxxxxxxxxxxxx>
> Subject: Re: [PATCH V3 3/5] smartpqi: add module param expose ld first
>
> External E-Mail
>
>
> Hello Don
>
> Patch applied. But a question below.
>
> On 8/26/19 5:41 PM, Don Brace wrote:
> > From: Gilbert Wu <gilbert.wu@xxxxxxxxxxxxx>
> >
> > Reviewed-by: Matt Perricone <matt.perricone@xxxxxxxxxxxxx>
> > Reviewed-by: Scott Benesh <scott.benesh@xxxxxxxxxxxxx>
> > Reviewed-by: Kevin Barnett <kevin.barnett@xxxxxxxxxxxxx>
> > Signed-off-by: Gilbert Wu <gilbert.wu@xxxxxxxxxxxxx>
> > Signed-off-by: Don Brace <don.brace@xxxxxxxxxxxxx>
> > ---
> >  man4/smartpqi.4 |    5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/man4/smartpqi.4 b/man4/smartpqi.4 index
> > dc9ef1020..5389b8453 100644
> > --- a/man4/smartpqi.4
> > +++ b/man4/smartpqi.4
> > @@ -15,6 +15,7 @@ smartpqi \- Microsemi Smart Family SCSI driver  .RB
> > [ disable_heartbeat= { 0 | 1 }]  .RB [ disable_ctrl_shutdown= { 0 | 1
> > }]  .RB [ lockup_action= { none | reboot | panic }]
> > +.RB [ expose_ld_first= { 0 | 1 }]
> >  .YS
> >  .SH DESCRIPTION
> >  .B smartpqi
> > @@ -91,6 +92,10 @@ parameter  action
> >  \fBreboot\fP reboot the system
> >  \fBpanic\fP  panic the system
> >  .TE
> > +.TP
> > +.BR expose_ld_first= { 0 | 1 }
> > +This option enables support for exposing logical devices to OS before physical devices.
>
> Should "OS' instead be "the operating system"?
>
> "The operating system" sounds better.
> Thanks,
> Don Brace

Thanks. Page amended.

Cheers,

Michael