Re: [PATCH v2 00/10] iio mount matrix - revitalize missing bindings documentation and provide code for bmc150, bmg160, bma180, itg3200, hmc584x

Linus Walleij
2019-02-22T16:24:26Z

On Thu, Feb 21, 2019 at 6:03 PM H. Nikolaus Schaller <hns@goldelico.com> wrote:

> Fixes V2:
> * make get_mount_matrix() functions more readable (use temp variable)
> (suggested by Jonathan and Andy)
> * add these readability improvements also for ak8975 and mpu6050
> (suggested by Jonathan and Andy)
> * squash bindings documentation into single commit for better discussion
> (suggested by Linus)
> * FOR DISCUSSION: add some more clarifications to the bindings documentation
> and an attempt to define the magnetometer orientation
> * add "iio: Allow to read mount matrix from ACPI" to the beginning of
> the series to make it compile
> (suggested by Andy)
> * replace of_iio_read_mount_matrix() by iio_read_mount_matrix()
> (required by "iio: Allow to read mount matrix from ACPI")
> * drop patch to convert bma180 to devm (potential race)
> (suggested by Jonathan)

I'm a big fan of this series and I really like what you did with my old
patch!
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij