Hi Simon, On Mon, Jan 25, 2021 at 1:51 AM Simon Glass <s...@chromium.org> wrote: > > While static configuration is useful it cannot cover every case. Sometimes > board revisions are encoded in resistor straps and must be read at > runtime. > > The easiest way to provide this information is via sysinfo, since the > board can then provide a driver to read whatever is needed. > > Add some standard sysinfo options for this, and use them to obtain the > required information. > > Signed-off-by: Simon Glass <s...@chromium.org> >
I guess you used patman to automatically collect the Reviewed-by tag? Is this blank line inserted by patman? If yes, we should fix patman to avoid this. > Reviewed-by: Bin Meng <bmeng...@gmail.com> > --- > > Changes in v3: > - Use SMBIOS_STR_MAX for the max sysinfo string length > > include/sysinfo.h | 11 +++++++++++ > lib/smbios.c | 32 +++++++++++++++++++++++++++++--- > 2 files changed, 40 insertions(+), 3 deletions(-) > Regards, Bin