[PATCH v5 1/6] sysinfo: Add IDs for board id and revision

Heinrich Schuchardt xypron.glpk at gmx.de
Mon Oct 16 19:37:40 CEST 2023


On 10/2/23 17:20, Detlev Casanova wrote:
> These IDs will be used by the sysinfo command. The new IDs are:
>   * SYSINFO_ID_BOARD_ID: The board ID as an integer
>   * SYSINFO_ID_BOARD_REV_MAJOR: The board major revision as int
>   * SYSINFO_ID_BOARD_REV_MINOR: The board minor revision as int

Integers will not work in the generic case. E.g. for the VisionFive 2
board we have seen board revisions:

1.2A, 1.2B, 1.3B

For the Wandboard:

B1, D1

>
> Reviewed-by: Marek Vasut <marek.vasut+renesas at mailbox.org>
> Signed-off-by: Detlev Casanova <detlev.casanova at collabora.com>
> ---
>   include/sysinfo.h | 5 +++++
>   1 file changed, 5 insertions(+)
>
> diff --git a/include/sysinfo.h b/include/sysinfo.h
> index b140d742e93..13815600ae6 100644
> --- a/include/sysinfo.h
> +++ b/include/sysinfo.h
> @@ -47,6 +47,11 @@ enum sysinfo_id {
>   	/* For show_board_info() */
>   	SYSINFO_ID_BOARD_MODEL,
>
> +	/* For sysinfo command (all int) */

Please, provide a Sphinx style documentation for every value. Cf.
https://www.kernel.org/doc/html/v5.3/doc-guide/kernel-doc.html#structure-union-and-enumeration-documentation

Best regards

Heinrich

> +	SYSINFO_ID_BOARD_ID,
> +	SYSINFO_ID_BOARD_REV_MAJOR,
> +	SYSINFO_ID_BOARD_REV_MINOR,
> +
>   	/* First value available for downstream/board used */
>   	SYSINFO_ID_USER = 0x1000,
>   };



More information about the U-Boot mailing list