From: Daniel Drake <drake@endlessm.com>
To: Denis CIOCCA <denis.ciocca@st.com>
Cc: Jonathan Cameron <jic23@kernel.org>,
"knaack.h@gmx.de" <knaack.h@gmx.de>,
"lars@metafoo.de" <lars@metafoo.de>,
"pmeerw@pmeerw.net" <pmeerw@pmeerw.net>,
"linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
"linux@endlessm.com" <linux@endlessm.com>,
"lorenzo.bianconi83@gmail.com" <lorenzo.bianconi83@gmail.com>,
"hadess@hadess.net" <hadess@hadess.net>,
"hdegoede@redhat.com" <hdegoede@redhat.com>
Subject: Re: [PATCH 2/2] iio: st_accel: use ACPI orientation data
Date: Tue, 15 Jan 2019 11:09:51 +0800 [thread overview]
Message-ID: <CAD8Lp47XPzqQrVpAMnyHKE92Zj7XB8fZnj5HW_TTddgHzUCmOQ@mail.gmail.com> (raw)
In-Reply-To: <49fe1df26d70489ca3b3c52cb5cf561c@SFHDAG2NODE1.st.com>
Hi Denis,
On Tue, Jan 15, 2019 at 8:18 AM Denis CIOCCA <denis.ciocca@st.com> wrote:
> I am not familiar with ACPI and I am not sure why axis orientation is not following the standard there.
> What I wonder is if this is related to SMO8840, Acer or in general to ST accelerometers integration. Your patch seems to be fine technically but are you able to confirm it will works even outside the 3 parts tested? The other accel supported by ACPI probing SMO8A90 is affected also?
The _ONT data format was explained to us by a vendor support contact
at ST. (however, the fact that the _ONT transformation must be applied
after first applying another translation was determined experimentally
by me).
We have seen this _ONT data present in ACPI tables from multiple
product vendors, on SMO8820, SMO8821 and SMO8840 devices, so I believe
there is no doubt that it is a ST-specific thing.
The specific transformation behaviour in my patch has only been tested
on products with SMO8840 though, since those are the only ones we have
on hand.
Daniel
next prev parent reply other threads:[~2019-01-15 3:10 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-20 6:59 [PATCH 1/2] iio: drop const typing from iio_mount_matrix rotation Daniel Drake
2018-12-20 6:59 ` [PATCH 2/2] iio: st_accel: use ACPI orientation data Daniel Drake
2018-12-22 18:09 ` Jonathan Cameron
2019-01-12 19:13 ` Jonathan Cameron
2019-01-15 0:18 ` Denis CIOCCA
2019-01-15 3:09 ` Daniel Drake [this message]
2019-01-19 16:50 ` Jonathan Cameron
2019-01-26 17:27 ` Jonathan Cameron
2019-01-19 16:49 ` [PATCH 1/2] iio: drop const typing from iio_mount_matrix rotation Jonathan Cameron
2019-01-21 8:42 ` Daniel Drake
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=CAD8Lp47XPzqQrVpAMnyHKE92Zj7XB8fZnj5HW_TTddgHzUCmOQ@mail.gmail.com \
--to=drake@endlessm.com \
--cc=denis.ciocca@st.com \
--cc=hadess@hadess.net \
--cc=hdegoede@redhat.com \
--cc=jic23@kernel.org \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux@endlessm.com \
--cc=lorenzo.bianconi83@gmail.com \
--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).