diff options
| author | Simon Glass <sjg@chromium.org> | 2020-11-04 09:57:29 -0700 |
|---|---|---|
| committer | Bin Meng <bmeng.cn@gmail.com> | 2020-11-06 09:51:29 +0800 |
| commit | dd30c5bb574aba55e99284dbdfe0d6033bf44d7f (patch) | |
| tree | 9e3a21ede0f6748d5a18dd7238f315e016a7013b | |
| parent | a0ed800376b0db57428f9bf6401db36789ff1aa6 (diff) | |
| download | u-boot-dd30c5bb574aba55e99284dbdfe0d6033bf44d7f.zip u-boot-dd30c5bb574aba55e99284dbdfe0d6033bf44d7f.tar.gz u-boot-dd30c5bb574aba55e99284dbdfe0d6033bf44d7f.tar.bz2 | |
x86: sound: Correct error handling
A few functions have changed to return pin numbers or I2C addresses. The
error checking for some of the callers is therefore wrong. Fix them.
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
| -rw-r--r-- | drivers/sound/da7219.c | 4 | ||||
| -rw-r--r-- | drivers/sound/max98357a.c | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/drivers/sound/da7219.c b/drivers/sound/da7219.c index 6bc1ad0..8d674bc 100644 --- a/drivers/sound/da7219.c +++ b/drivers/sound/da7219.c @@ -54,13 +54,13 @@ static int da7219_acpi_fill_ssdt(const struct udevice *dev, acpigen_write_name(ctx, "_CRS"); acpigen_write_resourcetemplate_header(ctx); ret = acpi_device_write_i2c_dev(ctx, dev); - if (ret) + if (ret < 0) return log_msg_ret("i2c", ret); /* Use either Interrupt() or GpioInt() */ ret = acpi_device_write_interrupt_or_gpio(ctx, (struct udevice *)dev, "req-gpios"); - if (ret) + if (ret < 0) return log_msg_ret("irq_gpio", ret); acpigen_write_resourcetemplate_footer(ctx); diff --git a/drivers/sound/max98357a.c b/drivers/sound/max98357a.c index 827262d..b3d27a3 100644 --- a/drivers/sound/max98357a.c +++ b/drivers/sound/max98357a.c @@ -69,7 +69,7 @@ static int max98357a_acpi_fill_ssdt(const struct udevice *dev, acpigen_write_name(ctx, "_CRS"); acpigen_write_resourcetemplate_header(ctx); ret = acpi_device_write_gpio_desc(ctx, &priv->sdmode_gpio); - if (ret) + if (ret < 0) return log_msg_ret("gpio", ret); acpigen_write_resourcetemplate_footer(ctx); |