Hi Simon, On Fri, Mar 4, 2022 at 10:38 AM Simon Glass <s...@chromium.org> wrote: > > 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.
Better put some comments. Reviewed-by: Bin Meng <bmeng...@gmail.com> Regards, Bin