aboutsummaryrefslogtreecommitdiff
path: root/src/drivers
diff options
context:
space:
mode:
authorMatt DeVillier <matt.devillier@puri.sm>2020-11-19 18:29:44 -0600
committerPatrick Georgi <pgeorgi@google.com>2020-11-22 22:22:00 +0000
commit86dce8fa75233c9e6639962e0f402adcd6530e66 (patch)
tree6408e7a085b1ba6bea313e381149e4d12a50680a /src/drivers
parent9226fe9aee7c06bce69881ca41393fdc8964f08f (diff)
drivers/i2c/generic: Only write DDN field if description not empty
DDN field isn't required, no point in writing an empty string to it. Change-Id: Ifea6e48c324598f114178e86a79f519ee35f5258 Signed-off-by: Matt DeVillier <matt.devillier@puri.sm> Reviewed-on: https://review.coreboot.org/c/coreboot/+/47781 Reviewed-by: Furquan Shaikh <furquan@google.com> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/drivers')
-rw-r--r--src/drivers/i2c/generic/generic.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/drivers/i2c/generic/generic.c b/src/drivers/i2c/generic/generic.c
index cd7406893b..1c7267e897 100644
--- a/src/drivers/i2c/generic/generic.c
+++ b/src/drivers/i2c/generic/generic.c
@@ -72,7 +72,8 @@ void i2c_generic_fill_ssdt(const struct device *dev,
if (config->cid)
acpigen_write_name_string("_CID", config->cid);
acpigen_write_name_integer("_UID", config->uid);
- acpigen_write_name_string("_DDN", config->desc);
+ if (config->desc)
+ acpigen_write_name_string("_DDN", config->desc);
acpigen_write_STA(acpi_device_status(dev));
/* Resources */