Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Peng Fan <peng.fan@nxp.com>
To:     Abel Vesa <abel.vesa@nxp.com>, Stephen Boyd <sboyd@kernel.org>,
        Rob Herring <robh@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Mike Turquette <mturquette@baylibre.com>
CC:     Pengutronix Kernel Team <kernel@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        dl-linux-imx <linux-imx@nxp.com>, Frank Li <frank.li@nxp.com>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        "linux-clk@vger.kernel.org" <linux-clk@vger.kernel.org>,
        David Wolfe <david.wolfe@nxp.com>
Subject: RE: [PATCH 2/2] clk: imx: Remove audio_mclk_root_clk
Thread-Topic: [PATCH 2/2] clk: imx: Remove audio_mclk_root_clk
Thread-Index: AQHYE4fT0bbV2fYEaUGLH6MPqVy0J6ye63xg
Date:   Tue, 22 Feb 2022 01:12:59 +0000
Message-ID: <DU0PR04MB941759AC363C5B162ED35DEF883B9@DU0PR04MB9417.eurprd04.prod.outlook.com>
References: <20220127141052.1900174-1-abel.vesa@nxp.com>
 <20220127141052.1900174-2-abel.vesa@nxp.com>
In-Reply-To: <20220127141052.1900174-2-abel.vesa@nxp.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?IlmX9G5gI2BgjfNaoe+88iQWHy65+WArlLyZ8m0B1rjz2/rgIAqhJmPYnz1O?=
 =?us-ascii?Q?80K7siJgAR0UX5vi2NDZQ66MTsxEw25S439cvm/YMFdVVekrPE5GmnNxfmBV?=
 =?us-ascii?Q?kDyay+8W1n+gdGdYyFejInDiv0OXpBcRkazK5xHnDWaIVSSGZFQ1ICZik22O?=
 =?us-ascii?Q?GYfa1Jk9qmCaGfPheKz1OItKnW/QLnoiO0MVW40MXU7N6vI5qMJ3uy2/3Ij8?=
 =?us-ascii?Q?/QnWtQecaUQ+PF8uL7rynLl+M9kV/p7zCU6S1As4c0UQrkZH+MDh2emVNwtn?=
 =?us-ascii?Q?DLeDQQ2iyhuWI0+W94y+aoDR+kPyueOhRIEZeIEslmGQhqg9hDLFDfQ0kc8n?=
 =?us-ascii?Q?gqO7rpzty/sGAkPMFuA3iTMpPUrlza0eePvaRmxh9qQ/+nMIzpprKycOvJv0?=
 =?us-ascii?Q?RJTnqor71AmKEP/5dntTtcRgDo2xtPbtFD25hZxtG/XQh54kHWl5Swzpo14k?=
 =?us-ascii?Q?59/LkkQtobmdm02VjiD9n+BkO+H150plvk5Z0kPxbGhLGgXE5pw14hXk0tNF?=
 =?us-ascii?Q?GZUApBrhNzj4mgRv1sd6nyEAk1HFLMfuzRPSP6919XD0oRj9er5UNBZ6laZ1?=
 =?us-ascii?Q?xokrhQbWRanAVrDbVIc9udfJ1HufZz5q+qqaqCHfYTlqT3WpBhkca/2XH1l6?=
 =?us-ascii?Q?WIrWl++depCc0XVO+LFoA0rnuRwNwb9JU7mEHBY5soWSFOS+LEdO4RQL0zcZ?=
 =?us-ascii?Q?T0JGbwWiODtLSfQ4gFkeVxWfe2mYDoUZcSNy8xEkns+NtWuFajddPfDx4MTA?=
 =?us-ascii?Q?Yex2ZX2we3eEholTNq5/m45Miw41sQMYo+Txfl/LgUWOi6YvVguFvPeaznPB?=
 =?us-ascii?Q?X/JdQqE13BxkEomFj8o9ZrGnLRwOr7b9TJmx0pVqP2nLeub146Ea/f1ahORY?=
 =?us-ascii?Q?EHTqlOQj2l1BdWYOZ2P16EEixcxtRgf89lemloBm1Ky7XyW7Uq9pxRKPWPzY?=
 =?us-ascii?Q?+lIrvT1uxuXmGVeLHNY1I7p0Pk9OFbTnAciJA8WjJMpbpi6/J0xqxqaUMJES?=
 =?us-ascii?Q?ehkEIlPNE078EAvy1QKLpQ5LA/1ordM1xeDY5+IE3gmL4gEylEfwTCgnErxx?=
 =?us-ascii?Q?21aScSE8PeHHITfAu15PSKz33PpCh0l7PBWRrVVK/CXGUICIGARzDUsf1mwL?=
 =?us-ascii?Q?gd90MkHkuBQWOBhrGqsDSBof04UEs5n82p/aSMpyDakQb/ZnvhfjN8wYhdzG?=
 =?us-ascii?Q?AluNgFjhGdO/0ijvsdrjdZzu/n/2sfX3ZJA22ndVcVBap+aosXPgoCzfTe5y?=
 =?us-ascii?Q?LwGeCW6fI+tCPitbzeQMST9zUokaF1HRj0Wdv/5PM2r4d6pwna0CUAcF9C05?=
 =?us-ascii?Q?OQQDqIAy0SqxntITq4q8YSq2nEtyBTImu6dl2o39rtUcLFyBBnm4XZ/5bxKt?=
 =?us-ascii?Q?NYMh2Kl0eC7H4AyPRr7GlgBbR1mPjDkoOp1VH6yu+Ze+D4m1UBD5EKR5m1Lo?=
 =?us-ascii?Q?ON2lM5elJ7KvSvfjzSeeefUYJRte8tyHTu1OFK+4WJZcIsWJts2WR/WE3PVD?=
 =?us-ascii?Q?BwV8yEBF0UIJrMIMXnoeqmPFy6BI5Z0s61cSUeA2OenZwAZ9zjPZGANx0Rwl?=
 =?us-ascii?Q?9ofTePLPSfIv6H3Gvw+eanbW3PdnMIdf7n8W5uXSF1tG/Oy71DpeBr27t6at?=
 =?us-ascii?Q?4w=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DU0PR04MB9417.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f1c042d2-2e57-41c3-74cc-08d9f5a07777
X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Feb 2022 01:12:59.7057
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: n2lCs7Plg4M1dhVpsgDykZSHCSUCy7XnlrZgrJ69toaty4ezI1/QjS/e2/s/gkZ/Voms6km/996Bg05Xl2bIhQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB7PR04MB5097
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> Subject: [PATCH 2/2] clk: imx: Remove audio_mclk_root_clk
>=20
> The audio_mclk_root_clk was added as a gate with the CCGR121 (0x4790),
> but according to the reference manual, there is no such gate. The
> CCGR121 belongs to ECSPI2 and it is not shared.
>=20
> Fixes: 8f6d8094b215b57 ("ARM: imx: add imx7d clk tree support")
> Reported-by: David Wolfe <david.wolfe@nxp.com>
> Signed-off-by: Abel Vesa <abel.vesa@nxp.com>

Reviewed-by: Peng Fan <peng.fan@nxp.com>

> ---
>  drivers/clk/imx/clk-imx7d.c | 1 -
>  1 file changed, 1 deletion(-)
>=20
> diff --git a/drivers/clk/imx/clk-imx7d.c b/drivers/clk/imx/clk-imx7d.c in=
dex
> c4e0f1c07192..3f6fd7ef2a68 100644
> --- a/drivers/clk/imx/clk-imx7d.c
> +++ b/drivers/clk/imx/clk-imx7d.c
> @@ -849,7 +849,6 @@ static void __init imx7d_clocks_init(struct
> device_node *ccm_node)
>  	hws[IMX7D_WDOG4_ROOT_CLK] =3D
> imx_clk_hw_gate4("wdog4_root_clk", "wdog_post_div", base + 0x49f0, 0);
>  	hws[IMX7D_KPP_ROOT_CLK] =3D imx_clk_hw_gate4("kpp_root_clk",
> "ipg_root_clk", base + 0x4aa0, 0);
>  	hws[IMX7D_CSI_MCLK_ROOT_CLK] =3D
> imx_clk_hw_gate4("csi_mclk_root_clk", "csi_mclk_post_div", base + 0x4490,
> 0);
> -	hws[IMX7D_AUDIO_MCLK_ROOT_CLK] =3D
> imx_clk_hw_gate4("audio_mclk_root_clk", "audio_mclk_post_div", base +
> 0x4790, 0);
>  	hws[IMX7D_WRCLK_ROOT_CLK] =3D imx_clk_hw_gate4("wrclk_root_clk",
> "wrclk_post_div", base + 0x47a0, 0);
>  	hws[IMX7D_USB_CTRL_CLK] =3D imx_clk_hw_gate4("usb_ctrl_clk",
> "ahb_root_clk", base + 0x4680, 0);
>  	hws[IMX7D_USB_PHY1_CLK] =3D imx_clk_hw_gate4("usb_phy1_clk",
> "pll_usb1_main_clk", base + 0x46a0, 0);
> --
> 2.31.1