From: Jonathan Cameron <jic23@kernel.org>
To: Bobby Jones <rjones@gateworks.com>
Cc: linux-iio@vger.kernel.org
Subject: Re: FXOS8700 Accelerometer/Magnetometer Support
Date: Sun, 14 Jul 2019 16:16:23 +0100 [thread overview]
Message-ID: <20190714161623.0127bbcb@archlinux> (raw)
In-Reply-To: <CALAE=UBN7ZPGTHB_jgSje0zG3mCvE8auW1FTrbjn=KbJzN_How@mail.gmail.com>
On Wed, 3 Jul 2019 15:48:29 -0700
Bobby Jones <rjones@gateworks.com> wrote:
> Hello linux-iio,
>
> I'm interested in adding support for the FXOS8700 accel/mag device. I
> was curious if there had already been an effort in mainline to add
> support and could not find anything in the latest pulled version of
> the testing branch. However looking at the list archive I saw an email
> back in 2015 from Utkarsh who asked about implementing a driver for
> the same device [https://marc.info/?l=linux-iio&m=142920413702776]. My
> question for Utkarsh (if you're still reading this, I get a bounce on
> your address) and the greater linux-iio mailing list is if there had
> been any further effort since then that I can help finish?
>
> Otherwise as someone new to driver development in general, I'm looking
> for any advice to get me started. I did see a NXP aka Freescale driver
> [https://source.codeaurora.org/external/imx/linux-imx/tree/drivers/misc/fxos8700.c?h=imx_4.19.35_1.0.0]
> for the device but it doesn't appear to follow the iio api from what I
> can tell. Is it worth taking that driver and trying to get it to fit
> into the IMU api or should I start from scratch?
I would certainly start by seeing if that driver works to at least
some extent as it can be extremely handy to have a working reference.
Then it's up to you on how you prefer to work. Either you can take that
code and modify it step by step until you get something you are happy
with, or start from scratch keeping that as an additional source of
information only.
It's a fair way away from where it needs to end up, so starting from
scratch might be more efficient in this case. Start simple and build
up the fancy features later. From the reply I sent to that original
thread you reference I gather this is a complex device, so we may want
to have a reasonably feature complete driver to figure out if the
overall structure will work before we merge it.
Good luck and looking forward to seeing the code!
Thanks,
Jonathan
>
> Thanks,
> Bobby Jones
next prev parent reply other threads:[~2019-07-14 15:16 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-03 22:48 FXOS8700 Accelerometer/Magnetometer Support Bobby Jones
2019-07-14 15:16 ` Jonathan Cameron [this message]
2019-09-10 21:36 ` Bobby Jones
-- strict thread matches above, loose matches on Subject: below --
2019-07-03 22:39 Bobby Jones
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=20190714161623.0127bbcb@archlinux \
--to=jic23@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=rjones@gateworks.com \
/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).