[PATCH 1/1] doc: add partition API to HTML documentation
Simon Glass
sjg at chromium.org
Tue Aug 15 16:44:25 CEST 2023
Hi Heinrich,
On Tue, 15 Aug 2023 at 04:32, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> * Convert comments in part.h to Sphinx style.
> * Create documentation page for the partition API.
> * Add the partition API page to the API index page.
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
> doc/api/index.rst | 1 +
> doc/api/part.rst | 6 ++
> include/part.h | 210 ++++++++++++++++++++++++----------------------
> 3 files changed, 119 insertions(+), 98 deletions(-)
> create mode 100644 doc/api/part.rst
Reviewed-by: Simon Glass <sjg at chromium.org>
Q below
>
> diff --git a/doc/api/index.rst b/doc/api/index.rst
> index a9338cfef9..3a80ae0635 100644
> --- a/doc/api/index.rst
> +++ b/doc/api/index.rst
> @@ -15,6 +15,7 @@ U-Boot API documentation
> lmb
> logging
> nvmem
> + part
> pinctrl
> rng
> sandbox
> diff --git a/doc/api/part.rst b/doc/api/part.rst
> new file mode 100644
> index 0000000000..d1df1d8494
> --- /dev/null
> +++ b/doc/api/part.rst
> @@ -0,0 +1,6 @@
> +.. SPDX-License-Identifier: GPL-2.0+
> +
> +Partition API
> +=============
> +[..]
> +/**
> + * struct part_driver - partition driver
> + */
> struct part_driver {
> + /** @name: partition name */
> const char *name;
> + /** @part_type: (MBR) partition type */
> int part_type;
> - const int max_entries; /* maximum number of entries to search */
> -
> + /** @max_entries: maximum number of partition table entries */
> + const int max_entries;
> /**
> - * get_info() - Get information about a partition
> + * @get_info: Get information about a partition
Are we not allowed the () at end? I much prefer brackets as they show
it is a function.
> *
> - * @dev_desc: Block device descriptor
> - * @part: Partition number (1 = first)
> - * @info: Returns partition information
> + * @get_info.dev_desc: Block device descriptor
> + * @get_info.part: Partition number (1 = first)
> + * @get_info.info: Returns partition information
> */
> int (*get_info)(struct blk_desc *dev_desc, int part,
> struct disk_partition *info);
[..]
Regards,
Simon
More information about the U-Boot
mailing list