RE: [PATCH 05/28] drm: Add support for 3x3 CTM

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Pekka Paalanen <pekka.paalanen@xxxxxxxxxxxxx>
> Sent: Tuesday, February 13, 2024 2:45 PM
> To: Shankar, Uma <uma.shankar@xxxxxxxxx>
> Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; dri-devel@xxxxxxxxxxxxxxxxxxxxx;
> ville.syrjala@xxxxxxxxxxxxxxx; contact@xxxxxxxxxxx; harry.wentland@xxxxxxx;
> mwen@xxxxxxxxxx; jadahl@xxxxxxxxxx; sebastian.wick@xxxxxxxxxx;
> shashank.sharma@xxxxxxx; agoins@xxxxxxxxxx; joshua@xxxxxxxxx;
> mdaenzer@xxxxxxxxxx; aleixpol@xxxxxxx; xaver.hugl@xxxxxxxxx;
> victoria@xxxxxxxxxxxx; daniel@xxxxxxxx; quic_naseer@xxxxxxxxxxx;
> quic_cbraga@xxxxxxxxxxx; quic_abhinavk@xxxxxxxxxxx; arthurgrillo@xxxxxxxxxx;
> marcan@xxxxxxxxx; Liviu.Dudau@xxxxxxx; sashamcintosh@xxxxxxxxxx;
> sean@xxxxxxxxxx; Borah, Chaitanya Kumar <chaitanya.kumar.borah@xxxxxxxxx>
> Subject: Re: [PATCH 05/28] drm: Add support for 3x3 CTM
> 
> On Tue, 13 Feb 2024 12:18:12 +0530
> Uma Shankar <uma.shankar@xxxxxxxxx> wrote:
> 
> > From: Chaitanya Kumar Borah <chaitanya.kumar.borah@xxxxxxxxx>
> >
> > Add support for 3x3 Color Transformation Matrices in Color Pipeline.
> >
> > Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@xxxxxxxxx>
> > Signed-off-by: Uma Shankar <uma.shankar@xxxxxxxxx>
> > ---
> >  drivers/gpu/drm/drm_atomic_uapi.c | 3 +++
> >  drivers/gpu/drm/drm_colorop.c     | 2 +-
> >  include/uapi/drm/drm_mode.h       | 1 +
> >  3 files changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/gpu/drm/drm_atomic_uapi.c
> > b/drivers/gpu/drm/drm_atomic_uapi.c
> > index e7bf1fb054af..c54b0d6c133e 100644
> > --- a/drivers/gpu/drm/drm_atomic_uapi.c
> > +++ b/drivers/gpu/drm/drm_atomic_uapi.c
> > @@ -716,6 +716,9 @@ static int drm_atomic_color_set_data_property(struct
> drm_colorop *colorop,
> >  	case DRM_COLOROP_CTM_3X4:
> >  		size = sizeof(struct drm_color_ctm_3x4);
> >  		break;
> > +	case DRM_COLOROP_CTM_3X3:
> > +		size = sizeof(struct drm_color_ctm);
> > +		break;
> >  	default:
> >  		/* should never get here */
> >  		return -EINVAL;
> > diff --git a/drivers/gpu/drm/drm_colorop.c
> > b/drivers/gpu/drm/drm_colorop.c index 462ffec42cdf..6bae6dc8e54b
> > 100644
> > --- a/drivers/gpu/drm/drm_colorop.c
> > +++ b/drivers/gpu/drm/drm_colorop.c
> > @@ -107,7 +107,7 @@ int drm_colorop_init(struct drm_device *dev, struct
> drm_colorop *colorop,
> >  				   0);
> >
> >  	/* data */
> > -	if (type == DRM_COLOROP_CTM_3X4) {
> > +	if (type == DRM_COLOROP_CTM_3X4 || type ==
> DRM_COLOROP_CTM_3X3) {
> >  		prop = drm_property_create(dev, DRM_MODE_PROP_ATOMIC |
> DRM_MODE_PROP_BLOB,
> >  					   "DATA", 0);
> >  		if (!prop)
> > diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
> > index f16318f1785f..68696253867e 100644
> > --- a/include/uapi/drm/drm_mode.h
> > +++ b/include/uapi/drm/drm_mode.h
> > @@ -868,6 +868,7 @@ struct drm_color_lut {
> >
> >  enum drm_colorop_type {
> >  	DRM_COLOROP_1D_CURVE,
> > +	DRM_COLOROP_CTM_3X3,
> >  	DRM_COLOROP_CTM_3X4,
> >  };
> >
> 
> Hi,
> 
> where are the docs for DRM_COLOROP_CTM_3X3?

Hi Pekka,
Sorry, we missed this in the current version. Will update the same in next revision.

Regards,
Uma Shankar

> 
> Thanks,
> pq




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux