All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Bernhard Rosenkränzer" <bero@baylibre.com>
To: devicetree@vger.kernel.org
Cc: linux-mediatek@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org,
	linux-stm32@st-md-mailman.stormreply.com,
	mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com,
	krzysztof.kozlowski@linaro.org, matthias.bgg@gmail.com,
	angelogioacchino.delregno@collabora.com, khilman@baylibre.com,
	linux-gpio@vger.kernel.org, linus.walleij@linaro.org
Subject: [PATCH v3 0/7] Remove the pins-are-numbered DT property
Date: Tue, 29 Nov 2022 03:33:54 +0100	[thread overview]
Message-ID: <20221129023401.278780-1-bero@baylibre.com> (raw)

During the review of my MT8365 support patchset
(https://lore.kernel.org/linux-mediatek/20221117210356.3178578-1-bero@baylibre.com/),
the issue of the "pins-are-numbered" DeviceTree property has come up.

This property is unique to Mediatek MT65xx and STM32 pinctrls, and
doesn't seem to serve any purpose (both the Mediatek and STM32 drivers
simply refuse to deal with a device unless pins-are-numbered is set to
true).

There is no other use of this property in the kernel or in other projects
using DeviceTrees (checked u-boot and FreeBSD -- in both of those, the
flag is present in Mediatek and STM devicetrees, but not used anywhere).

There is also no known use in userspace (in fact, a userland application
relying on the property would be broken because it would get true on
any Mediatek or STM chipset and false on all others, even though other
chipsets use numbered pins).

This patchset removes all uses of pins-are-numbered and marks the
property as deprecated.

v3:
  - No functional changes; add recent Reviewed-Bys and Acked-Bys,
    add linux-gpio to Cc

v2:
  - Deprecate the property instead of removing it completely from
    schemas
  - squash some related commits


Bernhard Rosenkränzer (7):
  pinctrl: mediatek: common: Remove check for pins-are-numbered
  pinctrl: stm32: Remove check for pins-are-numbered
  dt-bindings: pinctrl: mediatek,mt65xx: Deprecate pins-are-numbered
  dt-bindings: pinctrl: st,stm32: Deprecate pins-are-numbered
  arm64: dts: mediatek: Remove pins-are-numbered property
  ARM: dts: mediatek: Remove pins-are-numbered property
  ARM: dts: stm32: Remove the pins-are-numbered property

 .../bindings/pinctrl/mediatek,mt65xx-pinctrl.yaml          | 5 ++---
 .../devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml      | 7 +++----
 arch/arm/boot/dts/mt2701.dtsi                              | 1 -
 arch/arm/boot/dts/mt7623.dtsi                              | 1 -
 arch/arm/boot/dts/mt8135.dtsi                              | 1 -
 arch/arm/boot/dts/stm32f4-pinctrl.dtsi                     | 1 -
 arch/arm/boot/dts/stm32f7-pinctrl.dtsi                     | 1 -
 arch/arm/boot/dts/stm32h743.dtsi                           | 1 -
 arch/arm/boot/dts/stm32mp131.dtsi                          | 1 -
 arch/arm/boot/dts/stm32mp151.dtsi                          | 2 --
 arch/arm64/boot/dts/mediatek/mt2712e.dtsi                  | 1 -
 arch/arm64/boot/dts/mediatek/mt8167.dtsi                   | 1 -
 arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi               | 1 -
 arch/arm64/boot/dts/mediatek/mt8173.dtsi                   | 1 -
 arch/arm64/boot/dts/mediatek/mt8516.dtsi                   | 1 -
 drivers/pinctrl/mediatek/pinctrl-mtk-common.c              | 6 ------
 drivers/pinctrl/stm32/pinctrl-stm32.c                      | 5 -----
 17 files changed, 5 insertions(+), 32 deletions(-)

-- 
2.38.1


WARNING: multiple messages have this Message-ID (diff)
From: "Bernhard Rosenkränzer" <bero@baylibre.com>
To: devicetree@vger.kernel.org
Cc: linux-mediatek@lists.infradead.org,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org,
	linux-stm32@st-md-mailman.stormreply.com,
	mcoquelin.stm32@gmail.com, alexandre.torgue@foss.st.com,
	krzysztof.kozlowski@linaro.org, matthias.bgg@gmail.com,
	angelogioacchino.delregno@collabora.com, khilman@baylibre.com,
	linux-gpio@vger.kernel.org, linus.walleij@linaro.org
Subject: [PATCH v3 0/7] Remove the pins-are-numbered DT property
Date: Tue, 29 Nov 2022 03:33:54 +0100	[thread overview]
Message-ID: <20221129023401.278780-1-bero@baylibre.com> (raw)

During the review of my MT8365 support patchset
(https://lore.kernel.org/linux-mediatek/20221117210356.3178578-1-bero@baylibre.com/),
the issue of the "pins-are-numbered" DeviceTree property has come up.

This property is unique to Mediatek MT65xx and STM32 pinctrls, and
doesn't seem to serve any purpose (both the Mediatek and STM32 drivers
simply refuse to deal with a device unless pins-are-numbered is set to
true).

There is no other use of this property in the kernel or in other projects
using DeviceTrees (checked u-boot and FreeBSD -- in both of those, the
flag is present in Mediatek and STM devicetrees, but not used anywhere).

There is also no known use in userspace (in fact, a userland application
relying on the property would be broken because it would get true on
any Mediatek or STM chipset and false on all others, even though other
chipsets use numbered pins).

This patchset removes all uses of pins-are-numbered and marks the
property as deprecated.

