From: Rob Herring <robh@kernel.org>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: devicetree@vger.kernel.org, Lars-Peter Clausen <lars@metafoo.de>,
Jonathan Cameron <jic23@kernel.org>,
Hartmut Knaack <knaack.h@gmx.de>,
Peter Meerwald-Stadler <pmeerw@pmeerw.net>,
linux-iio@vger.kernel.org, phone-devel@vger.kernel.org
Subject: Re: [PATCH 1/2 v4] iio: accel: yamaha-yas530: Add DT bindings
Date: Mon, 30 Nov 2020 19:07:46 -0700 [thread overview]
Message-ID: <20201201020746.GA3492504@robh.at.kernel.org> (raw)
In-Reply-To: <20201129215421.1177990-1-linus.walleij@linaro.org>
On Sun, 29 Nov 2020 22:54:20 +0100, Linus Walleij wrote:
> This adds device tree bindings for the Yamaha YAS530
> family of magnetometers/compass sensors.
>
> Cc: devicetree@vger.kernel.org
> Cc: phone-devel@vger.kernel.org
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> ChangeLog v3->v4:
> - Rename the bindings and file yas530 after the first
> introduced component in the family.
> - Simplify conditional logic using the if: not: YAML
> constructions.
> - Use the possibility to set a property to false to just
> remvove reset GPIOs and interrupts from variants that
> do not have them.
> ChangeLog v2->v3:
> - Restrict to cover the YAS53x variants, it turns out that
> YAS529 is a very different component from the others so
> keep that for a separate document when/if needed.
> - Rename the file and binding yamaha,53x.yaml
> - Use - if: clauses to restrict some properties.
> - Fix some spelling mistakes.
> - Restrict the nodename to be "magnetometer@[0-9a-f]"
> ChangeLog v1->v2:
> - Add Yamaha to the vendor list, I was surprised to find
> they were not yet listed.
> ---
> .../iio/magnetometer/yamaha,yas530.yaml | 112 ++++++++++++++++++
> .../devicetree/bindings/vendor-prefixes.yaml | 2 +
> 2 files changed, 114 insertions(+)
> create mode 100644 Documentation/devicetree/bindings/iio/magnetometer/yamaha,yas530.yaml
>
Reviewed-by: Rob Herring <robh@kernel.org>
prev parent reply other threads:[~2020-12-01 2:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-11-29 21:54 [PATCH 1/2 v4] iio: accel: yamaha-yas530: Add DT bindings Linus Walleij
2020-11-29 21:54 ` [PATCH 2/2 v4] iio: magnetometer: Add driver for Yamaha YAS530 Linus Walleij
2020-11-30 11:43 ` Andy Shevchenko
2020-12-01 2:07 ` Rob Herring [this message]
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=20201201020746.GA3492504@robh.at.kernel.org \
--to=robh@kernel.org \
--cc=devicetree@vger.kernel.org \
--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=phone-devel@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).