From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 12C9DC3DA7F for ; Wed, 31 Jul 2024 14:26:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Subject:Message-Id: References:In-Reply-To:Cc:To:From:MIME-Version:Content-Transfer-Encoding: Content-Type:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=9k9pKhcV4hzdCa7yldwdOsClofbj0HinzHXHoHBvOz0=; b=C4EzMyaZIzCGiU12bFJk6UrlaZ y0fz/HlEfLoQfhWy8JqVu6hyW9rkWQz617OMj/f2CipNwQxvWszORBrlLUV8jyPvQMrT4lrqsNAWR kpmQ2NMwY0spEB7crnwB7w/CTIjO/fyDkOZIMcqeioXdEG6QNhIA0Cd6tyNcNFJhLyIwsu+7AW+Ry /hpiQ6+RrOgJi36FNXNKAn4ZsKWycskU4wFoTLe02CSh4rnT5y4s7nqPafKniFLriQn+8p7xv7ZSN G1XDW7gggL2YtrFfXbO9ZWWixu0Sdfv5x18DpfnD/vnOK3pajMcXp6TSSHQT1Y4hT66Lftk7h0Ur6 pye1XuHQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sZAHq-00000001Tuh-1nB3; Wed, 31 Jul 2024 14:26:42 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sZAGu-00000001Tea-2tax for linux-arm-kernel@lists.infradead.org; Wed, 31 Jul 2024 14:25:46 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 3188762454; Wed, 31 Jul 2024 14:25:44 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id CA43CC116B1; Wed, 31 Jul 2024 14:25:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1722435944; bh=nFrpTEL72ncMiEVg5MV9Nz+lBTH5NVX5OD0iZeNpyeU=; h=Date:From:To:Cc:In-Reply-To:References:Subject:From; b=r9YDVoVA7qtB6NcQdETFq1n/dfAOcEQ8dx7N7ML/Gafy35jGg9aslRRlc2xf5XOvQ /F6wh2TawekzopDISnehOA8tcEQjDg32yO7lUzT7MR5Sdb3nuF5MBSnXy0tklQo8VZ KlxEuPm1DrEjOJAS+8OiNNUQVabfy0xVPVAt6LelLPKro4RA7UEHM4vUzEmYYSruc5 Uyz7fRuElmCSM2tzwOrtwWlel3xqsKYg80uEfu0pkr642vPx4vAcq+UsBkcBf9F+28 3Mju46PwxTMhsACVTkEYodGGtcfL/fdrR+n2Ob+HKe36ElySGEzxplzTq8pDD5zBKp BGz/z1314XxTQ== Date: Wed, 31 Jul 2024 08:25:42 -0600 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit MIME-Version: 1.0 From: "Rob Herring (Arm)" To: "Ivan T. Ivanov" Cc: conor+dt@kernel.org, devicetree@vger.kernel.org, andrea.porta@suse.com, florian.fainelli@broadcom.com, linux-gpio@vger.kernel.org, krzk+dt@kernel.org, wahrenst@gmx.net, linux-arm-kernel@lists.infradead.org, linus.walleij@linaro.org In-Reply-To: <20240731062814.215833-1-iivanov@suse.de> References: <20240731062814.215833-1-iivanov@suse.de> Message-Id: <172243537599.718775.994017602551237055.robh@kernel.org> Subject: Re: [PATCH 0/7] Add pin control driver for BCM2712 SoC X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240731_072544_871290_A8213BD9 X-CRM114-Status: GOOD ( 26.45 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, 31 Jul 2024 09:28:07 +0300, Ivan T. Ivanov wrote: > Hi, > > The following patches add a pin control driver for the BCM2712 SoC and few > pin/gpio Devicetree nodes for Raspberry Pi 5. > > Device driver is follow up version on what Andrea posted in April [1]. > > It is based on sources from here [2]. I just made few cosmetic changes > and addressed review comments from earlier submission. I don't have > documentation for this controller. > > Patch 3 was already posted by Andrea and it is in Broadcom integration tree[3]. > Unfortunately it is still not in the Linus tree[4]. So I added it here as base > for my changes and to easy building and testing. > > Patch 5 wire up power button on RPi5 > > Patch 6 adds WiFi Devicetree node for RPi5 > > Patch 7 adds Bluetooth Devicetree node for RPi5 > > They are few complaints from checkpatch.pl. Like few lines over 100 columns, > which I keep that way for better readability or usage of EOPNOTSUPP, > but it is according pin control API document. > > All this have been tested as kernel was directly booted RPi5 via > kernel= config.txt option and cmdline.txt file with following content > (Note I am using Tumbleweed RPi raw images) > > # cat /boot/efi/cmdline.txt > root=/dev/mmcblk0p3 rootwait rw systemd.show_status=1 console=tty ignore_loglevel earlycon console=ttyAMA10,115200 > > With all these patches Bluetooth and Wifi are working fine (tm) with > firmware files provided by openSUSE Tumbleweed. > > All comments and suggestions are welcome! > > Happy hacking! > Ivan > > [1] https://lore.kernel.org/lkml/f6601f73-cb22-4ba3-88c5-241be8421fc3@broadcom.com/ > [2] https://github.com/raspberrypi/linux/blob/rpi-6.6.y/drivers/pinctrl/bcm/pinctrl-bcm2712.c > [3] https://lore.kernel.org/lkml/20240605120712.3523290-1-florian.fainelli@broadcom.com/#t > [4] https://lore.kernel.org/all/bfc60a7e-54d2-48a6-a288-4fe76d66507a@gmx.net/ > > > Andrea della Porta (1): > arm64: dts: broadcom: Add support for BCM2712 > > Ivan T. Ivanov (6): > dt-bindings: pinctrl: Add support for Broadcom STB pin controller > pinctrl: bcm: Add STB family pin controller driver > arm64: dts: broadcom: bcm2712: Add pin controller nodes > arm64: dts: broadcom: bcm2712: Add one more GPIO node > arm64: dts: broadcom: bcm2712: Add second SDHCI controller node > arm64: dts: broadcom: bcm2712: Add UARTA controller node. > > .../pinctrl/brcm,brcmstb-pinctrl.yaml | 73 + > arch/arm64/boot/dts/broadcom/Makefile | 1 + > .../boot/dts/broadcom/bcm2712-rpi-5-b.dts | 203 +++ > arch/arm64/boot/dts/broadcom/bcm2712.dtsi | 357 +++++ > drivers/pinctrl/bcm/Kconfig | 13 + > drivers/pinctrl/bcm/Makefile | 1 + > drivers/pinctrl/bcm/pinctrl-brcmstb.c | 1217 +++++++++++++++++ > 7 files changed, 1865 insertions(+) > create mode 100644 Documentation/devicetree/bindings/pinctrl/brcm,brcmstb-pinctrl.yaml > create mode 100644 arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dts > create mode 100644 arch/arm64/boot/dts/broadcom/bcm2712.dtsi > create mode 100644 drivers/pinctrl/bcm/pinctrl-brcmstb.c > > > base-commit: dc1c8034e31b14a2e5e212104ec508aec44ce1b9 > -- > 2.43.0 > > > My bot found new DTB warnings on the .dts files added or changed in this series. Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings are fixed by another series. Ultimately, it is up to the platform maintainer whether these warnings are acceptable or not. No need to reply unless the platform maintainer has comments. If you already ran DT checks and didn't see these error(s), then make sure dt-schema is up to date: pip3 install dtschema --upgrade New warnings running 'make CHECK_DTBS=y broadcom/bcm2712-rpi-5-b.dtb' for 20240731062814.215833-1-iivanov@suse.de: arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: mmc@1001100000: compatible: 'oneOf' conditional failed, one must be fixed: ['brcm,bcm2712-sdhci'] is too short 'brcm,bcm2712-sdhci' is not one of ['brcm,bcm7216-sdhci'] from schema $id: http://devicetree.org/schemas/mmc/brcm,sdhci-brcmstb.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: mmc@1001100000: 'clock-frequency' is a required property from schema $id: http://devicetree.org/schemas/mmc/brcm,sdhci-brcmstb.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: mmc@1001100000: 'clock-names' is a required property from schema $id: http://devicetree.org/schemas/mmc/brcm,sdhci-brcmstb.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: mmc@1001100000: Unevaluated properties are not allowed ('compatible', 'sdhci-caps', 'sdhci-caps-mask' were unexpected) from schema $id: http://devicetree.org/schemas/mmc/brcm,sdhci-brcmstb.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: wifi@1: Unevaluated properties are not allowed ('local-mac-address' was unexpected) from schema $id: http://devicetree.org/schemas/net/wireless/brcm,bcm4329-fmac.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: /soc@107c000000/timer@7c003000: failed to match any schema with compatible: ['brcm,bcm2835-system-timer'] arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: /soc@107c000000/local-intc@7cd00000: failed to match any schema with compatible: ['brcm,bcm2836-l1-intc'] arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: intc@7d508400: $nodename:0: 'intc@7d508400' does not match '^interrupt-controller(@[0-9a-f,]+)*$' from schema $id: http://devicetree.org/schemas/interrupt-controller/brcm,l2-intc.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: gpio@7d508500: compatible:0: 'brcm,brcmstb-gpio' is not one of ['brcm,bcm7445-gpio'] from schema $id: http://devicetree.org/schemas/gpio/brcm,brcmstb-gpio.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: gpio@7d508500: compatible: ['brcm,brcmstb-gpio'] is too short from schema $id: http://devicetree.org/schemas/gpio/brcm,brcmstb-gpio.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: gpio@7d508500: 'brcm,gpio-direct' does not match any of the regexes: 'pinctrl-[0-9]+' from schema $id: http://devicetree.org/schemas/gpio/brcm,brcmstb-gpio.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: serial@7d50c000: 'clocks' is a required property from schema $id: http://devicetree.org/schemas/serial/brcm,bcm7271-uart.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: serial@7d50c000: 'clock-names' is a required property from schema $id: http://devicetree.org/schemas/serial/brcm,bcm7271-uart.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: serial@7d50c000: 'interrupt-names' is a required property from schema $id: http://devicetree.org/schemas/serial/brcm,bcm7271-uart.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: serial@7d50c000: Unevaluated properties are not allowed ('auto-flow-control', 'clock-frequency', 'reg-io-width', 'reg-shift', 'skip-init' were unexpected) from schema $id: http://devicetree.org/schemas/serial/brcm,bcm7271-uart.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: bluetooth: 'local-bd-address' does not match any of the regexes: 'pinctrl-[0-9]+' from schema $id: http://devicetree.org/schemas/net/broadcom-bluetooth.yaml# arch/arm64/boot/dts/broadcom/bcm2712-rpi-5-b.dtb: pwr-button: 'pwr' does not match any of the regexes: '^(button|event|key|switch|(button|event|key|switch)-[a-z0-9-]+|[a-z0-9-]+-(button|event|key|switch))$', 'pinctrl-[0-9]+' from schema $id: http://devicetree.org/schemas/input/gpio-keys.yaml#