> > 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 >
