On Mon, May 17, 2021 at 02:57:33PM +0300, Jani Nikula wrote:
On Mon, 12 Apr 2021, Matthew Auld <matthew.auld@xxxxxxxxx> wrote:
From: Anshuman Gupta <anshuman.gupta@xxxxxxxxx>
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
