* [PATCH v4 0/2] x86: Minor improvements mostly for image loading
@ 2021-02-06 16:13 Simon Glass
2021-02-06 16:13 ` [PATCH v4 1/2] sysinfo: Allow showing model info from sysinfo Simon Glass
2021-02-06 16:13 ` [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup Simon Glass
0 siblings, 2 replies; 7+ messages in thread
From: Simon Glass @ 2021-02-06 16:13 UTC (permalink / raw
To: u-boot
This series provides a few improvements for loading of images. It also
provides a way to show more detailed model information as well as an
of-platdata fix noticed recently.
Note that this series depends on the GPIO series here:
http://patchwork.ozlabs.org/project/uboot/list/?series=226118
Changes in v4:
- Rebase to x86/master (but will not build until GPIO series lands)
- Update cover letter
Changes in v3:
- Rebase to master
- Rebase to master
- Drop patches previously applied
Changes in v2:
- Fix two missing asterisks in comments
Simon Glass (2):
sysinfo: Allow showing model info from sysinfo
x86: coral: Show memory config and SKU ID on startup
arch/x86/dts/chromebook_coral.dts | 11 ++
board/google/chromebook_coral/coral.c | 139 +++++++++++++++++-
board/google/chromebook_coral/variant_gpio.h | 6 -
common/board_info.c | 37 ++++-
.../sysinfo/google,coral.txt | 37 +++++
include/sysinfo.h | 4 +
6 files changed, 212 insertions(+), 22 deletions(-)
create mode 100644 doc/device-tree-bindings/sysinfo/google,coral.txt
--
2.30.0.478.g8a0d178c01-goog
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v4 1/2] sysinfo: Allow showing model info from sysinfo
2021-02-06 16:13 [PATCH v4 0/2] x86: Minor improvements mostly for image loading Simon Glass
@ 2021-02-06 16:13 ` Simon Glass
2021-02-06 16:13 ` [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup Simon Glass
1 sibling, 0 replies; 7+ messages in thread
From: Simon Glass @ 2021-02-06 16:13 UTC (permalink / raw
To: u-boot
Some boards may want to show the SKU ID or other information obtained at
runtime. Allow this to come from sysinfo. The board can then provide a
sysinfo driver to provide it.
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
---
(no changes since v3)
Changes in v3:
- Rebase to master
common/board_info.c | 37 +++++++++++++++++++++++++++++--------
include/sysinfo.h | 4 ++++
2 files changed, 33 insertions(+), 8 deletions(-)
diff --git a/common/board_info.c b/common/board_info.c
index a6db087f960..20a2dea1f35 100644
--- a/common/board_info.c
+++ b/common/board_info.c
@@ -1,30 +1,51 @@
// SPDX-License-Identifier: GPL-2.0+
#include <common.h>
+#include <dm.h>
#include <init.h>
+#include <sysinfo.h>
#include <linux/libfdt.h>
#include <linux/compiler.h>
+DECLARE_GLOBAL_DATA_PTR;
+
int __weak checkboard(void)
{
return 0;
}
/*
- * If the root node of the DTB has a "model" property, show it.
+ * Check sysinfo for board information. Failing that if the root node of the DTB
+ * has a "model" property, show it.
+ *
* Then call checkboard().
*/
int __weak show_board_info(void)
{
-#ifdef CONFIG_OF_CONTROL
- DECLARE_GLOBAL_DATA_PTR;
- const char *model;
+ if (IS_ENABLED(CONFIG_OF_CONTROL)) {
+ struct udevice *dev;
+ const char *model;
+ char str[80];
+ int ret = -ENOSYS;
+
+ if (IS_ENABLED(CONFIG_SYSINFO)) {
+ /* This might provide more detail */
+ ret = uclass_first_device_err(UCLASS_SYSINFO, &dev);
+ if (!ret)
+ ret = sysinfo_get_str(dev,
+ SYSINFO_ID_BOARD_MODEL,
+ sizeof(str), str);
+ }
- model = fdt_getprop(gd->fdt_blob, 0, "model", NULL);
+ /* Fail back to the main 'model' if available */
+ if (ret)
+ model = fdt_getprop(gd->fdt_blob, 0, "model", NULL);
+ else
+ model = str;
- if (model)
- printf("Model: %s\n", model);
-#endif
+ if (model)
+ printf("Model: %s\n", model);
+ }
return checkboard();
}
diff --git a/include/sysinfo.h b/include/sysinfo.h
index 743f3554659..f4ffb1a3503 100644
--- a/include/sysinfo.h
+++ b/include/sysinfo.h
@@ -35,9 +35,13 @@
enum sysinfo_id {
SYSINFO_ID_NONE,
+ /* For SMBIOS tables */
SYSINFO_ID_SMBIOS_SYSTEM_VERSION,
SYSINFO_ID_SMBIOS_BASEBOARD_VERSION,
+ /* For show_board_info() */
+ SYSINFO_ID_BOARD_MODEL,
+
/* First value available for downstream/board used */
SYSINFO_ID_USER = 0x1000,
};
--
2.30.0.478.g8a0d178c01-goog
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup
2021-02-06 16:13 [PATCH v4 0/2] x86: Minor improvements mostly for image loading Simon Glass
2021-02-06 16:13 ` [PATCH v4 1/2] sysinfo: Allow showing model info from sysinfo Simon Glass
@ 2021-02-06 16:13 ` Simon Glass
2021-03-09 2:33 ` Bin Meng
1 sibling, 1 reply; 7+ messages in thread
From: Simon Glass @ 2021-02-06 16:13 UTC (permalink / raw
To: u-boot
Provide the model information through sysinfo so that it shows up on
boot. For memconfig 4 pins are provided, for 16 combinations. For SKU
ID there are two options:
- two pins provided in a ternary arrangement, for 9 combinations.
- reading from the EC
Add a binding doc and drop the unused #defines as well.
Example:
U-Boot 2021.01-rc5
CPU: Intel(R) Celeron(R) CPU N3450 @ 1.10GHz
DRAM: 3.9 GiB
MMC: sdmmc at 1b,0: 1, emmc at 1c,0: 2
Video: 1024x768x32 @ b0000000
Model: Google Coral (memconfig 5, SKU 3)
This depends on the GPIO series:
http://patchwork.ozlabs.org/project/uboot/list/?series=228126
Signed-off-by: Simon Glass <sjg@chromium.org>
Acked-by: Bin Meng <bmeng.cn@gmail.com>
---
Changes in v4:
- Rebase to x86/master (but will not build until GPIO series lands)
- Update cover letter
Changes in v3:
- Rebase to master
- Drop patches previously applied
Changes in v2:
- Fix two missing asterisks in comments
arch/x86/dts/chromebook_coral.dts | 11 ++
board/google/chromebook_coral/coral.c | 139 +++++++++++++++++-
board/google/chromebook_coral/variant_gpio.h | 6 -
.../sysinfo/google,coral.txt | 37 +++++
4 files changed, 179 insertions(+), 14 deletions(-)
create mode 100644 doc/device-tree-bindings/sysinfo/google,coral.txt
diff --git a/arch/x86/dts/chromebook_coral.dts b/arch/x86/dts/chromebook_coral.dts
index 2ffe3b423c3..18bbafe5981 100644
--- a/arch/x86/dts/chromebook_coral.dts
+++ b/arch/x86/dts/chromebook_coral.dts
@@ -55,6 +55,17 @@
recovery-gpios = <&gpio_nw (-1) GPIO_ACTIVE_LOW>;
write-protect-gpios = <&gpio_nw GPIO_75 GPIO_ACTIVE_HIGH>;
phase-enforce-gpios = <&gpio_n GPIO_10 GPIO_ACTIVE_HIGH>;
+ memconfig-gpios = <&gpio_nw GPIO_101 GPIO_ACTIVE_HIGH
+ &gpio_nw GPIO_102 GPIO_ACTIVE_HIGH
+ &gpio_n GPIO_38 GPIO_ACTIVE_HIGH
+ &gpio_n GPIO_45 GPIO_ACTIVE_HIGH>;
+
+ /*
+ * This is used for reef only:
+ *
+ * skuconfig-gpios = <&gpio_nw GPIO_16 GPIO_ACTIVE_HIGH
+ * &gpio_nw GPIO_17 GPIO_ACTIVE_HIGH>;
+ */
smbios {
/* Type 1 table */
system {
diff --git a/board/google/chromebook_coral/coral.c b/board/google/chromebook_coral/coral.c
index f9fb3f163f0..77a6cca4497 100644
--- a/board/google/chromebook_coral/coral.c
+++ b/board/google/chromebook_coral/coral.c
@@ -3,9 +3,12 @@
* Copyright 2019 Google LLC
*/
+#define LOG_CATEGORY UCLASS_SYSINFO
+
#include <common.h>
#include <bloblist.h>
#include <command.h>
+#include <cros_ec.h>
#include <dm.h>
#include <log.h>
#include <sysinfo.h>
@@ -15,6 +18,7 @@
#include <asm/intel_gnvs.h>
#include <asm/intel_pinctrl.h>
#include <dm/acpi.h>
+#include <linux/delay.h>
#include "variant_gpio.h"
struct cros_gpio_info {
@@ -29,10 +33,125 @@ int arch_misc_init(void)
return 0;
}
-/* This function is needed if CONFIG_CMDLINE is not enabled */
-int board_run_command(const char *cmdline)
+static int get_memconfig(struct udevice *dev)
{
- printf("No command line\n");
+ struct gpio_desc gpios[4];
+ int cfg;
+ int ret;
+
+ ret = gpio_request_list_by_name(dev, "memconfig-gpios", gpios,
+ ARRAY_SIZE(gpios),
+ GPIOD_IS_IN | GPIOD_PULL_UP);
+ if (ret < 0) {
+ log_debug("Cannot get GPIO list '%s' (%d)\n", dev->name, ret);
+ return ret;
+ }
+
+ /* Give the lines time to settle */
+ udelay(10);
+
+ ret = dm_gpio_get_values_as_int(gpios, ARRAY_SIZE(gpios));
+ if (ret < 0)
+ return log_msg_ret("get", ret);
+ cfg = ret;
+
+ ret = gpio_free_list(dev, gpios, ARRAY_SIZE(gpios));
+ if (ret)
+ return log_msg_ret("free", ret);
+
+ return cfg;
+}
+
+/**
+ * get_skuconfig() - Get the SKU number either from pins or the EC
+ *
+ * Two options are supported:
+ * skuconfig-gpios - two pins in the device tree (tried first)
+ * EC - reading from the EC (backup)
+ *
+ * @dev: sysinfo device to use
+ * @return SKU ID, or -ve error if not found
+ */
+static int get_skuconfig(struct udevice *dev)
+{
+ struct gpio_desc gpios[2];
+ int cfg;
+ int ret;
+
+ ret = gpio_request_list_by_name(dev, "skuconfig-gpios", gpios,
+ ARRAY_SIZE(gpios),
+ GPIOD_IS_IN);
+ if (ret != ARRAY_SIZE(gpios)) {
+ struct udevice *cros_ec;
+
+ log_debug("Cannot get GPIO list '%s' (%d)\n", dev->name, ret);
+
+ /* Try the EC */
+ ret = uclass_first_device_err(UCLASS_CROS_EC, &cros_ec);
+ if (ret < 0) {
+ log_err("Cannot find EC for SKU details\n");
+ return log_msg_ret("sku", ret);
+ }
+ ret = cros_ec_get_sku_id(cros_ec);
+ if (ret < 0) {
+ log_err("Cannot read SKU details\n");
+ return log_msg_ret("sku", ret);
+ }
+
+ return ret;
+ }
+
+ ret = dm_gpio_get_values_as_int_base3(gpios, ARRAY_SIZE(gpios));
+ if (ret < 0)
+ return log_msg_ret("get", ret);
+ cfg = ret;
+
+ ret = gpio_free_list(dev, gpios, ARRAY_SIZE(gpios));
+ if (ret)
+ return log_msg_ret("free", ret);
+
+ return cfg;
+}
+
+static int coral_get_str(struct udevice *dev, int id, size_t size, char *val)
+{
+ int ret;
+
+ if (IS_ENABLED(CONFIG_SPL_BUILD))
+ return -ENOSYS;
+
+ switch (id) {
+ case SYSINFO_ID_SMBIOS_SYSTEM_VERSION:
+ case SYSINFO_ID_SMBIOS_BASEBOARD_VERSION: {
+ ret = get_skuconfig(dev);
+
+ if (ret < 0)
+ return ret;
+ if (size < 15)
+ return -ENOSPC;
+ sprintf(val, "rev%d", ret);
+ break;
+ }
+ case SYSINFO_ID_BOARD_MODEL: {
+ int mem_config, sku_config;
+ const char *model;
+
+ ret = get_memconfig(dev);
+ if (ret < 0)
+ log_warning("Unable to read memconfig (err=%d)\n", ret);
+ mem_config = ret;
+ ret = get_skuconfig(dev);
+ if (ret < 0)
+ log_warning("Unable to read skuconfig (err=%d)\n", ret);
+ sku_config = ret;
+ model = fdt_getprop(gd->fdt_blob, 0, "model", NULL);
+ snprintf(val, size, "%s (memconfig %d, SKU %d)", model,
+ mem_config, sku_config);
+ break;
+ }
+ default:
+ return -ENOENT;
+ }
return 0;
}
@@ -45,12 +164,15 @@ int chromeos_get_gpio(const struct udevice *dev, const char *prop,
int ret;
ret = gpio_request_by_name((struct udevice *)dev, prop, 0, &desc, 0);
- if (ret == -ENOTBLK)
+ if (ret == -ENOTBLK) {
info->gpio_num = CROS_GPIO_VIRTUAL;
- else if (ret)
+ log_debug("GPIO '%s' is virtual\n", prop);
+ } else if (ret) {
return log_msg_ret("gpio", ret);
- else
+ } else {
info->gpio_num = desc.offset;
+ dm_gpio_free((struct udevice *)dev, &desc);
+ }
info->linux_name = dev_read_string(desc.dev, "linux-name");
if (!info->linux_name)
return log_msg_ret("linux-name", -ENOENT);
@@ -81,11 +203,11 @@ static int chromeos_acpi_gpio_generate(const struct udevice *dev,
ret = chromeos_get_gpio(dev, "write-protect-gpios", CROS_GPIO_WP,
&info[1]);
if (ret)
- return log_msg_ret("rec", ret);
+ return log_msg_ret("wp", ret);
ret = chromeos_get_gpio(dev, "phase-enforce-gpios", CROS_GPIO_PE,
&info[2]);
if (ret)
- return log_msg_ret("rec", ret);
+ return log_msg_ret("phase", ret);
acpigen_write_scope(ctx, "\\");
acpigen_write_name(ctx, "OIPG");
acpigen_write_package(ctx, count);
@@ -145,6 +267,7 @@ struct acpi_ops coral_acpi_ops = {
};
struct sysinfo_ops coral_sysinfo_ops = {
+ .get_str = coral_get_str,
};
#if !CONFIG_IS_ENABLED(OF_PLATDATA)
diff --git a/board/google/chromebook_coral/variant_gpio.h b/board/google/chromebook_coral/variant_gpio.h
index f516d88be5c..403e2419a71 100644
--- a/board/google/chromebook_coral/variant_gpio.h
+++ b/board/google/chromebook_coral/variant_gpio.h
@@ -34,12 +34,6 @@
/* Determine if board is in final shipping mode. */
#define GPIO_SHIP_MODE GPIO_10
-/* Memory SKU GPIOs. */
-#define MEM_CONFIG3 GPIO_45
-#define MEM_CONFIG2 GPIO_38
-#define MEM_CONFIG1 GPIO_102
-#define MEM_CONFIG0 GPIO_101
-
/* DMIC_CONFIG_PIN: High for 1-DMIC and low for 4-DMIC's */
#define DMIC_CONFIG_PIN GPIO_17
diff --git a/doc/device-tree-bindings/sysinfo/google,coral.txt b/doc/device-tree-bindings/sysinfo/google,coral.txt
new file mode 100644
index 00000000000..d8a1a79687e
--- /dev/null
+++ b/doc/device-tree-bindings/sysinfo/google,coral.txt
@@ -0,0 +1,37 @@
+Google Coral sysinfo information
+================================
+
+This binding allows information about the board to be described. It includes
+the SMBIOS binding as well.
+
+Required properties:
+
+ - compatible: "google,coral"
+ - recovery-gpios: GPIO to use for recovery button (-1 if none)
+ - wite-protect-gpios: GPIO to use for write-protect screw
+ - phase-enforce-gpios: GPIO to indicate the board is in final ship mode
+ - memconfig-gpios: 4 GPIOs to use to read memory config (as base2 int)
+
+Optional properties:
+ - skuconfig-gpios: 2 GPIOs to use to read SKU ID. If not present, the
+ Chromium OS EC SKU_ID is used instead
+
+Example:
+
+board: board {
+ compatible = "google,coral";
+ recovery-gpios = <&gpio_nw (-1) GPIO_ACTIVE_LOW>;
+ write-protect-gpios = <&gpio_nw GPIO_75 GPIO_ACTIVE_HIGH>;
+ phase-enforce-gpios = <&gpio_n GPIO_10 GPIO_ACTIVE_HIGH>;
+ memconfig-gpios = <&gpio_nw GPIO_101 GPIO_ACTIVE_HIGH
+ &gpio_nw GPIO_102 GPIO_ACTIVE_HIGH
+ &gpio_n GPIO_38 GPIO_ACTIVE_HIGH
+ &gpio_n GPIO_45 GPIO_ACTIVE_HIGH>;
+
+ /*
+ * This is used for reef only:
+ *
+ * skuconfig-gpios = <&gpio_nw GPIO_16 GPIO_ACTIVE_HIGH
+ * &gpio_nw GPIO_17 GPIO_ACTIVE_HIGH>;
+ */
+ };
--
2.30.0.478.g8a0d178c01-goog
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup
2021-02-06 16:13 ` [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup Simon Glass
@ 2021-03-09 2:33 ` Bin Meng
2021-03-15 5:30 ` Simon Glass
0 siblings, 1 reply; 7+ messages in thread
From: Bin Meng @ 2021-03-09 2:33 UTC (permalink / raw
To: u-boot
Hi Simon,
On Sun, Feb 7, 2021 at 12:13 AM Simon Glass <sjg@chromium.org> wrote:
>
> Provide the model information through sysinfo so that it shows up on
> boot. For memconfig 4 pins are provided, for 16 combinations. For SKU
> ID there are two options:
>
> - two pins provided in a ternary arrangement, for 9 combinations.
> - reading from the EC
>
> Add a binding doc and drop the unused #defines as well.
>
> Example:
>
> U-Boot 2021.01-rc5
>
> CPU: Intel(R) Celeron(R) CPU N3450 @ 1.10GHz
> DRAM: 3.9 GiB
> MMC: sdmmc at 1b,0: 1, emmc at 1c,0: 2
> Video: 1024x768x32 @ b0000000
> Model: Google Coral (memconfig 5, SKU 3)
>
> This depends on the GPIO series:
>
> http://patchwork.ozlabs.org/project/uboot/list/?series=228126
It looks this series is applied as patchwork returns nothing?
But the CI still fails
https://source.denx.de/u-boot/custodians/u-boot-x86/-/jobs/235619
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
>
> Acked-by: Bin Meng <bmeng.cn@gmail.com>
> ---
>
> Changes in v4:
> - Rebase to x86/master (but will not build until GPIO series lands)
> - Update cover letter
Regards,
Bin
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup
2021-03-09 2:33 ` Bin Meng
@ 2021-03-15 5:30 ` Simon Glass
2021-03-21 3:51 ` Simon Glass
0 siblings, 1 reply; 7+ messages in thread
From: Simon Glass @ 2021-03-15 5:30 UTC (permalink / raw
To: u-boot
Hi Bin,
On Tue, 9 Mar 2021 at 15:34, Bin Meng <bmeng.cn@gmail.com> wrote:
>
> Hi Simon,
>
> On Sun, Feb 7, 2021 at 12:13 AM Simon Glass <sjg@chromium.org> wrote:
> >
> > Provide the model information through sysinfo so that it shows up on
> > boot. For memconfig 4 pins are provided, for 16 combinations. For SKU
> > ID there are two options:
> >
> > - two pins provided in a ternary arrangement, for 9 combinations.
> > - reading from the EC
> >
> > Add a binding doc and drop the unused #defines as well.
> >
> > Example:
> >
> > U-Boot 2021.01-rc5
> >
> > CPU: Intel(R) Celeron(R) CPU N3450 @ 1.10GHz
> > DRAM: 3.9 GiB
> > MMC: sdmmc at 1b,0: 1, emmc at 1c,0: 2
> > Video: 1024x768x32 @ b0000000
> > Model: Google Coral (memconfig 5, SKU 3)
> >
> > This depends on the GPIO series:
> >
> > http://patchwork.ozlabs.org/project/uboot/list/?series=228126
>
> It looks this series is applied as patchwork returns nothing?
>
> But the CI still fails
> https://source.denx.de/u-boot/custodians/u-boot-x86/-/jobs/235619
Can you apply it to u-boot-x86/next perhaps? The GPIO series was
applied to -next, not -master.
Regards,
Simon
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup
2021-03-15 5:30 ` Simon Glass
@ 2021-03-21 3:51 ` Simon Glass
2021-03-22 1:17 ` Bin Meng
0 siblings, 1 reply; 7+ messages in thread
From: Simon Glass @ 2021-03-21 3:51 UTC (permalink / raw
To: u-boot
Hi Bin,
On Mon, 15 Mar 2021 at 18:30, Simon Glass <sjg@chromium.org> wrote:
>
> Hi Bin,
>
> On Tue, 9 Mar 2021 at 15:34, Bin Meng <bmeng.cn@gmail.com> wrote:
> >
> > Hi Simon,
> >
> > On Sun, Feb 7, 2021 at 12:13 AM Simon Glass <sjg@chromium.org> wrote:
> > >
> > > Provide the model information through sysinfo so that it shows up on
> > > boot. For memconfig 4 pins are provided, for 16 combinations. For SKU
> > > ID there are two options:
> > >
> > > - two pins provided in a ternary arrangement, for 9 combinations.
> > > - reading from the EC
> > >
> > > Add a binding doc and drop the unused #defines as well.
> > >
> > > Example:
> > >
> > > U-Boot 2021.01-rc5
> > >
> > > CPU: Intel(R) Celeron(R) CPU N3450 @ 1.10GHz
> > > DRAM: 3.9 GiB
> > > MMC: sdmmc at 1b,0: 1, emmc at 1c,0: 2
> > > Video: 1024x768x32 @ b0000000
> > > Model: Google Coral (memconfig 5, SKU 3)
> > >
> > > This depends on the GPIO series:
> > >
> > > http://patchwork.ozlabs.org/project/uboot/list/?series=228126
> >
> > It looks this series is applied as patchwork returns nothing?
> >
> > But the CI still fails
> > https://source.denx.de/u-boot/custodians/u-boot-x86/-/jobs/235619
>
> Can you apply it to u-boot-x86/next perhaps? The GPIO series was
> applied to -next, not -master.
I've sent a new version, v5, which should apply on the U-Boot -next
branch. Are you keeping a -next for x86? If not I am happy to apply it
via dm.
Regards,
Simon
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup
2021-03-21 3:51 ` Simon Glass
@ 2021-03-22 1:17 ` Bin Meng
0 siblings, 0 replies; 7+ messages in thread
From: Bin Meng @ 2021-03-22 1:17 UTC (permalink / raw
To: u-boot
Hi Simon,
On Sun, Mar 21, 2021 at 11:51 AM Simon Glass <sjg@chromium.org> wrote:
>
> Hi Bin,
>
> On Mon, 15 Mar 2021 at 18:30, Simon Glass <sjg@chromium.org> wrote:
> >
> > Hi Bin,
> >
> > On Tue, 9 Mar 2021 at 15:34, Bin Meng <bmeng.cn@gmail.com> wrote:
> > >
> > > Hi Simon,
> > >
> > > On Sun, Feb 7, 2021 at 12:13 AM Simon Glass <sjg@chromium.org> wrote:
> > > >
> > > > Provide the model information through sysinfo so that it shows up on
> > > > boot. For memconfig 4 pins are provided, for 16 combinations. For SKU
> > > > ID there are two options:
> > > >
> > > > - two pins provided in a ternary arrangement, for 9 combinations.
> > > > - reading from the EC
> > > >
> > > > Add a binding doc and drop the unused #defines as well.
> > > >
> > > > Example:
> > > >
> > > > U-Boot 2021.01-rc5
> > > >
> > > > CPU: Intel(R) Celeron(R) CPU N3450 @ 1.10GHz
> > > > DRAM: 3.9 GiB
> > > > MMC: sdmmc at 1b,0: 1, emmc at 1c,0: 2
> > > > Video: 1024x768x32 @ b0000000
> > > > Model: Google Coral (memconfig 5, SKU 3)
> > > >
> > > > This depends on the GPIO series:
> > > >
> > > > http://patchwork.ozlabs.org/project/uboot/list/?series=228126
> > >
> > > It looks this series is applied as patchwork returns nothing?
> > >
> > > But the CI still fails
> > > https://source.denx.de/u-boot/custodians/u-boot-x86/-/jobs/235619
> >
> > Can you apply it to u-boot-x86/next perhaps? The GPIO series was
> > applied to -next, not -master.
>
> I've sent a new version, v5, which should apply on the U-Boot -next
> branch. Are you keeping a -next for x86? If not I am happy to apply it
> via dm.
No, I did not track u-boot/next in the x86 repo. Feel free to apply
this series via dm.
Regards,
Bin
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-03-22 1:17 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-02-06 16:13 [PATCH v4 0/2] x86: Minor improvements mostly for image loading Simon Glass
2021-02-06 16:13 ` [PATCH v4 1/2] sysinfo: Allow showing model info from sysinfo Simon Glass
2021-02-06 16:13 ` [PATCH v4 2/2] x86: coral: Show memory config and SKU ID on startup Simon Glass
2021-03-09 2:33 ` Bin Meng
2021-03-15 5:30 ` Simon Glass
2021-03-21 3:51 ` Simon Glass
2021-03-22 1:17 ` Bin Meng
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.