On Mon, May 17, 2021 at 02:57:33PM +0300, Jani Nikula wrote:
On Mon, 12 Apr 2021, Matthew Auld matthew.auld@intel.com wrote:
From: Anshuman Gupta anshuman.gupta@intel.com
Sanitize OPROM header, CPD signature and OPROM PCI version. OPROM_HEADER, EXPANSION_ROM_HEADER and OPROM_MEU_BLOB structures and PCI struct offsets are provided by GSC counterparts. These are yet to be Documented in B.Spec. After successful sanitization, extract VBT from opregion image.
So I don't understand what the point is with two consecutive patches where the latter rewrites a lot of the former.
I actually wonder what's the point of this. Getting it from spi is already the fallback and looks much more complex. Yes, it's pretty detailed and document the format pretty well, but it still looks more complex than the initial code. Do you see additional benefit in this one?
Lucas De Marchi