From: leif.lindholm@linaro.org (Leif Lindholm)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm64: dmi: Add SMBIOS/DMI support
Date: Thu, 31 Jul 2014 17:15:28 +0100 [thread overview]
Message-ID: <20140731161528.GH4179@bivouac.eciton.net> (raw)
In-Reply-To: <20140731155338.GC26853@arm.com>
On Thu, Jul 31, 2014 at 04:53:38PM +0100, Will Deacon wrote:
> > >> The patch works well on FVP model, and here are the boot log and
> > >> dmidecode+lshw result:
> > >>
> > > Thanks Yi!
> > >
> > > Can we help Will out if he wants to test this himself?
> > @Will: which platform do you have on hand? I am using FVP model
> > for testing.
> > First, you need to merge SMBIOS support into UEFI.
> > Then, compile and install the dmidecode(open source) on the linux
> > ,and run it.
>
> I've got some stuff from Leif for testing this, but what I'm really after is
> something that would have spotted this problem. Are there any compliance
> tests, for example?
The patch as previously submitted (and accepted) was functional on all
platforms it has been tested on. It is technically not compliant with
the UEFI specification, but it works with the images we had.
The worst-case failure would have been the lookup function returning
NULL and the SMBIOS tables being unavailable even though they existed.
What you are asking for would be a test for UEFI, not for the kernel.
(And this one might be useful to add to the fwts if it's not already
part of it.)
/
Leif
next prev parent reply other threads:[~2014-07-31 16:15 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-31 12:47 [PATCH] arm64: dmi: Add SMBIOS/DMI support Ard Biesheuvel
2014-07-31 12:48 ` Ard Biesheuvel
2014-07-31 13:01 ` Will Deacon
2014-09-17 22:14 ` Ard Biesheuvel
2014-09-19 16:47 ` Will Deacon
[not found] ` <53DA509A.8070304@linaro.org>
2014-07-31 14:26 ` Ard Biesheuvel
2014-07-31 15:51 ` Yi Li
2014-07-31 15:53 ` Will Deacon
2014-07-31 16:15 ` Leif Lindholm [this message]
2014-07-31 18:44 ` Ard Biesheuvel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20140731161528.GH4179@bivouac.eciton.net \
--to=leif.lindholm@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.