Message ID | 20211115154201.46579-3-andriy.shevchenko@linux.intel.com |
---|---|
State | Accepted |
Headers | show |
Series | [v1,1/3] i2c: mux: gpio: Replace custom acpi_get_local_address() | expand |
On Mon, Nov 15, 2021 at 7:42 AM Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote: > > Use array_size() helper to aid in 2-factor allocation instances. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Thanks for the cleanup series! Reviewed-by: Evan Green <evgreen@chromium.org>
On 2021-11-15 16:42, Andy Shevchenko wrote: > Use array_size() helper to aid in 2-factor allocation instances. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Acked-by: Peter Rosin <peda@axentia.se> Cheers, Peter
On Mon, Nov 15, 2021 at 05:42:01PM +0200, Andy Shevchenko wrote: > Use array_size() helper to aid in 2-factor allocation instances. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> Applied to for-next, thanks!
diff --git a/drivers/i2c/muxes/i2c-mux-gpio.c b/drivers/i2c/muxes/i2c-mux-gpio.c index b09c10f36ddb..73a23e117ebe 100644 --- a/drivers/i2c/muxes/i2c-mux-gpio.c +++ b/drivers/i2c/muxes/i2c-mux-gpio.c @@ -7,6 +7,7 @@ #include <linux/i2c.h> #include <linux/i2c-mux.h> +#include <linux/overflow.h> #include <linux/platform_data/i2c-mux-gpio.h> #include <linux/platform_device.h> #include <linux/module.h> @@ -152,7 +153,7 @@ static int i2c_mux_gpio_probe(struct platform_device *pdev) return -EPROBE_DEFER; muxc = i2c_mux_alloc(parent, &pdev->dev, mux->data.n_values, - ngpios * sizeof(*mux->gpios), 0, + array_size(ngpios, sizeof(*mux->gpios)), 0, i2c_mux_gpio_select, NULL); if (!muxc) { ret = -ENOMEM;
Use array_size() helper to aid in 2-factor allocation instances. Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> --- drivers/i2c/muxes/i2c-mux-gpio.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)