[PATCH 1/2] x86: Add an enum name for the GNVS firmware type

This enum is currently anonymous. Add a name so it can be used in the code.
Signed-off-by: Simon Glass sjg@chromium.org ---
arch/x86/include/asm/intel_gnvs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/include/asm/intel_gnvs.h b/arch/x86/include/asm/intel_gnvs.h index fc743dc928..b12d7f9485 100644 --- a/arch/x86/include/asm/intel_gnvs.h +++ b/arch/x86/include/asm/intel_gnvs.h @@ -47,7 +47,7 @@ enum { BINF_RW_B = 2 };
-enum { +enum cros_fw_type_t { FIRMWARE_TYPE_AUTO_DETECT = -1, FIRMWARE_TYPE_RECOVERY = 0, FIRMWARE_TYPE_NORMAL = 1,

This board has its own config header file. Correct it.
Signed-off-by: Simon Glass sjg@chromium.org ---
board/coreboot/coreboot/MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/board/coreboot/coreboot/MAINTAINERS b/board/coreboot/coreboot/MAINTAINERS index a05673bb0b..ee12d32ce7 100644 --- a/board/coreboot/coreboot/MAINTAINERS +++ b/board/coreboot/coreboot/MAINTAINERS @@ -2,12 +2,12 @@ COREBOOT BOARD M: Simon Glass sjg@chromium.org S: Maintained F: board/coreboot/coreboot/ -F: include/configs/chromebook_link.h +F: include/configs/coreboot.h F: configs/coreboot_defconfig
COREBOOT64 BOARD M: Simon Glass sjg@chromium.org S: Maintained F: board/coreboot/coreboot/ -F: include/configs/chromebook_link.h +F: include/configs/coreboot.h F: configs/coreboot64_defconfig

On Tue, Mar 1, 2022 at 6:18 AM Simon Glass sjg@chromium.org wrote:
This board has its own config header file. Correct it.
Signed-off-by: Simon Glass sjg@chromium.org
board/coreboot/coreboot/MAINTAINERS | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Bin Meng bmeng.cn@gmail.com

Hi Simon,
On Tue, Mar 1, 2022 at 6:18 AM Simon Glass sjg@chromium.org wrote:
This enum is currently anonymous. Add a name so it can be used in the code.
Signed-off-by: Simon Glass sjg@chromium.org
arch/x86/include/asm/intel_gnvs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/include/asm/intel_gnvs.h b/arch/x86/include/asm/intel_gnvs.h index fc743dc928..b12d7f9485 100644 --- a/arch/x86/include/asm/intel_gnvs.h +++ b/arch/x86/include/asm/intel_gnvs.h @@ -47,7 +47,7 @@ enum { BINF_RW_B = 2 };
-enum { +enum cros_fw_type_t {
The name indicates this is chrome os only, but it should be a generic name, no?
FIRMWARE_TYPE_AUTO_DETECT = -1, FIRMWARE_TYPE_RECOVERY = 0, FIRMWARE_TYPE_NORMAL = 1,
Regards, Bin

Hi Bin,
On Thu, 3 Mar 2022 at 01:49, Bin Meng bmeng.cn@gmail.com wrote:
Hi Simon,
On Tue, Mar 1, 2022 at 6:18 AM Simon Glass sjg@chromium.org wrote:
This enum is currently anonymous. Add a name so it can be used in the code.
Signed-off-by: Simon Glass sjg@chromium.org
arch/x86/include/asm/intel_gnvs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/include/asm/intel_gnvs.h b/arch/x86/include/asm/intel_gnvs.h index fc743dc928..b12d7f9485 100644 --- a/arch/x86/include/asm/intel_gnvs.h +++ b/arch/x86/include/asm/intel_gnvs.h @@ -47,7 +47,7 @@ enum { BINF_RW_B = 2 };
-enum { +enum cros_fw_type_t {
The name indicates this is chrome os only, but it should be a generic name, no?
FIRMWARE_TYPE_AUTO_DETECT = -1, FIRMWARE_TYPE_RECOVERY = 0, FIRMWARE_TYPE_NORMAL = 1,
Well, this field is only used on Chromium OS. It has its own section of the GNVS.
Regards, Simon

Hi Simon,
On Fri, Mar 4, 2022 at 10:38 AM Simon Glass sjg@chromium.org wrote:
Hi Bin,
On Thu, 3 Mar 2022 at 01:49, Bin Meng bmeng.cn@gmail.com wrote:
Hi Simon,
On Tue, Mar 1, 2022 at 6:18 AM Simon Glass sjg@chromium.org wrote:
This enum is currently anonymous. Add a name so it can be used in the code.
Signed-off-by: Simon Glass sjg@chromium.org
arch/x86/include/asm/intel_gnvs.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/x86/include/asm/intel_gnvs.h b/arch/x86/include/asm/intel_gnvs.h index fc743dc928..b12d7f9485 100644 --- a/arch/x86/include/asm/intel_gnvs.h +++ b/arch/x86/include/asm/intel_gnvs.h @@ -47,7 +47,7 @@ enum { BINF_RW_B = 2 };
-enum { +enum cros_fw_type_t {
The name indicates this is chrome os only, but it should be a generic name, no?
FIRMWARE_TYPE_AUTO_DETECT = -1, FIRMWARE_TYPE_RECOVERY = 0, FIRMWARE_TYPE_NORMAL = 1,
Well, this field is only used on Chromium OS. It has its own section of the GNVS.
Better put some comments.
Reviewed-by: Bin Meng bmeng.cn@gmail.com
Regards, Bin
participants (2)
-
Bin Meng
-
Simon Glass