Re: [PATCH] staging: iio: add lsm303dlh magnetometer driver

[Srinidhi Kasagar <srinidhi.kasagar@stericsson.com>]

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@stericsson.com>
> > Acked-by: Linus Walleij <linus.walleij@linaro.org>
> 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