> > +static IIO_DEVICE_ATTR(mode, S_IRUGO | S_IWUSR, show_mode, > store_mode, 0); > > +static IIO_DEVICE_ATTR(magn_x_calibscale, S_IRUGO, > show_calibscale, NULL, 0); > > +static IIO_DEVICE_ATTR(magn_y_calibscale, S_IRUGO, > show_calibscale, NULL, 1); > > +static IIO_DEVICE_ATTR(magn_z_calibscale, S_IRUGO, > show_calibscale, NULL, 2); > > +static IIO_DEV_ATTR_MAGN_X(show_raw, AK8975_REG_HXL); > > +static IIO_DEV_ATTR_MAGN_Y(show_raw, AK8975_REG_HYL); > > +static IIO_DEV_ATTR_MAGN_Z(show_raw, AK8975_REG_HZL); > > This seems odd as an interface as it's raw when the maths to provide > non-raw (and thus abstract and easy for user space) data is trivial > enough to do in kernel IIO guys want to do normalization maths above the kernel-level magnetometer IIO layer. This interface came before me, so I'm just following current conventions.-- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html