v3:
  - No functional changes; add recent Reviewed-Bys and Acked-Bys,
    add linux-gpio to Cc

v2:
  - Deprecate the property instead of removing it completely from
    schemas
  - squash some related commits


Bernhard Rosenkränzer (7):
  pinctrl: mediatek: common: Remove check for pins-are-numbered
  pinctrl: stm32: Remove check for pins-are-numbered
  dt-bindings: pinctrl: mediatek,mt65xx: Deprecate pins-are-numbered
  dt-bindings: pinctrl: st,stm32: Deprecate pins-are-numbered
  arm64: dts: mediatek: Remove pins-are-numbered property
  ARM: dts: mediatek: Remove pins-are-numbered property
  ARM: dts: stm32: Remove the pins-are-numbered property

 .../bindings/pinctrl/mediatek,mt65xx-pinctrl.yaml          | 5 ++---
 .../devicetree/bindings/pinctrl/st,stm32-pinctrl.yaml      | 7 +++----
 arch/arm/boot/dts/mt2701.dtsi                              | 1 -
 arch/arm/boot/dts/mt7623.dtsi                              | 1 -
 arch/arm/boot/dts/mt8135.dtsi                              | 1 -
 arch/arm/boot/dts/stm32f4-pinctrl.dtsi                     | 1 -
 arch/arm/boot/dts/stm32f7-pinctrl.dtsi                     | 1 -
 arch/arm/boot/dts/stm32h743.dtsi                           | 1 -
 arch/arm/boot/dts/stm32mp131.dtsi                          | 1 -
 arch/arm/boot/dts/stm32mp151.dtsi                          | 2 --
 arch/arm64/boot/dts/mediatek/mt2712e.dtsi                  | 1 -
 arch/arm64/boot/dts/mediatek/mt8167.dtsi                   | 1 -
 arch/arm64/boot/dts/mediatek/mt8173-elm.dtsi               | 1 -
 arch/arm64/boot/dts/mediatek/mt8173.dtsi                   | 1 -
 arch/arm64/boot/dts/mediatek/mt8516.dtsi                   | 1 -
 drivers/pinctrl/mediatek/pinctrl-mtk-common.c              | 6 ------
 drivers/pinctrl/stm32/pinctrl-stm32.c                      | 5 -----
 17 files changed, 5 insertions(+), 32 deletions(-)

-- 
2.38.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

             reply	other threads:[~2022-11-29  2:34 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-29  2:33 Bernhard Rosenkränzer [this message]
2022-11-29  2:33 ` [PATCH v3 0/7] Remove the pins-are-numbered DT property Bernhard Rosenkränzer
2022-11-29  2:33 ` [PATCH v3 1/7] pinctrl: mediatek: common: Remove check for pins-are-numbered Bernhard Rosenkränzer
2022-11-29  2:33   ` Bernhard Rosenkränzer
2022-11-29  2:33 ` [PATCH v3 2/7] pinctrl: stm32: " Bernhard Rosenkränzer
2022-11-29  2:33   ` Bernhard Rosenkränzer
2022-11-29  2:33 ` [PATCH v3 3/7] dt-bindings: pinctrl: mediatek,mt65xx: Deprecate pins-are-numbered Bernhard Rosenkränzer
2022-11-29  2:33   ` Bernhard Rosenkränzer
2022-11-29  8:29   ` AngeloGioacchino Del Regno
2022-11-29  8:29     ` AngeloGioacchino Del Regno
2022-11-29  2:33 ` [PATCH v3 4/7] dt-bindings: pinctrl: st,stm32: " Bernhard Rosenkränzer
2022-11-29  2:33   ` Bernhard Rosenkränzer
2022-11-29  2:33 ` [PATCH v3 5/7] arm64: dts: mediatek: Remove pins-are-numbered property Bernhard Rosenkränzer
2022-11-29  2:33   ` Bernhard Rosenkränzer
2022-12-05 13:20   ` Kevin Hilman
2022-12-05 13:20     ` Kevin Hilman
2022-12-16 11:50     ` Matthias Brugger
2022-12-16 11:50       ` Matthias Brugger
2022-11-29  2:34 ` [PATCH v3 6/7] ARM: " Bernhard Rosenkränzer
2022-11-29  2:34   ` Bernhard Rosenkränzer
2022-11-29  2:34 ` [PATCH v3 7/7] ARM: dts: stm32: Remove the " Bernhard Rosenkränzer
2022-11-29  2:34   ` Bernhard Rosenkränzer
2022-12-05 13:21   ` Kevin Hilman
2022-12-05 13:21     ` Kevin Hilman
2023-01-03  7:59     ` Alexandre TORGUE
2023-01-03  7:59       ` Alexandre TORGUE
2022-12-03  9:22 ` [PATCH v3 0/7] Remove the pins-are-numbered DT property Linus Walleij
2022-12-03  9:22   ` Linus Walleij
2022-12-06 16:25   ` Bernhard Rosenkränzer
2022-12-06 16:25     ` Bernhard Rosenkränzer

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=20221129023401.278780-1-bero@baylibre.com \
    --to=bero@baylibre.com \
    --cc=alexandre.torgue@foss.st.com \
    --cc=angelogioacchino.delregno@collabora.com \
    --cc=devicetree@vger.kernel.org \
    --cc=khilman@baylibre.com \
    --cc=krzysztof.kozlowski@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-stm32@st-md-mailman.stormreply.com \
    --cc=matthias.bgg@gmail.com \
    --cc=mcoquelin.stm32@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 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.