linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* unexpected indentation warning in Documentation/ABI/testing/sysfs-*
@ 2022-04-04 11:58 Bagas Sanjaya
  2022-04-04 12:02 ` Greg Kroah-Hartman
  2022-04-04 12:54 ` Hans de Goede
  0 siblings, 2 replies; 6+ messages in thread
From: Bagas Sanjaya @ 2022-04-04 11:58 UTC (permalink / raw)
  To: linux-doc, 'Linux Kernel',
	platform-driver-x86, Dell.Client.Kernel, linux-iio
  Cc: Gao Xiang, Huang Jianan, Chao Yu, Divya Bharathi, Mark Pearson,
	Jonathan Cameron, Lars-Peter Clausen, Gwendal Grignou,
	Kajol Jain, Andrew Morton, Daniel Borkmann, Greg Kroah-Hartman,
	David S. Miller, David E. Box, Mark Gross, Hans de Goede

Hi,

Doing "make htmldocs" for Linux v5.18-rc1, I get new warnings:

<path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Unexpected indentation.
<path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Block quote ends without a blank line; unexpected unindent.
<path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Definition list ends without a blank line; unexpected unindent.
<path/to/linux>/Documentation/ABI/testing/sysfs-bus-nvdimm:11: WARNING: Unexpected indentation.
<path/to/linux>/Documentation/ABI/testing/sysfs-bus-iio-sx9324:2: WARNING: Unexpected indentation.
<path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
<path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
<path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
<path/to/linux>/Documentation/ABI/testing/sysfs-fs-erofs:10: WARNING: Unexpected indentation.
<path/to/linux>/Documentation/ABI/testing/sysfs-fs-erofs:10: WARNING: Block quote ends without a blank line; unexpected unindent.

Introduced by 2546c60004309e (platform/x86: Add Intel Software Defined
Silicon driver, 2022-02-11), 2bec6d9aa89cbe (docs: ABI: sysfs-bus-nvdimm:
Document sysfs event format entries for nvdimm pmu, 2022-02-25),
e8a60aa7404bfe (platform/x86: Introduce support for Systems Management
Driver over WMI for Dell Systems), 40452ffca3c1a0 (erofs: add sysfs
node to control sync decompression strategy, 2021-12-06), and
4c18a890dff8d9 (iio:proximity:sx9324: Add SX9324 support, 2022-01-01).

Presumably because Sphinx mistakes these documentation files without
extensions for .rst files? I dunno.

Reported-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: unexpected indentation warning in Documentation/ABI/testing/sysfs-*
  2022-04-04 11:58 unexpected indentation warning in Documentation/ABI/testing/sysfs-* Bagas Sanjaya
@ 2022-04-04 12:02 ` Greg Kroah-Hartman
  2022-04-04 12:26   ` Bagas Sanjaya
  2022-04-04 12:54 ` Hans de Goede
  1 sibling, 1 reply; 6+ messages in thread
From: Greg Kroah-Hartman @ 2022-04-04 12:02 UTC (permalink / raw)
  To: Bagas Sanjaya
  Cc: linux-doc, 'Linux Kernel',
	platform-driver-x86, Dell.Client.Kernel, linux-iio, Gao Xiang,
	Huang Jianan, Chao Yu, Divya Bharathi, Mark Pearson,
	Jonathan Cameron, Lars-Peter Clausen, Gwendal Grignou,
	Kajol Jain, Andrew Morton, Daniel Borkmann, David S. Miller,
	David E. Box, Mark Gross, Hans de Goede

On Mon, Apr 04, 2022 at 06:58:16PM +0700, Bagas Sanjaya wrote:
> Hi,
> 
> Doing "make htmldocs" for Linux v5.18-rc1, I get new warnings:
> 
> <path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Block quote ends without a blank line; unexpected unindent.
> <path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Definition list ends without a blank line; unexpected unindent.
> <path/to/linux>/Documentation/ABI/testing/sysfs-bus-nvdimm:11: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-bus-iio-sx9324:2: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-fs-erofs:10: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-fs-erofs:10: WARNING: Block quote ends without a blank line; unexpected unindent.
> 
> Introduced by 2546c60004309e (platform/x86: Add Intel Software Defined
> Silicon driver, 2022-02-11), 2bec6d9aa89cbe (docs: ABI: sysfs-bus-nvdimm:
> Document sysfs event format entries for nvdimm pmu, 2022-02-25),
> e8a60aa7404bfe (platform/x86: Introduce support for Systems Management
> Driver over WMI for Dell Systems), 40452ffca3c1a0 (erofs: add sysfs
> node to control sync decompression strategy, 2021-12-06), and
> 4c18a890dff8d9 (iio:proximity:sx9324: Add SX9324 support, 2022-01-01).
> 
> Presumably because Sphinx mistakes these documentation files without
> extensions for .rst files? I dunno.
> 
> Reported-by: Bagas Sanjaya <bagasdotme@gmail.com>

Care to send patches to fix these up?

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: unexpected indentation warning in Documentation/ABI/testing/sysfs-*
  2022-04-04 12:02 ` Greg Kroah-Hartman
