Pretty sure the current code is right. If the GAMMA_LUT property can't be set, it tries to set gamma the old-fashioned way.
On Tue, Mar 3, 2020 at 8:12 AM Devarsh Thakkar <devar...@xilinx.com> wrote: > > Hi Rohit, > > This makes sense to me as gamma was implemented as optional property. > Reviewed-By: "Devarsh Thakkar <devarsh.thak...@xilinx.com>" > > @emil.veli...@collabora.com, @imir...@alum.mit.edu, @Ville Syrjälä, Could you > please ack and help merge this patch if it also look good to you ? > > Regards, > Devarsh > > > -----Original Message----- > > From: Rohit Visavalia > > Sent: 27 February 2020 00:40 > > To: Rohit Visavalia <rvisa...@xilinx.com>; dri-devel@lists.freedesktop.org; > > imir...@alum.mit.edu; emil.veli...@collabora.com > > Cc: Hyun Kwon <hy...@xilinx.com>; Ranganathan Sk <r...@xilinx.com>; Dhaval > > Rajeshbhai Shah <ds...@xilinx.com>; Varunkumar Allagadapa > > <varun...@xilinx.com>; Devarsh Thakkar <devar...@xilinx.com> > > Subject: RE: [PATCH libdrm] modetest: call drmModeCrtcSetGamma() only if > > add_property_optional returns true > > > > Gentle reminder. > > > > + Ilia Mirkin, +Emil Velikov. > > > > Thanks & Regards, > > Rohit > > > > > -----Original Message----- > > > From: Rohit Visavalia [mailto:rohit.visava...@xilinx.com] > > > Sent: Tuesday, February 25, 2020 3:08 PM > > > To: dri-devel@lists.freedesktop.org > > > Cc: Hyun Kwon <hy...@xilinx.com>; Ranganathan Sk <r...@xilinx.com>; > > > Dhaval Rajeshbhai Shah <ds...@xilinx.com>; Varunkumar Allagadapa > > > <varun...@xilinx.com>; Devarsh Thakkar <devar...@xilinx.com>; Rohit > > > Visavalia <rvisa...@xilinx.com> > > > Subject: [PATCH libdrm] modetest: call drmModeCrtcSetGamma() only if > > > add_property_optional returns true > > > > > > gamma is a optional property then also it prints error message, so set > > > gamma only if add_property_optional() returns true. > > > > > > Signed-off-by: Rohit Visavalia <rohit.visava...@xilinx.com> > > > --- > > > tests/modetest/modetest.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c > > > index b907ab3..379b9ea 100644 > > > --- a/tests/modetest/modetest.c > > > +++ b/tests/modetest/modetest.c > > > @@ -1138,7 +1138,7 @@ static void set_gamma(struct device *dev, > > > unsigned crtc_id, unsigned fourcc) > > > > > > add_property_optional(dev, crtc_id, "DEGAMMA_LUT", 0); > > > add_property_optional(dev, crtc_id, "CTM", 0); > > > - if (!add_property_optional(dev, crtc_id, "GAMMA_LUT", blob_id)) { > > > + if (add_property_optional(dev, crtc_id, "GAMMA_LUT", blob_id)) { > > > uint16_t r[256], g[256], b[256]; > > > > > > for (i = 0; i < 256; i++) { > > > -- > > > 2.7.4 > _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
