iio: dac: Convert powerdown read callbacks to sysfs_emit()
Update DAC drivers powerdown attribute show callback to use the new sysfs_emit() function. sysfs_emit() is preferred over raw s*printf() for sysfs attributes since it knows about the sysfs buffer specifics and has some built-in sanity checks. Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Link: https://lore.kernel.org/r/20210320071405.9347-5-lars@metafoo.de Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
This commit is contained in:
parent
6b92ba0a30
commit
f46ac00978
@ -277,7 +277,7 @@ static ssize_t ad5064_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5064_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", st->pwr_down[chan->channel]);
|
||||
return sysfs_emit(buf, "%d\n", st->pwr_down[chan->channel]);
|
||||
}
|
||||
|
||||
static ssize_t ad5064_write_dac_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -255,7 +255,7 @@ static ssize_t ad5360_read_dac_powerdown(struct device *dev,
|
||||
struct iio_dev *indio_dev = dev_to_iio_dev(dev);
|
||||
struct ad5360_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", (bool)(st->ctrl & AD5360_SF_CTRL_PWR_DOWN));
|
||||
return sysfs_emit(buf, "%d\n", (bool)(st->ctrl & AD5360_SF_CTRL_PWR_DOWN));
|
||||
}
|
||||
|
||||
static int ad5360_update_ctrl(struct iio_dev *indio_dev, unsigned int set,
|
||||
|
@ -85,7 +85,7 @@ static ssize_t ad5380_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5380_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", st->pwr_down);
|
||||
return sysfs_emit(buf, "%d\n", st->pwr_down);
|
||||
}
|
||||
|
||||
static ssize_t ad5380_write_dac_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -100,7 +100,7 @@ static ssize_t ad5446_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5446_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", st->pwr_down);
|
||||
return sysfs_emit(buf, "%d\n", st->pwr_down);
|
||||
}
|
||||
|
||||
static ssize_t ad5446_write_dac_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -170,8 +170,8 @@ static ssize_t ad5504_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5504_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n",
|
||||
!(st->pwr_down_mask & (1 << chan->channel)));
|
||||
return sysfs_emit(buf, "%d\n",
|
||||
!(st->pwr_down_mask & (1 << chan->channel)));
|
||||
}
|
||||
|
||||
static ssize_t ad5504_write_dac_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -117,8 +117,8 @@ static ssize_t ad5624r_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5624r_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n",
|
||||
!!(st->pwr_down_mask & (1 << chan->channel)));
|
||||
return sysfs_emit(buf, "%d\n",
|
||||
!!(st->pwr_down_mask & (1 << chan->channel)));
|
||||
}
|
||||
|
||||
static ssize_t ad5624r_write_dac_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -57,7 +57,7 @@ static ssize_t ad5686_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5686_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", !!(st->pwr_down_mask &
|
||||
return sysfs_emit(buf, "%d\n", !!(st->pwr_down_mask &
|
||||
(0x3 << (chan->channel * 2))));
|
||||
}
|
||||
|
||||
|
@ -399,8 +399,8 @@ static ssize_t ad5755_read_powerdown(struct iio_dev *indio_dev, uintptr_t priv,
|
||||
{
|
||||
struct ad5755_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n",
|
||||
(bool)(st->pwr_down & (1 << chan->channel)));
|
||||
return sysfs_emit(buf, "%d\n",
|
||||
(bool)(st->pwr_down & (1 << chan->channel)));
|
||||
}
|
||||
|
||||
static ssize_t ad5755_write_powerdown(struct iio_dev *indio_dev, uintptr_t priv,
|
||||
|
@ -574,7 +574,7 @@ static ssize_t ad5758_read_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5758_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", st->pwr_down);
|
||||
return sysfs_emit(buf, "%d\n", st->pwr_down);
|
||||
}
|
||||
|
||||
static ssize_t ad5758_write_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -433,7 +433,7 @@ static ssize_t ad5770r_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5770r_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", st->ch_pwr_down[chan->channel]);
|
||||
return sysfs_emit(buf, "%d\n", st->ch_pwr_down[chan->channel]);
|
||||
}
|
||||
|
||||
static ssize_t ad5770r_write_dac_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -177,7 +177,7 @@ static ssize_t ad5791_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad5791_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", st->pwr_down);
|
||||
return sysfs_emit(buf, "%d\n", st->pwr_down);
|
||||
}
|
||||
|
||||
static ssize_t ad5791_write_dac_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -65,7 +65,7 @@ static ssize_t ad7303_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ad7303_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", (bool)(st->config &
|
||||
return sysfs_emit(buf, "%d\n", (bool)(st->config &
|
||||
AD7303_CFG_POWER_DOWN(chan->channel)));
|
||||
}
|
||||
|
||||
|
@ -135,8 +135,8 @@ static ssize_t ltc2632_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ltc2632_state *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n",
|
||||
!!(st->powerdown_cache_mask & (1 << chan->channel)));
|
||||
return sysfs_emit(buf, "%d\n",
|
||||
!!(st->powerdown_cache_mask & (1 << chan->channel)));
|
||||
}
|
||||
|
||||
static ssize_t ltc2632_write_dac_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -84,7 +84,7 @@ static ssize_t max5821_read_dac_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct max5821_data *st = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", st->powerdown[chan->channel]);
|
||||
return sysfs_emit(buf, "%d\n", st->powerdown[chan->channel]);
|
||||
}
|
||||
|
||||
static int max5821_sync_powerdown_mode(struct max5821_data *data,
|
||||
|
@ -167,7 +167,7 @@ static ssize_t mcp4725_read_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct mcp4725_data *data = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", data->powerdown);
|
||||
return sysfs_emit(buf, "%d\n", data->powerdown);
|
||||
}
|
||||
|
||||
static ssize_t mcp4725_write_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -210,7 +210,7 @@ static ssize_t stm32_dac_read_powerdown(struct iio_dev *indio_dev,
|
||||
if (ret < 0)
|
||||
return ret;
|
||||
|
||||
return sprintf(buf, "%d\n", ret ? 0 : 1);
|
||||
return sysfs_emit(buf, "%d\n", ret ? 0 : 1);
|
||||
}
|
||||
|
||||
static ssize_t stm32_dac_write_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -121,7 +121,7 @@ static ssize_t ti_dac_read_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ti_dac_chip *ti_dac = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", ti_dac->powerdown);
|
||||
return sysfs_emit(buf, "%d\n", ti_dac->powerdown);
|
||||
}
|
||||
|
||||
static ssize_t ti_dac_write_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -166,7 +166,7 @@ static ssize_t dac5571_read_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct dac5571_data *data = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", data->powerdown[chan->channel]);
|
||||
return sysfs_emit(buf, "%d\n", data->powerdown[chan->channel]);
|
||||
}
|
||||
|
||||
static ssize_t dac5571_write_powerdown(struct iio_dev *indio_dev,
|
||||
|
@ -110,7 +110,7 @@ static ssize_t ti_dac_read_powerdown(struct iio_dev *indio_dev,
|
||||
{
|
||||
struct ti_dac_chip *ti_dac = iio_priv(indio_dev);
|
||||
|
||||
return sprintf(buf, "%d\n", ti_dac->powerdown);
|
||||
return sysfs_emit(buf, "%d\n", ti_dac->powerdown);
|
||||
}
|
||||
|
||||
static ssize_t ti_dac_write_powerdown(struct iio_dev *indio_dev,
|
||||
|
Loading…
Reference in New Issue
Block a user