RE: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-22 Thread Shankar, Uma
quic_nas...@quicinc.com; > quic_cbr...@quicinc.com; quic_abhin...@quicinc.com; mar...@marcan.st; > liviu.du...@arm.com; sashamcint...@google.com; Borah, Chaitanya Kumar > ; louis.chau...@bootlin.com > Subject: Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color > pipeline > &

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-22 Thread Simon Ser
On Wednesday, May 21st, 2025 at 21:18, Harry Wentland wrote: > On 2025-05-20 16:13, Harry Wentland wrote: > > > On 2025-05-19 19:43, Simon Ser wrote: > > > > > On Sunday, May 18th, 2025 at 00:32, Xaver Hugl xaver.h...@gmail.com wrote: > > > > > > > > We can always make the property mutable on

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-21 Thread Harry Wentland
On 2025-05-20 16:13, Harry Wentland wrote: > > > On 2025-05-19 19:43, Simon Ser wrote: >> On Sunday, May 18th, 2025 at 00:32, Xaver Hugl wrote: >> We can always make the property mutable on drivers that support it in >>> the future, much like the zpos property. I think we should kee

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-20 Thread Harry Wentland
On 2025-05-19 19:43, Simon Ser wrote: > On Sunday, May 18th, 2025 at 00:32, Xaver Hugl wrote: > >>> We can always make the property mutable on drivers that support it in >> >>> the future, much like the zpos property. I think we should keep it >>> immutable for now. >> >> Sure, but I don't see

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-19 Thread Simon Ser
On Sunday, May 18th, 2025 at 00:32, Xaver Hugl wrote: > > We can always make the property mutable on drivers that support it in > > > the future, much like the zpos property. I think we should keep it > > immutable for now. > > Sure, but I don't see any reason for immutability with an enum > pr

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-17 Thread Xaver Hugl
> We can always make the property mutable on drivers that support it in > the future, much like the zpos property. I think we should keep it > immutable for now. Sure, but I don't see any reason for immutability with an enum property - it can just limit the possible values to what it supports, and

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-17 Thread Autumn Ashton
On 5/17/25 2:22 AM, Xaver Hugl wrote: Am Do., 27. März 2025 um 00:58 Uhr schrieb Alex Hung : It is to be used to enable HDR by allowing userpace to create and pass 3D LUTs to kernel and hardware. new drm_colorop_type: DRM_COLOROP_3D_LUT. Signed-off-by: Alex Hung --- v8: - Fix typo in subj

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-17 Thread Simon Ser
On Saturday, May 17th, 2025 at 03:23, Xaver Hugl wrote: > Do we ever expect this to be something with multiple options that > userspace could select? I think it would be good to keep our options > open and make this property not immutable (properties that are > sometimes but not always immutable

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-16 Thread Xaver Hugl
Am Do., 27. März 2025 um 00:58 Uhr schrieb Alex Hung : > > It is to be used to enable HDR by allowing userpace to create and pass > 3D LUTs to kernel and hardware. > > new drm_colorop_type: DRM_COLOROP_3D_LUT. > > Signed-off-by: Alex Hung > --- > v8: > - Fix typo in subject (Simon Ser) > - Updat

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-05-12 Thread Alex Hung
On 4/25/25 07:50, Leandro Ribeiro wrote: On 3/26/25 20:47, Alex Hung wrote: It is to be used to enable HDR by allowing userpace to create and pass 3D LUTs to kernel and hardware. new drm_colorop_type: DRM_COLOROP_3D_LUT. Signed-off-by: Alex Hung --- v8: - Fix typo in subject (Simon Ser

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-04-25 Thread Leandro Ribeiro
On 3/26/25 20:47, Alex Hung wrote: > It is to be used to enable HDR by allowing userpace to create and pass > 3D LUTs to kernel and hardware. > > new drm_colorop_type: DRM_COLOROP_3D_LUT. > > Signed-off-by: Alex Hung > --- > v8: > - Fix typo in subject (Simon Ser) > - Update documentation f

Re: [PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-04-05 Thread Simon Ser
Reviewed-by: Simon Ser

[PATCH V8 40/43] drm/colorop: Add 3D LUT support to color pipeline

2025-03-26 Thread Alex Hung
It is to be used to enable HDR by allowing userpace to create and pass 3D LUTs to kernel and hardware. new drm_colorop_type: DRM_COLOROP_3D_LUT. Signed-off-by: Alex Hung --- v8: - Fix typo in subject (Simon Ser) - Update documentation for DRM_COLOROP_3D_LUT (Simon Ser) - Delete empty lines (S