> -----Original Message-----
> From: Joshua Ashton <joshua@xxxxxxxxx>
> Sent: Wednesday, November 8, 2023 7:13 PM
> To: Shankar, Uma <uma.shankar@xxxxxxxxx>; Harry Wentland
> <harry.wentland@xxxxxxx>; dri-devel@xxxxxxxxxxxxxxxxxxxxx
> Cc: wayland-devel@xxxxxxxxxxxxxxxxxxxxx; Ville Syrjala
> <ville.syrjala@xxxxxxxxxxxxxxx>; Pekka Paalanen
> <pekka.paalanen@xxxxxxxxxxxxx>; Simon Ser <contact@xxxxxxxxxxx>; Melissa
> Wen <mwen@xxxxxxxxxx>; Jonas Ådahl <jadahl@xxxxxxxxxx>; Sebastian Wick
> <sebastian.wick@xxxxxxxxxx>; Shashank Sharma
> <shashank.sharma@xxxxxxx>; Alexander Goins <agoins@xxxxxxxxxx>; Michel
> Dänzer <mdaenzer@xxxxxxxxxx>; Aleix Pol <aleixpol@xxxxxxx>; Xaver Hugl
> <xaver.hugl@xxxxxxxxx>; Victoria Brekenfeld <victoria@xxxxxxxxxxxx>; Sima
> <daniel@xxxxxxxx>; Naseer Ahmed <quic_naseer@xxxxxxxxxxx>; Christopher
> Braga <quic_cbraga@xxxxxxxxxxx>; Abhinav Kumar
> <quic_abhinavk@xxxxxxxxxxx>; Arthur Grillo <arthurgrillo@xxxxxxxxxx>; Hector
> Martin <marcan@xxxxxxxxx>; Liviu Dudau <Liviu.Dudau@xxxxxxx>; Sasha
> McIntosh <sashamcintosh@xxxxxxxxxx>
> Subject: Re: [RFC PATCH v2 06/17] drm/doc/rfc: Describe why prescriptive color
> pipeline is needed
>
>
>
> On 11/8/23 12:18, Shankar, Uma wrote:
> >
> >
> >> -----Original Message-----
> >> From: Harry Wentland <harry.wentland@xxxxxxx>
> >> Sent: Friday, October 20, 2023 2:51 AM
> >> To: dri-devel@xxxxxxxxxxxxxxxxxxxxx
> >> Cc: wayland-devel@xxxxxxxxxxxxxxxxxxxxx; Harry Wentland
> >> <harry.wentland@xxxxxxx>; Ville Syrjala
> >> <ville.syrjala@xxxxxxxxxxxxxxx>; Pekka Paalanen
> >> <pekka.paalanen@xxxxxxxxxxxxx>; Simon Ser <contact@xxxxxxxxxxx>;
> >> Melissa Wen <mwen@xxxxxxxxxx>; Jonas Ådahl <jadahl@xxxxxxxxxx>;
> >> Sebastian Wick <sebastian.wick@xxxxxxxxxx>; Shashank Sharma
> >> <shashank.sharma@xxxxxxx>; Alexander Goins <agoins@xxxxxxxxxx>;
> >> Joshua Ashton <joshua@xxxxxxxxx>; Michel Dänzer
> >> <mdaenzer@xxxxxxxxxx>; Aleix Pol <aleixpol@xxxxxxx>; Xaver Hugl
> >> <xaver.hugl@xxxxxxxxx>; Victoria Brekenfeld <victoria@xxxxxxxxxxxx>;
> >> Sima <daniel@xxxxxxxx>; Shankar, Uma <uma.shankar@xxxxxxxxx>; Naseer
> >> Ahmed <quic_naseer@xxxxxxxxxxx>; Christopher Braga
> >> <quic_cbraga@xxxxxxxxxxx>; Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>;
> >> Arthur Grillo <arthurgrillo@xxxxxxxxxx>; Hector Martin
> >> <marcan@xxxxxxxxx>; Liviu Dudau <Liviu.Dudau@xxxxxxx>; Sasha McIntosh
> >> <sashamcintosh@xxxxxxxxxx>
> >> Subject: [RFC PATCH v2 06/17] drm/doc/rfc: Describe why prescriptive
> >> color pipeline is needed
> >>
> >> v2:
> >> - Update colorop visualizations to match reality (Sebastian, Alex Hung)
> >> - Updated wording (Pekka)
> >> - Change BYPASS wording to make it non-mandatory (Sebastian)
> >> - Drop cover-letter-like paragraph from COLOR_PIPELINE Plane Property
> >> section (Pekka)
> >> - Use PQ EOTF instead of its inverse in Pipeline Programming example
> (Melissa)
> >> - Add "Driver Implementer's Guide" section (Pekka)
> >> - Add "Driver Forward/Backward Compatibility" section (Sebastian,
> >> Pekka)
> >>
> >> Signed-off-by: Harry Wentland <harry.wentland@xxxxxxx>
> >> Cc: Ville Syrjala <ville.syrjala@xxxxxxxxxxxxxxx>
> >> Cc: Pekka Paalanen <pekka.paalanen@xxxxxxxxxxxxx>
> >> Cc: Simon Ser <contact@xxxxxxxxxxx>
> >> Cc: Harry Wentland <harry.wentland@xxxxxxx>
> >> Cc: Melissa Wen <mwen@xxxxxxxxxx>
> >> Cc: Jonas Ådahl <jadahl@xxxxxxxxxx>
> >> Cc: Sebastian Wick <sebastian.wick@xxxxxxxxxx>
> >> Cc: Shashank Sharma <shashank.sharma@xxxxxxx>
> >> Cc: Alexander Goins <agoins@xxxxxxxxxx>
> >> Cc: Joshua Ashton <joshua@xxxxxxxxx>
> >> Cc: Michel Dänzer <mdaenzer@xxxxxxxxxx>
> >> Cc: Aleix Pol <aleixpol@xxxxxxx>
> >> Cc: Xaver Hugl <xaver.hugl@xxxxxxxxx>
> >> Cc: Victoria Brekenfeld <victoria@xxxxxxxxxxxx>
> >> Cc: Sima <daniel@xxxxxxxx>
> >> Cc: Uma Shankar <uma.shankar@xxxxxxxxx>
> >> Cc: Naseer Ahmed <quic_naseer@xxxxxxxxxxx>
> >> Cc: Christopher Braga <quic_cbraga@xxxxxxxxxxx>
> >> Cc: Abhinav Kumar <quic_abhinavk@xxxxxxxxxxx>
> >> Cc: Arthur Grillo <arthurgrillo@xxxxxxxxxx>
> >> Cc: Hector Martin <marcan@xxxxxxxxx>
> >> Cc: Liviu Dudau <Liviu.Dudau@xxxxxxx>
> >> Cc: Sasha McIntosh <sashamcintosh@xxxxxxxxxx>
> >> ---
> >> Documentation/gpu/rfc/color_pipeline.rst | 347 +++++++++++++++++++++++
> >> 1 file changed, 347 insertions(+)
> >> create mode 100644 Documentation/gpu/rfc/color_pipeline.rst
> >>
> >> diff --git a/Documentation/gpu/rfc/color_pipeline.rst
> >> b/Documentation/gpu/rfc/color_pipeline.rst
> >> new file mode 100644
> >> index 000000000000..af5f2ea29116
> >> --- /dev/null
> >> +++ b/Documentation/gpu/rfc/color_pipeline.rst
> >> @@ -0,0 +1,347 @@
> >> +========================
> >> +Linux Color Pipeline API
> >> +========================
> >> +
> >> +What problem are we solving?
> >> +============================
> >> +
> >> +We would like to support pre-, and post-blending complex color
> >> +transformations in display controller hardware in order to allow for
> >> +HW-supported HDR use-cases, as well as to provide support to
> >> +color-managed applications, such as video or image editors.
> >> +
> >> +It is possible to support an HDR output on HW supporting the
> >> +Colorspace and HDR Metadata drm_connector properties, but that
> >> +requires the compositor or application to render and compose the
> >> +content into one final buffer intended for display. Doing so is costly.
> >> +
> >> +Most modern display HW offers various 1D LUTs, 3D LUTs, matrices,
> >> +and other operations to support color transformations. These
> >> +operations are often implemented in fixed-function HW and therefore
> >> +much more power efficient than performing similar operations via shaders or
> CPU.
> >> +
> >> +We would like to make use of this HW functionality to support
> >> +complex color transformations with no, or minimal CPU or shader load.
> >> +
> >> +
> >> +How are other OSes solving this problem?
> >> +========================================
> >> +
> >> +The most widely supported use-cases regard HDR content, whether
> >> +video or gaming.
> >> +
> >> +Most OSes will specify the source content format (color gamut,
> >> +encoding transfer function, and other metadata, such as max and
> >> +average light levels) to a
> >> driver.
> >> +Drivers will then program their fixed-function HW accordingly to map
> >> +from a source content buffer's space to a display's space.
> >> +
> >> +When fixed-function HW is not available the compositor will assemble
> >> +a shader to ask the GPU to perform the transformation from the
> >> +source content format to the display's format.
> >> +
> >> +A compositor's mapping function and a driver's mapping function are
> >> +usually entirely separate concepts. On OSes where a HW vendor has no
> >> +insight into closed-source compositor code such a vendor will tune
> >> +their color management code to visually match the compositor's. On
> >> +other OSes, where both mapping functions are open to an implementer
> >> +they will
> >> ensure both mappings match.
> >> +
> >> +This results in mapping algorithm lock-in, meaning that no-one alone
> >> +can experiment with or introduce new mapping algorithms and achieve
> >> +consistent results regardless of which implementation path is taken.
> >> +
> >> +Why is Linux different?
> >> +=======================
> >> +
> >> +Unlike other OSes, where there is one compositor for one or more
> >> +drivers, on Linux we have a many-to-many relationship. Many
> >> +compositors;
> >> many drivers.
> >> +In addition each compositor vendor or community has their own view
> >> +of how color management should be done. This is what makes Linux so
> beautiful.
> >> +
> >> +This means that a HW vendor can now no longer tune their driver to
> >> +one compositor, as tuning it to one could make it look fairly
> >> +different from another compositor's color mapping.
> >> +
> >> +We need a better solution.
> >> +
> >> +
> >> +Descriptive API
> >> +===============
> >> +
> >> +An API that describes the source and destination colorspaces is a
> >> +descriptive API. It describes the input and output color spaces but
> >> +does not describe how precisely they should be mapped. Such a
> >> +mapping includes many minute design decision that can greatly affect
> >> +the look of the final
> >> result.
> >> +
> >> +It is not feasible to describe such mapping with enough detail to
> >> +ensure the same result from each implementation. In fact, these
> >> +mappings are a very active research area.
> >> +
> >> +
> >> +Prescriptive API
> >> +================
> >> +
> >> +A prescriptive API describes not the source and destination
> >> +colorspaces. It instead prescribes a recipe for how to manipulate
> >> +pixel values to arrive at the desired outcome.
> >> +
> >> +This recipe is generally an ordered list of straight-forward
> >> +operations, with clear mathematical definitions, such as 1D LUTs, 3D
> >> +LUTs, matrices, or other operations that can be described in a precise manner.
> >> +
> >> +
> >> +The Color Pipeline API
> >> +======================
> >> +
> >> +HW color management pipelines can significantly differ between HW
> >> +vendors in terms of availability, ordering, and capabilities of HW
> >> +blocks. This makes a common definition of color management blocks
> >> +and their ordering nigh impossible. Instead we are defining an API
> >> +that allows user space to discover the HW capabilities in a generic
> >> +manner, agnostic of specific drivers and hardware.
> >> +
> >> +
> >> +drm_colorop Object & IOCTLs
> >> +===========================
> >> +
> >> +To support the definition of color pipelines we define the DRM core
> >> +object type drm_colorop. Individual drm_colorop objects will be
> >> +chained via the NEXT property of a drm_colorop to constitute a color
> pipeline.
> >> +Each drm_colorop object is unique, i.e., even if multiple color
> >> +pipelines have the same operation they won't share the same
> >> +drm_colorop object to describe that operation.
> >> +
> >> +Note that drivers are not expected to map drm_colorop objects
> >> +statically to specific HW blocks. The mapping of drm_colorop objects
> >> +is entirely a driver-internal detail and can be as dynamic or static
> >> +as a driver needs it to be. See more in the Driver Implementation
> >> +Guide section
> >> below.
> >> +
> >> +Just like other DRM objects the drm_colorop objects are discovered
> >> +via
> >> +IOCTLs:
> >> +
> >> +DRM_IOCTL_MODE_GETCOLOROPRESOURCES: This IOCTL is used to
> retrieve
> >> the
> >> +number of all drm_colorop objects.
> >> +
> >> +DRM_IOCTL_MODE_GETCOLOROP: This IOCTL is used to read one
> drm_colorop.
> >> +It includes the ID for the colorop object, as well as the plane_id
> >> +of the associated plane. All other values should be registered as
> >> +properties.
> >> +
> >> +Each drm_colorop has three core properties:
> >> +
> >> +TYPE: The type of transformation, such as
> >> +* enumerated curve
> >> +* custom (uniform) 1D LUT
> >> +* 3x3 matrix
> >> +* 3x4 matrix
> >> +* 3D LUT
> >> +* etc.
> >> +
> >> +Depending on the type of transformation other properties will
> >> +describe more details.
> >> +
> >> +BYPASS: A boolean property that can be used to easily put a block
> >> +into bypass mode. While setting other properties might fail atomic
> >> +check, setting the BYPASS property to true should never fail. The
> >> +BYPASS property is not mandatory for a colorop, as long as the
> >> +entire pipeline can get bypassed by setting the COLOR_PIPELINE on a plane
> to '0'.
> >> +
> >> +NEXT: The ID of the next drm_colorop in a color pipeline, or 0 if
> >> +this drm_colorop is the last in the chain.
> >> +
> >> +An example of a drm_colorop object might look like one of these::
> >> +
> >> + /* 1D enumerated curve */
> >> + Color operation 42
> >> + ├─ "TYPE": immutable enum {1D enumerated curve, 1D LUT, 3x3
> >> + matrix, 3x4
> >> matrix, 3D LUT, etc.} = 1D enumerated curve
> >> + ├─ "BYPASS": bool {true, false}
> >> + ├─ "CURVE_1D_TYPE": enum {sRGB EOTF, sRGB inverse EOTF, PQ EOTF,
> >> + PQ
> >> inverse EOTF, …}
> >
> > Having the fixed function enum for some targeted input/output may not
> > be scalable for all usecases. There are multiple colorspaces and
> > transfer functions possible, so it will not be possible to cover all
> > these by any enum definitions. Also, this will depend on the capabilities of
> respective hardware from various vendors.
>
> The reason this exists is such that certain HW vendors such as AMD have transfer
> functions implemented in HW. It is important to take advantage of these for both
> precision and power reasons.
Issue we see here is that, it will be too usecase and vendor specific.
There will be BT601, BT709, BT2020, SRGB, HDR EOTF and many more. Not to forget
we will need linearization and non-linearization enums for each of these. Also
a CTM indication to convert colospace. Also, if the underlying hardware block is
programmable, its not limited to be used only for the colorspace management but
can be used for other color enhancements as well by a capable client.
Hence, we feel that it is bordering on being descriptive with too many possible
combinations (not easy to generalize). So, if hardware is programmable, lets
expose its capability through a blob and be generic.
For any fixed function hardware where Lut etc is stored in ROM and just a control/enable
bit is provided to driver, we can define a pipeline with a vendor specific color block. This
can be identified with a flag (better ways can be discussed).
For example, on some of the Intel platform, we had a fixed function to convert colorspaces
directly with a bit setting. These kinds of things should be vendor specific and not be part
of generic userspace implementation.
For reference:
001b YUV601 to RGB601 YUV BT.601 to RGB BT.601 conversion.
010b YUV709 to RGB709 YUV BT.709 to RGB BT.709 conversion.
011b YUV2020 to RGB2020 YUV BT.2020 to RGB BT.2020 conversion.
100b RGB709 to RGB2020 RGB BT.709 to RGB BT.2020 conversion.
> Additionally, not every vendor implements bucketed/segemented LUTs the same
> way, so it's not feasible to expose that in a way that's particularly useful or not
> vendor-specific.
If the underlying hardware is programmable, the structure which we propose to advertise
the capability of the block to userspace will be sufficient to compute the LUT coefficients.
The caps can be :
1. Number of segments in Lut
2. Precision of lut
3. Starting and ending point of the segment
4. Number of samples in the segment.
5. Any other flag which could be useful in this computation.
This way we can compute LUT's generically and send to driver. This will be scalable for all
colorspaces, configurations and vendors.
> Thus we decided to have a regular 1D LUT modulated onto a known curve.
> This is the only real cross-vendor solution here that allows HW curve
> implementations to be taken advantage of and also works with
> bucketing/segemented LUTs.
> (Including vendors we are not aware of yet).
>
> This also means that vendors that only support HW curves at some stages without
> an actual LUT are also serviced.
Any fixed function vendor implementation should be supported but with a vendor
specific color block. Trying to come up with enums which aligns with some underlying
hardware may not be scalable.
> You are right that there *might* be some usecase not covered by this right now,
> and that it would need kernel churn to implement new curves, but unfortunately
> that's the compromise that we (so-far) have decided on in order to ensure
> everyone can have good, precise, power-efficient support.
Yes, we are aligned on this. But believe programmable hardware should be able to
expose its caps. Fixed function hardware should be non-generic and vendor specific.
> It is always possible for us to extend the uAPI at a later date for other curves, or
> other properties that might expose a generic segmented LUT interface (such as
> what you have proposed for a while) for vendors that can support it.
> (With the whole color pipeline thing, we can essentially do 'versioning'
> with that, if we wanted a new 1D LUT type.)
Most of the hardware vendors have programmable luts (including AMD), so it would be
good to have this as a default generic compositor implementation. And yes, any new color
block with a type can be added to the existing API's as the need arises without breaking
compatibility.
Regards,
Uma Shankar
>
> Thanks!
> - Joshie 🐸✨
>
> >
> >> + └─ "NEXT": immutable color operation ID = 43
> >> +
> >> + /* custom 4k entry 1D LUT */
> >> + Color operation 52
> >> + ├─ "TYPE": immutable enum {1D enumerated curve, 1D LUT, 3x3
> >> + matrix, 3x4
> >> matrix, 3D LUT, etc.} = 1D LUT
> >> + ├─ "BYPASS": bool {true, false}
> >> + ├─ "LUT_1D_SIZE": immutable range = 4096
> >
> > For the size and capability of individual LUT block, it would be good
> > to add this as a blob as defined in the blob approach we were planning
> > earlier. So just taking that part of the series to have this capability detection
> generic. Refer below:
> > https://patchwork.freedesktop.org/patch/554855/?series=123023&rev=1
> >
> > Basically, use this structure for lut capability and arrangement:
> > struct drm_color_lut_range {
> > /* DRM_MODE_LUT_* */
> > __u32 flags;
> > /* number of points on the curve */
> > __u16 count;
> > /* input/output bits per component */
> > __u8 input_bpc, output_bpc;
> > /* input start/end values */
> > __s32 start, end;
> > /* output min/max values */
> > __s32 min, max;
> > };
> >
> > If the intention is to have just 1 segment with 4096, it can be easily described
> there.
> > Additionally, this can also cater to any kind of lut arrangement, PWL, segmented
> or logarithmic.
> >
> >> + ├─ "LUT_1D": blob
> >> + └─ "NEXT": immutable color operation ID = 0
> >> +
> >> + /* 17^3 3D LUT */
> >> + Color operation 72
> >> + ├─ "TYPE": immutable enum {1D enumerated curve, 1D LUT, 3x3
> >> + matrix, 3x4
> >> matrix, 3D LUT, etc.} = 3D LUT
> >> + ├─ "BYPASS": bool {true, false}
> >> + ├─ "LUT_3D_SIZE": immutable range = 17
> >> + ├─ "LUT_3D": blob
> >> + └─ "NEXT": immutable color operation ID = 73
> >> +
> >> +
> >> +COLOR_PIPELINE Plane Property
> >> +=============================
> >> +
> >> +Color Pipelines are created by a driver and advertised via a new
> >> +COLOR_PIPELINE enum property on each plane. Values of the property
> >> +always include '0', which is the default and means all color
> >> +processing is disabled. Additional values will be the object IDs of
> >> +the first drm_colorop in a pipeline. A driver can create and
> >> +advertise none, one, or more possible color pipelines. A DRM client
> >> +will select a color pipeline by setting the COLOR PIPELINE to the respective
> value.
> >> +
> >> +In the case where drivers have custom support for pre-blending color
> >> +processing those drivers shall reject atomic commits that are trying
> >> +to use both the custom color properties, as well as the
> >> +COLOR_PIPELINE property.
> >> +
> >> +An example of a COLOR_PIPELINE property on a plane might look like this::
> >> +
> >> + Plane 10
> >> + ├─ "type": immutable enum {Overlay, Primary, Cursor} = Primary
> >> + ├─ …
> >> + └─ "color_pipeline": enum {0, 42, 52} = 0
> >> +
> >> +
> >> +Color Pipeline Discovery
> >> +========================
> >> +
> >> +A DRM client wanting color management on a drm_plane will:
> >> +
> >> +1. Read all drm_colorop objects
> >> +2. Get the COLOR_PIPELINE property of the plane 3. iterate all
> >> +COLOR_PIPELINE enum values 4. for each enum value walk the color
> >> +pipeline (via the NEXT pointers)
> >> + and see if the available color operations are suitable for the
> >> + desired color management operations
> >> +
> >> +An example of chained properties to define an AMD pre-blending color
> >> +pipeline might look like this::
> >> +
> >> + Plane 10
> >> + ├─ "TYPE" (immutable) = Primary
> >> + └─ "COLOR_PIPELINE": enum {0, 44} = 0
> >> +
> >> + Color operation 44
> >> + ├─ "TYPE" (immutable) = 1D enumerated curve
> >> + ├─ "BYPASS": bool
> >> + ├─ "CURVE_1D_TYPE": enum {sRGB EOTF, PQ EOTF} = sRGB EOTF
> >> + └─ "NEXT" (immutable) = 45
> >> +
> >> + Color operation 45
> >> + ├─ "TYPE" (immutable) = 3x4 Matrix
> >> + ├─ "BYPASS": bool
> >> + ├─ "MATRIX_3_4": blob
> >> + └─ "NEXT" (immutable) = 46
> >> +
> >> + Color operation 46
> >> + ├─ "TYPE" (immutable) = 1D enumerated curve
> >> + ├─ "BYPASS": bool
> >> + ├─ "CURVE_1D_TYPE": enum {sRGB Inverse EOTF, PQ Inverse EOTF} =
> >> + sRGB
> >> EOTF
> >> + └─ "NEXT" (immutable) = 47
> >> +
> >> + Color operation 47
> >> + ├─ "TYPE" (immutable) = 1D LUT
> >> + ├─ "LUT_1D_SIZE": immutable range = 4096
> >> + ├─ "LUT_1D_DATA": blob
> >> + └─ "NEXT" (immutable) = 48
> >> +
> >> + Color operation 48
> >> + ├─ "TYPE" (immutable) = 3D LUT
> >> + ├─ "LUT_3D_SIZE" (immutable) = 17
> >> + ├─ "LUT_3D_DATA": blob
> >> + └─ "NEXT" (immutable) = 49
> >> +
> >> + Color operation 49
> >> + ├─ "TYPE" (immutable) = 1D enumerated curve
> >> + ├─ "BYPASS": bool
> >> + ├─ "CURVE_1D_TYPE": enum {sRGB EOTF, PQ EOTF} = sRGB EOTF
> >> + └─ "NEXT" (immutable) = 0
> >> +
> >> +
> >> +Color Pipeline Programming
> >> +==========================
> >> +
> >> +Once a DRM client has found a suitable pipeline it will:
> >> +
> >> +1. Set the COLOR_PIPELINE enum value to the one pointing at the first
> >> + drm_colorop object of the desired pipeline 2. Set the properties
> >> +for all drm_colorop objects in the pipeline to the
> >> + desired values, setting BYPASS to true for unused drm_colorop blocks,
> >> + and false for enabled drm_colorop blocks 3. Perform
> >> +atomic_check/commit as desired
> >> +
> >> +To configure the pipeline for an HDR10 PQ plane and blending in
> >> +linear space, a compositor might perform an atomic commit with the
> >> +following property values::
> >> +
> >> + Plane 10
> >> + └─ "COLOR_PIPELINE" = 42
> >> +
> >> + Color operation 42 (input CSC)
> >> + └─ "BYPASS" = true
> >> +
> >> + Color operation 44 (DeGamma)
> >> + └─ "BYPASS" = true
> >> +
> >> + Color operation 45 (gamut remap)
> >> + └─ "BYPASS" = true
> >> +
> >> + Color operation 46 (shaper LUT RAM)
> >> + └─ "BYPASS" = true
> >> +
> >> + Color operation 47 (3D LUT RAM)
> >> + └─ "LUT_3D_DATA" = Gamut mapping + tone mapping + night mode
> >> +
> >> + Color operation 48 (blend gamma)
> >> + └─ "CURVE_1D_TYPE" = PQ EOTF
> >> +
> >> +
> >> +Driver Implementer's Guide
> >> +==========================
> >> +
> >> +What does this all mean for driver implementations? As noted above
> >> +the colorops can map to HW directly but don't need to do so. Here
> >> +are some suggestions on how to think about creating your color pipelines:
> >> +
> >> +- Try to expose pipelines that use already defined colorops, even if
> >> + your hardware pipeline is split differently. This allows existing
> >> + userspace to immediately take advantage of the hardware.
> >> +
> >> +- Additionally, try to expose your actual hardware blocks as colorops.
> >> + Define new colorop types where you believe it can offer
> >> +significant
> >> + benefits if userspace learns to program them.
> >> +
> >> +- Avoid defining new colorops for compound operations with very
> >> +narrow
> >> + scope. If you have a hardware block for a special operation that
> >> + cannot be split further, you can expose that as a new colorop type.
> >> + However, try to not define colorops for "use cases", especially if
> >> + they require you to combine multiple hardware blocks.
> >> +
> >> +- Design new colorops as prescriptive, not descriptive; by the
> >> + mathematical formula, not by the assumed input and output.
> >> +
> >> +A defined colorop type must be deterministic. Its operation can
> >> +depend only on its properties and input and nothing else, allowed
> >> +error tolerance notwithstanding.
> >> +
> >> +
> >> +Driver Forward/Backward Compatibility
> >> +=====================================
> >> +
> >> +As this is uAPI drivers can't regress color pipelines that have been
> >> +introduced for a given HW generation. New HW generations are free to
> >> +abandon color pipelines advertised for previous generations.
> >> +Nevertheless, it can be beneficial to carry support for existing
> >> +color pipelines forward as those will likely already have support in
> >> +DRM clients.
> >> +
> >> +Introducing new colorops to a pipeline is fine, as long as they can
> >> +be disabled or are purely informational. DRM clients implementing
> >> +support for the pipeline can always skip unknown properties as long
> >> +as they can be confident that doing so will not cause unexpected results.
> >> +
> >> +If a new colorop doesn't fall into one of the above categories
> >> +(bypassable or informational) the modified pipeline would be
> >> +unusable for user space. In this case a new pipeline should be defined.
> >
> > Thanks again for this nice documentation and capturing all the details clearly.
> >
> > Regards,
> > Uma Shankar
> >
> >> +
> >> +References
> >> +==========
> >> +
> >> +1.
> >> +https://lore.kernel.org/dri-devel/QMers3awXvNCQlyhWdTtsPwkp5ie9bze_h
> >> +D5n
> >>
> +AccFW7a_RXlWjYB7MoUW_8CKLT2bSQwIXVi5H6VULYIxCdgvryZoAoJnC5lZgyK1
> >> QWn488=
> >> +@emersion.fr/
> >> \ No newline at end of file
> >> --
> >> 2.42.0
> >
>
