Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/58174 )
Change subject: drivers/intel/dptf: return package with value ......................................................................
drivers/intel/dptf: return package with value
Return the package with a value for the dptf user space service. This is required in write tpch method for pch device under dptf driver.
BUG=b:198582766 BRANCH=None TEST=Build FW and test on brya0 board
Change-Id: I64e1bb04a6115c7f93c84a5d6644101ac1d3d8ba Signed-off-by: Sumeet Pawnikar sumeet.r.pawnikar@intel.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/58174 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Tim Wawrzynczak twawrzynczak@chromium.org --- M src/drivers/intel/dptf/dptf.c 1 file changed, 2 insertions(+), 1 deletion(-)
Approvals: build bot (Jenkins): Verified Tim Wawrzynczak: Looks good to me, approved
diff --git a/src/drivers/intel/dptf/dptf.c b/src/drivers/intel/dptf/dptf.c index 4a65127..c6ead0f 100644 --- a/src/drivers/intel/dptf/dptf.c +++ b/src/drivers/intel/dptf/dptf.c @@ -227,7 +227,8 @@ acpigen_write_zero(); /* The reason for returning a value here is a W/A for the ESIF shell */ acpigen_emit_byte(RETURN_OP); - acpigen_write_package(0); + acpigen_write_package(1); + acpigen_write_zero(); acpigen_write_package_end(); acpigen_write_method_end(); }