@ 2022-04-04 12:26   ` Bagas Sanjaya
  0 siblings, 0 replies; 6+ messages in thread
From: Bagas Sanjaya @ 2022-04-04 12:26 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: linux-doc, 'Linux Kernel',
	platform-driver-x86, Dell.Client.Kernel, linux-iio, Gao Xiang,
	Huang Jianan, Chao Yu, Divya Bharathi, Mark Pearson,
	Jonathan Cameron, Lars-Peter Clausen, Gwendal Grignou,
	Kajol Jain, Andrew Morton, Daniel Borkmann, David S. Miller,
	David E. Box, Mark Gross, Hans de Goede

On 04/04/22 19.02, Greg Kroah-Hartman wrote:
>> Presumably because Sphinx mistakes these documentation files without
>> extensions for .rst files? I dunno.
>>
>> Reported-by: Bagas Sanjaya <bagasdotme@gmail.com>
> 
> Care to send patches to fix these up?
> 
> thanks,
> 
> greg k-h

Hi Greg,

I'm sorry I couldn't do any fixes for now. I'm just reporting.

-- 
An old man doll... just what I always wanted! - Clara

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: unexpected indentation warning in Documentation/ABI/testing/sysfs-*
  2022-04-04 11:58 unexpected indentation warning in Documentation/ABI/testing/sysfs-* Bagas Sanjaya
  2022-04-04 12:02 ` Greg Kroah-Hartman
@ 2022-04-04 12:54 ` Hans de Goede
  2022-04-05  7:52   ` Bagas Sanjaya
  1 sibling, 1 reply; 6+ messages in thread
From: Hans de Goede @ 2022-04-04 12:54 UTC (permalink / raw)
  To: Bagas Sanjaya, linux-doc, 'Linux Kernel',
	platform-driver-x86, Dell.Client.Kernel, linux-iio
  Cc: Gao Xiang, Huang Jianan, Chao Yu, Divya Bharathi, Mark Pearson,
	Jonathan Cameron, Lars-Peter Clausen, Gwendal Grignou,
	Kajol Jain, Andrew Morton, Daniel Borkmann, Greg Kroah-Hartman,
	David S. Miller, David E. Box, Mark Gross

Hi,

On 4/4/22 13:58, Bagas Sanjaya wrote:
> Hi,
> 
> Doing "make htmldocs" for Linux v5.18-rc1, I get new warnings:
> 
> <path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Block quote ends without a blank line; unexpected unindent.
> <path/to/linux>/Documentation/ABI/testing/sysfs-driver-intel_sdsi:2: WARNING: Definition list ends without a blank line; unexpected unindent.
> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.

These 6 are all fixed in this series:

> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-class-firmware-attributes:130: WARNING: Unexpected indentation.

https://lore.kernel.org/platform-driver-x86/20220324164737.21765-1-hdegoede@redhat.com/T/#t

Which will be included in my next fixes pull-req to Linus.

> <path/to/linux>/Documentation/ABI/testing/sysfs-fs-erofs:10: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-fs-erofs:10: WARNING: Block quote ends without a blank line; unexpected unindent.

These 2 are fixed by this (pending) patch:
https://lore.kernel.org/linux-kernel/20220324165918.22005-4-hdegoede@redhat.com/

And I also have this pending patch for a pre-existing warning:
https://lore.kernel.org/linux-kernel/20220324165918.22005-3-hdegoede@redhat.com/

I'm not sure who should pick these 2 up ?

> <path/to/linux>/Documentation/ABI/testing/sysfs-bus-iio-sx9324:2: WARNING: Unexpected indentation.
> <path/to/linux>/Documentation/ABI/testing/sysfs-bus-nvdimm:11: WARNING: Unexpected indentation.

These 2 warnings are new to me (I tested with pdx86/for-next, so they probably
came in through another tree).

Regards,

HaNS





