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 ABCE3C27C7B for ; Mon, 17 Jun 2024 07:33:44 +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:MIME-Version: Content-Transfer-Encoding:Content-Type:References:In-Reply-To:Date:Cc:To:From :Subject:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=jkSFmq+QIQxRLTqnlNiAfSbUBGRyyeuuOZTApiqXJqc=; b=WiQGHHkIEz5qYtJ1PaL+bgrnoe 4rY5S01PZd19syR3hZYzTxvnUsjX/79yY5BkkFUxhVrwR8IAVIg7ZJBDkh13DcGvL4hH0B+8JJDMV s1kZH+GC8F2B7Io0+q+/ZtLO7CEnEyfuRZwQ9Ff78ReHZ7vOORc+y6+wvEgo117pzE5oJwXAqOmJI smd8nAVRZUJME5iWqVOUaRhQs3Ol5rGDYX6KByX8B6X5zf4mWF557RCkDFAqlegMc4EMTDA46CSqF +kkqlayWV6sUhj4G8wzydFj9KW/uiZIui5ZaI46uzH7sAotYITkgZ7FEY/bLZ6VPM/GktTUjpMipC MMZkbPow==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sJ6rr-00000009fyq-2Bsj; Mon, 17 Jun 2024 07:33:31 +0000 Received: from pi.codeconstruct.com.au ([203.29.241.158] helo=codeconstruct.com.au) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sJ6rb-00000009fo4-2zmj for linux-arm-kernel@lists.infradead.org; Mon, 17 Jun 2024 07:33:19 +0000 Received: from [192.168.68.112] (ppp118-210-79-194.adl-adc-lon-bras32.tpg.internode.on.net [118.210.79.194]) by mail.codeconstruct.com.au (Postfix) with ESMTPSA id ED8492014A; Mon, 17 Jun 2024 15:33:11 +0800 (AWST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=codeconstruct.com.au; s=2022a; t=1718609593; bh=jkSFmq+QIQxRLTqnlNiAfSbUBGRyyeuuOZTApiqXJqc=; h=Subject:From:To:Cc:Date:In-Reply-To:References; b=EdcpG68asKfnCXSZa0TL+Pz/gdPTcrOZB/FtOK/Gn90lLys2gXSwb5movxxiDGJWK 5PsoNoMQTYyPQlIWwrJH6OKy/Qscz5huoNcSp/EQgHKio1fyUwPTibZuswPQWfQoB2 p2LtdxCikIBvRO6DGbZLlarl+XaRVgAXglbfQbuleQ8Z5weadT63R41slT1vomX6R6 0Ooknl61y7WVDnhVkETZ7FuUY++DCxrNPNj81eo4n4IcSpcV9S4/krnq49ia/P4C0v KF8cPqTOZFzU6BZmr0sCrVGX2IYxknsDuK6xrbxuHxIDDXh9rTdde6vQ5KljDEqfet CEPxpgWlt7Z4g== Message-ID: <946f44526e3016f595bfe463cf0a7f5b4eaa084a.camel@codeconstruct.com.au> Subject: Re: [PATCH 1/1] pinctrl: aspeed-g6: Add NCSI pin group config From: Andrew Jeffery To: Potin Lai , Linus Walleij , Joel Stanley Cc: linux-aspeed@lists.ozlabs.org, openbmc@lists.ozlabs.org, linux-gpio@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Patrick Williams , Cosmo Chou , Potin Lai Date: Mon, 17 Jun 2024 17:03:11 +0930 In-Reply-To: <20240613080725.2531580-1-potin.lai.pt@gmail.com> References: <20240613080725.2531580-1-potin.lai.pt@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.46.4-2 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240617_003316_382853_A949FE17 X-CRM114-Status: GOOD ( 14.75 ) 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 Thu, 2024-06-13 at 16:07 +0800, Potin Lai wrote: > In the NCSI pin table, the reference clock output pin (RMIIXRCLKO) is not > needed on the management controller side. >=20 > To optimize pin usage, add new NCSI pin groupis that excludes RMIIXRCLKO, > reducing the number of required pins. Hmm, I'm not convinced this is specific to NCSI (and it's an unfortunate mistake on my part), but we do need to call the groups something different than RMII[34]. Did you have any other suggestions? >=20 > Signed-off-by: Potin Lai > --- > drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) >=20 > diff --git a/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c b/drivers/pinctrl= /aspeed/pinctrl-aspeed-g6.c > index 7938741136a2c..31e4e0b342a00 100644 > --- a/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c > +++ b/drivers/pinctrl/aspeed/pinctrl-aspeed-g6.c > @@ -249,7 +249,9 @@ PIN_DECL_2(E26, GPIOD3, RGMII3RXD3, RMII3RXER); > =20 > FUNC_GROUP_DECL(RGMII3, H24, J22, H22, H23, G22, F22, G23, G24, F23, F26= , F25, > E26); > -FUNC_GROUP_DECL(RMII3, H24, J22, H22, H23, G23, F23, F26, F25, E26); > +GROUP_DECL(RMII3, H24, J22, H22, H23, G23, F23, F26, F25, E26); > +GROUP_DECL(NCSI3, J22, H22, H23, G23, F23, F26, F25, E26); > +FUNC_DECL_2(RMII3, RMII3, NCSI3); > =20 > #define F24 28 > SIG_EXPR_LIST_DECL_SESG(F24, NCTS3, NCTS3, SIG_DESC_SET(SCU410, 28)); > @@ -355,7 +357,9 @@ FUNC_GROUP_DECL(NRTS4, B24); > =20 > FUNC_GROUP_DECL(RGMII4, F24, E23, E24, E25, D26, D24, C25, C26, C24, B26= , B25, > B24); > -FUNC_GROUP_DECL(RMII4, F24, E23, E24, E25, C25, C24, B26, B25, B24); > +GROUP_DECL(RMII4, F24, E23, E24, E25, C25, C24, B26, B25, B24); > +GROUP_DECL(NCSI4, E23, E24, E25, C25, C24, B26, B25, B24); > +FUNC_DECL_2(RMII4, RMII4, NCSI4); > =20 > #define D22 40 > SIG_EXPR_LIST_DECL_SESG(D22, SD1CLK, SD1, SIG_DESC_SET(SCU414, 8)); > @@ -1976,6 +1980,8 @@ static const struct aspeed_pin_group aspeed_g6_grou= ps[] =3D { > ASPEED_PINCTRL_GROUP(MDIO2), > ASPEED_PINCTRL_GROUP(MDIO3), > ASPEED_PINCTRL_GROUP(MDIO4), > + ASPEED_PINCTRL_GROUP(NCSI3), > + ASPEED_PINCTRL_GROUP(NCSI4), You will need to update the binding document as well. I've poked Linus W about a series I sent that re-formats the binding function and group lists - it would be nice if you rework the patch on top of that: https://lore.kernel.org/lkml/5bf8e1dddd2b958a102e7b1b9f9c080a34f9deff.camel= @codeconstruct.com.au/ Cheers, Andrew