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 604C4C433EF for ; Sun, 20 Feb 2022 11:37:33 +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=P5/orrGY8gE4TuOr4kvG8zE2VtB0HcaA1cC0AaAv5MQ=; b=4vAY1p2AXO8NEb eTaGsWX39fb9QQURPDW+5vGYg+er+1Bfjc49pKvWagNpuz3F/cfANPCjRr31/aDsLs0xD3b/ZWtkC bBmSTWUk8R1hnBAFlPx+akKJ8Xgtv8LIXrc28Qn9f6rhau8n5LmJ27dASJDyY2iGrOqeAkEor2cx6 SfYuwIHnZdqXztIYRDtMczhxsYc3YDMVg2Dc9ruW2i0S/QC4XFEk2fXnKIFqKicDF3Wbe/0ndBIc3 baAtErgUQW8vffoTaA1rVz+AC5RkNdDYx3Ph7B0O+9n1Ow4jQN2WoaZNbqVfVVhOyNuYtNfZlWu6c DylMqEwSSj1YUdcHWWdA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nLkVN-00199Q-RE; Sun, 20 Feb 2022 11:35:53 +0000 Received: from perceval.ideasonboard.com ([213.167.242.64]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nLkVK-00198K-5T for linux-arm-kernel@lists.infradead.org; Sun, 20 Feb 2022 11:35:52 +0000 Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id B8BA925B; Sun, 20 Feb 2022 12:35:45 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1645356946; bh=h3TjgobjOErw+y9+H+t1kNGzyjvsPkb5YVbgo6gsbGc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nbCzhblTnjBafjEob+BihYn6lcKAyV309qHrF9VHUekgFEdE+g/zk0Z/bCtrbK2qp mS2v5+FmNzeBXbS2i64JcAvKUxr943i7X2NFHV7Xa5CEVA6Ey0kEhW1AJffa2zyByJ cTGWWHxvlvl1sfjjSPlRGtHi/rtxHR0o2sn/7r3E= Date: Sun, 20 Feb 2022 13:35:36 +0200 From: Laurent Pinchart To: Lucas Stach Cc: Shawn Guo , Rob Herring , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, patchwork-lst@pengutronix.de Subject: Re: [PATCH v2 3/9] soc: imx: gpcv2: add support for i.MX8MP power domains Message-ID: References: <20220207192547.1997549-1-l.stach@pengutronix.de> <20220207192547.1997549-3-l.stach@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220220_033550_387001_AC790103 X-CRM114-Status: GOOD ( 21.50 ) 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 Hi Lucas, On Sat, Feb 19, 2022 at 01:02:21AM +0200, Laurent Pinchart wrote: > On Mon, Feb 07, 2022 at 08:25:41PM +0100, Lucas Stach wrote: > > This adds driver support for all the GPC power domains found on > > the i.MX8MP SoC. > > > > Signed-off-by: Lucas Stach > > --- > > drivers/soc/imx/gpcv2.c | 387 +++++++++++++++++++++++++++++++++++++++- > > 1 file changed, 386 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/soc/imx/gpcv2.c b/drivers/soc/imx/gpcv2.c > > index 01f46b078df3..a7c92bdfc53b 100644 > > --- a/drivers/soc/imx/gpcv2.c > > +++ b/drivers/soc/imx/gpcv2.c [snip] > > @@ -137,6 +183,21 @@ > > #define IMX8MN_DISPMIX_HSK_PWRDNREQN BIT(7) > > #define IMX8MN_HSIO_HSK_PWRDNREQN BIT(5) > > > > +#define IMX8MP_MEDIAMIX_PWRDNACKN BIT(3) > > This should be bit 30. With this fixed, Reviewed-by: Laurent Pinchart Tested-by: Laurent Pinchart with a soon to be posted driver for the MEDIA_BLK_CTRL. While this shouldn't be a blocker, I'm wondering how we should deal with the NOC configuration that TF-A handles in the power domain code ([1]). The reference manual doesn't document the registers, which doesn't help. There are also two registers in the MEDIA_BLK_CTRL that are specific to the LCDIF and ISI, see [2]. Would you recommend dealing with them in the imx8m-blk-ctrl driver (maybe in the power domain notifier, the same way we set bit 8 in the CLK_EN register), or through a syscon phandle directly in the LCDIF and ISI drivers ? [1] https://source.codeaurora.org/external/qoriq/qoriq-components/atf/tree/plat/imx/imx8m/imx8mp/gpc.c?h=lf-5.10.72-2.2.0#n156 [2] https://source.codeaurora.org/external/qoriq/qoriq-components/atf/tree/plat/imx/imx8m/imx8mp/gpc.c?h=lf-5.10.72-2.2.0#n146 > > +#define IMX8MP_HDMIMIX_PWRDNACKN BIT(29) > > +#define IMX8MP_HSIOMIX_PWRDNACKN BIT(28) > > +#define IMX8MP_VPUMIX_PWRDNACKN BIT(26) > > +#define IMX8MP_GPUMIX_PWRDNACKN BIT(25) > > +#define IMX8MP_MLMIX_PWRDNACKN (BIT(23) | BIT(24)) > > +#define IMX8MP_AUDIOMIX_PWRDNACKN (BIT(20) | BIT(31)) > > +#define IMX8MP_MEDIAMIX_PWRDNREQN BIT(14) > > +#define IMX8MP_HDMIMIX_PWRDNREQN BIT(13) > > +#define IMX8MP_HSIOMIX_PWRDNREQN BIT(12) > > +#define IMX8MP_VPUMIX_PWRDNREQN BIT(10) > > +#define IMX8MP_GPUMIX_PWRDNREQN BIT(9) > > +#define IMX8MP_MLMIX_PWRDNREQN (BIT(7) | BIT(8)) > > +#define IMX8MP_AUDIOMIX_PWRDNREQN (BIT(4) | BIT(15)) > > + > > /* > > * The PGC offset values in Reference Manual > > * (Rev. 1, 01/2018 and the older ones) GPC chapter's [snip] -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel