Re: [PATCH 1/7] dt-bindings: pinctrl: Add support for Broadcom STB pin controller
From: Stefan Wahren <wahrenst@gmx.net>
Date: 2024-08-02 18:09:31
Also in:
linux-devicetree, linux-gpio
Hi, [add official Raspberry Pi kernel developer list] Am 31.07.24 um 08:28 schrieb Ivan T. Ivanov:
It looks like they are few revisions of this chip which varies by number of pins. Perhaps not all of them are available on the market. Perhaps some of them where early engineering samples, I don't know. I decided to keep all of them just in case.
The BCM2711 had also some revisions and we avoided successfully multiple versions of the RPi 4B DTS. So it would be nice if someone can explain if C0 & D0 are available in the market? Otherwise we may end up with multiple versions of the RPi 5 DTS. I'm missing an explanation in the commit message, what's the difference between brcm,bcm2712-pinctrl and brcm,bcm2712-aon-pinctrl? According to the driver brcm,bcm2712-pinctrl is the same as brcm,bcm2712c0-pinctrl. So the former is more a fallback? Thanks
quoted hunk ↗ jump to hunk
Cc: Andrea della Porta <andrea.porta@suse.com> Signed-off-by: Ivan T. Ivanov <redacted> --- .../pinctrl/brcm,brcmstb-pinctrl.yaml | 73 +++++++++++++++++++ 1 file changed, 73 insertions(+) create mode 100644 Documentation/devicetree/bindings/pinctrl/brcm,brcmstb-pinctrl.yamldiff --git a/Documentation/devicetree/bindings/pinctrl/brcm,brcmstb-pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/brcm,brcmstb-pinctrl.yaml new file mode 100644 index 000000000000..c5afdb49d784 --- /dev/null +++ b/Documentation/devicetree/bindings/pinctrl/brcm,brcmstb-pinctrl.yaml@@ -0,0 +1,73 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/pinctrl/brcm,brcmstb-pinctrl.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Broadcom STB family pin controller + +maintainers: + - Ivan T. Ivanov <iivanov@suse.de> + +description: + Broadcom's STB family memory-mapped pin controller. + +properties: + compatible: + enum: + - brcm,bcm2712-pinctrl + - brcm,bcm2712-aon-pinctrl + - brcm,bcm2712c0-pinctrl + - brcm,bcm2712c0-aon-pinctrl + - brcm,bcm2712d0-pinctrl + - brcm,bcm2712d0-aon-pinctrl +