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 25C9FC5478C for ; Tue, 27 Feb 2024 16:26:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=gYbLegbd4HHovzbzAOoouwphQT5E0DxeVHcLfE9UPVw=; b=TGT930b08RGs5T 3R9bPmo0q2puJObRJuIaRc57fYKvfHiqD1mSI6nK/XEGZyXJQ19DL9Qxn+a7NXNOt5dFboG/lCQFV 4o6f2WMD+4XjKCyzF7XtPDIBGV5W3AdMRWtCNW8ln6IFNdnuLrDJemcdyTsGIjJNp/zW4iuEfRFw5 tHe9rT+Pbu6vXFYqCz1aFWM0vHDRrOdY9CUZYJjeOmR5KDWw2LlM++OC1qrIirDehU8IWP4H1bOe9 /R4n069TpiTah7IyQ59SOHEsFxee/L6AT/3K9Zl53BBZufD8d48qyh/ndICjzf//jtnMbRcD03NOv A2I/pq1MOe4Ec/V7ANMw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rf0HQ-00000005xlu-0u7h; Tue, 27 Feb 2024 16:26:08 +0000 Received: from relay1-d.mail.gandi.net ([2001:4b98:dc4:8::221]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rf0HL-00000005xj7-2dnZ for linux-arm-kernel@lists.infradead.org; Tue, 27 Feb 2024 16:26:06 +0000 Received: by mail.gandi.net (Postfix) with ESMTPSA id BBC97240010; Tue, 27 Feb 2024 16:25:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1709051158; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=5EL5MTBosVVpIu4g1wb/cAbzOuqrbIGwThm3GnF6MkU=; b=Rpr67OC1kRV904+6zCmMNThxo+4lX0RuA05YUSCsxPgZpJqnPom867O6bpfxMhFT6Yltsn B//pM+2XaXNfouWq45mJrypBW1Bh8wkCjA7vIB2AVyh8ojgmrAE8x18GvaP920C5gLr/i5 aYEyo3Ksc2s0ukh/U4tqI8yGiMqxrYEGL56cphIXiqP6QfM/VJGmX76C1PN3Z6cj+cJNeq BigspfVeLFEbaDd1oacroFL+YC9e8p+mGcp66xxH3KtjTtcs8XpskZAVRK7EqMMdv0m5t0 Bl21c9JFSiGphOuRXfJaTmGUf0REI1iXi3x9WQPx45I/OZCoD39Wt7R4WtDXRQ== From: Gregory CLEMENT To: Josua Mayer , Andrew Lunn , Sebastian Hesselbarth , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Russell King Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Josua Mayer Subject: Re: [PATCH v5 00/10] dt-bindings: marvell: a38x: add solidrun armada 388 clearfog boards In-Reply-To: <20240104-support-clearfog-gtr-l8-sfp-v5-0-52be60fc54e3@solid-run.com> References: <20240104-support-clearfog-gtr-l8-sfp-v5-0-52be60fc54e3@solid-run.com> Date: Tue, 27 Feb 2024 17:25:56 +0100 Message-ID: <874jdtkh63.fsf@BL-laptop> MIME-Version: 1.0 X-GND-Sasl: gregory.clement@bootlin.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240227_082603_985992_BE35A964 X-CRM114-Status: GOOD ( 19.67 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Josua Mayer, > Dear Maintainers, > > The initially merged device-tree for Clearfog GTR devices contained > various subtle mistakes and omissions: > > - missing board-specific compatible strings > - missing pinctrl entries > - missing second sfp connector > - invalid sfp loss-of-signal gpio > - mismatch of labels between dsa ports and enclosure > > Most notably this had caused functional issues with the sfp connectors. > > This patch-set first converts the existing armada-38x dt-bindings to > yaml, replacing invalid soc-only compatibles with specific boards that > already exist in tree and represent the three SoCs (380,385,388). > > Secondly for clearfog gtr pinctrl nodes are added for all referenced > gpios for independence from bootloader defaults. U-Boot is shared > between armada-388 clearfog and armada-385 clearfog gtr. > > Further remove an invalid io from the first sfp connector description, > and add descriptions for the secondary sfp connector which is driven > by dsa switch port number 9. > > Finally labels of dsa switch ports were updated to match the enclosure. > That patch is not suitable for stable. > > Signed-off-by: Josua Mayer Rebased on v6.8-rc1, fixed merge conflict and applied on mvebu/dt Thanks, Gregory > --- > Changes in v5: > - remove empty lines between description and items yaml entries > (reported by Krzysztof Kozlowski ) > - bindings text to yaml conversion is now single commit > (reported by Krzysztof Kozlowski ) > - added bindings for some existing a380, a385 and a388 boards > - removed code from commit description > (reported by Krzysztof Kozlowski ) > - Link to v4: https://lore.kernel.org/r/20231230-support-clearfog-gtr-l8-sfp-v4-0-1d7f0e2c7128@solid-run.com > > Changes in v4: > - dropped invalid soc-only armada-38x (txt) bindings > (reported by Krzysztof Kozlowski ) > - add bindings for armada 388 helios-4 > - updated yaml whitespace indentation count > (reported by Rob Hering's bot) > - Link to v3: https://lore.kernel.org/r/20231226-support-clearfog-gtr-l8-sfp-v3-0-fd1ac2d6bf2e@solid-run.com > > Changes in v3: > - armada-38x.yaml: removed '|', no need to prerserve formatting > (reported by Conor Dooley conor+dt@kernel.org) > - update commit descriptions to clarify confusing board names and > compatible strings > (reported by Krzysztof Kozlowski ) > - send to all relevant lists > (reported by Krzysztof Kozlowski ) > - remove duplicate binding for clearfog / clearfog > - Link to v2 (patches 0-2): > https://lore.kernel.org/r/20231224143750.5604-1-josua@solid-run.com > - Link to v2 (patches 3-8): > https://lore.kernel.org/r/20231224143850.5671-3-josua@solid-run.com > > Changes in v2: > - removed changes changes to gpio numbers because they were wrong > - added bindings documentation > - Link to v1: https://lore.kernel.org/r/20231223212930.14624-1-josua@solid-run.com > > --- > Josua Mayer (10): > dt-bindings: marvell: a38x: convert soc compatibles to yaml > dt-bindings: marvell: a38x: add solidrun armada 388 clearfog boards > dt-bindings: marvell: a38x: add kobol helios-4 board > dt-bindings: marvell: a38x: add solidrun armada 385 clearfog gtr boards > arm: dts: marvell: clearfog: add pro variant compatible in legacy dts > arm: dts: marvell: clearfog-gtr: add board-specific compatible strings > arm: dts: marvell: clearfog-gtr: sort pinctrl nodes alphabetically > arm: dts: marvell: clearfog-gtr: add missing pinctrl for all used gpios > arm: dts: marvell: clearfog-gtr-l8: add support for second sfp connector > arm: dts: marvell: clearfog-gtr-l8: align port numbers with enclosure > > .../devicetree/bindings/arm/marvell/armada-38x.txt | 27 ------- > .../bindings/arm/marvell/armada-38x.yaml | 70 ++++++++++++++++++ > .../dts/marvell/armada-385-clearfog-gtr-l8.dts | 38 +++++++--- > .../dts/marvell/armada-385-clearfog-gtr-s4.dts | 2 + > .../boot/dts/marvell/armada-385-clearfog-gtr.dtsi | 84 ++++++++++++++++------ > arch/arm/boot/dts/marvell/armada-388-clearfog.dts | 5 +- > 6 files changed, 167 insertions(+), 59 deletions(-) > --- > base-commit: 861deac3b092f37b2c5e6871732f3e11486f7082 > change-id: 20231226-support-clearfog-gtr-l8-sfp-d87ae715a787 > > Sincerely, > -- > Josua Mayer > -- Gregory Clement, Bootlin Embedded Linux and Kernel engineering http://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel