Re: [PATCH v2] Documentation: Update after split of <linux/mod_devicetable.h>

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The kernel-doc for the ACPI_DEVICE_CLASS macro is lost because include/linux/device-id/acpi.h is omitted from the new explicit header list in basics.rst.
--

commit c40d3b75cc5cb1e55623d0db318301a1e8676b9b
Author: Uwe Kleine-König (The Capable Hub) <u.kleine-koenig@baylibre.com>

Documentation: Update after split of <linux/mod_devicetable.h>

Most structs that were defined in include/linux/mod_devicetable.h before were
moved to a bus specific header. This commit follows the movement to get the
documentation back by using an explicit list of per-subsystem headers.

> diff --git a/Documentation/driver-api/basics.rst b/Documentation/driver-api/basics.rst
> index 8b6a5888cb11d..647e14254cfb8 100644
> --- a/Documentation/driver-api/basics.rst
> +++ b/Documentation/driver-api/basics.rst
> @@ -10,9 +10,36 @@ Driver Entry and Exit points
>  Driver device table
>  -------------------
>  
> -.. kernel-doc:: include/linux/mod_devicetable.h
> +.. kernel-doc:: include/linux/device-id/amba.h
> +   :internal:

[Severity: Low]
Does this new explicit list of headers miss include/linux/device-id/acpi.h?

It appears that leaving it out drops the kernel-doc for the ACPI_DEVICE_CLASS
macro from the generated documentation.

> +.. kernel-doc:: include/linux/device-id/cdx.h
> +   :internal:

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/ede7ce64f20e10660ebf6bc1b41836dad750ae3b.1783349758.git.ukleinek@kernel.org?part=1