Re: [PATCH] scsi: libsas: Fix interval notation

[John Garry]

On 11/3/26 14:25, Manuel Ebner wrote:
> On Wed, 2026-09-16 at 14:40 +0100, John Garry wrote:
> > On 9/16/26 13:44, Manuel Ebner wrote:
> > > Remove needless ']' from interval boundary.
> >
> > This documentation is wrong for this structure description (not shown) -
> > it should be for asd_sas_phy, not sas_phy
> >
> > Same for sas_port -> asd_sas_port
>
> Does replacing all sas_phy -> asd_sas_phy and sas_port -> asd_sas_port
> fix this? Then I'll send a patch.

uh, I don't think that anyone would be checking this documentation from 
now on, as I don't expect any drivers for new SAS HBAs which may even 
want to use libsas will appear.

John

> Thanks>   Manuel
> > > Signed-off-by: Manuel Ebner <manuelebnerli@xxxxxxxxxxx>
> > > ---
> > >    Documentation/scsi/libsas.rst | 2 +-
> > >    1 file changed, 1 insertion(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/scsi/libsas.rst b/Documentation/scsi/libsas.rst
> > > index 305a253d5..9ebf2bf28 100644
> > > --- a/Documentation/scsi/libsas.rst
> > > +++ b/Documentation/scsi/libsas.rst
> > > @@ -75,7 +75,7 @@ enabled
> > >        - must be set (0/1)
> > >    
> > >    id
> > > -    - must be set [0,MAX_PHYS)]
> > > +    - must be set [0,MAX_PHYS)
> >
> > this looks ok, even though it is so trivial
> >
> > >    
> > >    class, proto, type, role, oob_mode, linkrate
> > >        - must be set