[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [dmidecode] [PATCH] dmioem: decode HPE UEFI type 219 Misc Features
From: |
Jerry Hoemann |
Subject: |
Re: [dmidecode] [PATCH] dmioem: decode HPE UEFI type 219 Misc Features |
Date: |
Tue, 10 Apr 2018 10:03:30 -0600 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Tue, Apr 10, 2018 at 08:31:57AM +0200, Jean Delvare wrote:
> Hi Jerry,
>
> On Mon, 9 Apr 2018 18:15:33 -0600, Jerry Hoemann wrote:
> > The incorrect "Misc Feature" bit in type 219 table was being used to
> > identify that the platform is capable of booting with UEFI.
> >
> > For cooresponding kernel change please see:
> >
> > commit(c42cbe41727a)
> >
> > in linux-stable.
>
> In linux. linux-stable has backports of the commits, that's not
> relevant in this case.
Will drop the cross reference.
>
> >
> > Signed-off-by: Jerry Hoemann <address@hidden>
> > ---
> > dmioem.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/dmioem.c b/dmioem.c
> > index 8a72ac9..dfd14c7 100644
> > --- a/dmioem.c
> > +++ b/dmioem.c
> > @@ -268,7 +268,7 @@ static int dmi_decode_hp(const struct dmi_header *h)
> > feat = DWORD(data + 0x10);
> > printf("\tMisc. Features: 0x%08x\n", feat);
> > printf("\t\tiCRU: %s\n", feat & 0x0001 ? "Yes" : "No");
> > - printf("\t\tUEFI: %s\n", feat & 0x0408 ? "Yes" : "No");
> > + printf("\t\tUEFI: %s\n", feat & 0x01400 ? "Yes" : "No");
> > break;
> >
> > default:
>
> What's the rationale for using a 5-digit mask instead of 4-digit? It
> makes both the change and the resulting code harder to read IMHO.
Will fix.
--
-----------------------------------------------------------------------------
Jerry Hoemann Software Engineer Hewlett Packard Enterprise
-----------------------------------------------------------------------------