From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Jonathan Cameron <jic23@kernel.org>
Cc: Hartmut Knaack <knaack.h@gmx.de>,
Lars-Peter Clausen <lars@metafoo.de>,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
linux-iio@vger.kernel.org,
Linus Walleij <linus.walleij@linaro.org>,
"H. Nikolaus Schaller" <hns@goldelico.com>
Subject: Re: [PATCH v1] iio: Allow to read mount matrix from ACPI
Date: Wed, 20 Feb 2019 18:41:40 +0200 [thread overview]
Message-ID: <20190220164140.GC9224@smile.fi.intel.com> (raw)
In-Reply-To: <20190220160239.79f2566c@archlinux>
On Wed, Feb 20, 2019 at 04:02:39PM +0000, Jonathan Cameron wrote:
> On Wed, 20 Feb 2019 18:20:52 +0300
> Andy Shevchenko <andriy.shevchenko@linux.intel.com> wrote:
>
> > Currently mount matrix is allowed in Device Tree, though there is
> > no technical issue to extend it to support ACPI.
> >
> > Convert the function to use device_property_read_string_array() and
> > thus allow to read mount matrix from ACPI if available.
> >
> > At the same time drop the "of" prefix from its name and
> > convert current users.
> >
> > No functional change intended.
> >
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
>
> Hi Andy,
>
> I sort of agree that there is no obvious reason to limit it to
> DT. One corner that is unfortunate is that we have ACPI DSDT with
> _OFT providing the same thing. Neither is 'standard' as such.
He-h, that _OFT should be parsed on per device basis. I'm tired of firmware
writers that are (abusing / not using in a clever way) ACPI features.
> Also, could we have an example of what the DSDT would look like
> to provide this for a device? I'm too lazy to figure it out for
> myself given you probably already know off the top of your head ;)
Sure.
Since it's an array of strings it would be as simple as
Name (_DSD, Package ()
{
ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"),
Package ()
{
Package () { "mount-matrix", Package() {
"1", "0", "0",
"0", "0.866", "0.5",
"0", "-0.5", "0.866"
} }
}
})
Should I send a version with extended commit message?
P.S. This is basically the example of any property that can be provided
via ACPI. Just add / replace name and value(s).
>
> Thanks,
>
> Jonathan
>
> > ---
> > drivers/iio/accel/kxsd9.c | 2 +-
> > drivers/iio/gyro/mpu3050-core.c | 2 +-
> > drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 2 +-
> > drivers/iio/industrialio-core.c | 46 +++++++++-------------
> > drivers/iio/magnetometer/ak8974.c | 2 +-
> > drivers/iio/magnetometer/ak8975.c | 2 +-
> > include/linux/iio/iio.h | 4 +-
> > 7 files changed, 25 insertions(+), 35 deletions(-)
> >
> > diff --git a/drivers/iio/accel/kxsd9.c b/drivers/iio/accel/kxsd9.c
> > index 0c0df4fce420..f73404ca715e 100644
> > --- a/drivers/iio/accel/kxsd9.c
> > +++ b/drivers/iio/accel/kxsd9.c
> > @@ -420,7 +420,7 @@ int kxsd9_common_probe(struct device *dev,
> > indio_dev->available_scan_masks = kxsd9_scan_masks;
> >
> > /* Read the mounting matrix, if present */
> > - ret = of_iio_read_mount_matrix(dev,
> > + ret = iio_read_mount_matrix(dev,
> > "mount-matrix",
> > &st->orientation);
> > if (ret)
> > diff --git a/drivers/iio/gyro/mpu3050-core.c b/drivers/iio/gyro/mpu3050-core.c
> > index 77fac81a3adc..0a7d0dde020e 100644
> > --- a/drivers/iio/gyro/mpu3050-core.c
> > +++ b/drivers/iio/gyro/mpu3050-core.c
> > @@ -1149,7 +1149,7 @@ int mpu3050_common_probe(struct device *dev,
> > mpu3050->divisor = 99;
> >
> > /* Read the mounting matrix, if present */
> > - ret = of_iio_read_mount_matrix(dev, "mount-matrix",
> > + ret = iio_read_mount_matrix(dev, "mount-matrix",
> > &mpu3050->orientation);
> > if (ret)
> > return ret;
> > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c
> > index 1e428c196a82..e2c8e8ca324a 100644
> > --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c
> > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c
> > @@ -990,7 +990,7 @@ int inv_mpu_core_probe(struct regmap *regmap, int irq, const char *name,
> >
> > pdata = dev_get_platdata(dev);
> > if (!pdata) {
> > - result = of_iio_read_mount_matrix(dev, "mount-matrix",
> > + result = iio_read_mount_matrix(dev, "mount-matrix",
> > &st->orientation);
> > if (result) {
> > dev_err(dev, "Failed to retrieve mounting matrix %d\n",
> > diff --git a/drivers/iio/industrialio-core.c b/drivers/iio/industrialio-core.c
> > index 4f5cd9f60870..3baede140d69 100644
> > --- a/drivers/iio/industrialio-core.c
> > +++ b/drivers/iio/industrialio-core.c
> > @@ -19,6 +19,7 @@
> > #include <linux/device.h>
> > #include <linux/fs.h>
> > #include <linux/poll.h>
> > +#include <linux/property.h>
> > #include <linux/sched.h>
> > #include <linux/wait.h>
> > #include <linux/cdev.h>
> > @@ -525,8 +526,8 @@ ssize_t iio_show_mount_matrix(struct iio_dev *indio_dev, uintptr_t priv,
> > EXPORT_SYMBOL_GPL(iio_show_mount_matrix);
> >
> > /**
> > - * of_iio_read_mount_matrix() - retrieve iio device mounting matrix from
> > - * device-tree "mount-matrix" property
> > + * iio_read_mount_matrix() - retrieve iio device mounting matrix from
> > + * device "mount-matrix" property
> > * @dev: device the mounting matrix property is assigned to
> > * @propname: device specific mounting matrix property name
> > * @matrix: where to store retrieved matrix
> > @@ -536,40 +537,29 @@ EXPORT_SYMBOL_GPL(iio_show_mount_matrix);
> > *
> > * Return: 0 if success, or a negative error code on failure.
> > */
> > -#ifdef CONFIG_OF
> > -int of_iio_read_mount_matrix(const struct device *dev,
> > - const char *propname,
> > - struct iio_mount_matrix *matrix)
> > +int iio_read_mount_matrix(struct device *dev, const char *propname,
> > + struct iio_mount_matrix *matrix)
> > {
> > - if (dev->of_node) {
> > - int err = of_property_read_string_array(dev->of_node,
> > - propname, matrix->rotation,
> > - ARRAY_SIZE(iio_mount_idmatrix.rotation));
> > + size_t len = ARRAY_SIZE(iio_mount_idmatrix.rotation);
> > + int err;
> >
> > - if (err == ARRAY_SIZE(iio_mount_idmatrix.rotation))
> > - return 0;
> > + err = device_property_read_string_array(dev, propname,
> > + matrix->rotation, len);
> > + if (err == len)
> > + return 0;
> >
> > - if (err >= 0)
> > - /* Invalid number of matrix entries. */
> > - return -EINVAL;
> > + if (err >= 0)
> > + /* Invalid number of matrix entries. */
> > + return -EINVAL;
> >
> > - if (err != -EINVAL)
> > - /* Invalid matrix declaration format. */
> > - return err;
> > - }
> > + if (err != -EINVAL)
> > + /* Invalid matrix declaration format. */
> > + return err;
> >
> > /* Matrix was not declared at all: fallback to identity. */
> > return iio_setup_mount_idmatrix(dev, matrix);
> > }
> > -#else
> > -int of_iio_read_mount_matrix(const struct device *dev,
> > - const char *propname,
> > - struct iio_mount_matrix *matrix)
> > -{
> > - return iio_setup_mount_idmatrix(dev, matrix);
> > -}
> > -#endif
> > -EXPORT_SYMBOL(of_iio_read_mount_matrix);
> > +EXPORT_SYMBOL(iio_read_mount_matrix);
> >
> > static ssize_t __iio_format_value(char *buf, size_t len, unsigned int type,
> > int size, const int *vals)
> > diff --git a/drivers/iio/magnetometer/ak8974.c b/drivers/iio/magnetometer/ak8974.c
> > index 93be1f4c0f27..9bf21e5fe3fc 100644
> > --- a/drivers/iio/magnetometer/ak8974.c
> > +++ b/drivers/iio/magnetometer/ak8974.c
> > @@ -733,7 +733,7 @@ static int ak8974_probe(struct i2c_client *i2c,
> > ak8974->i2c = i2c;
> > mutex_init(&ak8974->lock);
> >
> > - ret = of_iio_read_mount_matrix(&i2c->dev,
> > + ret = iio_read_mount_matrix(&i2c->dev,
> > "mount-matrix",
> > &ak8974->orientation);
> > if (ret)
> > diff --git a/drivers/iio/magnetometer/ak8975.c b/drivers/iio/magnetometer/ak8975.c
> > index d430b80808ef..dca20df1310e 100644
> > --- a/drivers/iio/magnetometer/ak8975.c
> > +++ b/drivers/iio/magnetometer/ak8975.c
> > @@ -911,7 +911,7 @@ static int ak8975_probe(struct i2c_client *client,
> > data->eoc_irq = 0;
> >
> > if (!pdata) {
> > - err = of_iio_read_mount_matrix(&client->dev,
> > + err = iio_read_mount_matrix(&client->dev,
> > "mount-matrix",
> > &data->orientation);
> > if (err)
> > diff --git a/include/linux/iio/iio.h b/include/linux/iio/iio.h
> > index a74cb177dc6f..bb10c1bee301 100644
> > --- a/include/linux/iio/iio.h
> > +++ b/include/linux/iio/iio.h
> > @@ -130,8 +130,8 @@ struct iio_mount_matrix {
> >
> > ssize_t iio_show_mount_matrix(struct iio_dev *indio_dev, uintptr_t priv,
> > const struct iio_chan_spec *chan, char *buf);
> > -int of_iio_read_mount_matrix(const struct device *dev, const char *propname,
> > - struct iio_mount_matrix *matrix);
> > +int iio_read_mount_matrix(struct device *dev, const char *propname,
> > + struct iio_mount_matrix *matrix);
> >
> > typedef const struct iio_mount_matrix *
> > (iio_get_mount_matrix_t)(const struct iio_dev *indio_dev,
>
--
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2019-02-20 16:42 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-20 15:20 [PATCH v1] iio: Allow to read mount matrix from ACPI Andy Shevchenko
2019-02-20 16:02 ` Jonathan Cameron
2019-02-20 16:41 ` Andy Shevchenko [this message]
2019-02-20 17:07 ` Jonathan Cameron
2019-02-20 16:08 ` H. Nikolaus Schaller
2019-02-20 16:19 ` Andy Shevchenko
2019-02-20 17:08 ` Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190220164140.GC9224@smile.fi.intel.com \
--to=andriy.shevchenko@linux.intel.com \
--cc=hns@goldelico.com \
--cc=jic23@kernel.org \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=linus.walleij@linaro.org \
--cc=linux-iio@vger.kernel.org \
--cc=pmeerw@pmeerw.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).