Arthur Heymans has submitted this change. ( https://review.coreboot.org/c/coreboot/+/66264 )
Change subject: soc/intel/meteorlake: Fix set but unused variable ......................................................................
soc/intel/meteorlake: Fix set but unused variable
Clang complains about this.
Change-Id: Ibe1de3057c17b4aa8ecbd87fac598e43294584e3 Signed-off-by: Arthur Heymans arthur@aheymans.xyz Reviewed-on: https://review.coreboot.org/c/coreboot/+/66264 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Angel Pons th3fanbus@gmail.com --- M src/soc/intel/meteorlake/systemagent.c 1 file changed, 17 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Angel Pons: Looks good to me, approved
diff --git a/src/soc/intel/meteorlake/systemagent.c b/src/soc/intel/meteorlake/systemagent.c index 675f9fb..c6c1ada 100644 --- a/src/soc/intel/meteorlake/systemagent.c +++ b/src/soc/intel/meteorlake/systemagent.c @@ -83,6 +83,8 @@ return; }
+ /* Remove once commented line below is enabled */ + (void)soc_config; /* UPDATEME: Need to enable later */ //set_power_limits(MOBILE_SKU_PL1_TIME_SEC, soc_config); }