Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/83301?usp=email )
Change subject: drivers/spi/acpi: Update generic property list ......................................................................
drivers/spi/acpi: Update generic property list
Update generic property list for build test result fail https://qa.coreboot.org/job/coreboot-gerrit/259702/
BUG=b:342932183 BRANCH=None TEST=emerge-brox coreboot
Change-Id: Iecd8573343706184dce5edfc12fe7a143390e0e9 Signed-off-by: Kun Liu liukun11@huaqin.corp-partner.google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/83301 Reviewed-by: Subrata Banik subratabanik@google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Jian Tong tongjian@huaqin.corp-partner.google.com Reviewed-by: cong yang yangcong5@huaqin.corp-partner.google.com --- M src/drivers/spi/acpi/acpi.c 1 file changed, 2 insertions(+), 1 deletion(-)
Approvals: Jian Tong: Looks good to me, but someone else must approve build bot (Jenkins): Verified Subrata Banik: Looks good to me, approved cong yang: Looks good to me, approved
diff --git a/src/drivers/spi/acpi/acpi.c b/src/drivers/spi/acpi/acpi.c index 687dd52..bd3f89a 100644 --- a/src/drivers/spi/acpi/acpi.c +++ b/src/drivers/spi/acpi/acpi.c @@ -149,7 +149,8 @@ enable_gpio_index, 0, config->enable_gpio.active_low); /* Add generic property list */ - acpi_dp_add_property_list(dsd, config->property_list, + if (config->property_count > 0) + acpi_dp_add_property_list(dsd, config->property_list, config->property_count); acpi_dp_write(dsd); }