linux-next: build warning after merge of the iio tree

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 307 bytes --]

Hi all,

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

Documentation/ABI/testing/sysfs-bus-iio-sx9324:2: WARNING: Unexpected indentation.

Introduced by commit

  4c18a890dff8 ("iio:proximity:sx9324: Add SX9324 support")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 386 bytes --]

Hi all,

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

include/linux/iio/iio.h:319: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
 * Device operating modes

Introduced by commit

  831d87089ca8 ("iio: core: Clarify the modes")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the iio tree

[Miquel Raynal]

Hello,

sfr@canb.auug.org.au wrote on Thu, 28 Apr 2022 17:37:58 +1000:

> Hi all,
> 
> After merging the iio tree, today's linux-next build (htmldocs) produced
> this warning:
> 
> include/linux/iio/iio.h:319: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
>  * Device operating modes
> 
> Introduced by commit
> 
>   831d87089ca8 ("iio: core: Clarify the modes")

Looks like there is no kdoc specific format for #define's
(Documentation/doc-guide/kernel-doc.rst), which is disappointing, I
really hoped this could be brought to the developer's attention this
way.

But that's not a big deal. Jonathan, if you don't mind, you may just
drop that extra start s|/**|/*| to make the htmldocs target happy
again.

Thanks,
Miquèl

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 961 bytes --]

Hi all,

After merging the iio tree, today's linux-next build (x86_64 allmodconfig)
produced this warning:

WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc_tm5_temp_volt_scale from namespace IIO_QCOM_VADC, but does not import it.
WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_prescaling_from_dt from namespace IIO_QCOM_VADC, but does not import it.
WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_hw_settle_time_from_dt from namespace IIO_QCOM_VADC, but does not import it.
WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_avg_samples_from_dt from namespace IIO_QCOM_VADC, but does not import it.
WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_decimation_from_dt from namespace IIO_QCOM_VADC, but does not import it.

Introduced by commit

  ec9b269f61cc ("iio: adc: qcom-vadc: Move symbol exports to IIO_QCOM_VADC namespace")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 1176 bytes --]

Hi all,

On Fri, 1 Jul 2022 15:29:44 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
\> 
> After merging the iio tree, today's linux-next build (x86_64 allmodconfig)
> produced this warning:
> 
> WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc_tm5_temp_volt_scale from namespace IIO_QCOM_VADC, but does not import it.
> WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_prescaling_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_hw_settle_time_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_avg_samples_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_decimation_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> 
> Introduced by commit
> 
>   ec9b269f61cc ("iio: adc: qcom-vadc: Move symbol exports to IIO_QCOM_VADC namespace")

This became a build failure in the arm64 defconfig build, so I have
reverted that commit for today.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the iio tree

[Jonathan Cameron]

On Fri, 1 Jul 2022 18:28:49 +1000
Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi all,
> 
> On Fri, 1 Jul 2022 15:29:44 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> \> 
> > After merging the iio tree, today's linux-next build (x86_64 allmodconfig)
> > produced this warning:
> > 
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc_tm5_temp_volt_scale from namespace IIO_QCOM_VADC, but does not import it.
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_prescaling_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_hw_settle_time_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_avg_samples_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_decimation_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> > 
> > Introduced by commit
> > 
> >   ec9b269f61cc ("iio: adc: qcom-vadc: Move symbol exports to IIO_QCOM_VADC namespace")  
> 
> This became a build failure in the arm64 defconfig build, so I have
> reverted that commit for today.
> 

Sorry about that. I'm guessing some commits crossed and it didn't show
in my local build tests or 0-day. Will fix it up later today.

Thanks,

Jonathan

Re: linux-next: build warning after merge of the iio tree

[Jonathan Cameron]

On Fri, 1 Jul 2022 18:28:49 +1000
Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi all,
> 
> On Fri, 1 Jul 2022 15:29:44 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> \> 
> > After merging the iio tree, today's linux-next build (x86_64 allmodconfig)
> > produced this warning:
> > 
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc_tm5_temp_volt_scale from namespace IIO_QCOM_VADC, but does not import it.
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_prescaling_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_hw_settle_time_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_avg_samples_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> > WARNING: modpost: module qcom-spmi-adc-tm5 uses symbol qcom_adc5_decimation_from_dt from namespace IIO_QCOM_VADC, but does not import it.
> > 
> > Introduced by commit
> > 
> >   ec9b269f61cc ("iio: adc: qcom-vadc: Move symbol exports to IIO_QCOM_VADC namespace")  
> 
> This became a build failure in the arm64 defconfig build, so I have
> reverted that commit for today.
> 

As the original patch didn't have any thermal related parts / review, I've dropped
it for now rather than fixing up.  Will circle round with a new version in due time.

Thanks,

Jonathan

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 301 bytes --]

Hi all,

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

Documentation/iio/adis16475.rst:317: ERROR: Unexpected indentation.

Introduced by commit

  38033fdf3b09 ("docs: iio: add documentation for adis16475 driver")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 1494 bytes --]

Hi all,

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

Documentation/iio/adis16480.rst:419: WARNING: Lexing literal_block 'root:/sys/bus/iio/devices/iio:device0> hexdump -C /dev/iio\\:device0\n...\n00006aa0  09 62 00 00 ff ff fc a4  00 00 01 69 00 03 3c 08  |.b.........i..<.|\n00006ab0  09 61 00 00 00 00 02 96  00 00 02 8f 00 03 37 50  |.a............7P|\n00006ac0  09 61 00 00 00 00 12 3d  00 00 0b 89 00 03 2c 0b  |.a.....=......,.|\n00006ad0  09 61 00 00 00 00 1e dc  00 00 16 dd 00 03 25 bf  |.a............%.|\n00006ae0  09 61 00 00 00 00 1e e3  00 00 1b bf 00 03 27 0b  |.a............\'.|\n00006af0  09 61 00 00 00 00 15 50  00 00 19 44 00 03 30 fd  |.a.....P...D..0.|\n00006b00  09 61 00 00 00 00 09 0e  00 00 14 41 00 03 3d 7f  |.a.........A..=.|\n00006b10  09 61 00 00 ff ff ff f0  00 00 0e bc 00 03 48 d0  |.a............H.|\n00006b20  09 63 00 00 00 00 00 9f  00 00 0f 37 00 03 4c fe  |.c.........7..L.|\n00006b30  09 64 00 00 00 00 0b f6  00 00 18 92 00 03 43 22  |.d............C"|\n00006b40  09 64 00 00 00 00 18 df  00 00 22 33 00 03 33 ab  |.d........"3..3.|\n00006b50  09 63 00 00 00 00 1e 81  00 00 26 be 00 03 29 60  |.c........&...)`|\n00006b60  09 63 00 00 00 00 1b 13  00 00 22 2f 00 03 23 91  |.c........"/..#.|\n...' as "bash" resulted in an error at token: "'". Retrying in relaxed mode.

Introduced by commit

  d6de8052f4a0 ("docs: iio: add documentation for adis16480 driver")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 416 bytes --]

Hi all,

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

Warning: /sys/bus/iio/devices/iio:deviceX/in_resistance_calibbias is defined 2 times:  Documentation/ABI/testing/sysfs-bus-iio-chemical-sgp40:18  Documentation/ABI/testing/sysfs-bus-iio:541

Introduced by commit

  0f718e10da81 ("iio: ABI: add missing calibbias attributes")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the iio tree

[Jonathan Cameron]

On Tue, 20 Aug 2024 13:47:39 +1000
Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi all,
> 
> After merging the iio tree, today's linux-next build (htmldocs) produced
> this warning:
> 
> Warning: /sys/bus/iio/devices/iio:deviceX/in_resistance_calibbias is defined 2 times:  Documentation/ABI/testing/sysfs-bus-iio-chemical-sgp40:18  Documentation/ABI/testing/sysfs-bus-iio:541
> 
> Introduced by commit
> 
>   0f718e10da81 ("iio: ABI: add missing calibbias attributes")
Thanks.

I think it's a case of just deleting the sgp40 version.
We will loose a bit of information, but I don't think it's critical
info to be worth a special chunk in the main docs.

Jonathan


> 

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 299 bytes --]

Hi all,

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

Documentation/iio/ad7625.rst: WARNING: document isn't included in any toctree

Introduced by commit

  b4a35432dbd0 ("docs: iio: new docs for ad7625 driver")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 379 bytes --]

Hi all,

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

include/linux/iio/iio.h:555: warning: Function parameter or struct member 'read_avail_release_resource' not described in 'iio_info'

Introduced by commit

  8a63e3033e72 ("iio: core: add read_avail_release_resource callback to fix race")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the iio tree

[Jonathan Cameron]

On Tue, 29 Oct 2024 15:50:22 +1100
Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi all,
> 
> After merging the iio tree, today's linux-next build (htmldocs) produced
> this warning:
> 
> include/linux/iio/iio.h:555: warning: Function parameter or struct member 'read_avail_release_resource' not described in 'iio_info'
> 
> Introduced by commit
> 
>   8a63e3033e72 ("iio: core: add read_avail_release_resource callback to fix race")
> 

Thanks.  Sorry for number of times I've missed things this cycle!

Will fix this up when on right machine.  Matteo, feel free to send a
patch if you have time, if not I'll make something suitable up.

Jonathan

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 434 bytes --]

Hi all,

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

include/linux/iio/iio.h:628: warning: Function parameter or struct member '__private' not described in 'iio_dev'
include/linux/iio/iio.h:628: warning: Excess struct member 'priv' description in 'iio_dev'

Introduced by commit

  9a5a2483bc60 ("iio: Mark iio_dev::priv member with __private")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the iio tree

[Jonathan Cameron]

On Thu, 7 Nov 2024 22:40:12 +1100
Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> Hi all,
> 
> After merging the iio tree, today's linux-next build (htmldocs) produced
> this warning:
> 
> include/linux/iio/iio.h:628: warning: Function parameter or struct member '__private' not described in 'iio_dev'
> include/linux/iio/iio.h:628: warning: Excess struct member 'priv' description in 'iio_dev'
> 
> Introduced by commit
> 
>   9a5a2483bc60 ("iio: Mark iio_dev::priv member with __private")
> 

I guess we need to teach the kernel-doc script about __private.
May be just a case of doing the same as done for __aligned, __packed etc.

Also curious why I've not seen warnings in the past about the masklength field
in the same structure.  I guess the test coverage has improved (or I missed
the reports!).

 So a 
   $members =~ s/\s*__private\s*\([^;]*\)/ /gos;

about here:
https://elixir.free-electrons.com/linux/v6.11.6/source/scripts/kernel-doc#L1148

Jon, does that make sense to you?  

Completely untested as I'm on wrong computer at the moment and my approach
to these scripts is cut and paste a similar smelling line until it works ;)

Jonathan

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 981 bytes --]

Hi all,

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

/home/sfr/next/next/Documentation/iio/ad3552r.rst:55: ERROR: Malformed table.

+-----------------------+------------------------------------------------------+
| Debugfs device files  | Description                                          |
+-----------------------+------------------------------------------------------+
| data_source           | The used data source, as                             |
|                       | ``normal``, ``ramp-16bit``, etc.                        |
+-----------------------+------------------------------------------------------+
| data_source_available | The available data sources.                          |
+-----------------------+------------------------------------------------------+ [docutils]

Introduced by commit

  9a259b51e3ea ("docs: iio: add documentation for ad3552r driver")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the iio tree

[Angelo Dureghello]

Hi Stephen,

thanks, fixing that.

On 15.04.2025 16:28, Stephen Rothwell wrote:
> Hi all,
> 
> After merging the iio tree, today's linux-next build (htmldocs) produced
> this warning:
> 
> /home/sfr/next/next/Documentation/iio/ad3552r.rst:55: ERROR: Malformed table.
> 
> +-----------------------+------------------------------------------------------+
> | Debugfs device files  | Description                                          |
> +-----------------------+------------------------------------------------------+
> | data_source           | The used data source, as                             |
> |                       | ``normal``, ``ramp-16bit``, etc.                        |
> +-----------------------+------------------------------------------------------+
> | data_source_available | The available data sources.                          |
> +-----------------------+------------------------------------------------------+ [docutils]
> 
> Introduced by commit
> 
>   9a259b51e3ea ("docs: iio: add documentation for ad3552r driver")
> 
> -- 
> Cheers,
> Stephen Rothwell


Regards,
angelo

linux-next: build warning after merge of the iio tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 308 bytes --]

Hi all,

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

Documentation/iio/adxl345.rst:161: ERROR: Unexpected indentation. [docutils]

Introduced by commit

  fdcb9cb9178a ("docs: iio: add documentation for adxl345 driver")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]