From patchwork Mon Oct 16 09:12:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Linus Walleij X-Patchwork-Id: 13422800 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 66D28CDB483 for ; Mon, 16 Oct 2023 09:13:48 +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:Cc:To:MIME-Version:Message-Id:Date: Subject:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=dCqcIWkLOFvRnfMSJbVFQTTZ2tmjThjMKVsFlFYEsS8=; b=pXKf9ROIrQtOeQ zhfzZVWEOyAHVFtSx+AA2/QrnEIsEHJ4IIrAh7H/ReRiODwniYJ3HUeTt4V4tfHDzebEIzXQ6npWG Eyt5h2rnvDLCrOmoYGmCtLaSxJ0FKhuOUCLpXGEYJ2irKrdpd7gz9iT4j0ACrdXcNwJ6hIyTNuRLj 9LDJ9sTq1+EdUZSNaMU7iazf7O9sR5TgxDxpU+EsMYqSo9VPtVF009l5whYGZRL6QQdX7jLHPwlcg KnilRP0ij6kxY94J3/1lw8V+8nRq0KJZchc1vD2l5Cr9RbVLxCShXHwB01Ni2f9KgaMWhpZ9HCSAB 1XLdWinVxUT2FXszDRMA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qsJf8-008woE-2d; Mon, 16 Oct 2023 09:13:22 +0000 Received: from mail-lf1-x134.google.com ([2a00:1450:4864:20::134]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qsJf0-008wfO-1i for linux-arm-kernel@lists.infradead.org; Mon, 16 Oct 2023 09:13:16 +0000 Received: by mail-lf1-x134.google.com with SMTP id 2adb3069b0e04-507adc3381cso1168163e87.3 for ; Mon, 16 Oct 2023 02:12:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1697447575; x=1698052375; darn=lists.infradead.org; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:from:to:cc:subject:date:message-id:reply-to; bh=WZ8ecWrZiBw7tKD+R4W1tepO49kifDs5CDZYCV5hVF8=; b=dHn9c3weHzGGsSTUWznSHHKogBxsiDCVp1u/Uq2Sbg9w8mUqc9d3jW4w0aU3BdfqZz NR2qqV5kdko0c4KB5DULr3UCpmaxONjPmdKZlBane1OtweaGHULzNxD8InBbpPLSYLQZ pRGkS9e4hnRht+bPk0A3w6E+QTkVh8jKG+enUavENcSYxsas9iltJF/dPQqp9eAGrpnN QbgNJ1lpeIWa3U3noCOlYicCLAhUHH8wuCpSNF2Uh0FcnAqEJWHPEGlaIQosf93lsZTZ mTfF/jDPdHeag3Ipih8fZGRvgLsO0WQ1Rfzro1APBmT10lFWkkK/diOWr18tOWHudW4W BIFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1697447575; x=1698052375; h=cc:to:content-transfer-encoding:mime-version:message-id:date :subject:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=WZ8ecWrZiBw7tKD+R4W1tepO49kifDs5CDZYCV5hVF8=; b=Bkd7pv86dP7IVJhEqaEJQAOe9vpo2O2twPEHyGRjGDBwpAcvAkcPHgD9ZliDKMi3nN b3jFWPC4kaHc7Ls3Wvbk3dcM24cC5+DILM6pB3zZ0q6EPwwp2snuwKoqbGsv5hI3H6RW hIJQ1cU+fk9lNF0uw5pHNcu6KEEmcjx/FMrfEq2Ng228oKfGVjmJnpP24n2CnEv2zUc+ 2GXUVl7yp1LtLrmBnfgu5sL2F70J0VtixPX0BgVUhMzqRut1klwIEvQ7D7prLcvy2Dmy l1Luo0YGrwevoPoekKMpixYO1qQ/Bme0NrpM0jje+7S083GndVYGoQv+mAk9IQOhhW9V DW3g== X-Gm-Message-State: AOJu0Yy8IU34dlydbmoe+uDLBU6fg+cY4xyDcBM/+U4QDulSRM2RELl5 /EUVaOh0pgFnH2XtCneotL/4lw== X-Google-Smtp-Source: AGHT+IEyNwIV4lQLtApT9PUXRaVGf07GyDdtakoBSXy1ah0wDPOGkKQGeCoFWnPvGcaojFu4SR19dQ== X-Received: by 2002:a05:6512:328f:b0:503:1c07:f7f9 with SMTP id p15-20020a056512328f00b005031c07f7f9mr24887142lfe.29.1697447575471; Mon, 16 Oct 2023 02:12:55 -0700 (PDT) Received: from [127.0.1.1] ([85.235.12.238]) by smtp.gmail.com with ESMTPSA id br34-20020a056512402200b005068e7a2e7dsm4160986lfb.77.2023.10.16.02.12.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 16 Oct 2023 02:12:55 -0700 (PDT) From: Linus Walleij Subject: [PATCH net-next v3 0/6] Create a binding for the Marvell MV88E6xxx DSA switches Date: Mon, 16 Oct 2023 11:12:53 +0200 Message-Id: <20231016-marvell-88e6152-wan-led-v3-0-38cd449dfb15@linaro.org> MIME-Version: 1.0 X-B4-Tracking: v=1; b=H4sIAJX+LGUC/3XNyw6CMBCF4Vchs7amFy7Fle9hXJR2gCZYTEsqh vDuNnWjC5Z/TuabDQJ6iwEuxQYeow12dinEqQA9KjcgsSY1cMoFo1SSh/IRp4lIiTWrOHkpRyY 0qXUpuqY3vDeQrp8ee7tm+QYOF+JwXeCeltGGZfbv/DKyvH91Jg71yAgltGG8U20rq0ZfJ+uUn 8+zHzIa+S9UHkM8QU2vFZVdLWXZ/kH7vn8AZKge/xIBAAA= To: Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Russell King , Florian Fainelli , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni Cc: Christian Marangi , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Linus Walleij X-Mailer: b4 0.12.3 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231016_021314_569389_E67E1A60 X-CRM114-Status: GOOD ( 15.80 ) 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 This shows the path we could take with this, deprecating the weird external bus thing. I don't know what to do about the irq lines with a pointless type flag that should be onecell:ed. I need proper schema checking to add LED support to the Marvell switch. Just how it is, it can't go on like this. Some Device Tree fixes are included in the series, these remove the major and most annoying warnings fallout noise: some warnings remain, and these are of more serious nature, such as missing phy-mode. Signed-off-by: Linus Walleij --- Changes in v3: - Fix up a related mvusb example in a different binding that the scripts were complaining about. - Fix up the wording on internal vs external MDIO buses in the mv88e6xxx binding document. - Remove pointless label and put the right rev-mii into the MV88E6060 schema. - Link to v2: https://lore.kernel.org/r/20231014-marvell-88e6152-wan-led-v2-0-7fca08b68849@linaro.org Changes in v2: - Break out a separate Marvell MV88E6060 binding file. I stand corrected. - Drop the idea to rely on nodename mdio-external for the external MDIO bus, keep the compatible, drop patch for the driver. - Fix more Marvell DT mistakes. - Fix NXP DT mistakes in a separate patch. - Fix Marvell ARM64 mistakes in a separate patch. - Link to v1: https://lore.kernel.org/r/20231013-marvell-88e6152-wan-led-v1-0-0712ba99857c@linaro.org --- Linus Walleij (6): dt-bindings: marvell: Rewrite MV88E6xxx in schema dt-bindings: marvell: Add Marvell MV88E6060 DSA schema dt-bindings: net: mvusb: Fix up DSA example ARM: dts: marvell: Fix some common switch mistakes ARM: dts: nxp: Fix some common switch mistakes ARM64: dts: marvell: Fix some common switch mistakes .../bindings/net/dsa/marvell,mv88e6060.yaml | 105 +++++++++ .../bindings/net/dsa/marvell,mv88e6xxx.yaml | 243 +++++++++++++++++++++ .../devicetree/bindings/net/dsa/marvell.txt | 109 --------- .../devicetree/bindings/net/marvell,mvusb.yaml | 6 + MAINTAINERS | 3 +- arch/arm/boot/dts/marvell/armada-370-rd.dts | 2 - .../dts/marvell/armada-381-netgear-gs110emx.dts | 6 +- .../dts/marvell/armada-385-clearfog-gtr-l8.dts | 2 +- .../dts/marvell/armada-385-clearfog-gtr-s4.dts | 2 +- arch/arm/boot/dts/marvell/armada-385-linksys.dtsi | 2 - .../boot/dts/marvell/armada-385-turris-omnia.dts | 16 +- arch/arm/boot/dts/marvell/armada-388-clearfog.dts | 2 - .../boot/dts/marvell/armada-xp-linksys-mamba.dts | 2 - arch/arm/boot/dts/nxp/vf/vf610-zii-cfu1.dts | 2 +- arch/arm/boot/dts/nxp/vf/vf610-zii-scu4-aib.dts | 8 +- arch/arm/boot/dts/nxp/vf/vf610-zii-spb4.dts | 2 +- arch/arm/boot/dts/nxp/vf/vf610-zii-ssmb-dtu.dts | 4 +- arch/arm/boot/dts/nxp/vf/vf610-zii-ssmb-spu3.dts | 2 +- .../boot/dts/marvell/armada-3720-espressobin.dtsi | 4 +- .../boot/dts/marvell/armada-3720-gl-mv1000.dts | 4 +- .../boot/dts/marvell/armada-3720-turris-mox.dts | 32 +-- .../boot/dts/marvell/armada-7040-mochabin.dts | 2 - .../dts/marvell/armada-8040-clearfog-gt-8k.dts | 2 +- arch/arm64/boot/dts/marvell/cn9130-crb.dtsi | 6 +- 24 files changed, 397 insertions(+), 171 deletions(-) --- base-commit: 0bb80ecc33a8fb5a682236443c1e740d5c917d1d change-id: 20231008-marvell-88e6152-wan-led-88c43b7fd2fd Best regards,