On 04/13/2017 11:54 PM, plroskin@xxxxxxxxx wrote: > From: Pavel Roskin <plroskin@xxxxxxxxx> > > realbits, storagebits and shift should be numbers, not ASCII characters. > > Signed-off-by: Pavel Roskin <plroskin@xxxxxxxxx> Thanks for the fix. Reviewed-by: Lars-Peter Clausen <lars@xxxxxxxxxx> You really have to wonder how somebody would be able to get that wrong in the first place... ;) > --- > drivers/iio/dac/ad7303.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/iio/dac/ad7303.c b/drivers/iio/dac/ad7303.c > index e690dd11e99f..4b0f942b8914 100644 > --- a/drivers/iio/dac/ad7303.c > +++ b/drivers/iio/dac/ad7303.c > @@ -184,9 +184,9 @@ static const struct iio_chan_spec_ext_info ad7303_ext_info[] = { > .address = (chan), \ > .scan_type = { \ > .sign = 'u', \ > - .realbits = '8', \ > - .storagebits = '8', \ > - .shift = '0', \ > + .realbits = 8, \ > + .storagebits = 8, \ > + .shift = 0, \ > }, \ > .ext_info = ad7303_ext_info, \ > } > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html