> 
> Introduced by 2546c60004309e (platform/x86: Add Intel Software Defined
> Silicon driver, 2022-02-11), 2bec6d9aa89cbe (docs: ABI: sysfs-bus-nvdimm:
> Document sysfs event format entries for nvdimm pmu, 2022-02-25),
> e8a60aa7404bfe (platform/x86: Introduce support for Systems Management
> Driver over WMI for Dell Systems), 40452ffca3c1a0 (erofs: add sysfs
> node to control sync decompression strategy, 2021-12-06), and
> 4c18a890dff8d9 (iio:proximity:sx9324: Add SX9324 support, 2022-01-01).
> 
> Presumably because Sphinx mistakes these documentation files without
> extensions for .rst files? I dunno.
> 
> Reported-by: Bagas Sanjaya <bagasdotme@gmail.com>
> 


^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: unexpected indentation warning in Documentation/ABI/testing/sysfs-*
  2022-04-04 12:54 ` Hans de Goede
@ 2022-04-05  7:52   ` Bagas Sanjaya
  2022-04-05  9:32     ` Gao Xiang
  0 siblings, 1 reply; 6+ messages in thread
From: Bagas Sanjaya @ 2022-04-05  7:52 UTC (permalink / raw)
  To: Hans de Goede, linux-doc, 'Linux Kernel',
	platform-driver-x86, Dell.Client.Kernel, linux-iio
  Cc: Gao Xiang, Huang Jianan, Chao Yu, Divya Bharathi, Mark Pearson,
	Jonathan Cameron, Lars-Peter Clausen, Gwendal Grignou,
	Kajol Jain, Andrew Morton, Daniel Borkmann, Greg Kroah-Hartman,
	David S. Miller, David E. Box, Mark Gross

On 04/04/22 19.54, Hans de Goede wrote:
> 
> These 2 are fixed by this (pending) patch:
> https://lore.kernel.org/linux-kernel/20220324165918.22005-4-hdegoede@redhat.com/
> 
> And I also have this pending patch for a pre-existing warning:
> https://lore.kernel.org/linux-kernel/20220324165918.22005-3-hdegoede@redhat.com/
> 
> I'm not sure who should pick these 2 up ?
> 

I think linux-doc can pick them, if no subsystems do it.

-- 
An old man doll... just what I always wanted! - Clara

^ permalink raw reply	[flat|nested] 6+ messages in thread
* Re: unexpected indentation warning in Documentation/ABI/testing/sysfs-*
  2022-04-05  7:52   ` Bagas Sanjaya
@ 2022-04-05  9:32     ` Gao Xiang
  0 siblings, 0 replies; 6+ messages in thread
From: Gao Xiang @ 2022-04-05  9:32 UTC (permalink / raw)
  To: Bagas Sanjaya
  Cc: Hans de Goede, linux-doc, 'Linux Kernel',
	platform-driver-x86, Dell.Client.Kernel, linux-iio, Huang Jianan,
	Chao Yu, Divya Bharathi, Mark Pearson, Jonathan Cameron,
	Lars-Peter Clausen, Gwendal Grignou, Kajol Jain, Andrew Morton,
	Daniel Borkmann, Greg Kroah-Hartman, David S. Miller,
	David E. Box, Mark Gross

Hi,

On Tue, Apr 05, 2022 at 02:52:18PM +0700, Bagas Sanjaya wrote:
> On 04/04/22 19.54, Hans de Goede wrote:
> > 
> > These 2 are fixed by this (pending) patch:
> > https://lore.kernel.org/linux-kernel/20220324165918.22005-4-hdegoede@redhat.com/
> > 
> > And I also have this pending patch for a pre-existing warning:
> > https://lore.kernel.org/linux-kernel/20220324165918.22005-3-hdegoede@redhat.com/
> > 
> > I'm not sure who should pick these 2 up ?
> > 
> 
> I think linux-doc can pick them, if no subsystems do it.
> 

btw, I can pick the erofs one up though, but since it's a doc-specific
series, I think it can go with linux-doc too.

Thanks,
Gao Xiang

> -- 
> An old man doll... just what I always wanted! - Clara

^ permalink raw reply	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-04-05 19:53 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-04 11:58 unexpected indentation warning in Documentation/ABI/testing/sysfs-* Bagas Sanjaya
2022-04-04 12:02 ` Greg Kroah-Hartman
2022-04-04 12:26   ` Bagas Sanjaya
2022-04-04 12:54 ` Hans de Goede
2022-04-05  7:52   ` Bagas Sanjaya
2022-04-05  9:32     ` Gao Xiang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).