2021-06-01 08:31:50

by Stephen Rothwell

[permalink] [raw]
Subject: linux-next: build warning after merge of the usb tree

Hi all,

After merging the usb tree, today's linux-next build (htmldocs) produced
this warning:

Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Unexpected indentation.
Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Block quote ends without a blank line; unexpected unindent.

Introduced by commit

70f400d4d957 ("driver core: Move the "removable" attribute from USB to core")

--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature

2021-06-01 17:43:22

by Rajat Jain

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the usb tree

Hello,


On Tue, Jun 1, 2021 at 1:30 AM Stephen Rothwell <[email protected]> wrote:
>
> Hi all,
>
> After merging the usb tree, today's linux-next build (htmldocs) produced
> this warning:
>
> Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Unexpected indentation.
> Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Block quote ends without a blank line; unexpected unindent.

I'd be happy to send a patch to fix this, but I didn't really
understand what needs to be done.

Here is the relevant documentation update in the patch:

+What: /sys/devices/.../removable
+Date: May 2021
+Contact: Rajat Jain <[email protected]>
+Description:
+ Information about whether a given device can be removed from the
+ platform by the user. This is determined by its subsystem in a
+ bus / platform-specific way. This attribute is only present for
+ devices that can support determining such information:
+
+ "removable": device can be removed from the platform by the user
+ "fixed": device is fixed to the platform / cannot be removed
+ by the user.
+ "unknown": The information is unavailable / cannot be deduced.
+
+ Currently this is only supported by USB (which infers the
+ information from a combination of hub descriptor bits and
+ platform-specific data such as ACPI).

I'd be happy to send a patch if you can point me what needs to be done
(or let Stephen / Greg / some one else do it if it is easier to just
do it instead of guiding me).

Thanks,

Rajat

>
> Introduced by commit
>
> 70f400d4d957 ("driver core: Move the "removable" attribute from USB to core")
>
> --
> Cheers,
> Stephen Rothwell

2021-06-04 13:39:29

by Greg KH

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the usb tree

On Tue, Jun 01, 2021 at 10:39:42AM -0700, Rajat Jain wrote:
> Hello,
>
>
> On Tue, Jun 1, 2021 at 1:30 AM Stephen Rothwell <[email protected]> wrote:
> >
> > Hi all,
> >
> > After merging the usb tree, today's linux-next build (htmldocs) produced
> > this warning:
> >
> > Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Unexpected indentation.
> > Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Block quote ends without a blank line; unexpected unindent.
>
> I'd be happy to send a patch to fix this, but I didn't really
> understand what needs to be done.
>
> Here is the relevant documentation update in the patch:
>
> +What: /sys/devices/.../removable
> +Date: May 2021
> +Contact: Rajat Jain <[email protected]>
> +Description:
> + Information about whether a given device can be removed from the
> + platform by the user. This is determined by its subsystem in a
> + bus / platform-specific way. This attribute is only present for
> + devices that can support determining such information:
> +
> + "removable": device can be removed from the platform by the user
> + "fixed": device is fixed to the platform / cannot be removed
> + by the user.
> + "unknown": The information is unavailable / cannot be deduced.
> +
> + Currently this is only supported by USB (which infers the
> + information from a combination of hub descriptor bits and
> + platform-specific data such as ACPI).
>
> I'd be happy to send a patch if you can point me what needs to be done
> (or let Stephen / Greg / some one else do it if it is easier to just
> do it instead of guiding me).

I do not know what the "right" thing to do here is, sorry. Maybe one of
the kerneldoc people know?

greg k-h

2021-07-08 02:24:03

by Stephen Rothwell

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the usb tree

Hi all,

On Tue, 1 Jun 2021 18:30:02 +1000 Stephen Rothwell <[email protected]> wrote:
>
> After merging the usb tree, today's linux-next build (htmldocs) produced
> this warning:
>
> Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Unexpected indentation.
> Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Block quote ends without a blank line; unexpected unindent.
>
> Introduced by commit
>
> 70f400d4d957 ("driver core: Move the "removable" attribute from USB to core")

I am still getting these warnings.

--
Cheers,
Stephen Rothwell


Attachments:
(No filename) (499.00 B)
OpenPGP digital signature

2021-07-08 08:11:15

by Greg KH

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the usb tree

On Thu, Jul 08, 2021 at 12:22:06PM +1000, Stephen Rothwell wrote:
> Hi all,
>
> On Tue, 1 Jun 2021 18:30:02 +1000 Stephen Rothwell <[email protected]> wrote:
> >
> > After merging the usb tree, today's linux-next build (htmldocs) produced
> > this warning:
> >
> > Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Unexpected indentation.
> > Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Block quote ends without a blank line; unexpected unindent.
> >
> > Introduced by commit
> >
> > 70f400d4d957 ("driver core: Move the "removable" attribute from USB to core")
>
> I am still getting these warnings.

Rajat, can you send me a patch to fix this up?

thanks,

greg k-h

2021-07-13 21:05:24

by Rajat Jain

[permalink] [raw]
Subject: Re: linux-next: build warning after merge of the usb tree

[email protected]
+Jonathan Corbet

Hi Jonathan / kernel-docs people,

On Fri, Jun 4, 2021 at 6:37 AM Greg KH <[email protected]> wrote:
>
> On Tue, Jun 01, 2021 at 10:39:42AM -0700, Rajat Jain wrote:
> > Hello,
> >
> >
> > On Tue, Jun 1, 2021 at 1:30 AM Stephen Rothwell <[email protected]> wrote:
> > >
> > > Hi all,
> > >
> > > After merging the usb tree, today's linux-next build (htmldocs) produced
> > > this warning:
> > >
> > > Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Unexpected indentation.
> > > Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Block quote ends without a blank line; unexpected unindent.
> >
> > I'd be happy to send a patch to fix this, but I didn't really
> > understand what needs to be done.
> >
> > Here is the relevant documentation update in the patch:
> >
> > +What: /sys/devices/.../removable
> > +Date: May 2021
> > +Contact: Rajat Jain <[email protected]>
> > +Description:
> > + Information about whether a given device can be removed from the
> > + platform by the user. This is determined by its subsystem in a
> > + bus / platform-specific way. This attribute is only present for
> > + devices that can support determining such information:
> > +
> > + "removable": device can be removed from the platform by the user
> > + "fixed": device is fixed to the platform / cannot be removed
> > + by the user.
> > + "unknown": The information is unavailable / cannot be deduced.
> > +
> > + Currently this is only supported by USB (which infers the
> > + information from a combination of hub descriptor bits and
> > + platform-specific data such as ACPI).
> >
> > I'd be happy to send a patch if you can point me what needs to be done
> > (or let Stephen / Greg / some one else do it if it is easier to just
> > do it instead of guiding me).
>
> I do not know what the "right" thing to do here is, sorry. Maybe one of
> the kerneldoc people know?

Can someone familiar with the documentation and html docs let me know
what is wrong with Documentation/ABI/testing/sysfs-devices-removable
that is resulting in this warning when doing "make htmldocs":

> > > Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Unexpected indentation.
> > > Documentation/ABI/testing/sysfs-devices-removable:2: WARNING: Block quote ends without a blank line; unexpected unindent.

I looked at the file line number 2 and it just seems inline with other
similar files in that documentation folder.

Thanks,

Rajat

> >
>
> greg k-h