On Tue, Dec 03, 2024 at 09:14:58AM +0000, Shankar, Uma wrote: > > > > -----Original Message----- > > From: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > > Sent: Saturday, November 30, 2024 3:17 PM > > To: Shankar, Uma <uma.shankar@xxxxxxxxx> > > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx; intel-gfx@xxxxxxxxxxxxxxxxxxxxx; intel- > > xe@xxxxxxxxxxxxxxxxxxxxx; ville.syrjala@xxxxxxxxxxxxxxx; > > harry.wentland@xxxxxxx; pekka.paalanen@xxxxxxxxxxxxx; > > sebastian.wick@xxxxxxxxxx; jadahl@xxxxxxxxxx; mwen@xxxxxxxxxx; > > contact@xxxxxxxxxxx; Kumar, Naveen1 <naveen1.kumar@xxxxxxxxx>; Borah, > > Chaitanya Kumar <chaitanya.kumar.borah@xxxxxxxxx> > > Subject: Re: [v2 09/25] drm: Add helper to initialize segmented 1D LUT > > > > On Tue, Nov 26, 2024 at 06:57:14PM +0530, Uma Shankar wrote: > > > From: Chaitanya Kumar Borah <chaitanya.kumar.borah@xxxxxxxxx> > > > > > > Add helper to initialize 1D segmented LUT > > > > > > Signed-off-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@xxxxxxxxx> > > > Signed-off-by: Uma Shankar <uma.shankar@xxxxxxxxx> > > > --- > > > drivers/gpu/drm/drm_colorop.c | 27 ++++++++++++++++++++++++++- > > > include/drm/drm_colorop.h | 4 ++++ > > > 2 files changed, 30 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/gpu/drm/drm_colorop.c > > > b/drivers/gpu/drm/drm_colorop.c index 111517c08216..871d5660e3b2 > > > 100644 > > > --- a/drivers/gpu/drm/drm_colorop.c > > > +++ b/drivers/gpu/drm/drm_colorop.c > > > @@ -104,7 +104,6 @@ static int drm_create_colorop_capability_prop(struct > > drm_device *dev, > > > return 0; > > > } > > > > > > -__maybe_unused > > > > Squash all three patches so that you don't have to play with __maybe_unused. > > Then please expand commit message to describe the problem that you are > > solving. > > Idea was to logically separate for ease of review. But we can squash if that looks better. I think the patches are simple enough, but it's just my 2c. > > > > static int drm_colorop_lutcaps_init(struct drm_colorop *colorop, > > > struct drm_plane *plane, > > > const struct drm_color_lut_range *ranges, -- With best wishes Dmitry
