| Message ID | 20240928161108.163647-1-abhashkumarjha123@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | iio: adc: mt6360-adc: Converted to use get_unaligned_be16() | expand |
On Sat, 28 Sep 2024 21:41:08 +0530 Abhash Jha <abhashkumarjha123@gmail.com> wrote: > Changed the manual shifting and adding of bytes to use > get_unaligned_be16() api instead. > > Signed-off-by: Abhash Jha <abhashkumarjha123@gmail.com> Looks simple so I've applied it, but plenty of time for additional reviews to come in. Jonathan > --- > drivers/iio/adc/mt6360-adc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/adc/mt6360-adc.c b/drivers/iio/adc/mt6360-adc.c > index 3710473e5..91befe2cd 100644 > --- a/drivers/iio/adc/mt6360-adc.c > +++ b/drivers/iio/adc/mt6360-adc.c > @@ -124,7 +124,7 @@ static int mt6360_adc_read_channel(struct mt6360_adc_data *mad, int channel, int > usleep_range(ADC_LOOP_TIME_US / 2, ADC_LOOP_TIME_US); > } > > - *val = rpt[1] << 8 | rpt[2]; > + *val = get_unaligned_be16(&rpt[1]); > ret = IIO_VAL_INT; > > out_adc_conv:
diff --git a/drivers/iio/adc/mt6360-adc.c b/drivers/iio/adc/mt6360-adc.c index 3710473e5..91befe2cd 100644 --- a/drivers/iio/adc/mt6360-adc.c +++ b/drivers/iio/adc/mt6360-adc.c @@ -124,7 +124,7 @@ static int mt6360_adc_read_channel(struct mt6360_adc_data *mad, int channel, int usleep_range(ADC_LOOP_TIME_US / 2, ADC_LOOP_TIME_US); } - *val = rpt[1] << 8 | rpt[2]; + *val = get_unaligned_be16(&rpt[1]); ret = IIO_VAL_INT; out_adc_conv:
Changed the manual shifting and adding of bytes to use get_unaligned_be16() api instead. Signed-off-by: Abhash Jha <abhashkumarjha123@gmail.com> --- drivers/iio/adc/mt6360-adc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)