<div dir="auto"><div> <div class="gmail_quote"><div dir="ltr">Am Fr., 28. Dez. 2018, 02:09 hat Stéphane Marchesin <<a href="mailto:marcheu@chromium.org">marcheu@chromium.org</a>> geschrieben: </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Dec 27, 2018 at 4:45 PM Matt Roper <<a href="mailto:matthew.d.roper@intel.com" target="_blank" rel="noreferrer">matthew.d.roper@intel.com</a>> wrote: > > On Thu, Dec 27, 2018 at 04:22:28PM -0800, Stéphane Marchesin wrote: > > On Thu, Dec 27, 2018 at 3:49 PM Li, Wei C <<a href="mailto:wei.c.li@intel.com" target="_blank" rel="noreferrer">wei.c.li@intel.com</a>> wrote: > > > > > > Matt, > > > > > > Is that possible for you to get some time to review <a href="https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1387366" rel="noreferrer noreferrer" target="_blank">https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1387366</a> and confirm the FROMLIST change if it looks good to you so we could move forward? > > > > > > > To be more precise, I am trying to assess what's needed before this > > patch goes usptream, and in particular, I'd like to know if we are > > blocked on any Chrome-side thing. > > > > Stéphane > > > > Hi Stéphane. On the Chrome side of things, I believe we need an Ack > from the ChromeOS userspace team on the ABI, plus a pointer to where the > final, reviewed userspace patches are that correspond to it (mailing > list link or gerrit or whatever). I have an old gerrit link to some > in-development chromium/ozone patches for this from November 2nd, but > I'm not sure if there's a newer one now. IMO from user space the ABI is good. I think the question is more whether other hw would be fine with it. For example if we notice that other platforms can only do black as a background color, how can we make this API standard. Hopefully other folks can chime in here about other hw capabilities. Stéphane > > Aside from satisfying the ABI/userspace requirements, we still need all > the patches to get reviewed by the upstream kernel devs. An older > version of patch #2 has a r-b from Sean, but patches 1 and 3 haven't > been accepted yet. Actually it looks like I never sent the latest > version of my series that incorporates some additional feedback from > Brian Starkey to the list; I'll do that tomorrow. I think a lot of > people are still out on holiday vacation at the moment, so if necessary > I'll start pinging IRC for reviews in a week or two when people are back > in office. </blockquote></div></div><div dir="auto"> </div><div dir="auto"> </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> > > > Matt > > > > > > BTW, I've backported your kernel patch into Chrome OS and verified it using the TEST=drm-tests/atomictest -t crtc_background_color on a Google Pixelbook with KBL graphics. > > > > > > Thanks, > > > Wei > > > > > > -----Original Message----- > > > From: Stéphane Marchesin [mailto:<a href="mailto:marcheu@chromium.org" target="_blank" rel="noreferrer">marcheu@chromium.org</a>] > > > Sent: Thursday, December 27, 2018 3:27 PM > > > To: Roper, Matthew D <<a href="mailto:matthew.d.roper@intel.com" target="_blank" rel="noreferrer">matthew.d.roper@intel.com</a>> > > > Cc: intel-gfx <<a href="mailto:intel-gfx@lists.freedesktop.org" target="_blank" rel="noreferrer">intel-gfx@lists.freedesktop.org</a>>; Li, Wei C <<a href="mailto:wei.c.li@intel.com" target="_blank" rel="noreferrer">wei.c.li@intel.com</a>>; dri-devel <<a href="mailto:dri-devel@lists.freedesktop.org" target="_blank" rel="noreferrer">dri-devel@lists.freedesktop.org</a>> > > > Subject: Re: [Intel-gfx] [PATCH v3 0/3] CRTC background color > > > > > > Hey, > > > > > > Is there anything missing on the Chrome side to move forward with this series? > > > > > > Stéphane > > > > > > On Thu, Nov 15, 2018 at 2:14 PM Matt Roper <<a href="mailto:matthew.d.roper@intel.com" target="_blank" rel="noreferrer">matthew.d.roper@intel.com</a>> wrote: > > > > > > > > Third version of the series previously posted here: > > > > > > > > <a href="https://lists.freedesktop.org/archives/intel-gfx/2018-November/181777" rel="noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/archives/intel-gfx/2018-November/181777</a>. > > > > html > > > > > > > > This version incorporates review feedback from Ville and Sean Paul. > > > > > > > > The first patch here can be merged whenever it receives review approval. > > > > The second and third patches still need to wait for opensource > > > > userspace to be ready before merging (there's ChromeOS work underway). > > > > > > > > Cc: <a href="mailto:dri-devel@lists.freedesktop.org" target="_blank" rel="noreferrer">dri-devel@lists.freedesktop.org</a> > > > > Cc: Wei C Li <<a href="mailto:wei.c.li@intel.com" target="_blank" rel="noreferrer">wei.c.li@intel.com</a>> > > > > Cc: Sean Paul <sean@poorly.run> > > > > Cc: Ville Syrjälä <<a href="mailto:ville.syrjala@linux.intel.com" target="_blank" rel="noreferrer">ville.syrjala@linux.intel.com</a>> > > > > > > > > Matt Roper (3): > > > > drm/i915: Force background color to black for gen9+ (v2) > > > > drm: Add CRTC background color property (v2) > > > > drm/i915/gen9+: Add support for pipe background color (v3) > > > > > > > > drivers/gpu/drm/drm_atomic_state_helper.c | 1 + > > > > drivers/gpu/drm/drm_atomic_uapi.c | 5 ++++ > > > > drivers/gpu/drm/drm_blend.c | 21 +++++++++++++--- > > > > drivers/gpu/drm/drm_mode_config.c | 6 +++++ > > > > drivers/gpu/drm/i915/i915_debugfs.c | 9 +++++++ > > > > drivers/gpu/drm/i915/i915_reg.h | 6 +++++ > > > > drivers/gpu/drm/i915/intel_display.c | 40 +++++++++++++++++++++++++++++++ > > > > include/drm/drm_blend.h | 1 + > > > > include/drm/drm_crtc.h | 17 +++++++++++++ > > > > include/drm/drm_mode_config.h | 5 ++++ > > > > include/uapi/drm/drm_mode.h | 28 ++++++++++++++++++++++ > > > > 11 files changed, 136 insertions(+), 3 deletions(-) > > > > > > > > -- > > > > 2.14.4 > > > > > > > > _______________________________________________ > > > > Intel-gfx mailing list > > > > <a href="mailto:Intel-gfx@lists.freedesktop.org" target="_blank" rel="noreferrer">Intel-gfx@lists.freedesktop.org</a> > > > > <a href="https://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/intel-gfx</a> > > -- > Matt Roper > Graphics Software Engineer > IoTG Platform Enabling & Development > Intel Corporation > (916) 356-2795 _______________________________________________ dri-devel mailing list <a href="mailto:dri-devel@lists.freedesktop.org" target="_blank" rel="noreferrer">dri-devel@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/dri-devel" rel="noreferrer noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/dri-devel</a> </blockquote></div></div></div>