[PATCH] arm: dts: rockchip: rk3399: nanopi-r4s: Provide smbios sysinfo

[Simon Glass]

Hi Christian,

On Thu, 29 Sept 2022 at 08:30, Christian Kohlschütter
<christian at kohlschutter.com> wrote:
>
> > On 29. Sep 2022, at 04:36, Simon Glass <sjg at chromium.org> wrote:
> >
> > On Wed, 28 Sept 2022 at 18:15, Christian Kohlschütter
> > <christian at kohlschutter.com> wrote:
> >>
> >> Provide human-readable manufacturer and product names for the
> >> FriendlyELEC NanoPi R4S.
> >>
> >> Enable CONFIG_SYSINFO and CONFIG_SYSINFO_SMBIOS by default.
> >>
> >> Signed-off-by: Christian Kohlschütter <christian at kohlschutter.com>
> >> ---
> >> arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi | 22 ++++++++++++++++++++++
> >> configs/nanopi-r4s-rk3399_defconfig        |  2 ++
> >> 2 files changed, 24 insertions(+)
> >
> > Reviewed-by: Simon Glass <sjg at chromium.org>
> >
> >>
> >> diff --git a/arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi b/arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi
> >> index cd1642527b..69800cc368 100644
> >> --- a/arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi
> >> +++ b/arch/arm/dts/rk3399-nanopi-r4s-u-boot.dtsi
> >> @@ -14,3 +14,25 @@
> >>
> >> #include "rk3399-nanopi4-u-boot.dtsi"
> >> #include "rk3399-sdram-lpddr4-100.dtsi"
> >> +
> >> +/ {
> >> +       smbios {
> >> +               compatible = "u-boot,sysinfo-smbios";
> >> +
> >> +               smbios {
> >> +                       system {
> >> +                               manufacturer = "FriendlyELEC";
> >> +                               product = "NanoPi R4S";
> >> +                       };
> >> +
> >> +                       baseboard {
> >> +                               manufacturer = "FriendlyELEC";
> >> +                               product = "NanoPi R4S";
> >> +                       };
> >> +
> >> +                       chassis {
> >> +                               manufacturer = "FriendlyELEC";
> >> +                       };
> >> +               };
> >> +       };
> >> +};
> >> diff --git a/configs/nanopi-r4s-rk3399_defconfig b/configs/nanopi-r4s-rk3399_defconfig
> >> index dd2579d2e0..502f007e6e 100644
> >> --- a/configs/nanopi-r4s-rk3399_defconfig
> >> +++ b/configs/nanopi-r4s-rk3399_defconfig
> >> @@ -75,3 +75,5 @@ CONFIG_MISC=y
> >> CONFIG_MISC_INIT_R=y
> >> CONFIG_ROCKCHIP_EFUSE=y
> >> CONFIG_ROCKCHIP_OTP=y
> >> +CONFIG_SYSINFO=y
> >> +CONFIG_SYSINFO_SMBIOS=y
> >
> > Is the ordering right, here?
> >
> > Regards,
> > Simon
>
> Is there a tool to sort these values in a certain order?

$ echo nanopi-r4s-rk3399_defconfig >asc
$ ./tools/moveconfig.py -s -d asc

(it could use a new flag to just specific the board, if you want to
send a patch)

Should these not be in alphabetical order?

No it is in order of the Kconfig tree (use 'make menuconfig' or 'make
xconfig' to see it).

Regards,
Simon