On Thu, Jun 9, 2022 at 1:44 AM Jakob Hauser <jahau@xxxxxxxxxxxxxx> wrote: > > The "hard_offsets" are currently unsigned u8 but they should be signed as they > can get negative. They are signed in function yas5xx_meaure_offsets() and in the > Yamaha drivers [1][2]. > > [1] https://github.com/NovaFusion/android_kernel_samsung_golden/blob/cm-12.1/drivers/sensor/compass/yas.h#L156 > [2] https://github.com/msm8916-mainline/android_kernel_qcom_msm8916/blob/GT-I9195I/drivers/iio/magnetometer/yas_mag_drv-yas532.c#L91 > > Fixes: de8860b1ed47 ("iio: magnetometer: Add driver for Yamaha YAS530") The series should be organized like this: 1) definite fixes; 2) cleanups / refactoring; 3) new features / documentation. It will give a maintainer easier times to handle it. Code looks good to me. Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > Cc: Linus Walleij <linus.walleij@xxxxxxxxxx> > Signed-off-by: Jakob Hauser <jahau@xxxxxxxxxxxxxx> > --- > drivers/iio/magnetometer/yamaha-yas530.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/iio/magnetometer/yamaha-yas530.c b/drivers/iio/magnetometer/yamaha-yas530.c > index 2e8d20b05217..9bfb3b573907 100644 > --- a/drivers/iio/magnetometer/yamaha-yas530.c > +++ b/drivers/iio/magnetometer/yamaha-yas530.c > @@ -133,7 +133,7 @@ struct yas5xx { > unsigned int version; > char name[16]; > struct yas5xx_calibration calibration; > - u8 hard_offsets[3]; > + s8 hard_offsets[3]; > struct iio_mount_matrix orientation; > struct regmap *map; > struct regulator_bulk_data regs[2]; > -- > 2.35.1 > -- With Best Regards, Andy Shevchenko
