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 25589CD5BA4 for ; Tue, 19 May 2026 10:38:41 +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:CC:To: Content-Transfer-Encoding:Content-Type: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=FWtkLp/R5WFv80whuUO7KL0k05VJ7QqBO/vDoN3VrT0=; b=gXlZNwNFDo7QAd IREdjcp/5Ksb5ug2qizHlsY2G+WqiLYl5DxIS8Oz6J00V6PML5sXMwUvhk/ce8dXmxIDdGbLrhRkd vjuP1QMT0mQV43fA2y/R34b+zjbBKNgLyVmIwfD7ApPsSWa3qirAxI8674L5Ik11gU7fKqMZ/LeFB Qysg+TFbGYX1WXGogdYyIZRE+RTFpTSGcJHeSTj7I6Jl0uN2KcCrnwGssHUiPIXN5LzS0n2SDWkiG 1T+Xca5s6+brg4VkkQ2lqsXF8zsow8D14jXY7KzdIukpxg+Z7ixn6yBPWNSGnqJW7YCrfd4I8A7c+ 5N8YD9jY0JQt9cFPoy5w==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1wPHqL-000000017LF-3QJz; Tue, 19 May 2026 10:38:33 +0000 Received: from mail.aspeedtech.com ([211.20.114.72] helo=twmbx01.aspeedtech.com) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1wPHqG-000000017EX-2Ucn for linux-arm-kernel@lists.infradead.org; Tue, 19 May 2026 10:38:30 +0000 Received: from TWMBX01.aspeed.com (192.168.0.62) by TWMBX01.aspeed.com (192.168.0.62) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1748.10; Tue, 19 May 2026 18:38:20 +0800 Received: from [127.0.1.1] (192.168.10.13) by TWMBX01.aspeed.com (192.168.0.62) with Microsoft SMTP Server id 15.2.1748.10 via Frontend Transport; Tue, 19 May 2026 18:38:20 +0800 From: Billy Tsai Subject: [PATCH v4 0/3] pinctrl: aspeed: Add AST2700 SoC1 support Date: Tue, 19 May 2026 18:38:11 +0800 Message-ID: <20260519-pinctrl-single-bit-v4-0-5fe568a8ffde@aspeedtech.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-B4-Tracking: v=1; b=H4sIAJQ9DGoC/23NTQqDMBCG4atI1k3JTPztqvcoXcQ40QGrkoi0F O/eVCgu6vL9YJ55i0CeKYhL8haeFg48DjHSUyJsZ4aWJDexBSrMFQLIiQc7+14GHtqeZM2zbAy CdliirTIRDydPjp8bervH7jjMo39tPxb4rj9OH3ELSCVtrhy6ulTk6qsJE1Ezk+3OdnyIr7ng7 miVHzoYnaKqoLTO1HkKh47enQyyQ0dHh6qiMakGBJX+Oeu6fgAMHxhKSQEAAA== X-Change-ID: 20260211-pinctrl-single-bit-da213f282c95 To: Linus Walleij , Tony Lindgren , "Rob Herring" , Krzysztof Kozlowski , "Conor Dooley" , Joel Stanley , Andrew Jeffery , Bartosz Golaszewski , "Lee Jones" , Ryan Chen CC: , , , , , , , , Andrew Jeffery , , Billy Tsai , Conor Dooley X-Mailer: b4 0.14.3 X-Developer-Signature: v=1; a=ed25519-sha256; t=1779187100; l=5847; i=billy_tsai@aspeedtech.com; s=20251118; h=from:subject:message-id; bh=L1iLfDQrZwxvnw6/O3AJV6ahyxO2sv21yD4ZkfxrN4k=; b=yZ+Y7S6UBimAwm6wBTS+wu7ylZooUUWR9axObgWXnkPI/Jy4ScwWNWwQ/oxN2gtcn+HpJoFaY ms+njnXKTm3CoP/9uIqxvjO8JBWuxqkTALyd4RU0IwVRomo0NoCI+tE X-Developer-Key: i=billy_tsai@aspeedtech.com; a=ed25519; pk=/A8qvgZ6CPfnwKgT6/+k+nvXOkN477MshEGJvVdzeeQ= X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260519_033828_644083_AF0274C8 X-CRM114-Status: GOOD ( 16.96 ) 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 Legacy ASPEED pin controllers have historically not had a coherent register interface. Control fields often had no consistent mapping to individual pins, and configuring a function frequently required coordinating multiple control bits across several registers. As a result, the existing ASPEED pinctrl drivers rely on complex macro infrastructure to describe the dependencies between pins, functions, and register fields. The pin controller for SoC1 in the AST2700 breaks from this legacy design. For SoC1, each pin maps directly to a dedicated function field in the SCU register space that determines the active mux function for that pin. This results in a much more regular register layout compared to previous generations. While the behaviour is conceptually similar to pinctrl-single, the register layout and configuration model differ enough that reusing pinctrl-single directly is not practical. Therefore this driver is implemented as a SoC-specific pinctrl driver using static data tables to describe the register layout. The binding reuses the standard pinmux and generic pin configuration schemas and does not introduce any custom Devicetree properties. Signed-off-by: Billy Tsai --- Changes in v4: - Fix SGMII0 mux register: SGMII0's control bit is at SCU47C[0], outside the contiguous pin-indexed MUX register range. Handle it as a special case in set_mux (analogous to PCIERC2_PERST) using field.mask = 0x1 to avoid clobbering adjacent bits in SCU47C. - Use devm_pinctrl_register_and_init() and pinctrl_enable(): populate all groups and functions before enabling the controller, closing the race window where a consumer could observe zero groups/functions. - pin_config_get: return -EINVAL when BIAS_DISABLE is not active; return -ENOTSUPP for BIAS_PULL_UP and BIAS_PULL_DOWN because the hardware has a single 1-bit enable/disable field per pin and pull direction cannot be read back from the register. - Probe: preserve const qualification in the pinctrl_generic_add_group() pin array cast. - Binding: retain additionalProperties: false for the state-node schema. The state-node properties block re-declares the four pincfg properties the hardware supports (bias-disable, bias-pull-up, bias-pull-down, drive-strength) and intentionally rejects all others. unevaluatedProperties: false was suggested as an alternative, but that would permit the full set of pincfg-node.yaml properties (input-schmitt-enable, slew-rate, etc.) even though the hardware does not support them. additionalProperties: false is the correct mechanism when the goal is to restrict the accepted properties to a known subset. - Link to v3: https://lore.kernel.org/r/20260515-pinctrl-single-bit-v3-0-e97da4312104@aspeedtech.com Changes in v3: - Added pin configuration group support for AST2700 SoC1 by implementing `pin_config_group_get()` and `pin_config_group_set()`. - Restricted AST2700 SoC1 `drive-strength` settings to the supported 4/8/12/16 mA values in the pinctrl driver. - Convert kernel-doc comment for aspeed_g7_soc1_drv_map to a regular comment to avoid kernel-doc warning reported by kernel test robot. - Update the AST2700 SoC1 pinctrl binding to describe the `reg` property and require it. - Allow standard pinconf properties in pin state nodes. - Add a binding example for the AST2700 SoC1 pinctrl node. - Add state-node description, function+groups dependency constraint, and oneOf groups/pins constraint to the binding, matching the SoC0 binding style. - Add pins enum (212 entries) to the binding to cover all physical pins that support per-pin configuration. - Add UART modem-line signals as independent functions/groups: NCTS0/1/5/6, NDCD0/1/5/6, NDSR0/1/5/6, NDTR0/1/5/6, NRI0/1/5/6, NRTS0/1/5/6; remove those pins from UART0/1/5/6 groups. - Add LTPI_PS_I2C0/1/2/3 functions/groups for I2C-over-LTPI; extend I2C0/1/2/3 functions with the new LTPI groups. - Fix typo: rename RMII0RCKO/RMII1RCKO to RMII0RCLKO/RMII1RCLKO. - Fix wrong index: rename DSGPM1 to DSGPM0. - Kconfig: use "Aspeed G7 SoC1 pin control" to match neighbouring entries. - pin_config_get: fix BIAS_DISABLE readback (val=!val must be skipped for BIAS_DISABLE since hardware bit=1 means pull disabled). - set_mux: remove dead null check on grp; propagate regmap_update_bits() return value. - gpio_request_enable: propagate regmap_update_bits() return value. - Link to v2: https://lore.kernel.org/r/20260306-pinctrl-single-bit-v2-0-79918cfab641@aspeedtech.com Changes in v2: - Updated the series title to focus on AST2700 SoC1 support. - Reworked implementation to use static SoC-specific layout tables instead of a generic packed-field model. - Dropped the generic "pinctrl-packed" driver approach. - Removed custom Devicetree properties. - Updated binding to reuse standard pinmux and generic pin configuration schemas. - Link to v1: https://lore.kernel.org/r/20260213-pinctrl-single-bit-v1-0-c60f2fb80efb@aspeedtech.com --- Billy Tsai (3): dt-bindings: mfd: aspeed,ast2x00-scu: Support AST2700 SoC1 pinctrl dt-bindings: pinctrl: Add aspeed,ast2700-soc1-pinctrl pinctrl: aspeed: Add AST2700 SoC1 support .../bindings/mfd/aspeed,ast2x00-scu.yaml | 1 + .../pinctrl/aspeed,ast2700-soc1-pinctrl.yaml | 760 +++++++++ drivers/pinctrl/aspeed/Kconfig | 14 + drivers/pinctrl/aspeed/Makefile | 1 + drivers/pinctrl/aspeed/pinctrl-aspeed-g7-soc1.c | 1754 ++++++++++++++++++++ 5 files changed, 2530 insertions(+) --- base-commit: e532a5a81d0db872acd2c0a92d2639580ca3da44 change-id: 20260211-pinctrl-single-bit-da213f282c95 Best regards, -- Billy Tsai