On Fri, Apr 09, 2021 at 03:13:25PM +0200, Fabian Vogt wrote: > Am Freitag, 9. April 2021, 14:35:10 CEST schrieb matthias....@kernel.org: > > From: Matthias Brugger <mbrug...@suse.com> > > > > At present SMBIOS tables are emtpy, > ^ typo > > > which breaks some use-cases that rely on that. > Can you give an example?
Management software that want's to know what HW a node is based on. > > > Add some minimal information to > Isn't it possible to get the complete information from the firmware here? > That would allow to have the correct product and serial at least. That's the plan for the long run. > If that's not possible or too complex, then I suggest to use the proper > strings, e.g. "Raspberry Pi", as those are user visible. I'd like to stay backwards compatible to older U-Boot versions for now. > > > fullfill this. > ^ typo > > > Signed-off-by: Matthias Brugger <mbrug...@suse.com> > > --- > > > > arch/arm/dts/bcm283x-u-boot.dtsi | 20 ++++++++++++++++++++ > > 1 file changed, 20 insertions(+) > > > > diff --git a/arch/arm/dts/bcm283x-u-boot.dtsi > > b/arch/arm/dts/bcm283x-u-boot.dtsi > > index 68d03627f4..f5235cb083 100644 > > --- a/arch/arm/dts/bcm283x-u-boot.dtsi > > +++ b/arch/arm/dts/bcm283x-u-boot.dtsi > > @@ -6,6 +6,26 @@ > > * (C) Copyright 2016 Fabian Vogt <fv...@suse.com> > > */ > > > > +/ { > > + smbios { > > + compatible = "u-boot,sysinfo-smbios"; > > + smbios { > > + system { > > + manufacturer = "raspberrypi"; > > + product = "rpi"; > > + }; > > + baseboard { > > + manufacturer = "raspberrypi"; > > + product = "rpi"; > > + }; > > + chassis { > > + manufacturer = "raspberrypi"; > > + product = "rpi"; > > According to doc/device-tree-bindings/sysinfo/smbios.txt, "chassis" > doesn't have a "product". > You are correct. I'll fix the typos and this in the next version. Regards, Matthias > Cheers, > Fabian > > > + }; > > + }; > > + }; > > +}; > > + > > &uart0 { > > skip-init; > > u-boot,dm-pre-reloc; > > >