Arthur Heymans has submitted this change. ( https://review.coreboot.org/c/coreboot/+/84041?usp=email )
(
7 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: drivers/intel/opregion.c: Also set vbt_size if size is 0 ......................................................................
drivers/intel/opregion.c: Also set vbt_size if size is 0
Make sure size vbt_size is initialized. GCC LTO warns about this.
Change-Id: I4fcc6c02f898640e9b40d769e1165a4a0fb0fdf2 Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/84041 Reviewed-by: Nico Huber nico.h@gmx.de Reviewed-by: Angel Pons th3fanbus@gmail.com Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Nicholas Sudsgaard devel+coreboot@nsudsgaard.com --- M src/drivers/intel/gma/opregion.c 1 file changed, 1 insertion(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Nico Huber: Looks good to me, approved Angel Pons: Looks good to me, approved Nicholas Sudsgaard: Looks good to me, approved
diff --git a/src/drivers/intel/gma/opregion.c b/src/drivers/intel/gma/opregion.c index d551638..f97d373 100644 --- a/src/drivers/intel/gma/opregion.c +++ b/src/drivers/intel/gma/opregion.c @@ -49,7 +49,7 @@ size = 0;
out: - if (vbt_size && size) + if (vbt_size) *vbt_size = size; return data; }