Hi Bin, On Thu, 3 Mar 2022 at 01:49, Bin Meng <bmeng...@gmail.com> wrote: > > Hi Simon, > > On Tue, Mar 1, 2022 at 6:18 AM Simon Glass <s...@chromium.org> wrote: > > > > This enum is currently anonymous. Add a name so it can be used in the > > code. > > > > Signed-off-by: Simon Glass <s...@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