Re: [PATCH v2 0/9] add i2c controller support to st_lsm6dsx driver

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

 



>
> On Sun, 11 Nov 2018 15:15:27 +0100
> Lorenzo Bianconi <lorenzo.bianconi@xxxxxxxxxx> wrote:
>
> > Introduce i2c controller support to st_lsm6dsx driver for lsm6dso sensor.
> > st_lsm6dsx sensor-hub supports up to 3 concurrent slave devices.
> > Add register map for lis2mdl magnetometer sensor.
> > Add hw FIFO support to st_lsm6dsx sensorhub driver.
> Hi Lorenzo.

Hi Jonathan,

>
> A very nice series in general, though I did make a small change in patch 6
> so please check I didn't mess that up.  It seemed fairly obvious but I've
> been wrong on obvious things many times before :)
>

Thx for the fast review :) Looking at the code I think it is fine.
Moreover I guess we
can generalize it taking into account different channel lengths (e.g. if we want
to connect a pressure sensor). I will send a follow-up patch.

Regards,
Lorenzo

> Thanks,
>
> Jonathan
>



[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