Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: "Shankar, Uma" <uma.shankar@intel.com>
To: "Surendrakumar Upadhyay,
	TejaskumarX" <tejaskumarx.surendrakumar.upadhyay@intel.com>,
	"Srinivas, Vidya" <vidya.srinivas@intel.com>,
	"Vudum, Lakshminarayana" <lakshminarayana.vudum@intel.com>,
	"igt-dev@lists.freedesktop.org" <igt-dev@lists.freedesktop.org>,
	"Joshi, Kunal1" <kunal1.joshi@intel.com>
Cc: "Latvala, Petri" <petri.latvala@intel.com>
Subject: Re: [igt-dev] [PATCH i-g-t 0/2] Revert the wrong fix for coverage-premultiplied-constant
Date: Thu, 29 Jul 2021 07:42:36 +0000	[thread overview]
Message-ID: <83448a629c114a5394f98af83accee25@intel.com> (raw)
In-Reply-To: <SN6PR11MB3421F31D4487939E06DE3FB0DFEB9@SN6PR11MB3421.namprd11.prod.outlook.com>



> -----Original Message-----
> From: Surendrakumar Upadhyay, TejaskumarX
> <tejaskumarx.surendrakumar.upadhyay@intel.com>
> Sent: Thursday, July 29, 2021 12:09 PM
> To: Srinivas, Vidya <vidya.srinivas@intel.com>; Vudum, Lakshminarayana
> <lakshminarayana.vudum@intel.com>; igt-dev@lists.freedesktop.org; Joshi, Kunal1
> <kunal1.joshi@intel.com>
> Cc: Latvala, Petri <petri.latvala@intel.com>; Modem, Bhanuprakash
> <bhanuprakash.modem@intel.com>; Shankar, Uma <uma.shankar@intel.com>
> Subject: RE: [PATCH i-g-t 0/2] Revert the wrong fix for coverage-premultiplied-
> constant
> 
> Reviewed-by : Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@intel.com>

Request to all, please refrain from top posting. Have all your comments inline.

Regards,
Uma Shankar
> Thanks,
> Tejas
> 
> > -----Original Message-----
> > From: Srinivas, Vidya <vidya.srinivas@intel.com>
> > Sent: 29 July 2021 12:07
> > To: Vudum, Lakshminarayana <lakshminarayana.vudum@intel.com>; igt-
> > dev@lists.freedesktop.org; Joshi, Kunal1 <kunal1.joshi@intel.com>;
> > Surendrakumar Upadhyay, TejaskumarX
> > <tejaskumarx.surendrakumar.upadhyay@intel.com>
> > Cc: Latvala, Petri <petri.latvala@intel.com>; Modem, Bhanuprakash
> > <bhanuprakash.modem@intel.com>; Shankar, Uma <uma.shankar@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@intel.com>; igt-
> > > dev@lists.freedesktop.org
> > > Cc: Latvala, Petri <petri.latvala@intel.com>; Modem, Bhanuprakash
> > > <Bhanuprakash.Modem@intel.com>; Shankar, Uma
> > <uma.shankar@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@intel.com>
> > > > Sent: Friday, July 23, 2021 9:00 PM
> > > > To: Srinivas, Vidya <vidya.srinivas@intel.com>;
> > > > igt-dev@lists.freedesktop.org
> > > > Cc: Latvala, Petri <petri.latvala@intel.com>; Modem, Bhanuprakash
> > > > <bhanuprakash.modem@intel.com>; Shankar, Uma
> > > <uma.shankar@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@intel.com>
> > > > Sent: Friday, July 23, 2021 1:46 AM
> > > > To: igt-dev@lists.freedesktop.org
> > > > Cc: Latvala, Petri <petri.latvala@intel.com>; Modem, Bhanuprakash
> > > > <bhanuprakash.modem@intel.com>; Shankar, Uma
> > > <uma.shankar@intel.com>;
> > > > Vudum, Lakshminarayana <lakshminarayana.vudum@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@intel.com>
> > > > > Sent: Thursday, July 22, 2021 8:37 PM
> > > > > To: igt-dev@lists.freedesktop.org
> > > > > Cc: Latvala, Petri <petri.latvala@intel.com>; Modem,
> > > > > Bhanuprakash <bhanuprakash.modem@intel.com>; Shankar, Uma
> > > > <uma.shankar@intel.com>;
> > > > > Srinivas, Vidya <vidya.srinivas@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@intel.com>
> 
> Reviewed-by : Tejas Upadhyay <tejaskumarx.surendrakumar.upadhyay@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
> > > >
> > > >
> 

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

  reply	other threads:[~2021-07-29  7:42 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-07-22 15:06 [igt-dev] [PATCH i-g-t 0/2] Revert the wrong fix for coverage-premultiplied-constant Vidya Srinivas
2021-07-22 15:06 ` [igt-dev] [PATCH i-g-t 1/2] Revert "tests/kms_plane_alpha_blend: Fix a check in is_6bpc function" Vidya Srinivas
2021-07-29  8:00   ` Surendrakumar Upadhyay, TejaskumarX
2021-07-29  8:06   ` Surendrakumar Upadhyay, TejaskumarX
2021-07-22 15:07 ` [igt-dev] [PATCH i-g-t 2/2] Revert "tests/kms_plane_alpha_blend: Skip coverage-vs-premult-vs-none for 6bpc panels" Vidya Srinivas
2021-07-29  8:09   ` Surendrakumar Upadhyay, TejaskumarX
2021-07-22 16:15 ` [igt-dev] ✗ Fi.CI.BAT: failure for Revert the wrong fix for coverage-premultiplied-constant Patchwork
2021-07-23  8:45 ` [igt-dev] [PATCH i-g-t 0/2] " Srinivas, Vidya
2021-07-23 15:30   ` Vudum, Lakshminarayana
2021-07-24 16:08     ` Srinivas, Vidya
2021-07-29  6:13       ` Surendrakumar Upadhyay, TejaskumarX
2021-07-29  6:36       ` Srinivas, Vidya
2021-07-29  6:38         ` Surendrakumar Upadhyay, TejaskumarX
2021-07-29  7:42           ` Shankar, Uma [this message]
2021-07-23 15:15 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2021-07-23 22:49 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork
2021-07-29  6:33 ` [igt-dev] [PATCH i-g-t 0/2] " Surendrakumar Upadhyay, TejaskumarX

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83448a629c114a5394f98af83accee25@intel.com \
    --to=uma.shankar@intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    --cc=kunal1.joshi@intel.com \
    --cc=lakshminarayana.vudum@intel.com \
    --cc=petri.latvala@intel.com \
    --cc=tejaskumarx.surendrakumar.upadhyay@intel.com \
    --cc=vidya.srinivas@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox