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 49F4BC4167B for ; Mon, 2 Jan 2023 08:08:14 +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:References:In-Reply-To: Message-ID:Date: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=droU+BngMnb48N+6KeDQ9yxUYwp58jF9h5d9yYdSC3s=; b=tckAbeLN++W5Kn dfyYOEUc89C7cUjGCmayoi2ApSH/aVZmFmF4QwFKTn/l3lLlpGi/z87xBsXQ1Bd0uwyrDdH3fQpU0 562gVm0HQxqQM3ioLoNUl24g3CHm0AmZ5GFt7g+LKRXtmlFPVJlF1Hn0aV8yzi9Edx08O8gZt85P5 EkkbdUqNIsuYlqbPIUZz5bmUJJTSEfKzGCCJUGDo0BOSJ/LLmoT95OABZqWhl20deQk6OWT96AcK3 /O8ajET3yIbIZDaIQr2W5Fo2gWAJsQEvSP+qSnH5J+5vvFLLRRRIb2z8EAk0l9xPEs4aLrL/7bPmb +IecRI3/pOIhUerqr5gg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pCFqZ-009KLv-JQ; Mon, 02 Jan 2023 08:07:03 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pCFqK-009KHX-Ge for linux-arm-kernel@lists.infradead.org; Mon, 02 Jan 2023 08:06:52 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1672646808; x=1704182808; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=veKOufvzsDh8m+WG9BQ4mPSrOlaa4WGpJHiH80CC9b0=; b=MPrmRVZmmIyFX+Q6ygEIsadYcheS2QuXbYEVS+ShFmg7REGxbVBxdewp eOA8WqTRGo6WTvxs7dGPCvg+LICNIKwuypjhSVuoFbnqv0av0P8jDel5n D3TvU6ikg7H9e+hnA17ai5uJupCvEYOg/UFhbLecfccffts9ZsPEu+bFv wstvxY0oBfV0dLRuHpZvnzufbS0gJKe4/OHiYwT7kYzNMvo7RLAP4qgWI jOvP1yvWJd6QpNxz1rK1lm5W8s/WrVvMKnGSwJAZcSKG11eeylXAHQQMV oA9M10hzYUrU+sfSo9RbC7jMyzuWvgNZGHVNFPo1SDd02+6+9OoCOmYu4 Q==; X-IronPort-AV: E=Sophos;i="5.96,293,1665439200"; d="scan'208";a="28196760" Received: from unknown (HELO tq-pgp-pr1.tq-net.de) ([192.168.6.15]) by mx1-pgp.tq-group.com with ESMTP; 02 Jan 2023 09:06:42 +0100 Received: from mx1.tq-group.com ([192.168.6.7]) by tq-pgp-pr1.tq-net.de (PGP Universal service); Mon, 02 Jan 2023 09:06:42 +0100 X-PGP-Universal: processed; by tq-pgp-pr1.tq-net.de on Mon, 02 Jan 2023 09:06:42 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1672646802; x=1704182802; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=veKOufvzsDh8m+WG9BQ4mPSrOlaa4WGpJHiH80CC9b0=; b=edFioQnHHr7w9BjycbJNb05T+xPXI3pUtOicw1oee38Sc3+lT1mYj5l2 qdlDzvaHWO5rC+VXT7YZR5kFqpBpK3iCU/bkjUVkTDVMW0M5sb6CISXRJ 9GmYl8LHG+4uaqi59fOAvjwhe8WbpIacZh8opYZ/KWje8vhP3RhXc7rJz J6c83J3wEhas28Oo/Pkly0y8aTLyVfAyPUz8mOnY+kTQNByIEae18zd0w Wc2xocJV5Mlv/NSU5EdgWiiloufY08yMQjQOt4CR0FVI88yqyywFzHZeM KnkaqCFqw4D/iK9eKD+BNep4aP/wP/EGdeBwmvd2z9JqKRalYOQbzRUiS Q==; X-IronPort-AV: E=Sophos;i="5.96,293,1665439200"; d="scan'208";a="28196759" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 02 Jan 2023 09:06:41 +0100 Received: from steina-w.localnet (unknown [10.123.53.21]) by vtuxmail01.tq-net.de (Postfix) with ESMTP id A2C43280072; Mon, 2 Jan 2023 09:06:41 +0100 (CET) From: Alexander Stein To: aisheng.dong@nxp.com, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, shawnguo@kernel.org, s.hauer@pengutronix.de, ping.bai@nxp.com, linus.walleij@linaro.org, linux-gpio@vger.kernel.org, "Peng Fan (OSS)" Cc: kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, linux-i2c@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Peng Fan Subject: Re: [PATCH] dt-bindings: pinctrl: imx8m: Integrate duplicated i.MX 8M schemas Date: Mon, 02 Jan 2023 09:04:37 +0100 Message-ID: <3225978.44csPzL39Z@steina-w> Organization: TQ-Systems GmbH In-Reply-To: <20221223030708.91459-1-peng.fan@oss.nxp.com> References: <20221223030708.91459-1-peng.fan@oss.nxp.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230102_000649_191216_4C1319B2 X-CRM114-Status: GOOD ( 24.38 ) 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 Peng, Am Freitag, 23. Dezember 2022, 04:07:08 CET schrieb Peng Fan (OSS): > From: Peng Fan > > The i.MX8MM/N/P/Q IOMUXC schemas are basically the same, it does not to > have four schemas for almost the same binding. Nice, just one nit below. > Signed-off-by: Peng Fan > --- > ...mm-pinctrl.yaml => fsl,imx8m-pinctrl.yaml} | 16 ++-- > .../bindings/pinctrl/fsl,imx8mn-pinctrl.yaml | 84 ------------------- > .../bindings/pinctrl/fsl,imx8mp-pinctrl.yaml | 84 ------------------- > .../bindings/pinctrl/fsl,imx8mq-pinctrl.yaml | 84 ------------------- > 4 files changed, 10 insertions(+), 258 deletions(-) > rename Documentation/devicetree/bindings/pinctrl/{fsl,imx8mm-pinctrl.yaml > => fsl,imx8m-pinctrl.yaml} (82%) delete mode 100644 > Documentation/devicetree/bindings/pinctrl/fsl,imx8mn-pinctrl.yaml delete > mode 100644 > Documentation/devicetree/bindings/pinctrl/fsl,imx8mp-pinctrl.yaml delete > mode 100644 > Documentation/devicetree/bindings/pinctrl/fsl,imx8mq-pinctrl.yaml > > diff --git > a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mm-pinctrl.yaml > b/Documentation/devicetree/bindings/pinctrl/fsl,imx8m-pinctrl.yaml > similarity index 82% > rename from > Documentation/devicetree/bindings/pinctrl/fsl,imx8mm-pinctrl.yaml rename to > Documentation/devicetree/bindings/pinctrl/fsl,imx8m-pinctrl.yaml index > 6717f163390b..949d962a97b4 100644 > --- a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mm-pinctrl.yaml > +++ b/Documentation/devicetree/bindings/pinctrl/fsl,imx8m-pinctrl.yaml > @@ -1,13 +1,13 @@ > # SPDX-License-Identifier: GPL-2.0 > %YAML 1.2 > --- > -$id: http://devicetree.org/schemas/pinctrl/fsl,imx8mm-pinctrl.yaml# > +$id: http://devicetree.org/schemas/pinctrl/fsl,imx8m-pinctrl.yaml# > $schema: http://devicetree.org/meta-schemas/core.yaml# > > -title: Freescale IMX8MM IOMUX Controller > +title: Freescale IMX8M IOMUX Controller > > maintainers: > - - Anson Huang > + - Peng Fan > > description: > Please refer to fsl,imx-pinctrl.txt and pinctrl-bindings.txt in this > directory @@ -15,7 +15,11 @@ description: > > properties: > compatible: > - const: fsl,imx8mm-iomuxc > + enum: > + - fsl,imx8mm-iomuxc > + - fsl,imx8mn-iomuxc > + - fsl,imx8mp-iomuxc > + - fsl,imx8mq-iomuxc > > reg: > maxItems: 1 > @@ -34,9 +38,9 @@ patternProperties: > each entry consists of 6 integers and represents the mux and > config setting for one pin. The first 5 integers input_reg mux_val input_val> are specified using a PIN_FUNC_ID macro, which > can - be found in > . The last + be > found in . The last > integer CONFIG is the pad setting value like pull-up on this pin. Please - > refer to i.MX8M Mini Reference Manual for detailed CONFIG settings. > + refer to i.MX8M Quad/Mini/Nano/Plus Reference Manual for Please sort this alphabetically, similar to the compatible list. Thanks Alexander > detailed CONFIG settings. $ref: > /schemas/types.yaml#/definitions/uint32-matrix > items: > items: > diff --git > a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mn-pinctrl.yaml > b/Documentation/devicetree/bindings/pinctrl/fsl,imx8mn-pinctrl.yaml deleted > file mode 100644 > index b1cdbb56d4e4..000000000000 > --- a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mn-pinctrl.yaml > +++ /dev/null > @@ -1,84 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0 > -%YAML 1.2 > ---- > -$id: http://devicetree.org/schemas/pinctrl/fsl,imx8mn-pinctrl.yaml# > -$schema: http://devicetree.org/meta-schemas/core.yaml# > - > -title: Freescale IMX8MN IOMUX Controller > - > -maintainers: > - - Anson Huang > - > -description: > - Please refer to fsl,imx-pinctrl.txt and pinctrl-bindings.txt in this > directory - for common binding part and usage. > - > -properties: > - compatible: > - const: fsl,imx8mn-iomuxc > - > - reg: > - maxItems: 1 > - > -# Client device subnode's properties > -patternProperties: > - 'grp$': > - type: object > - description: > - Pinctrl node's client devices use subnodes for desired pin > configuration. - Client device subnodes use below standard properties. > - > - properties: > - fsl,pins: > - description: > - each entry consists of 6 integers and represents the mux and > config - setting for one pin. The first 5 integers conf_reg input_reg - mux_val input_val> are specified using a > PIN_FUNC_ID macro, which can - be found in > . The last - > integer CONFIG is the pad setting value like pull-up on this pin. Please - > refer to i.MX8M Nano Reference Manual for detailed CONFIG settings. > - $ref: /schemas/types.yaml#/definitions/uint32-matrix > - items: > - items: > - - description: | > - "mux_reg" indicates the offset of mux register. > - - description: | > - "conf_reg" indicates the offset of pad configuration > register. - - description: | > - "input_reg" indicates the offset of select input register. > - - description: | > - "mux_val" indicates the mux value to be applied. > - - description: | > - "input_val" indicates the select input value to be applied. > - - description: | > - "pad_setting" indicates the pad configuration value to be > applied. - > - required: > - - fsl,pins > - > - additionalProperties: false > - > -allOf: > - - $ref: "pinctrl.yaml#" > - > -required: > - - compatible > - - reg > - > -additionalProperties: false > - > -examples: > - # Pinmux controller node > - - | > - iomuxc: pinctrl@30330000 { > - compatible = "fsl,imx8mn-iomuxc"; > - reg = <0x30330000 0x10000>; > - > - pinctrl_uart2: uart2grp { > - fsl,pins = > - <0x23C 0x4A4 0x4FC 0x0 0x0 0x140>, > - <0x240 0x4A8 0x000 0x0 0x0 0x140>; > - }; > - }; > - > -... > diff --git > a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mp-pinctrl.yaml > b/Documentation/devicetree/bindings/pinctrl/fsl,imx8mp-pinctrl.yaml deleted > file mode 100644 > index 4eed3a4e153a..000000000000 > --- a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mp-pinctrl.yaml > +++ /dev/null > @@ -1,84 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0 > -%YAML 1.2 > ---- > -$id: http://devicetree.org/schemas/pinctrl/fsl,imx8mp-pinctrl.yaml# > -$schema: http://devicetree.org/meta-schemas/core.yaml# > - > -title: Freescale IMX8MP IOMUX Controller > - > -maintainers: > - - Anson Huang > - > -description: > - Please refer to fsl,imx-pinctrl.txt and pinctrl-bindings.txt in this > directory - for common binding part and usage. > - > -properties: > - compatible: > - const: fsl,imx8mp-iomuxc > - > - reg: > - maxItems: 1 > - > -# Client device subnode's properties > -patternProperties: > - 'grp$': > - type: object > - description: > - Pinctrl node's client devices use subnodes for desired pin > configuration. - Client device subnodes use below standard properties. > - > - properties: > - fsl,pins: > - description: > - each entry consists of 6 integers and represents the mux and > config - setting for one pin. The first 5 integers conf_reg input_reg - mux_val input_val> are specified using a > PIN_FUNC_ID macro, which can - be found in > . The last - > integer CONFIG is the pad setting value like pull-up on this pin. Please - > refer to i.MX8M Plus Reference Manual for detailed CONFIG settings. > - $ref: /schemas/types.yaml#/definitions/uint32-matrix > - items: > - items: > - - description: | > - "mux_reg" indicates the offset of mux register. > - - description: | > - "conf_reg" indicates the offset of pad configuration > register. - - description: | > - "input_reg" indicates the offset of select input register. > - - description: | > - "mux_val" indicates the mux value to be applied. > - - description: | > - "input_val" indicates the select input value to be applied. > - - description: | > - "pad_setting" indicates the pad configuration value to be > applied. - > - required: > - - fsl,pins > - > - additionalProperties: false > - > -allOf: > - - $ref: "pinctrl.yaml#" > - > -required: > - - compatible > - - reg > - > -additionalProperties: false > - > -examples: > - # Pinmux controller node > - - | > - iomuxc: pinctrl@30330000 { > - compatible = "fsl,imx8mp-iomuxc"; > - reg = <0x30330000 0x10000>; > - > - pinctrl_uart2: uart2grp { > - fsl,pins = > - <0x228 0x488 0x5F0 0x0 0x6 0x49>, > - <0x228 0x488 0x000 0x0 0x0 0x49>; > - }; > - }; > - > -... > diff --git > a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mq-pinctrl.yaml > b/Documentation/devicetree/bindings/pinctrl/fsl,imx8mq-pinctrl.yaml deleted > file mode 100644 > index d4a8ea5551a5..000000000000 > --- a/Documentation/devicetree/bindings/pinctrl/fsl,imx8mq-pinctrl.yaml > +++ /dev/null > @@ -1,84 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0 > -%YAML 1.2 > ---- > -$id: http://devicetree.org/schemas/pinctrl/fsl,imx8mq-pinctrl.yaml# > -$schema: http://devicetree.org/meta-schemas/core.yaml# > - > -title: Freescale IMX8MQ IOMUX Controller > - > -maintainers: > - - Anson Huang > - > -description: > - Please refer to fsl,imx-pinctrl.txt and pinctrl-bindings.txt in this > directory - for common binding part and usage. > - > -properties: > - compatible: > - const: fsl,imx8mq-iomuxc > - > - reg: > - maxItems: 1 > - > -# Client device subnode's properties > -patternProperties: > - 'grp$': > - type: object > - description: > - Pinctrl node's client devices use subnodes for desired pin > configuration. - Client device subnodes use below standard properties. > - > - properties: > - fsl,pins: > - description: > - each entry consists of 6 integers and represents the mux and > config - setting for one pin. The first 5 integers conf_reg input_reg - mux_val input_val> are specified using a > PIN_FUNC_ID macro, which can - be found in > . The last - > integer CONFIG is the pad setting value like pull-up on this pin. Please - > refer to i.MX8M Quad Reference Manual for detailed CONFIG settings. > - $ref: /schemas/types.yaml#/definitions/uint32-matrix > - items: > - items: > - - description: | > - "mux_reg" indicates the offset of mux register. > - - description: | > - "conf_reg" indicates the offset of pad configuration > register. - - description: | > - "input_reg" indicates the offset of select input register. > - - description: | > - "mux_val" indicates the mux value to be applied. > - - description: | > - "input_val" indicates the select input value to be applied. > - - description: | > - "pad_setting" indicates the pad configuration value to be > applied. - > - required: > - - fsl,pins > - > - additionalProperties: false > - > -allOf: > - - $ref: "pinctrl.yaml#" > - > -required: > - - compatible > - - reg > - > -additionalProperties: false > - > -examples: > - # Pinmux controller node > - - | > - iomuxc: pinctrl@30330000 { > - compatible = "fsl,imx8mq-iomuxc"; > - reg = <0x30330000 0x10000>; > - > - pinctrl_uart1: uart1grp { > - fsl,pins = > - <0x234 0x49C 0x4F4 0x0 0x0 0x49>, > - <0x238 0x4A0 0x4F4 0x0 0x0 0x49>; > - }; > - }; > - > -... _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel