From: Rob Herring <robh@kernel.org>
To: arinc9.unal@gmail.com
Cc: "Linus Walleij" <linus.walleij@linaro.org>,
"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
"Arınç ÜNAL" <arinc.unal@arinc9.com>,
linux-mediatek@lists.infradead.org, linux-mips@vger.kernel.org,
linux-gpio@vger.kernel.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
"Sergio Paracuellos" <sergio.paracuellos@gmail.com>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"Sean Wang" <sean.wang@kernel.org>,
"William Dean" <williamsukatube@gmail.com>,
"Daniel Golle" <daniel@makrotopia.org>,
"Daniel Santos" <daniel.santos@pobox.com>,
"Luiz Angelo Daros de Luca" <luizluca@gmail.com>,
"Frank Wunderlich" <frank-w@public-files.de>,
"Landen Chao" <Landen.Chao@mediatek.com>,
"DENG Qingfang" <dqfext@gmail.com>,
"Sean Wang" <sean.wang@mediatek.com>,
erkin.bozoglu@xeront.com
Subject: Re: [PATCH 07/20] dt-bindings: pinctrl: ralink: improve bindings
Date: Wed, 8 Mar 2023 14:42:43 -0600 [thread overview]
Message-ID: <20230308204243.GA3735537-robh@kernel.org> (raw)
In-Reply-To: <20230303002850.51858-8-arinc.unal@arinc9.com>
On Fri, Mar 03, 2023 at 03:28:36AM +0300, arinc9.unal@gmail.com wrote:
> From: Arınç ÜNAL <arinc.unal@arinc9.com>
>
> Move additionalProperties to the top. It's easier to read than after a long
> indented section.
>
> Drop the quotes from the referred schemas.
Not the greatest subject as every change improves something and bindings
is stated twice. Hard to come up with something better since you've
combined 2 different kinds of changes which you shouldn't do either. So
I'd split this patch into 2.
>
> Signed-off-by: Arınç ÜNAL <arinc.unal@arinc9.com>
> ---
> .../bindings/pinctrl/ralink,mt7620-pinctrl.yaml | 9 ++++-----
> .../bindings/pinctrl/ralink,mt7621-pinctrl.yaml | 9 ++++-----
> .../bindings/pinctrl/ralink,rt2880-pinctrl.yaml | 9 ++++-----
> .../bindings/pinctrl/ralink,rt305x-pinctrl.yaml | 9 ++++-----
> .../bindings/pinctrl/ralink,rt3883-pinctrl.yaml | 9 ++++-----
> 5 files changed, 20 insertions(+), 25 deletions(-)
next prev parent reply other threads:[~2023-03-08 20:42 UTC|newest]
Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-03 0:28 [PATCH 00/20] pinctrl: ralink: fix ABI, improve driver, move to mediatek, improve dt-bindings arinc9.unal
2023-03-03 0:28 ` [PATCH 01/20] pinctrl: ralink: reintroduce ralink,rt2880-pinmux compatible string arinc9.unal
2023-03-03 6:01 ` Sergio Paracuellos
2023-03-03 0:28 ` [PATCH 02/20] pinctrl: ralink: {mt7620,mt7621}: add new mediatek compatible strings arinc9.unal
2023-03-03 6:02 ` Sergio Paracuellos
2023-03-03 0:28 ` [PATCH 03/20] pinctrl: ralink: rt305x: add new compatible string for every SoC arinc9.unal
2023-03-03 6:02 ` Sergio Paracuellos
2023-03-03 0:28 ` [PATCH 04/20] pinctrl: ralink: mt7620: split out to mt76x8 arinc9.unal
2023-03-03 6:03 ` Sergio Paracuellos
2023-03-03 6:04 ` Sergio Paracuellos
2023-03-03 7:53 ` Arınç ÜNAL
2023-03-03 8:06 ` Sergio Paracuellos
2023-03-03 0:28 ` [PATCH 05/20] pinctrl: ralink: move to mediatek as mtmips arinc9.unal
2023-03-03 6:17 ` Sergio Paracuellos
2023-03-03 6:34 ` Sergio Paracuellos
2023-03-03 8:15 ` Arınç ÜNAL
2023-03-03 10:57 ` Sergio Paracuellos
2023-03-03 14:17 ` Arınç ÜNAL
2023-03-06 14:07 ` Sergio Paracuellos
2023-03-06 15:05 ` Arınç ÜNAL
2023-03-06 16:15 ` Sergio Paracuellos
2023-03-06 16:43 ` Arınç ÜNAL
2023-03-03 0:28 ` [PATCH 06/20] pinctrl: mediatek: remove OF_GPIO as reverse dependency arinc9.unal
2023-03-03 6:18 ` Sergio Paracuellos
2023-03-03 0:28 ` [PATCH 07/20] dt-bindings: pinctrl: ralink: improve bindings arinc9.unal
2023-03-08 20:42 ` Rob Herring [this message]
2023-03-03 0:28 ` [PATCH 08/20] dt-bindings: pinctrl: ralink: add new compatible strings arinc9.unal
2023-03-08 21:00 ` Rob Herring
2023-03-08 21:19 ` Arınç ÜNAL
2023-03-09 9:48 ` Krzysztof Kozlowski
2023-03-03 0:28 ` [PATCH 09/20] dt-bindings: pinctrl: ralink: {mt7620,mt7621}: rename to mediatek arinc9.unal
2023-03-08 21:05 ` Rob Herring
2023-03-08 21:19 ` Arınç ÜNAL
2023-03-09 7:53 ` Arınç ÜNAL
2023-03-09 9:52 ` Krzysztof Kozlowski
2023-03-09 10:34 ` Arınç ÜNAL
2023-03-09 11:33 ` Sergio Paracuellos
2023-03-09 21:08 ` Arınç ÜNAL
2023-03-10 7:05 ` Sergio Paracuellos
2023-03-10 7:45 ` Arınç ÜNAL
2023-03-03 0:28 ` [PATCH 10/20] dt-bindings: pinctrl: mediatek: mt6795: rename to mediatek,mt6795-pinctrl arinc9.unal
2023-03-03 0:28 ` [PATCH 11/20] dt-bindings: pinctrl: mediatek: mt8186: rename to mediatek,mt8186-pinctrl arinc9.unal
2023-03-03 0:28 ` [PATCH 12/20] dt-bindings: pinctrl: mediatek: mt8192: rename to mediatek,mt8192-pinctrl arinc9.unal
2023-03-03 0:28 ` [PATCH 13/20] dt-bindings: pinctrl: mediatek: mt8195: rename to mediatek,mt8195-pinctrl arinc9.unal
2023-03-03 0:28 ` [PATCH 14/20] dt-bindings: pinctrl: mediatek: fix naming inconsistency arinc9.unal
2023-03-08 21:05 ` Rob Herring
2023-03-03 0:28 ` [PATCH 15/20] dt-bindings: pinctrl: {mediatek,ralink}: fix formatting arinc9.unal
2023-03-08 21:11 ` Rob Herring
2023-03-08 21:30 ` Arınç ÜNAL
2023-03-03 0:28 ` [PATCH 16/20] dt-bindings: pinctrl: mediatek: drop quotes from referred schemas arinc9.unal
2023-03-03 0:28 ` [PATCH 17/20] dt-bindings: pinctrl: mediatek: mt7986: fix patternProperties regex arinc9.unal
2023-03-08 21:15 ` Rob Herring
2023-03-08 21:22 ` Arınç ÜNAL
2023-03-03 0:28 ` [PATCH 18/20] dt-bindings: pinctrl: ralink: rt305x: split binding arinc9.unal
2023-03-08 21:17 ` Rob Herring
2023-03-03 0:28 ` [PATCH 19/20] dt-bindings: pinctrl: mediatek: mt7620: " arinc9.unal
2023-03-08 21:22 ` Rob Herring
2023-03-09 9:54 ` Krzysztof Kozlowski
2023-03-09 21:03 ` Arınç ÜNAL
2023-03-03 0:28 ` [PATCH 20/20] MAINTAINERS: move ralink pinctrl to mediatek mips pinctrl arinc9.unal
2023-03-03 6:21 ` Sergio Paracuellos
2023-03-06 13:50 ` [PATCH 00/20] pinctrl: ralink: fix ABI, improve driver, move to mediatek, improve dt-bindings Linus Walleij
2023-03-06 14:17 ` Sergio Paracuellos
2023-03-06 14:55 ` Arınç ÜNAL
2023-03-06 22:09 ` Linus Walleij
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=20230308204243.GA3735537-robh@kernel.org \
--to=robh@kernel.org \
--cc=Landen.Chao@mediatek.com \
--cc=arinc.unal@arinc9.com \
--cc=arinc9.unal@gmail.com \
--cc=daniel.santos@pobox.com \
--cc=daniel@makrotopia.org \
--cc=devicetree@vger.kernel.org \
--cc=dqfext@gmail.com \
--cc=erkin.bozoglu@xeront.com \
--cc=frank-w@public-files.de \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-mips@vger.kernel.org \
--cc=luizluca@gmail.com \
--cc=matthias.bgg@gmail.com \
--cc=sean.wang@kernel.org \
--cc=sean.wang@mediatek.com \
--cc=sergio.paracuellos@gmail.com \
--cc=williamsukatube@gmail.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).