From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Julius Werner <jwerner@chromium.org>
Cc: Guenter Roeck <linux@roeck-us.net>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
samuel@sholland.org, LKML <linux-kernel@vger.kernel.org>,
Coreboot <coreboot@coreboot.org>
Subject: Re: [PATCH 0/5] coreboot table bus and framebuffer driver
Date: Mon, 23 Apr 2018 13:37:53 +0200 [thread overview]
Message-ID: <20180423113753.GA8630@kroah.com> (raw)
In-Reply-To: <CAODwPW96LfXRbLFkX9OAUV7DH9t1MPogqvUUrCExrzk00Vy4_w@mail.gmail.com>
On Wed, Mar 14, 2018 at 09:22:04PM +0000, Julius Werner wrote:
> [resend in plain text]
>
> > It would be great to get some of the google developers to ack these, as
> > this touches their code...
>
> From the coreboot point of view I guess we're fine with it since it claims
> to maintain all of the existing functionality. It's just changing the
> kernel-level plumbing for these drivers and I don't really have the
> expertise to comment on whether this is better or worse than the old code
> (maybe Dmitry or Guenter will?). It seems a little odd to me to call this a
> "bus", but if we think that's the most fitting abstraction the kernel has
> for it, I'm okay with that. All I care about is that it will work (in all
> combinations... e.g. regardless of probe order and even if some parts are
> compiled as modules and loaded/unloaded at runtime).
Thanks for the response, patches are all now queued up.
greg k-h
prev parent reply other threads:[~2018-04-23 11:37 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-25 1:41 [PATCH 0/5] coreboot table bus and framebuffer driver Samuel Holland
2018-01-25 1:41 ` [PATCH 1/5] firmware: coreboot: Expose the coreboot table as a bus Samuel Holland
2018-01-25 1:41 ` [PATCH 2/5] firmware: memconsole: Probe via coreboot bus Samuel Holland
2018-01-25 1:41 ` [PATCH 3/5] firmware: vpd: " Samuel Holland
2018-01-25 1:41 ` [PATCH 4/5] firmware: coreboot: Remove unused coreboot_table_find Samuel Holland
2018-01-25 1:41 ` [PATCH 5/5] firmware: coreboot: Add coreboot framebuffer driver Samuel Holland
2018-03-14 18:08 ` [PATCH 0/5] coreboot table bus and " Greg Kroah-Hartman
2018-03-14 21:18 ` Julius Werner
2018-03-14 21:22 ` Julius Werner
2018-04-23 11:37 ` Greg Kroah-Hartman [this message]
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=20180423113753.GA8630@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=coreboot@coreboot.org \
--cc=dmitry.torokhov@gmail.com \
--cc=jwerner@chromium.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@roeck-us.net \
--cc=samuel@sholland.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.