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 B3BF8C25B08 for ; Sun, 21 Aug 2022 03:11:08 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=iX4CmDarkKX46x8x/XPcJOFBEtaWu2wkg77l3yv5/MU=; b=DkedMCx34czoPm ELflA76o1cD3Yz1nKgU1WMIc9OCmIXYEIXdg0VAjrhZ8ZIFTQ9+kBYS6NQiwMCYkMOheSGHjybebH wpEFucxaZuyxhgjbv+DxY2CSB4NGZBbeAb836XK/yeoXZeI/zhMBeypKATufu2AGFsgdOzlVaOrK2 uz0/A5ZmKB/nqRh+T4u4GbVspa6kjl1iIW3DcvYSQ3emxSwGbEXC12ganVuaW5NXtf/k3xoUAQA5S 3WWe9SZi4OS4h6xO2WpvmXpggRmEqxZoPih3i3CLFrEG3aj5xH7hXfNMc4ui8mHscT3ZNj/Upxqmu fQ8FB3iZCmxnLXiq0iZw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oPbLD-0021JR-2y; Sun, 21 Aug 2022 03:09:35 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oPbLA-0020EN-Ce for linux-arm-kernel@bombadil.infradead.org; Sun, 21 Aug 2022 03:09:32 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=sG3BaZC2qa/8JDThUZYvGLcJ896QqjZLkyotO/zUWiY=; b=SHO/BR77axAKx/td5vAe7NAp49 PcgGAM9HqflOgZCDm8xiz/Js7Wt6GTjl4udP6KDRV3DMibJPle6N95a+lpBB2b/78baLqRqiM8wbM QZrBMsG1WJTF3YLsTEQfp/4UrvTvGDQ1l7v2XMs0UcacYCcm/V9MmHLCFKy509U/Qbc0HkpWt9ECB aQdr3CltTXLlLt41JXlzzssJTsBG+Il10/WQmkQQQlGcviS4d6wkFqYXtFCIAurXsI9nOq/mgCSso lnV+BXsvw0RbtJYYKss9a79jyrnduTsfh+6jo7XraOqzPSTZIJ5eI1j3anQzICf5TaaU3fUnHyHgI KpJJMd4A==; Received: from dfw.source.kernel.org ([139.178.84.217]) by desiato.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oPb1u-004fXL-Eu for linux-arm-kernel@lists.infradead.org; Sun, 21 Aug 2022 02:49:41 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 18DC760C21; Sun, 21 Aug 2022 02:49:27 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DE789C433C1; Sun, 21 Aug 2022 02:49:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1661050166; bh=NQPXhFg5BdDMCFze1MKNJpNFjVXRqm3HsERWvMoSabM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=XKuC0yjUscAd/eUyEE1EjadDmF3GHpY/cvLE/6fWiT7zxCnfwPvuW7Xsr7VEuPNe8 MQlr6Q5BHAFakIxpZp/JQJURCXgFoTc+xCpcxTOjL4Q0NynwvyNqKneyoPTxZFzEOk KhujIUin8/g9aQVBhILKs1bAsC7wjDDv/cfda6E6HRjnjue/jNOdGmictDM98fbrD0 RUO1UJYDioF5NsXGB7eyQAiqg/V8dqFSVDlngU//RWUTC5vfNVC/9wekCZFg6lya3i F31/VjZwsRwy6+IXo7ifE4OvTw85JNfsivSa93ykKVd9+RP+iZ4pbWcv9x0YBXi4vu sggPxJQOfoTQQ== Date: Sun, 21 Aug 2022 10:49:19 +0800 From: Shawn Guo To: "Peng Fan (OSS)" Cc: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, s.hauer@pengutronix.de, l.stach@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, p.zabel@pengutronix.de, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, aisheng.dong@nxp.com, Peng Fan Subject: Re: [PATCH V6 3/6] soc: imx: add i.MX93 SRC power domain driver Message-ID: <20220821024919.GJ149610@dragon> References: <20220719073541.197788-1-peng.fan@oss.nxp.com> <20220719073541.197788-4-peng.fan@oss.nxp.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220719073541.197788-4-peng.fan@oss.nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220821_034939_000968_4DD76D29 X-CRM114-Status: GOOD ( 30.57 ) 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 On Tue, Jul 19, 2022 at 03:35:38PM +0800, Peng Fan (OSS) wrote: > From: Peng Fan > > Support controlling power domain managed by System Reset > Controller(SRC). Current supported power domain is mediamix power > domain. > > Signed-off-by: Peng Fan > --- > drivers/soc/imx/Kconfig | 8 ++ > drivers/soc/imx/Makefile | 1 + > drivers/soc/imx/imx93-pd.c | 163 ++++++++++++++++++++++++++++++++++++ > drivers/soc/imx/imx93-src.c | 32 +++++++ Shouldn't a reset driver go to drivers/reset/? > 4 files changed, 204 insertions(+) > create mode 100644 drivers/soc/imx/imx93-pd.c > create mode 100644 drivers/soc/imx/imx93-src.c > > diff --git a/drivers/soc/imx/Kconfig b/drivers/soc/imx/Kconfig > index a840494e849a..4b906791d6c7 100644 > --- a/drivers/soc/imx/Kconfig > +++ b/drivers/soc/imx/Kconfig > @@ -20,4 +20,12 @@ config SOC_IMX8M > support, it will provide the SoC info like SoC family, > ID and revision etc. > > +config SOC_IMX9 > + tristate "i.MX9 SoC family support" > + depends on ARCH_MXC || COMPILE_TEST > + default ARCH_MXC && ARM64 > + select SOC_BUS > + help > + If you say yes here, you get support for the NXP i.MX9 family > + > endmenu > diff --git a/drivers/soc/imx/Makefile b/drivers/soc/imx/Makefile > index 63cd29f6d4d2..a0baa2a01adb 100644 > --- a/drivers/soc/imx/Makefile > +++ b/drivers/soc/imx/Makefile > @@ -7,3 +7,4 @@ obj-$(CONFIG_IMX_GPCV2_PM_DOMAINS) += gpcv2.o > obj-$(CONFIG_SOC_IMX8M) += soc-imx8m.o > obj-$(CONFIG_SOC_IMX8M) += imx8m-blk-ctrl.o > obj-$(CONFIG_SOC_IMX8M) += imx8mp-blk-ctrl.o > +obj-$(CONFIG_SOC_IMX9) += imx93-src.o imx93-pd.o > diff --git a/drivers/soc/imx/imx93-pd.c b/drivers/soc/imx/imx93-pd.c > new file mode 100644 > index 000000000000..48437c303b78 > --- /dev/null > +++ b/drivers/soc/imx/imx93-pd.c > @@ -0,0 +1,163 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright 2022 NXP > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > +#include > + > +#define MIX_SLICE_SW_CTRL_OFF 0x20 > +#define SLICE_SW_CTRL_PSW_CTRL_OFF_MASK BIT(4) > +#define SLICE_SW_CTRL_PDN_SOFT_MASK BIT(31) > + > +#define MIX_FUNC_STAT_OFF 0xB4 > + > +#define FUNC_STAT_PSW_STAT_MASK BIT(0) > +#define FUNC_STAT_RST_STAT_MASK BIT(2) > +#define FUNC_STAT_ISO_STAT_MASK BIT(4) > + > +struct imx93_power_domain { > + struct generic_pm_domain genpd; > + struct device *dev; > + void __iomem *addr; > + struct clk_bulk_data *clks; > + int num_clks; > + bool init_off; > +}; > + > +#define to_imx93_pd(_genpd) container_of(_genpd, struct imx93_power_domain, genpd) > + > +static int imx93_pd_on(struct generic_pm_domain *genpd) > +{ > + struct imx93_power_domain *domain = to_imx93_pd(genpd); > + void __iomem *addr = domain->addr; > + u32 val; > + int ret; > + > + ret = clk_bulk_prepare_enable(domain->num_clks, domain->clks); > + if (ret) { > + dev_err(domain->dev, "failed to enable clocks for domain: %s\n", genpd->name); > + return ret; > + } > + > + val = readl(addr + MIX_SLICE_SW_CTRL_OFF); > + val &= ~SLICE_SW_CTRL_PDN_SOFT_MASK; > + writel(val, addr + MIX_SLICE_SW_CTRL_OFF); > + > + ret = readl_poll_timeout(addr + MIX_FUNC_STAT_OFF, val, > + !(val & FUNC_STAT_ISO_STAT_MASK), 1, 10000); > + if (ret) { > + dev_err(domain->dev, "pd_on timeout: name: %s, stat: %x\n", genpd->name, val); > + return ret; > + } > + > + return 0; > +} > + > +static int imx93_pd_off(struct generic_pm_domain *genpd) > +{ > + struct imx93_power_domain *domain = to_imx93_pd(genpd); > + void __iomem *addr = domain->addr; > + int ret; > + u32 val; > + > + /* Power off MIX */ > + val = readl(addr + MIX_SLICE_SW_CTRL_OFF); > + val |= SLICE_SW_CTRL_PDN_SOFT_MASK; > + writel(val, addr + MIX_SLICE_SW_CTRL_OFF); > + > + ret = readl_poll_timeout(addr + MIX_FUNC_STAT_OFF, val, > + val & FUNC_STAT_PSW_STAT_MASK, 1, 1000); > + if (ret) { > + dev_err(domain->dev, "pd_off timeout: name: %s, stat: %x\n", genpd->name, val); > + return ret; > + } > + > + clk_bulk_disable_unprepare(domain->num_clks, domain->clks); > + > + return 0; > +}; > + > +static int imx93_pd_remove(struct platform_device *pdev) > +{ > + struct imx93_power_domain *domain = platform_get_drvdata(pdev); > + struct device *dev = &pdev->dev; > + struct device_node *np = dev->of_node; > + > + if (!domain->init_off) > + clk_bulk_disable_unprepare(domain->num_clks, domain->clks); > + > + of_genpd_del_provider(np); > + pm_genpd_remove(&domain->genpd); > + > + return 0; > +} > + > +static int imx93_pd_probe(struct platform_device *pdev) > +{ > + struct device *dev = &pdev->dev; > + struct device_node *np = dev->of_node; > + struct imx93_power_domain *domain; > + int ret; > + > + domain = devm_kzalloc(dev, sizeof(*domain), GFP_KERNEL); > + if (!domain) > + return -ENOMEM; Have a newline. > + domain->addr = devm_platform_ioremap_resource(pdev, 0); > + if (IS_ERR(domain->addr)) > + return PTR_ERR(domain->addr); > + > + One newline is enough. > + domain->num_clks = devm_clk_bulk_get_all(dev, &domain->clks); > + if (domain->num_clks < 0) > + return dev_err_probe(dev, domain->num_clks, "Failed to get domain's clocks\n"); > + > + domain->genpd.name = dev_name(dev); > + domain->genpd.power_off = imx93_pd_off; > + domain->genpd.power_on = imx93_pd_on; > + domain->dev = dev; > + > + domain->init_off = readl(domain->addr + MIX_FUNC_STAT_OFF) & FUNC_STAT_ISO_STAT_MASK; > + /* Just to sync the status of hardware */ > + if (!domain->init_off) { > + ret = clk_bulk_prepare_enable(domain->num_clks, domain->clks); > + if (ret) { > + dev_err(domain->dev, "failed to enable clocks for domain: %s\n", > + domain->genpd.name); > + return 0; This is a case of success? > + } > + } > + > + ret = pm_genpd_init(&domain->genpd, NULL, domain->init_off); > + if (ret) > + return ret; > + > + platform_set_drvdata(pdev, domain); > + > + return of_genpd_add_provider_simple(np, &domain->genpd); > +} > + > +static const struct of_device_id imx93_dt_ids[] = { > + { .compatible = "fsl,imx93-src-slice" }, > + { } > +}; MODULE_DEVICE_TABLE()? Shawn > + > +static struct platform_driver imx93_power_domain_driver = { > + .driver = { > + .name = "imx93_power_domain", > + .owner = THIS_MODULE, > + .of_match_table = imx93_dt_ids, > + }, > + .probe = imx93_pd_probe, > + .remove = imx93_pd_remove, > +}; > +module_platform_driver(imx93_power_domain_driver); > + > +MODULE_AUTHOR("Peng Fan "); > +MODULE_DESCRIPTION("NXP i.MX93 power domain driver"); > +MODULE_LICENSE("GPL"); "GPL v2" since you have "SPDX-License-Identifier: GPL-2.0" claimed in the beginning? Shawn > diff --git a/drivers/soc/imx/imx93-src.c b/drivers/soc/imx/imx93-src.c > new file mode 100644 > index 000000000000..6f14c241538e > --- /dev/null > +++ b/drivers/soc/imx/imx93-src.c > @@ -0,0 +1,32 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright 2022 NXP > + */ > + > +#include > +#include > +#include > + > +static int imx93_src_probe(struct platform_device *pdev) > +{ > + return devm_of_platform_populate(&pdev->dev); > +} > + > +static const struct of_device_id imx93_dt_ids[] = { > + { .compatible = "fsl,imx93-src" }, > + { } > +}; > + > +static struct platform_driver imx93_src_driver = { > + .driver = { > + .name = "imx93_src", > + .owner = THIS_MODULE, > + .of_match_table = imx93_dt_ids, > + }, > + .probe = imx93_src_probe, > +}; > +module_platform_driver(imx93_src_driver); > + > +MODULE_AUTHOR("Peng Fan "); > +MODULE_DESCRIPTION("NXP i.MX93 src driver"); > +MODULE_LICENSE("GPL"); > -- > 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel