[PATCH v2 2/3] renesas: rcar3: Expose the board revision in sysinfo

[Marek Vasut]

On 6/12/23 21:51, Detlev Casanova wrote:
> The board revision is needed to determine which linux device tree to
> load.
> 
> Signed-off-by: Detlev Casanova <detlev.casanova at collabora.com>
> ---
>   .../arm/mach-rmobile/include/mach/sys_proto.h |  6 ++
>   drivers/sysinfo/rcar3.c                       | 60 +++++++++++--------
>   2 files changed, 40 insertions(+), 26 deletions(-)
> 
> diff --git a/arch/arm/mach-rmobile/include/mach/sys_proto.h b/arch/arm/mach-rmobile/include/mach/sys_proto.h
> index e020b24f7c2..1f204f06c17 100644
> --- a/arch/arm/mach-rmobile/include/mach/sys_proto.h
> +++ b/arch/arm/mach-rmobile/include/mach/sys_proto.h
> @@ -22,4 +22,10 @@
>   #define BOARD_EBISU_4D		0xD
>   #define BOARD_CONDOR_I		0x10
>   
> +/*
> + * Renesas sysinfo board revision
> + */
> +#define RCAR_SYSINFO_REV_MAJOR	SYSINFO_ID_USER
> +#define RCAR_SYSINFO_REV_MINOR	(SYSINFO_ID_USER + 1)

Please just introduce regular include/sysinfo.h board revision 
properties for this, not custom stuff.

>   #endif
> diff --git a/drivers/sysinfo/rcar3.c b/drivers/sysinfo/rcar3.c
> index b8b837341a2..3223875a99c 100644
> --- a/drivers/sysinfo/rcar3.c
> +++ b/drivers/sysinfo/rcar3.c
> @@ -22,6 +22,8 @@
>   struct sysinfo_rcar_priv {
>   	char	boardmodel[64];
>   	u8	board_id;
> +	u8	rev_major;
> +	u8	rev_minor;
>   	u8	val;
>   };

[...]