All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@kernel.org>
To: Samuel Holland <samuel@sholland.org>
Cc: Lars-Peter Clausen <lars@metafoo.de>,
	Denis Ciocca <denis.ciocca@st.com>,
	Linus Walleij <linus.walleij@linaro.org>,
	Rob Herring <robh+dt@kernel.org>,
	devicetree@vger.kernel.org, linux-iio@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/3] iio: Silan SC7A20 accelerometer support
Date: Sat, 5 Feb 2022 17:25:39 +0000	[thread overview]
Message-ID: <20220205172539.18e0ad49@jic23-huawei> (raw)
In-Reply-To: <20220202235049.8051-1-samuel@sholland.org>

On Wed,  2 Feb 2022 17:50:45 -0600
Samuel Holland <samuel@sholland.org> wrote:

> This chip is register-compatible with an ST part, so it works without
> any issues when added to the existing driver.
> 
> Previously I called it a clone, but it's not really a clone, because
> it's not a drop-in replacement for the other chips. The software
> interface matches the ST parts, but the pinout does not. Instead, the
> pinout matches the Kionix KX122 and KX132.
Weird :)

I'm fine with the series, but as it contains a new vendor ID and the
dt-binding update I'll leave it on list for a bit longer to give
Rob time to sanity check that.

Thanks,

Jonathan

> 
> Changes in v2:
>  - Move the new compatible to its own section
>  - Add a comment about the WAI value
>  - Update commit messages
> 
> Samuel Holland (3):
>   dt-bindings: vendor-prefixes: Add silan vendor prefix
>   dt-bindings: iio: st: Add Silan SC7A20 accelerometer
>   iio: accel: st_accel: Add support for Silan SC7A20
> 
>  .../bindings/iio/st,st-sensors.yaml           |  3 +
>  .../devicetree/bindings/vendor-prefixes.yaml  |  2 +
>  drivers/iio/accel/st_accel.h                  |  2 +
>  drivers/iio/accel/st_accel_core.c             | 83 +++++++++++++++++++
>  drivers/iio/accel/st_accel_i2c.c              |  5 ++
>  5 files changed, 95 insertions(+)
> 


  parent reply	other threads:[~2022-02-05 17:19 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-02 23:50 [PATCH v2 0/3] iio: Silan SC7A20 accelerometer support Samuel Holland
2022-02-02 23:50 ` [PATCH v2 1/3] dt-bindings: vendor-prefixes: Add silan vendor prefix Samuel Holland
2022-02-03 16:35   ` Linus Walleij
2022-02-09 21:59   ` Rob Herring
2022-02-02 23:50 ` [PATCH v2 2/3] dt-bindings: iio: st: Add Silan SC7A20 accelerometer Samuel Holland
2022-02-03 16:35   ` Linus Walleij
2022-02-09 21:59   ` Rob Herring
2022-02-02 23:50 ` [PATCH v2 3/3] iio: accel: st_accel: Add support for Silan SC7A20 Samuel Holland
2022-02-03 16:36   ` Linus Walleij
2022-02-05 17:25 ` Jonathan Cameron [this message]
2022-02-13 18:24 ` [PATCH v2 0/3] iio: Silan SC7A20 accelerometer support 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=20220205172539.18e0ad49@jic23-huawei \
    --to=jic23@kernel.org \
    --cc=denis.ciocca@st.com \
    --cc=devicetree@vger.kernel.org \
    --cc=lars@metafoo.de \
    --cc=linus.walleij@linaro.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=samuel@sholland.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.