linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: will.deacon@arm.com (Will Deacon)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm64: dmi: Add SMBIOS/DMI support
Date: Fri, 19 Sep 2014 17:47:34 +0100	[thread overview]
Message-ID: <20140919164734.GK20773@arm.com> (raw)
In-Reply-To: <CAKv+Gu8+qmpDpAyE3DkHihMmfCf7pf7EpnaMY_SrPFNn2yGLBQ@mail.gmail.com>

On Wed, Sep 17, 2014 at 11:14:24PM +0100, Ard Biesheuvel wrote:
> On 31 July 2014 06:01, Will Deacon <will.deacon@arm.com> wrote:
> > On Thu, Jul 31, 2014 at 01:48:36PM +0100, Ard Biesheuvel wrote:
> >> On 31 July 2014 14:47, Ard Biesheuvel <ard.biesheuvel@linaro.org> wrote:
> >> > From: Yi Li <yi.li@linaro.org>
> >> >
> >> > SMbios is important for server hardware vendors. It implements a spec for
> >> > providing descriptive information about the platform. Things like serial
> >> > numbers, physical layout of the ports, build configuration data, and the like.
> >> >
> >> > This has been tested by dmidecode and lshw tools.
> >> >
> >> > Signed-off-by: Yi Li <yi.li@linaro.org>
> >> > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >> > ---
> >> >
> >> > It turns out that efi_lookup_mapped_addr() is not appropriate after all for
> >> > remapping the address of the SMBIOS tables. The reason is that the UEFI spec
> >> > forbids virtual mappings being requested for configuration tables (which is not
> >> > currently honored by Tianocore/EDK2)
> >> >
> >>
> >> @Yi: could you retest yet again, please?
> >>
> >> @Will; this superseded the patch that is already queued up in
> >> for-next/core, so perhaps you could drop that while we get this
> >> tested?
> >
> > Reverted. I'd really appreciate help testing this stuff in future -- it was
> > a pig getting it going on the foundation model and the whole setup felt
> > rather contrived.
> >
> > Anyway, thanks for letting me know.
> >
> 
> Hi Will,
> 
> Are you ok to take this patch now, for 3.18? Surely, we could get it
> tested today or tomorrow if you would still like do it yourself? Yi
> has tested it on a recent RC so we could ask him to demonstrate if for
> us.

Any chance you could resend the fixed patch, please? Catalin's dealing with
the 3.18 tree, so he should be able to apply it then.

Will

  reply	other threads:[~2014-09-19 16:47 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 [this message]
     [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
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=20140919164734.GK20773@arm.com \
    --to=will.deacon@arm.com \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).