[coreboot-gerrit] Change in ...coreboot[master]: Revert "vboot: Fix linking error with USE_OPTION_TABLE enabled"

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30904 )

Change subject: Revert "vboot: Fix linking error with USE_OPTION_TABLE enabled"
......................................................................

Patch Set 3:
> Patch Set 3:
> > Patch Set 3:
> > > Patch Set 3:
> > > > Patch Set 3:
> > > >
> > > > Thinkpad xx20 series usb_always_on. Replying at the request of Hellsenberg.
> > >
> > > Apologies, can confirm that this fixes the broken USB_ALWAYS_ON option on Thinkpad xx20. Confirmed with a T420 and T520.
> >
> > Please post the commit hash and console log. The question is if the new (non SMM) method is affected, or the old (SMM) method is.
>
> Sorry it took so long, here's a paste of the CBMEM log of both Master and the Revert
>
> Master: https://paste.ee/p/pd1kG
> Revert: https://paste.ee/p/9JcpM
>
> I wasn't able prevent the log from being truncated, sorry. Both are around 68KB, so I hope this helps.

Great !
Both logs show that the cmos_layout.bin can be located using the different methods.
Please explain why this change is necessary, as the log(s) doesn't show any issue.