Re: [PATCH v10 0/2] Add XYUV format support

Saarinen, Jani
2018-09-14T14:34:10Z

Hi,

> -----Original Message-----
> From: Intel-gfx [mailto:intel-gfx-bounces@lists.freedesktop.org] On Behalf
> Of Lisovskiy, Stanislav
> Sent: perjantai 14. syyskuuta 2018 17.31
> To: ville.syrjala@linux.intel.com
> Cc: intel-gfx@lists.freedesktop.org; Syrjala, Ville <ville.syrjala@intel.com>;
> Heikkila, Juha-pekka <juha-pekka.heikkila@intel.com>; dri-
> devel@lists.freedesktop.org; Peres, Martin <martin.peres@intel.com>
> Subject: Re: [Intel-gfx] [PATCH v10 0/2] Add XYUV format support
>
> On Fri, 2018-09-14 at 16:47 +0300, Ville Syrjälä wrote:
> > On Fri, Sep 14, 2018 at 01:36:32PM +0000, Lisovskiy, Stanislav wrote:
> > > On Fri, 2018-09-07 at 11:45 +0300, Stanislav Lisovskiy wrote:
> > > > Introduced new XYUV scan-in format for framebuffer and added
> > > > support for it to i915(SkyLake+).
> > > >
> > > > Stanislav Lisovskiy (2):
> > > > drm: Introduce new DRM_FORMAT_XYUV
> > > > drm/i915: Adding YUV444 packed format support for skl+
> > > >
> > > > drivers/gpu/drm/drm_fourcc.c | 1 +
> > > > drivers/gpu/drm/i915/i915_reg.h | 2 +-
> > > > drivers/gpu/drm/i915/intel_display.c | 15 +++++++++++++++
> > > > drivers/gpu/drm/i915/intel_sprite.c | 2 ++
> > > > include/uapi/drm/drm_fourcc.h | 1 +
> > > > 5 files changed, 20 insertions(+), 1 deletion(-)
> > > >
> > >
> > > Ping. There is an IGT patch which got Reviewed-by Ville.
> > > This one left in order to get XYUV support.
> >
> > Did we figure out userspace for this?
> >
> > Was the conflict solved with the other guy (forgot who it is) trying
> > to add the same format with a different name?
>
> Currently for userspace we have VLC(checked with Juha-Pekka) and also IGT
> test case.
>
> I think those guys were from ARM and they were adding also support for
> XYUV. The only difference was that they called XYUV, like XYUV8888
> something.
> Our patches were otherwise identical regarding drm_fourcc.c part, I don't
> see their stuff merged, but I guess it really shouldn't matter, who does this
> first. i915 specific part doesn't conflict with their stuff. To be honest, I forgot
> the guy's name neither could find his email in my mailbox.
> So hopefully somebody shows up here.
Stan:
Alexandru-Cosmin Gheorghe <Alexandru-Cosmin.Gheorghe@arm.com> ?

>
> >
> --
> Best Regards,
>
> Lisovskiy Stanislav
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx