[PATCH 0/3] add regmap API support to st_lsm6dsx iio driver

[PATCH 0/3] add regmap API support to st_lsm6dsx iio driver

[Lorenzo Bianconi]

Lorenzo Bianconi (3):
  iio: imu: st_lsm6dsx: fix endianness in st_lsm6dsx_read_oneshot()
  iio: imu: st_lsm6dsx: introduce conf_lock mutex
  iio: imu: st_lsm6dsx: add regmap API support

 drivers/iio/imu/st_lsm6dsx/Kconfig             |   2 +
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h        |  33 ++------
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_buffer.c | 102 +++++++++++++++--------
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c   | 111 +++++++++++--------------
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c    |  55 ++++--------
 drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c    |  70 ++++------------
 6 files changed, 153 insertions(+), 220 deletions(-)

-- 
2.15.1

Re: [PATCH 0/3] add regmap API support to st_lsm6dsx iio driver

[Jonathan Cameron]

On Mon,  1 Jan 2018 19:54:41 +0100
Lorenzo Bianconi <lorenzo.bianconi@redhat.com> wrote:

Would have been nice to have a brief bit of text saying why...

Whilst I know all the advantages, that may not be the case for
someone new to the area who might read your series.

Thanks,

Jonathan


> Lorenzo Bianconi (3):
>   iio: imu: st_lsm6dsx: fix endianness in st_lsm6dsx_read_oneshot()
>   iio: imu: st_lsm6dsx: introduce conf_lock mutex
>   iio: imu: st_lsm6dsx: add regmap API support
> 
>  drivers/iio/imu/st_lsm6dsx/Kconfig             |   2 +
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h        |  33 ++------
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_buffer.c | 102 +++++++++++++++--------
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c   | 111 +++++++++++--------------
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c    |  55 ++++--------
>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c    |  70 ++++------------
>  6 files changed, 153 insertions(+), 220 deletions(-)
> 

Re: [PATCH 0/3] add regmap API support to st_lsm6dsx iio driver

[Lorenzo Bianconi]

> On Mon,  1 Jan 2018 19:54:41 +0100
> Lorenzo Bianconi <lorenzo.bianconi@redhat.com> wrote:
>
> Would have been nice to have a brief bit of text saying why...
>
> Whilst I know all the advantages, that may not be the case for
> someone new to the area who might read your series.
>

Ack, I will do in v2..can I send just regmap API patch since you have
just applied the first two?
Regards,

Lorenzo

> Thanks,
>
> Jonathan
>
>
>> Lorenzo Bianconi (3):
>>   iio: imu: st_lsm6dsx: fix endianness in st_lsm6dsx_read_oneshot()
>>   iio: imu: st_lsm6dsx: introduce conf_lock mutex
>>   iio: imu: st_lsm6dsx: add regmap API support
>>
>>  drivers/iio/imu/st_lsm6dsx/Kconfig             |   2 +
>>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h        |  33 ++------
>>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_buffer.c | 102 +++++++++++++++--------
>>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c   | 111 +++++++++++--------------
>>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c    |  55 ++++--------
>>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c    |  70 ++++------------
>>  6 files changed, 153 insertions(+), 220 deletions(-)
>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
UNIX is Sexy: who | grep -i blonde | talk; cd ~; wine; talk; touch;
unzip; touch; strip; gasp; finger; gasp; mount; fsck; more; yes; gasp;
umount; make clean; sleep

Re: [PATCH 0/3] add regmap API support to st_lsm6dsx iio driver

[Jonathan Cameron]

On Sat, 6 Jan 2018 13:31:15 +0100
Lorenzo Bianconi <lorenzo.bianconi83@gmail.com> wrote:

> > On Mon,  1 Jan 2018 19:54:41 +0100
> > Lorenzo Bianconi <lorenzo.bianconi@redhat.com> wrote:
> >
> > Would have been nice to have a brief bit of text saying why...
> >
> > Whilst I know all the advantages, that may not be the case for
> > someone new to the area who might read your series.
> >  
> 
> Ack, I will do in v2..can I send just regmap API patch since you have
> just applied the first two?
> Regards,
Actually I applied all 3 but forgot to say that on the 3rd.

ooops.  Stuff I suggested there wasn't about the content of the
patch afterall.

Jonathan
> 
> Lorenzo
> 
> > Thanks,
> >
> > Jonathan
> >
> >  
> >> Lorenzo Bianconi (3):
> >>   iio: imu: st_lsm6dsx: fix endianness in st_lsm6dsx_read_oneshot()
> >>   iio: imu: st_lsm6dsx: introduce conf_lock mutex
> >>   iio: imu: st_lsm6dsx: add regmap API support
> >>
> >>  drivers/iio/imu/st_lsm6dsx/Kconfig             |   2 +
> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h        |  33 ++------
> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_buffer.c | 102 +++++++++++++++--------
> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c   | 111 +++++++++++--------------
> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c    |  55 ++++--------
> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c    |  70 ++++------------
> >>  6 files changed, 153 insertions(+), 220 deletions(-)
> >>  
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html  
> 
> 
> 

Re: [PATCH 0/3] add regmap API support to st_lsm6dsx iio driver

[Lorenzo Bianconi]

> On Sat, 6 Jan 2018 13:31:15 +0100
> Lorenzo Bianconi <lorenzo.bianconi83@gmail.com> wrote:
>
>> > On Mon,  1 Jan 2018 19:54:41 +0100
>> > Lorenzo Bianconi <lorenzo.bianconi@redhat.com> wrote:
>> >
>> > Would have been nice to have a brief bit of text saying why...
>> >
>> > Whilst I know all the advantages, that may not be the case for
>> > someone new to the area who might read your series.
>> >
>>
>> Ack, I will do in v2..can I send just regmap API patch since you have
>> just applied the first two?
>> Regards,
> Actually I applied all 3 but forgot to say that on the 3rd.
>

Cool :) so I will fix sparse warning based on this series.
Regards,

Lorenzo

> ooops.  Stuff I suggested there wasn't about the content of the
> patch afterall.
>
> Jonathan
>>
>> Lorenzo
>>
>> > Thanks,
>> >
>> > Jonathan
>> >
>> >
>> >> Lorenzo Bianconi (3):
>> >>   iio: imu: st_lsm6dsx: fix endianness in st_lsm6dsx_read_oneshot()
>> >>   iio: imu: st_lsm6dsx: introduce conf_lock mutex
>> >>   iio: imu: st_lsm6dsx: add regmap API support
>> >>
>> >>  drivers/iio/imu/st_lsm6dsx/Kconfig             |   2 +
>> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx.h        |  33 ++------
>> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_buffer.c | 102 +++++++++++++++--------
>> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_core.c   | 111 +++++++++++--------------
>> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_i2c.c    |  55 ++++--------
>> >>  drivers/iio/imu/st_lsm6dsx/st_lsm6dsx_spi.c    |  70 ++++------------
>> >>  6 files changed, 153 insertions(+), 220 deletions(-)
>> >>
>> >
>> > --
>> > To unsubscribe from this list: send the line "unsubscribe linux-iio" in
>> > the body of a message to majordomo@vger.kernel.org
>> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
>>
>>
>



-- 
UNIX is Sexy: who | grep -i blonde | talk; cd ~; wine; talk; touch;
unzip; touch; strip; gasp; finger; gasp; mount; fsck; more; yes; gasp;
umount; make clean; sleep