Re: [PATCH v2] iio: cros_ec: Fix the maths for gyro scale calculation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Gwendal,

Thank you for the patch! Perhaps something to improve:

[auto build test WARNING on iio/togreg]
[also build test WARNING on v5.0 next-20190306]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Enric-Balletbo-i-Serra/iio-cros_ec-Fix-the-maths-for-gyro-scale-calculation/20190310-155057
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
config: i386-randconfig-x005-201910 (attached as .config)
compiler: gcc-8 (Debian 8.3.0-2) 8.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c: In function 'cros_ec_sensors_read':
>> drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c:107:4: warning: this decimal constant is unsigned only in ISO C90
       *val2 = div_s64(val64 * 3141592653,
       ^

vim +107 drivers/iio/common/cros_ec_sensors/cros_ec_sensors.c

    44	
    45	static int cros_ec_sensors_read(struct iio_dev *indio_dev,
    46				  struct iio_chan_spec const *chan,
    47				  int *val, int *val2, long mask)
    48	{
    49		struct cros_ec_sensors_state *st = iio_priv(indio_dev);
    50		s16 data = 0;
    51		s64 val64;
    52		int i;
    53		int ret;
    54		int idx = chan->scan_index;
    55	
    56		mutex_lock(&st->core.cmd_lock);
    57	
    58		switch (mask) {
    59		case IIO_CHAN_INFO_RAW:
    60			ret = st->core.read_ec_sensors_data(indio_dev, 1 << idx, &data);
    61			if (ret < 0)
    62				break;
    63			ret = IIO_VAL_INT;
    64			*val = data;
    65			break;
    66		case IIO_CHAN_INFO_CALIBBIAS:
    67			st->core.param.cmd = MOTIONSENSE_CMD_SENSOR_OFFSET;
    68			st->core.param.sensor_offset.flags = 0;
    69	
    70			ret = cros_ec_motion_send_host_cmd(&st->core, 0);
    71			if (ret < 0)
    72				break;
    73	
    74			/* Save values */
    75			for (i = CROS_EC_SENSOR_X; i < CROS_EC_SENSOR_MAX_AXIS; i++)
    76				st->core.calib[i] =
    77					st->core.resp->sensor_offset.offset[i];
    78			ret = IIO_VAL_INT;
    79			*val = st->core.calib[idx];
    80			break;
    81		case IIO_CHAN_INFO_SCALE:
    82			st->core.param.cmd = MOTIONSENSE_CMD_SENSOR_RANGE;
    83			st->core.param.sensor_range.data = EC_MOTION_SENSE_NO_VALUE;
    84	
    85			ret = cros_ec_motion_send_host_cmd(&st->core, 0);
    86			if (ret < 0)
    87				break;
    88	
    89			val64 = st->core.resp->sensor_range.ret;
    90			switch (st->core.type) {
    91			case MOTIONSENSE_TYPE_ACCEL:
    92				/*
    93				 * EC returns data in g, iio exepects m/s^2.
    94				 * Do not use IIO_G_TO_M_S_2 to avoid precision loss.
    95				 */
    96				*val = div_s64(val64 * 980665, 10);
    97				*val2 = 10000 << (CROS_EC_SENSOR_BITS - 1);
    98				ret = IIO_VAL_FRACTIONAL;
    99				break;
   100			case MOTIONSENSE_TYPE_GYRO:
   101				/*
   102				 * EC returns data in dps, iio expects rad/s.
   103				 * Do not use IIO_DEGREE_TO_RAD to avoid precision
   104				 * loss. Round to the nearest integer.
   105				 */
   106				*val = 0;
 > 107				*val2 = div_s64(val64 * 3141592653,
   108						180 << (CROS_EC_SENSOR_BITS - 1));
   109				ret = IIO_VAL_INT_PLUS_NANO;
   110				break;
   111			case MOTIONSENSE_TYPE_MAG:
   112				/*
   113				 * EC returns data in 16LSB / uT,
   114				 * iio expects Gauss
   115				 */
   116				*val = val64;
   117				*val2 = 100 << (CROS_EC_SENSOR_BITS - 1);
   118				ret = IIO_VAL_FRACTIONAL;
   119				break;
   120			default:
   121				ret = -EINVAL;
   122			}
   123			break;
   124		default:
   125			ret = cros_ec_sensors_core_read(&st->core, chan, val, val2,
   126							mask);
   127			break;
   128		}
   129		mutex_unlock(&st->core.cmd_lock);
   130	
   131		return ret;
   132	}
   133	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux