Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 8 Jul 2022 09:57:59 -0400
From:   Rodrigo Vivi <rodrigo.vivi@intel.com>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
CC:     Alexander Usyskin <alexander.usyskin@intel.com>,
        Jani Nikula <jani.nikula@linux.intel.com>,
        Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
        David Airlie <airlied@linux.ie>,
        "Daniel Vetter" <daniel@ffwll.ch>,
        Tvrtko Ursulin <tvrtko.ursulin@linux.intel.com>,
        Tomas Winkler <tomas.winkler@intel.com>,
        Vitaly Lubart <vitaly.lubart@intel.com>,
        <intel-gfx@lists.freedesktop.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH v5 00/14] GSC support for XeHP SDV and DG2 platforms
Message-ID: <Ysg350rph4UMgVdF@intel.com>
References: <20220706114345.1128018-1-alexander.usyskin@intel.com>
 <Ysgyc70ctolFl46S@kroah.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <Ysgyc70ctolFl46S@kroah.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?FKYV1PXSiuDPVr9l0Q+FRAA+TdJDKRb2+y/2hDBg8qt/H1RFfijHNuuL0GcT?=
 =?us-ascii?Q?zaOgb6twOx5rGMWrGJHXxCbDksgzGnBTfTGtIDKJxjmW0k17Wnn9X/oVs0rG?=
 =?us-ascii?Q?nxj7MNAFRTcyCyuM8JXG+mHTzi75CWn/LII0wZiJGMKRfcUeIgbpaKmS67V7?=
 =?us-ascii?Q?297RAdFACB87IGRk0Ioc5Nnpvtlt+dqD5dVFKliV1N3ev7yOKITpNdMVu6Lc?=
 =?us-ascii?Q?jwcrK7EFCn4qC7yEiRwtfuKK/RU4F9JBXoI4RWlD7Y7Te/KsiRmNsb5PCX9a?=
 =?us-ascii?Q?VDQEteOmQmMOiZtg/5te8UiodWJcr92p4H6SMNqxRmTto03VFfCSdjA5PQLj?=
 =?us-ascii?Q?e5u5jwRrfrShSQqffyY4XVq3S///3TqhLEXdIpQzwej6sUyZNohC1XbT0Pn9?=
 =?us-ascii?Q?eZ8dUoYyXgGgDaabUpwff1fJb2QGIy+er5YXu1sZScw0YU/LBvGfhzOOB9MW?=
 =?us-ascii?Q?eKCVhR1aDxxhBgYWAW6qwjLjOsXNwInRFXbIhsv5xP2H9suUE/AepvocD2to?=
 =?us-ascii?Q?vtlxLa85XMki3+RSHlUfUwUFRFspPQTane+wRymP/g7qVbFs7naDvjwCaDMF?=
 =?us-ascii?Q?SzqEBt36gtO4AxBGTE3hhPODeZc2icSTZFkoX6PtZs4x+Iw6+Pg2SLUnUWuN?=
 =?us-ascii?Q?F3dpemgkaOVfZ26DyNwHosDrVu11P6/fW5i3Ne84DW3ZRLiu+1tnzlp1QC+8?=
 =?us-ascii?Q?oHKD9hyDAV+6pwTojEeCanxaGEH/kSkY7a28VY+hyId1zkL1IqlpKtPjO+1H?=
 =?us-ascii?Q?9U7QE5ev6lU3fnl5yXL6EKakgAoF9RxbckVmXHW2Yb5zBdzc3DKHDiMx1m07?=
 =?us-ascii?Q?RA1Eqh+UNnUKlirIud6W0bc70ftDXnqSybfCfs49udo78hCFtqUQUdNU/usb?=
 =?us-ascii?Q?8CEHYjuPEeHqR4tQb3HQW3Ep2Bx2oYIrfPyXRSWmpvhqgLvhj9qnJwlRSMSb?=
 =?us-ascii?Q?2MOkCzC8W/2ORV4ueLFh/c5W5sTKQlvEthhUQv5QoUIwevV1krhhWKG/Y1Wx?=
 =?us-ascii?Q?M6Xcz2EaUSkJI2fVZRgEVMZy4bkMp9yDME6xk9W8r+Lf+6pb+gXrc957FeS2?=
 =?us-ascii?Q?hFz7PYrQQwekOM3oEmkXGeOTnLWA/DSmeAEgbGXPxWDEAi8aHUMQdIGFWRsg?=
 =?us-ascii?Q?JcitUfVODKItoc7u6rUgIgkQ7u/C9TG+Fv+iZsxONhViNyOmC2crrjie2GeB?=
 =?us-ascii?Q?UAV+DDU2MZgUx5tze5CfNQieeNODUYv8B/BHRJRjEHsI9wm5Z0fHY00386in?=
 =?us-ascii?Q?30r+/mcXy/GHKOoDHdTqCa7uIT/4rIegpNY/9nEBs5vnUsIqn0yqzI+bC6J2?=
 =?us-ascii?Q?W0HCFuGgYl2c1/0nYGtJvmnlVVyroC+F7P0HNEw11Z6O3QAndi0AETPUqlb/?=
 =?us-ascii?Q?Q7w948RSbg5xrwp3c+OAVwVTslHK0B9APpGPABgzAhDtrIavydcIlF4cvi0B?=
 =?us-ascii?Q?bim4/6Rk1mBINfzZ1yVa9DEnqK2z1P1pBdJ1anepGI+LONLsqjW7VWV13uJt?=
 =?us-ascii?Q?0Z8JTW9XYBqZrw0EazsiqJKrXB7UPnJFB0vjUg2gpOwVT9mn/yv9iqS1FSNn?=
 =?us-ascii?Q?kIqqtMjj4Cgbr57R/yb40v6qTJwPwahepfU8ve/Q7xgdSjki0QkD327LnAK2?=
 =?us-ascii?Q?5g=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 2efbfd4c-9549-4e4e-ec1e-08da60e9e0da
