linux-kernel - RE: [PATCH v7 0/9] Introducing firmware late binding

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CY5PR11MB6211D6829C6B3A39D2F48013954EA@CY5PR11MB6211.namprd11.prod.outlook.com>
Date: Tue, 8 Jul 2025 10:57:57 +0000
From: "Gupta, Anshuman" <anshuman.gupta@...el.com>
To: Greg KH <gregkh@...uxfoundation.org>, "Nilawar, Badal"
	<badal.nilawar@...el.com>
CC: "intel-xe@...ts.freedesktop.org" <intel-xe@...ts.freedesktop.org>,
	"dri-devel@...ts.freedesktop.org" <dri-devel@...ts.freedesktop.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "Vivi,
 Rodrigo" <rodrigo.vivi@...el.com>, "Usyskin, Alexander"
	<alexander.usyskin@...el.com>, "Ceraolo Spurio, Daniele"
	<daniele.ceraolospurio@...el.com>
Subject: RE: [PATCH v7 0/9] Introducing firmware late binding



> -----Original Message-----
> From: Greg KH <gregkh@...uxfoundation.org>
> Sent: Tuesday, July 8, 2025 12:19 PM
> To: Nilawar, Badal <badal.nilawar@...el.com>
> Cc: intel-xe@...ts.freedesktop.org; dri-devel@...ts.freedesktop.org; linux-
> kernel@...r.kernel.org; Gupta, Anshuman <anshuman.gupta@...el.com>;
> Vivi, Rodrigo <rodrigo.vivi@...el.com>; Usyskin, Alexander
> <alexander.usyskin@...el.com>; Ceraolo Spurio, Daniele
> <daniele.ceraolospurio@...el.com>
> Subject: Re: [PATCH v7 0/9] Introducing firmware late binding
> 
> On Tue, Jul 08, 2025 at 12:42:28AM +0530, Badal Nilawar wrote:
> > v7:
> >  xe_kmd:
> >  - resolved kernel doc warnings
> >  mei:
> >  - Address v6 review comments (greg kh)
> 
> I don't know what comments you addressed, please be specific, as in the past
> other ones have not been addressed (i.e. the .owner stuff) and I had to find
> that review again...
V7 series patch changelog has the details of addressed review comment.
https://patchwork.freedesktop.org/patch/662505/?series=151202&rev=3
---
v2:
 - Use generic naming (Jani)
 - Drop xe_late_bind_component struct to move to xe code (Daniele/Sasha)
v3:
 - Updated kconfig description
 - Move CSC late binding specific flags/defines to late_bind_mei_interface.h (Daniele)
 - Add match for PCI_CLASS_DISPLAY_OTHER to support headless cards (Anshuman)
v4:
 - Add fixes in push_config (Sasha)
 - Use INTEL_ prefix for component, refine doc,
   add status enum to headerlate_bind_mei_interface.h (Anshuman)
v5:
 - Addressed review comments
v7(greh kh):
 - dropped sub directory
 - dropped .owner from late_bind_component_ops
 - In csc_heci_late_bind_req/rsp headers
   - used __le32
   - updated doc for reserved fields
 - used conversion macros le32_to_cpu, cpu_to_le32
 - fixed duplicate print in dev_dbg
---

It seems Latest series[*] after dropping patch 10 does not have details of addressed review comment.
* https://patchwork.freedesktop.org/series/151290/ 

Thanks,
Anshuman
> 
> thanks,
> 
> greg k-h