On Sat, Jun 27, 2020 at 02:43:59PM +0100, Jonathan Cameron wrote: > On Mon, 22 Jun 2020 08:01:09 +0200 > Krzysztof Kozlowski <krzk@xxxxxxxxxx> wrote: > > > The driver supports also BMC156B and BMM150B but these compatibles had > > redundant suffix "_magn". Add existing compatibles marking them > > deprecated along with adding a new, proper one for this family of > > devices. > Thanks for tidying this up. > > The BMC156B is a dual magnetometer and accelerometer chip > so for that one the _magn postfix probably is needed. > > Or am I missing something? > > The way this does SPI is rather odd but seems to be the same as the bmc150 > (two chips with non overlapping register maps). You're right, only one compatible should be trimmed from suffix. I'll send v3. Best regards, Krzysztof