2 comments:
File src/drivers/intel/fsp1_1/logo.c:
Patch Set #4, Line 29: if (logo_size)
Should this be `*logo_size`? A null-pointer check after dereference seems odd.
Thanks for locating this issue. TI will create a patch to correct this issue.
File src/drivers/intel/fsp1_1/ramstage.c:
I doubt this will ever be used.
It is used in the Quark X1000 case. This doesn't provide the option to pass in a logo. Other than that you are right. I don't expect any new FSP1.1 based chipsets and the Braswell is covered.
To view, visit change 37791. To unsubscribe, or for help writing mail filters, visit settings.