[PATCH v2 3/8] sysinfo: Add sysinfo driver and data structure for smbios
Raymond Mao
raymond.mao at linaro.org
Thu Dec 5 18:28:33 CET 2024
Hi Simon,
On Mon, 28 Oct 2024 at 13:04, Simon Glass <sjg at chromium.org> wrote:
> Hi Raymond,
>
> On Tue, 22 Oct 2024 at 22:06, Raymond Mao <raymond.mao at linaro.org> wrote:
> >
> > Add sysinfo driver to retrieve smbios information (Type 4 and 7).
> > So that the smbios library can use it for getting values from the
> > hardware platform instead of device tree.
> >
> > Signed-off-by: Raymond Mao <raymond.mao at linaro.org>
> > ---
> > Changes in v2
> > - Move the changes to smbios.c instead of creating new file.
> > - Move the headfile to include dir.
> > - Combine with #6(v1) patch.
> > - Clean-up the private data structures.
> > - Clean-up the operations of the strings and common values.
> >
> > drivers/sysinfo/smbios.c | 228 +++++++++++++++++++++++++++++++++++++++
> > include/smbios.h | 60 +++++++++++
> > include/smbios_plat.h | 79 ++++++++++++++
> > include/sysinfo.h | 95 +++++++++++++++-
> > 4 files changed, 461 insertions(+), 1 deletion(-)
> > create mode 100644 include/smbios_plat.h
> >
> > diff --git a/drivers/sysinfo/smbios.c b/drivers/sysinfo/smbios.c
> > index a7ac8e3f072..3980845b3ba 100644
> > --- a/drivers/sysinfo/smbios.c
> > +++ b/drivers/sysinfo/smbios.c
> > @@ -5,14 +5,240 @@
> > */
> >
> > #include <dm.h>
> > +#include <smbios_plat.h>
> > #include <sysinfo.h>
> >
> > +/* platform information storage */
> > +struct processor_info processor_info;
> > +struct cache_info cache_info[SYSINFO_CACHE_LVL_MAX];
> > +struct sysinfo_plat sysinfo_smbios_p = {
> > + /* Processor Information */
> > + .processor = &processor_info,
> > + /* Cache Information */
> > + .cache = &cache_info[0],
> > +};
> > +
> > +/* structure for smbios private data storage */
> > +struct sysinfo_plat_priv {
> > + struct processor_info *t4;
> > + struct smbios_type7 t7[SYSINFO_CACHE_LVL_MAX];
> > + u16 cache_handles[SYSINFO_CACHE_LVL_MAX];
> > + u8 cache_level;
> > +};
> > +
> > +static void smbios_cache_info_dump(struct smbios_type7 *cache_info)
> > +{
> > + log_debug("SMBIOS Type 7 (Cache Information):\n");
> > + log_debug("Cache Configuration: 0x%04x\n",
> cache_info->config.data);
> > + log_debug("Maximum Cache Size: %u KB\n",
> cache_info->max_size.data);
> > + log_debug("Installed Size: %u KB\n", cache_info->inst_size.data);
> > + log_debug("Supported SRAM Type: 0x%04x\n",
>
> %#04x
>
> > + cache_info->supp_sram_type.data);
> > + log_debug("Current SRAM Type: 0x%04x\n",
> > + cache_info->curr_sram_type.data);
> > + log_debug("Cache Speed: %u\n", cache_info->speed);
> > + log_debug("Error Correction Type: %u\n",
> cache_info->err_corr_type);
> > + log_debug("System Cache Type: %u\n", cache_info->sys_cache_type);
> > + log_debug("Associativity: %u\n", cache_info->associativity);
> > + log_debug("Maximum Cache Size 2: %u KB\n",
> cache_info->max_size2.data);
> > + log_debug("Installed Cache Size 2: %u KB\n",
> > + cache_info->inst_size2.data);
> > +}
> > +
> > +/* weak function for the platforms not yet supported */
> > +__weak int sysinfo_get_cache_info(u8 level, struct cache_info
> *cache_info)
> > +{
> > + return -ENOSYS;
> > +}
> > +
> > +__weak int sysinfo_get_processor_info(struct processor_info *pinfo)
> > +{
> > + return -ENOSYS;
> > +}
> > +
> > +void sysinfo_cache_info_default(struct cache_info *ci)
> > +{
> > + memset(ci, 0, sizeof(*ci));
> > + ci->config.data = SMBIOS_CACHE_LOCATE_UNKNOWN |
> SMBIOS_CACHE_OP_UND;
> > + ci->supp_sram_type.fields.unknown = 1;
> > + ci->curr_sram_type.fields.unknown = 1;
> > + ci->speed = SMBIOS_CACHE_SPEED_UNKNOWN;
> > + ci->err_corr_type = SMBIOS_CACHE_ERRCORR_UNKNOWN;
> > + ci->cache_type = SMBIOS_CACHE_SYSCACHE_TYPE_UNKNOWN;
> > +}
> > +
> > +static int sysinfo_plat_detect(struct udevice *dev)
> > +{
> > + return 0;
> > +}
> > +
> > +static int sysinfo_plat_get_str(struct udevice *dev, int id,
> > + size_t size, char *val)
> > +{
> > + struct sysinfo_plat_priv *priv = dev_get_priv(dev);
> > + const char *str = NULL;
> > +
> > + switch (id) {
> > + case SYSINFO_ID_SMBIOS_PROCESSOR_MANUFACT:
>
> These are getting too long.
>
> How about SYSINFOSM_PROC_MANUF ?
>
> We can use SYSINFOSM as short for SYSINFO_ID_SMBIOS
>
> > + str = priv->t4->manufacturer;
> > + break;
> > + default:
> > + break;
> > + }
> > +
> > + if (!str)
> > + return -ENOSYS;
> > +
> > + strlcpy(val, str, size);
> > +
> > + return 0;
> > +}
> > +
> > +static int sysinfo_plat_get_int(struct udevice *dev, int id, int *val)
> > +{
> > + struct sysinfo_plat_priv *priv = dev_get_priv(dev);
> > + u8 i;
> > +
> > + if (id >= SYSINFO_ID_SMBIOS_CACHE_INFO_START &&
> > + id <= SYSINFO_ID_SMBIOS_CACHE_INFO_END) {
> > + /* For smbios type 7 */
> > + for (i = 0; i < priv->cache_level; i++) {
> > + switch (id - i) {
> > + case SYSINFO_ID_SMBIOS_CACHE_MAX_SIZE:
> > + *val = priv->t7[i].max_size.data;
> > + return 0;
> > + case SYSINFO_ID_SMBIOS_CACHE_INST_SIZE:
> > + *val = priv->t7[i].inst_size.data;
> > + return 0;
> > + case SYSINFO_ID_SMBIOS_CACHE_SCACHE_TYPE:
> > + *val = priv->t7[i].sys_cache_type;
> > + return 0;
> > + case SYSINFO_ID_SMBIOS_CACHE_ASSOC:
> > + *val = priv->t7[i].associativity;
> > + return 0;
> > + case SYSINFO_ID_SMBIOS_CACHE_MAX_SIZE2:
> > + *val = priv->t7[i].max_size2.data;
> > + return 0;
> > + case SYSINFO_ID_SMBIOS_CACHE_INST_SIZE2:
> > + *val = priv->t7[i].inst_size2.data;
> > + return 0;
> > + default:
> > + break;
> > + }
> > + }
> > + return -ENOSYS;
> > + }
> > +
> > + switch (id) {
> > + case SYSINFO_ID_SMBIOS_PROCESSOR_CORE_CNT:
> > + *val = priv->t4->core_count;
> > + break;
> > + case SYSINFO_ID_SMBIOS_PROCESSOR_CORE_EN:
> > + *val = priv->t4->core_enabled;
> > + break;
> > + case SYSINFO_ID_SMBIOS_PROCESSOR_CHARA:
> > + *val = priv->t4->characteristics;
> > + break;
> > + case SYSINFO_ID_SMBIOS_CACHE_LEVEL:
> > + if (!priv->cache_level) /* No cache detected */
> > + return -ENOSYS;
> > + *val = priv->cache_level - 1;
> > + break;
> > + default:
> > + return -ENOSYS;
> > + }
> > +
> > + return 0;
> > +}
> > +
> > +static int sysinfo_plat_get_data(struct udevice *dev, int id, uchar
> **buf,
>
> How about void **, so we don't need to cast below?
>
> > + size_t *size)
> > +{
> > + struct sysinfo_plat_priv *priv = dev_get_priv(dev);
> > +
> > + switch (id) {
> > + case SYSINFO_ID_SMBIOS_PROCESSOR_ID:
> > + *buf = (uchar *)priv->t4->id;
> > + *size = sizeof(priv->t4->id);
> > + break;
> > + case SYSINFO_ID_SMBIOS_CACHE_HANDLE:
> > + *buf = (uchar *)(&priv->cache_handles[0]);
>
> Isn't that the same as:
>
> *buf = (uchar *)&priv->cache_handles;
>
> > + *size = sizeof(priv->cache_handles);
> > + break;
> > + default:
> > + return -EOPNOTSUPP;
> > + }
> > + return 0;
> > +}
> > +
> > +static int sysinfo_plat_probe(struct udevice *dev)
> > +{
> > + struct sysinfo_plat_priv *priv = dev_get_priv(dev);
> > + struct sysinfo_plat *plat = &sysinfo_smbios_p;
> > + u8 level;
> > +
> > + if (!sysinfo_get_processor_info(plat->processor))
> > + priv->t4 = plat->processor;
> > +
> > + for (level = 0; level < SYSINFO_CACHE_LVL_MAX; level++) {
> > + struct cache_info *pcache = plat->cache + level;
> > +
> > + if (sysinfo_get_cache_info(level, pcache))
> > + break; /* no more levels */
> > +
> > + /*
> > + * Fill in the SMBIOS type 7 structure,
> > + * skip the header members (type, length, handle),
> > + * and the ones in DT smbios node.
> > + */
> > + priv->t7[level].sys_cache_type = pcache->cache_type;
> > + priv->t7[level].associativity = pcache->associativity;
> > +
> > + if (pcache->max_size > SMBIOS_CACHE_SIZE_EXT_KB) {
> > + priv->t7[level].max_size.data = 0xFFFF;
> > + priv->t7[level].max_size2.fields.size =
> > + pcache->max_size / 64;
> > + priv->t7[level].max_size2.fields.granu =
> > + SMBIOS_CACHE_GRANU_64K;
> > + } else {
> > + priv->t7[level].max_size.fields.size =
> pcache->max_size;
> > + priv->t7[level].max_size.fields.granu =
> > + SMBIOS_CACHE_GRANU_1K;
> > + priv->t7[level].max_size2.data = 0;
> > + }
> > + if (pcache->inst_size > SMBIOS_CACHE_SIZE_EXT_KB) {
> > + priv->t7[level].inst_size.data = 0xFFFF;
> > + priv->t7[level].inst_size2.fields.size =
> > + pcache->inst_size / 64;
> > + priv->t7[level].inst_size2.fields.granu =
> > + SMBIOS_CACHE_GRANU_64K;
> > + } else {
> > + priv->t7[level].inst_size.fields.size =
> > + pcache->inst_size;
> > + priv->t7[level].inst_size.fields.granu =
> > + SMBIOS_CACHE_GRANU_1K;
> > + priv->t7[level].inst_size2.data = 0;
> > + }
> > + smbios_cache_info_dump(&priv->t7[level]);
> > + }
> > + if (!level) /* no cache detected */
> > + return -ENOSYS;
> > +
> > + priv->cache_level = level;
> > +
> > + return 0;
> > +}
> > +
> > static const struct udevice_id sysinfo_smbios_ids[] = {
> > { .compatible = "u-boot,sysinfo-smbios" },
> > { /* sentinel */ }
> > };
> >
> > static const struct sysinfo_ops sysinfo_smbios_ops = {
> > + .detect = sysinfo_plat_detect,
> > + .get_str = sysinfo_plat_get_str,
> > + .get_int = sysinfo_plat_get_int,
> > + .get_data = sysinfo_plat_get_data,
> > };
> >
> > U_BOOT_DRIVER(sysinfo_smbios) = {
> > @@ -20,4 +246,6 @@ U_BOOT_DRIVER(sysinfo_smbios) = {
> > .id = UCLASS_SYSINFO,
> > .of_match = sysinfo_smbios_ids,
> > .ops = &sysinfo_smbios_ops,
> > + .priv_auto = sizeof(struct sysinfo_plat_priv),
> > + .probe = sysinfo_plat_probe,
> > };
> > diff --git a/include/smbios.h b/include/smbios.h
> > index 78fd14d881b..cb4b3e08b3a 100644
> > --- a/include/smbios.h
> > +++ b/include/smbios.h
> > @@ -187,6 +187,66 @@ struct __packed smbios_type4 {
> > char eos[SMBIOS_STRUCT_EOS_BYTES];
> > };
> >
> > +union cache_config {
> > + struct {
> > + u16 level:3;
> > + u16 bsocketed:1;
> > + u16 rsvd0:1;
> > + u16 locate:2;
> > + u16 benabled:1;
> > + u16 opmode:2;
> > + u16 rsvd1:6;
> > + } fields;
> > + u16 data;
> > +};
> > +
> > +union cache_size_word {
> > + struct {
> > + u16 size:15;
> > + u16 granu:1;
> > + } fields;
> > + u16 data;
> > +};
> > +
> > +union cache_size_dword {
> > + struct {
> > + u32 size:31;
> > + u32 granu:1;
> > + } fields;
> > + u32 data;
> > +};
> > +
> > +union cache_sram_type {
> > + struct {
> > + u16 other:1;
> > + u16 unknown:1;
> > + u16 nonburst:1;
> > + u16 burst:1;
> > + u16 plburst:1;
> > + u16 sync:1;
> > + u16 async:1;
> > + u16 rsvd:9;
> > + } fields;
> > + u16 data;
> > +};
> > +
> > +struct __packed smbios_type7 {
> > + struct smbios_header hdr;
> > + u8 socket_design;
> > + union cache_config config;
> > + union cache_size_word max_size;
> > + union cache_size_word inst_size;
> > + union cache_sram_type supp_sram_type;
> > + union cache_sram_type curr_sram_type;
> > + u8 speed;
> > + u8 err_corr_type;
> > + u8 sys_cache_type;
> > + u8 associativity;
> > + union cache_size_dword max_size2;
> > + union cache_size_dword inst_size2;
> > + char eos[SMBIOS_STRUCT_EOS_BYTES];
> > +};
> > +
> > struct __packed smbios_type32 {
> > u8 type;
> > u8 length;
> > diff --git a/include/smbios_plat.h b/include/smbios_plat.h
> > new file mode 100644
> > index 00000000000..70089d5a2ba
> > --- /dev/null
> > +++ b/include/smbios_plat.h
> > @@ -0,0 +1,79 @@
> > +/* SPDX-License-Identifier: GPL-2.0+ */
> > +/*
> > + * Copyright (c) 2024 Linaro Limited
> > + * Author: Raymond Mao <raymond.mao at linaro.org>
> > + */
> > +#ifndef __SMBIOS_PLAT_H
> > +#define __SMBIOS_PLAT_H
> > +
> > +#include <smbios.h>
> > +
> > +struct cache_info {
> > + union cache_config config;
> > + union cache_sram_type supp_sram_type;
> > + union cache_sram_type curr_sram_type;
> > + u32 line_size;
> > + u32 associativity;
> > + u32 max_size;
> > + u32 inst_size;
> > + u8 cache_type;
> > + u8 speed;
> > + u8 err_corr_type;
> > + char *socket_design;
> > +};
> > +
> > +struct processor_info {
> > + u32 id[2];
> > + u16 ext_clock;
> > + u16 max_speed;
> > + u16 curr_speed;
> > + u16 characteristics;
> > + u16 family2;
> > + u16 core_count2;
> > + u16 core_enabled2;
> > + u16 thread_count2;
> > + u16 thread_enabled;
> > + u8 type;
> > + u8 family;
> > + u8 voltage;
> > + u8 status;
> > + u8 upgrade;
> > + u8 core_count;
> > + u8 core_enabled;
> > + u8 thread_count;
> > + char *socket_design;
> > + char *manufacturer;
> > + char *version;
> > + char *sn;
> > + char *asset_tag;
> > + char *pn;
> > +};
> > +
> > +struct sysinfo_plat {
> > + struct processor_info *processor;
> > + struct cache_info *cache;
> > + /* add other sysinfo structure here */
> > +};
> > +
> > +#if defined CONFIG_SYSINFO_SMBIOS
> > +int sysinfo_get_cache_info(u8 level, struct cache_info *cache_info);
> > +void sysinfo_cache_info_default(struct cache_info *ci);
> > +int sysinfo_get_processor_info(struct processor_info *pinfo);
> > +#else
> > +static inline int sysinfo_get_cache_info(u8 level,
> > + struct cache_info *cache_info)
> > +{
> > + return -ENOSYS;
> > +}
> > +
> > +static inline void sysinfo_cache_info_default(struct cache_info *ci)
> > +{
> > +}
> > +
> > +static inline int sysinfo_get_processor_info(struct processor_info
> *pinfo)
> > +{
> > + return -ENOSYS;
> > +}
> > +#endif
> > +
> > +#endif /* __SMBIOS_PLAT_H */
> > diff --git a/include/sysinfo.h b/include/sysinfo.h
> > index 17b2b9c7111..cb08a691270 100644
> > --- a/include/sysinfo.h
> > +++ b/include/sysinfo.h
> > @@ -11,6 +11,8 @@
> >
> > struct udevice;
> >
> > +#define SYSINFO_CACHE_LVL_MAX 3
> > +
> > /*
> > * This uclass encapsulates hardware methods to gather information
> about a
> > * sysinfo or a specific device such as hard-wired GPIOs on GPIO
> expanders,
> > @@ -42,18 +44,109 @@ struct udevice;
> > enum sysinfo_id {
> > SYSINFO_ID_NONE,
> >
> > - /* For SMBIOS tables */
> > + /* BIOS Information (Type 0) */
> > + SYSINFO_ID_SMBIOS_BIOS_VENDOR,
> > + SYSINFO_ID_SMBIOS_BIOS_VER,
> > + SYSINFO_ID_SMBIOS_BIOS_REL_DATE,
> > +
> > + /* System Information (Type 1) */
> > SYSINFO_ID_SMBIOS_SYSTEM_MANUFACTURER,
> > SYSINFO_ID_SMBIOS_SYSTEM_PRODUCT,
> > SYSINFO_ID_SMBIOS_SYSTEM_VERSION,
> > SYSINFO_ID_SMBIOS_SYSTEM_SERIAL,
> > + SYSINFO_ID_SMBIOS_SYSTEM_WAKEUP,
> > SYSINFO_ID_SMBIOS_SYSTEM_SKU,
> > SYSINFO_ID_SMBIOS_SYSTEM_FAMILY,
> > +
> > + /* Baseboard (or Module) Information (Type 2) */
> > SYSINFO_ID_SMBIOS_BASEBOARD_MANUFACTURER,
> > SYSINFO_ID_SMBIOS_BASEBOARD_PRODUCT,
> > SYSINFO_ID_SMBIOS_BASEBOARD_VERSION,
> > SYSINFO_ID_SMBIOS_BASEBOARD_SERIAL,
> > SYSINFO_ID_SMBIOS_BASEBOARD_ASSET_TAG,
> > + SYSINFO_ID_SMBIOS_BASEBOARD_FEATURE,
> > + SYSINFO_ID_SMBIOS_BASEBOARD_CHASSIS_LOCAT,
> > + SYSINFO_ID_SMBIOS_BASEBOARD_TYPE,
> > + SYSINFO_ID_SMBIOS_BASEBOARD_OBJS_NUM,
> > + SYSINFO_ID_SMBIOS_BASEBOARD_OBJS_HANDLE,
> > +
> > + /* System Enclosure or Chassis (Type 3) */
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_MANUFACTURER,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_VERSION,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_SERIAL,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_ASSET_TAG,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_TYPE,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_BOOTUP,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_POW,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_THERMAL,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_SECURITY,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_OEM,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_HEIGHT,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_POWCORE_NUM,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_ELEMENT_CNT,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_ELEMENT_LEN,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_ELEMENTS,
> > + SYSINFO_ID_SMBIOS_ENCLOSURE_SKU,
> > +
> > + /* Processor Information (Type 4) */
> > + SYSINFO_ID_SMBIOS_PROCESSOR_SOCKET,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_TYPE,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_MANUFACT,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_ID,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_VERSION,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_VOLTAGE,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_EXT_CLOCK,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_MAX_SPEED,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_CUR_SPEED,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_STATUS,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_UPGRADE,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_SN,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_ASSET_TAG,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_PN,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_CORE_CNT,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_CORE_EN,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_THREAD_CNT,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_CHARA,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_FAMILY,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_FAMILY2,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_CORE_CNT2,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_CORE_EN2,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_THREAD_CNT2,
> > + SYSINFO_ID_SMBIOS_PROCESSOR_THREAD_EN,
> > +
> > + /*
> > + * Cache Information (Type 7)
> > + * Each of the id should reserve space for up to
> > + * SYSINFO_CACHE_LVL_MAX levels of cache
> > + */
> > + SYSINFO_ID_SMBIOS_CACHE_LEVEL,
> > + SYSINFO_ID_SMBIOS_CACHE_HANDLE,
> > + SYSINFO_ID_SMBIOS_CACHE_INFO_START,
> > + SYSINFO_ID_SMBIOS_CACHE_SOCKET =
> SYSINFO_ID_SMBIOS_CACHE_INFO_START,
> > + SYSINFO_ID_SMBIOS_CACHE_CONFIG =
> > + SYSINFO_ID_SMBIOS_CACHE_SOCKET + SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_MAX_SIZE =
> > + SYSINFO_ID_SMBIOS_CACHE_CONFIG + SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_INST_SIZE =
> > + SYSINFO_ID_SMBIOS_CACHE_MAX_SIZE + SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_SUPSRAM_TYPE =
> > + SYSINFO_ID_SMBIOS_CACHE_INST_SIZE +
> SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_CURSRAM_TYPE =
> > + SYSINFO_ID_SMBIOS_CACHE_SUPSRAM_TYPE +
> SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_SPEED =
> > + SYSINFO_ID_SMBIOS_CACHE_CURSRAM_TYPE +
> SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_ERRCOR_TYPE =
> > + SYSINFO_ID_SMBIOS_CACHE_SPEED + SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_SCACHE_TYPE =
> > + SYSINFO_ID_SMBIOS_CACHE_ERRCOR_TYPE +
> SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_ASSOC =
> > + SYSINFO_ID_SMBIOS_CACHE_SCACHE_TYPE +
> SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_MAX_SIZE2 =
> > + SYSINFO_ID_SMBIOS_CACHE_ASSOC + SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_INST_SIZE2 =
> > + SYSINFO_ID_SMBIOS_CACHE_MAX_SIZE2 +
> SYSINFO_CACHE_LVL_MAX,
> > + SYSINFO_ID_SMBIOS_CACHE_INFO_END =
> > + SYSINFO_ID_SMBIOS_CACHE_INST_SIZE2 +
> SYSINFO_CACHE_LVL_MAX - 1,
> >
>
> This seems to be allocating sequential values for each cache? Instead, we
> should add a 'seq' parameter to get_data()
>
> Sorry for not responding to this comment, actually I would prefer to use
get_str or get_val as it allows
to check whether values from sysinfo driver are missing and fallback to the
value from dt node one by one.
[snip]
Regards,
Raymond
More information about the U-Boot
mailing list