diff options
| author | Hartmut Knaack <knaack.h@gmx.de> | 2015-08-28 23:59:55 +0200 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2015-10-11 16:42:41 +0200 |
| commit | 7d2da8eaa92585769dfbafe285b94c8f743510f6 (patch) | |
| tree | b9fb5cac3567b9356ca005f0a8d4bd81f75cedc6 /drivers/iio | |
| parent | iio:dac:m62332: shutdown on remove (diff) | |
| download | linux-7d2da8eaa92585769dfbafe285b94c8f743510f6.tar.xz linux-7d2da8eaa92585769dfbafe285b94c8f743510f6.zip | |
iio:dac:m62332: use ARRAY_SIZE
Make use of ARRAY_SIZE to prevent buffer issues.
Signed-off-by: Hartmut Knaack <knaack.h@gmx.de>
Acked-by: Daniel Baluta <daniel.baluta@intel.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
Diffstat (limited to 'drivers/iio')
| -rw-r--r-- | drivers/iio/dac/m62332.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/iio/dac/m62332.c b/drivers/iio/dac/m62332.c index c61720de8606..fe750982b502 100644 --- a/drivers/iio/dac/m62332.c +++ b/drivers/iio/dac/m62332.c @@ -62,8 +62,8 @@ static int m62332_set_value(struct iio_dev *indio_dev, goto out; } - res = i2c_master_send(client, outbuf, 2); - if (res >= 0 && res != 2) + res = i2c_master_send(client, outbuf, ARRAY_SIZE(outbuf)); + if (res >= 0 && res != ARRAY_SIZE(outbuf)) res = -EIO; if (res < 0) goto out; @@ -212,7 +212,7 @@ static int m62332_probe(struct i2c_client *client, /* establish that the iio_dev is a child of the i2c device */ indio_dev->dev.parent = &client->dev; - indio_dev->num_channels = M62332_CHANNELS; + indio_dev->num_channels = ARRAY_SIZE(m62332_channels); indio_dev->channels = m62332_channels; indio_dev->modes = INDIO_DIRECT_MODE; indio_dev->info = &m62332_info; |