On Mon, Apr 09, 2012 at 22:51:45 +0200, Jonathan Cameron wrote: > On 04/09/2012 12:34 PM, Srinidhi KASAGAR wrote: > > Add support for lsm303dlh magnetometer device. > > > > Signed-off-by: srinidhi kasagar <srinidhi.kasagar@xxxxxxxxxxxxxx> > > Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx> > Hi Srinidhi, > > Basically a sound driver with a few easy bits to fix. > > Few nitpicks and the error paths in probe need another look. > Hi Jonathan, [...] > > We have the old question of _range attributes as well and their > interaction with scale. I've always been dubious about these, > mainly because it's not entirely obvious how to format the > weirder general cases. Do you have a pressing need for range > or could it just be dropped in favour of allowing write for the > scale parameter? If nothing else, scale is available for inkernel > users and range isn't. If I need to drop range in favor of allowing write access for scale as you mentioned above, do you mean to implement _write_raw with IIO_CHAN_INFO_CALIBSCALE abi? Does it mean that user should be able to pass the scale factors so that driver interprets it to the corresponding range? Srinidhi -- 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
