[igt-dev] [PATCH i-g-t 0/2] Revert the wrong fix for coverage-premultiplied-constant

[Surendrakumar Upadhyay, TejaskumarX]

Reviewed-by : Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay at intel.com>

Thanks,
Tejas

> -----Original Message-----
> From: Srinivas, Vidya <vidya.srinivas at intel.com>
> Sent: 29 July 2021 12:07
> To: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>; igt-
> dev at lists.freedesktop.org; Joshi, Kunal1 <kunal1.joshi at intel.com>;
> Surendrakumar Upadhyay, TejaskumarX
> <tejaskumarx.surendrakumar.upadhyay at intel.com>
> Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> <bhanuprakash.modem at intel.com>; Shankar, Uma
> <uma.shankar at intel.com>
> Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-
> premultiplied-constant
> 
> @Surendrakumar Upadhyay, TejaskumarX Can you kindly review the patch
> series - Please provide RB for full series? Thank you.
> Kunal, kindly help merge. Thank you.
> 
> Regards
> Vidya
> 
> > -----Original Message-----
> > From: Srinivas, Vidya
> > Sent: Saturday, July 24, 2021 9:39 PM
> > To: Vudum, Lakshminarayana <Lakshminarayana.Vudum at intel.com>; igt-
> > dev at lists.freedesktop.org
> > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> > <Bhanuprakash.Modem at intel.com>; Shankar, Uma
> <uma.shankar at intel.com>
> > Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-
> > premultiplied-constant
> >
> > Thank you so much Lakshmi. This time it has PASSED.
> >
> > Regards
> > Vidya
> >
> > > -----Original Message-----
> > > From: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>
> > > Sent: Friday, July 23, 2021 9:00 PM
> > > To: Srinivas, Vidya <vidya.srinivas at intel.com>;
> > > igt-dev at lists.freedesktop.org
> > > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> > > <bhanuprakash.modem at intel.com>; Shankar, Uma
> > <uma.shankar at intel.com>
> > > Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-
> > > premultiplied-constant
> > >
> > > Hi,
> > >
> > > Failures are addressed and results are re-reported.
> > >
> > > Lakshmi.
> > > -----Original Message-----
> > > From: Srinivas, Vidya <vidya.srinivas at intel.com>
> > > Sent: Friday, July 23, 2021 1:46 AM
> > > To: igt-dev at lists.freedesktop.org
> > > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> > > <bhanuprakash.modem at intel.com>; Shankar, Uma
> > <uma.shankar at intel.com>;
> > > Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>
> > > Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-
> > > premultiplied-constant
> > >
> > > Hello Lakshmi,
> > >
> > > We need to revert this series. But there is a false positive.
> > > Could you kindly help with re-run? Thank you so much.
> > >
> > > https://patchwork.freedesktop.org/series/92898/
> > >
> > > Regards
> > > Vidya
> > >
> > > > -----Original Message-----
> > > > From: Srinivas, Vidya <vidya.srinivas at intel.com>
> > > > Sent: Thursday, July 22, 2021 8:37 PM
> > > > To: igt-dev at lists.freedesktop.org
> > > > Cc: Latvala, Petri <petri.latvala at intel.com>; Modem, Bhanuprakash
> > > > <bhanuprakash.modem at intel.com>; Shankar, Uma
> > > <uma.shankar at intel.com>;
> > > > Srinivas, Vidya <vidya.srinivas at intel.com>
> > > > Subject: [PATCH i-g-t 0/2] Revert the wrong fix for
> > > > coverage-premultiplied- constant
> > > >
> > > > Subtest coverage-premultiplied-constant was failing due to buffer
> > > > alignment issue. It was not due to 6bpc.
> > > > Hence, we need to revert this patch which was wrongly submitted.
> > > >
> > > > Signed-off-by: Vidya Srinivas <vidya.srinivas at intel.com>

Reviewed-by : Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay at intel.com>

> > > >
> > > >
> > > > Vidya Srinivas (2):
> > > >   Revert "tests/kms_plane_alpha_blend: Fix a check in is_6bpc function"
> > > >   Revert "tests/kms_plane_alpha_blend: Skip
> > > > coverage-vs-premult-vs-
> > none
> > > >     for 6bpc panels"
> > > >
> > > >  tests/kms_plane_alpha_blend.c | 36
> > > > -----------------------------------
> > > >  1 file changed, 36 deletions(-)
> > > >
> > > > --
> > > > 2.32.0
> > >
> > >