RE: [PATCH v5 0/3] Support 64 bpp half float formats

2019-02-13 Thread Strasser, Kevin via dri-devel
Maarten Lankhorst wrote: > Op 13-02-2019 om 16:53 schreef Ville Syrjälä: > > On Fri, Feb 08, 2019 at 01:49:40PM -0800, Kevin Strasser wrote: > >> This series defines new formats and adds implementation to the i915 driver. > >> Since posting v1 I have removed the pixel normalize property, as it's no

RE: [Intel-gfx] [PATCH v4 0/3] Support 64 bpp half float formats

2019-02-08 Thread Strasser, Kevin
Maarten Lankhorst wrote: > It's nice that the igt parts are almost done. I think this might need rebasing > on top of > > I'm glad there are tests, but the internal accuracy of the CAIRO_FORMAT_RGB24 > is only 8 bits, so it wouldn't test this properly. > > Fortunately we will start using CAIRO_FORM

RE: [Intel-gfx] [PATCH v4 1/3] drm/fourcc: Add 64 bpp half float formats

2019-02-08 Thread Strasser, Kevin
Maarten Lankhorst wrote: > Hmmm, from include/drm/drm_fourcc.h > * @depth: > * > * Color depth (number of bits per pixel excluding padding bits), > * valid for a subset of RGB formats only. This is a legacy field, do > * not use in new code and set to 0

RE: [PATCH v2 RESEND 2/2] drm/i915/icl: Implement half float formats

2019-02-04 Thread Strasser, Kevin
Ville Syrjälä wrote: > Kevin Strasser wrote: > > Ville Syrjälä wrote: > > > is_hdr_plane() is around now, please use it. > > > > I don't think I can use icl_is_hdr_plane here without some refactoring. It > > requires the plane->base to be initialized through drm_universal_plane_init, > > which depe

RE: [PATCH v2 RESEND 2/2] drm/i915/icl: Implement half float formats

2019-02-01 Thread Strasser, Kevin
Ville Syrjälä wrote: > > @@ -1774,6 +1776,45 @@ static const u32 skl_planar_formats[] = { > >DRM_FORMAT_NV12, > > }; > > > > +static const uint32_t icl_hdr_plane_formats[] = { > > Please switch to u32 & co. We recently had a mass conversion in the > driver. Will do. Looks like the CI caug

RE: [Intel-gfx] [PATCH 0/3] Support 64 bpp half float formats

2018-11-29 Thread Strasser, Kevin
Ville Syrjälä wrote: > On Wed, Nov 28, 2018 at 10:38:10PM -0800, Kevin Strasser wrote: >> This series defines new formats and adds a plane property to be used for >> floating point framebuffer content. Implementation is then added to i915. >> >> I have shared an IGT branch which adds test coverage

RE: [PATCH 3/3] drm/i915: Implement half float formats and pixel normalize property

2018-11-29 Thread Strasser, Kevin
Ville Syrjälä wrote: > On Wed, Nov 28, 2018 at 10:38:13PM -0800, Kevin Strasser wrote: >> 64 bpp half float formats are supported on hdr planes only and are subject >> to the following restrictions: >> * 90/270 rotation not supported >> * Yf Tiling not supported >> * Frame Buffer Compression

RE: [PATCH 2/3] drm: Add optional PIXEL_NORMALIZE_RANGE property to drm_plane

2018-11-29 Thread Strasser, Kevin
Ville Syrjälä wrote: > On Wed, Nov 28, 2018 at 10:38:12PM -0800, Kevin Strasser wrote: >> Add an optional property to allow applications to indicate what range their >> floating point pixel data is normalized to. Drivers are free to choose what >> ranges they want to support and can attach this pro

RE: [PATCH 1/3] drm/fourcc: Add 64 bpp half float formats

2018-11-29 Thread Strasser, Kevin
Ville Syrjälä wrote: > On Wed, Nov 28, 2018 at 10:38:11PM -0800, Kevin Strasser wrote: >> + { .format = DRM_FORMAT_XRGB16161616H, .depth = 48, >> .num_planes = 1, .cpp = { 4, 0, 0 }, .hsub = 1, .vsub = 1, .is_fp = true }, >> + { .format = DRM_FORMAT_XBGR16161616H, .dept

RE: [PATCH 3/3] drm/i915: Implement half float formats and pixel normalize property

2018-11-29 Thread Strasser, Kevin
Daniel Vetter wrote: > Do we have end-to-end userspace for this? I have patches for IGT and I'm planning on adding usage code to Weston. Apart from that there is a Windows use case that Tina mentioned previously. I take it that you will need to see the Weston part before accepting this. Thanks, K