Re: [PATCH v1 07/13] iio: chemical: bme680: add regulators

From: Andy Shevchenko
Date: Fri Oct 11 2024 - 06:05:21 EST


On Thu, Oct 10, 2024 at 11:00:24PM +0200, vamoirid wrote:
> From: Vasileios Amoiridis <vassilisamir@xxxxxxxxx>
>
> Add support for the regulators described in the dt-binding.

...

> +static const char *const bme680_supply_names[] = {
> + "vdd", "vddio"

Leave trailing comma.

> +};

...

> + devm_add_action_or_reset(dev, bme680_regulators_disable, data->supplies);

No error check?! Why?

--
With Best Regards,
Andy Shevchenko