X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6059.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jul 2022 13:58:05.3094
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: BYxaSx1VY/vfO8fMR+s6fwg0tEg2SYfSEP6tpcCVHpujU+5hNRKX1jkL6C6ChgDpHV1ghW28/h+rpAxDS+/RjQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR1101MB2197
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,SPF_NONE,
        T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Fri, Jul 08, 2022 at 03:34:43PM +0200, Greg Kroah-Hartman wrote:
> On Wed, Jul 06, 2022 at 02:43:31PM +0300, Alexander Usyskin wrote:
> > Add GSC support for XeHP SDV and DG2 platforms.
> > 
> > The series includes changes for the mei driver:
> > - add ability to use polling instead of interrupts
> > - add ability to use extended timeouts
> > - setup extended operational memory for GSC
> > 
> > The series includes changes for the i915 driver:
> > - allocate extended operational memory for GSC
> > - GSC on XeHP SDV offsets and definitions
> > 
> > Greg KH, please review and ACK the MEI patches.
> > We are pushing these patches through gfx tree as
> > the auxiliary device belongs there.
> > 
> > V2: rebase over merged DG1 series and DG2 enablement patch,
> >     fix commit messages
> > 
> > V3: rebase over latest tip
> > 
> > V4: add missed changelog in pxp dbugfs patch
> > 
> > V5: rebase over latest tip
> >     fix changelog in pxp dbugfs patch
> >     put HAX patch last to the ease of merging
> 
> You did more than just this from v4 to v5 :(
> 
> It's as if you want to make it hard to review these...

I just checked the code and it looks the same to me.

well, yeap, changing the order of other commits during the rebase
was not mentioned. So we don't know the reason...

But at least now the HAX is the last patch what makes more sense.

But I don't believe this should block the review and require a v6
just to add this comment in the cover letter, or it should?

Rodrigo.

> 
> greg k-h