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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id D368EC433EF for ; Wed, 4 May 2022 08:48:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1343716AbiEDIvv (ORCPT ); Wed, 4 May 2022 04:51:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51122 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346633AbiEDIvu (ORCPT ); Wed, 4 May 2022 04:51:50 -0400 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B4C4924BF4 for ; Wed, 4 May 2022 01:48:14 -0700 (PDT) Received: from pendragon.ideasonboard.com (62-78-145-57.bb.dnainternet.fi [62.78.145.57]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 73B56E5; Wed, 4 May 2022 10:48:12 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1651654092; bh=B5Xx30lVZ8zBdiGMG2Kb3zu2ahKidGIx2CgC1h/qHlA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=wjS6/6oAZR3rHwkhAZPQdfeNs1tWtPywYoO1dKr64d+ciGMdK+TFdlg+CR+6qegXr xa2PXRwqkwqdHMwHoEbpNs69npUctJwX5JV4JMeI+ViCOr4xk8f+sKJX5uwKqD8kST dYqskr6TaKn6r9+kgVD2DFQ/n6CDy1E0ZHR5p2mw= Date: Wed, 4 May 2022 11:48:10 +0300 From: Laurent Pinchart To: Lucas Stach Cc: Shawn Guo , Rob Herring , Krzysztof Kozlowski , Marek Vasut , devicetree@vger.kernel.org, Paul Elder , NXP Linux Team , Pengutronix Kernel Team , Fabio Estevam , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v4 00/11] consolidated i.MX8MP HSIO/MEDIA/HDMI blk-ctrl series Message-ID: References: <20220406153402.1265474-1-l.stach@pengutronix.de> <8c29464ef118aae8a1ac2af28d4906a346b6028a.camel@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <8c29464ef118aae8a1ac2af28d4906a346b6028a.camel@pengutronix.de> Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Lucas, On Wed, May 04, 2022 at 10:27:01AM +0200, Lucas Stach wrote: > Hi Shawn, > > there were some comments about the implementation of the HDMI blk-ctrl, > which I don't know in which way to resolve, yet. In the meantime it > would be very helpful if you could take all but the last patch of this > series into your tree. They are all reviewed and tested and starting to > block further work on some parts of the i.MX8MP bringup. Can we decouple the HDMI blk-ctrl from the rest, to at least upstream the MEDIA blk-ctrl ? That part is ready. > Am Mittwoch, dem 06.04.2022 um 17:33 +0200 schrieb Lucas Stach: > > Hi all, > > > > this series adds a bunch more power domains that integrate with the blk-ctrls > > to the i.MX8MP. It depends on the i.MX8MP GPCv2 support series posted in [1]. > > > > The HSIO blk-ctrl bindings are already applied upstream, so they are not part > > of this series anymore. The DT description hasn't changed, but the > > implementation of the HSIO blk-ctrl driver is reworked quite a bit from the > > last round of patches, so I've dropped the review/tested-by tags. > > > > I've also picked up the pretty thoroughly reviewed and tested MEDIA blk-ctrl > > patches from Paul and Laurent into this series, as they would conflict with > > other patches from this series when applied separately. This should hopefully > > make it easier for Shawn to pick things up. > > > > Finally this now also adds the HDMI blk-ctrl, not part of any previous series. > > This one is pretty complex and the documentation in the reference manual > > appears to be inaccurate. I found at least the following issues: > > > > 1. GLOBAL_XTAL24M_CLK_EN does not actually gate the 24MHz reference clock, > > the PHY PLL happily locks and provides correct clocks without this clock being > > ungated. It seems to gate some clock input of the HDMI TX controller instead, > > register access to this controller fails without this clock. > > > > 2. HDMI_RTX_RESET_CTL0 bits 6 and 7 are marked as "reserved" in the manual, > > but they are actually required to be configured. Bit 6 seems to keep something > > in the display pixel clock path in inactive state and Bit 7 seems to reset > > the HDMI TX i2c controller. > > > > The HDMI blk-ctrl part is tested quite extensively with a PoC HDMI support > > patchset, which I will send out in a bit, so I'm pretty sure that the > > support as implemented is working. I've put the patches last in the series > > in case that this turns out to be controversial, so the other patches can > > be applied independent from the HDMI support. > > > > Regards, > > Lucas > > > > [1] https://lore.kernel.org/all/20220330104620.3600159-1-l.stach@pengutronix.de/ > > > > > > Laurent Pinchart (1): > > arm64: dts: imx8mp: Add MEDIAMIX power domains > > > > Lucas Stach (7): > > soc: imx: imx8m-blk-ctrl: set power device name > > soc: imx: add i.MX8MP HSIO blk-ctrl > > dt-bindings: power: imx8mp: add defines for HDMI blk-ctrl domains > > dt-bindings: soc: add binding for i.MX8MP HDMI blk-ctrl > > soc: imx: add i.MX8MP HDMI blk-ctrl > > arm64: dts: imx8mp: add HSIO power-domains > > arm64: dts: imx8mp: add HDMI power-domains > > > > Paul Elder (3): > > dt-bindings: soc: Add i.MX8MP media block control DT bindings > > soc: imx: imx8m-blk-ctrl: Add i.MX8MP media blk-ctrl > > arm64: dts: imx8mp: Add MEDIA_BLK_CTRL > > > > .../soc/imx/fsl,imx8mp-hdmi-blk-ctrl.yaml | 84 +++ > > .../soc/imx/fsl,imx8mp-media-blk-ctrl.yaml | 104 +++ > > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 153 +++- > > drivers/soc/imx/Makefile | 1 + > > drivers/soc/imx/imx8m-blk-ctrl.c | 124 +++- > > drivers/soc/imx/imx8mp-blk-ctrl.c | 696 ++++++++++++++++++ > > include/dt-bindings/power/imx8mp-power.h | 18 + > > 7 files changed, 1172 insertions(+), 8 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/soc/imx/fsl,imx8mp-hdmi-blk-ctrl.yaml > > create mode 100644 Documentation/devicetree/bindings/soc/imx/fsl,imx8mp-media-blk-ctrl.yaml > > create mode 100644 drivers/soc/imx/imx8mp-blk-ctrl.c -- Regards, Laurent Pinchart 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 8712EC4332F for ; Wed, 4 May 2022 08:49:39 +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=RniQpnYxI7I1qnlopET1GnalUV16SavqowywU78EQTo=; b=X7DTJ0BylmWbnz 1XH83Mm7YgOTqNWmzE9BdnGKAVnSYvdFUUGlPZEf50oh1GZRkCNuZpcfMv7JuFkbgv64lSDOYWxMl mxaQdc1nbv5ykxeMdf9I2fQRDT1//+kgkv9qPKhunBuZACzo9X7FMHKA4eksnxumvdKvgNMrhG0WU eyZ7hAuOsEJwhS9KSDJ6Mg3YM/q4CSoZwIPMgn4YUfiAmd7ScTxp1O5rU1+B0n0RCk/4KugH7joQJ 8F/j7GcrnNYrzsxlxMae/ARF2nQ/1ONlZOsz4Yxg1U9WDSkX+1n4R5kLkM7yjBbbTFpyPcEaGY9MT rvEBrvewbm0Bja7kmsPg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nmAgJ-009eLQ-Cg; Wed, 04 May 2022 08:48:23 +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 1nmAgF-009eJq-Bo for linux-arm-kernel@lists.infradead.org; Wed, 04 May 2022 08:48:21 +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 73B56E5; Wed, 4 May 2022 10:48:12 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1651654092; bh=B5Xx30lVZ8zBdiGMG2Kb3zu2ahKidGIx2CgC1h/qHlA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=wjS6/6oAZR3rHwkhAZPQdfeNs1tWtPywYoO1dKr64d+ciGMdK+TFdlg+CR+6qegXr xa2PXRwqkwqdHMwHoEbpNs69npUctJwX5JV4JMeI+ViCOr4xk8f+sKJX5uwKqD8kST dYqskr6TaKn6r9+kgVD2DFQ/n6CDy1E0ZHR5p2mw= Date: Wed, 4 May 2022 11:48:10 +0300 From: Laurent Pinchart To: Lucas Stach Cc: Shawn Guo , Rob Herring , Krzysztof Kozlowski , Marek Vasut , devicetree@vger.kernel.org, Paul Elder , NXP Linux Team , Pengutronix Kernel Team , Fabio Estevam , linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v4 00/11] consolidated i.MX8MP HSIO/MEDIA/HDMI blk-ctrl series Message-ID: References: <20220406153402.1265474-1-l.stach@pengutronix.de> <8c29464ef118aae8a1ac2af28d4906a346b6028a.camel@pengutronix.de> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <8c29464ef118aae8a1ac2af28d4906a346b6028a.camel@pengutronix.de> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220504_014819_581111_781F599B X-CRM114-Status: GOOD ( 37.93 ) 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 Wed, May 04, 2022 at 10:27:01AM +0200, Lucas Stach wrote: > Hi Shawn, > > there were some comments about the implementation of the HDMI blk-ctrl, > which I don't know in which way to resolve, yet. In the meantime it > would be very helpful if you could take all but the last patch of this > series into your tree. They are all reviewed and tested and starting to > block further work on some parts of the i.MX8MP bringup. Can we decouple the HDMI blk-ctrl from the rest, to at least upstream the MEDIA blk-ctrl ? That part is ready. > Am Mittwoch, dem 06.04.2022 um 17:33 +0200 schrieb Lucas Stach: > > Hi all, > > > > this series adds a bunch more power domains that integrate with the blk-ctrls > > to the i.MX8MP. It depends on the i.MX8MP GPCv2 support series posted in [1]. > > > > The HSIO blk-ctrl bindings are already applied upstream, so they are not part > > of this series anymore. The DT description hasn't changed, but the > > implementation of the HSIO blk-ctrl driver is reworked quite a bit from the > > last round of patches, so I've dropped the review/tested-by tags. > > > > I've also picked up the pretty thoroughly reviewed and tested MEDIA blk-ctrl > > patches from Paul and Laurent into this series, as they would conflict with > > other patches from this series when applied separately. This should hopefully > > make it easier for Shawn to pick things up. > > > > Finally this now also adds the HDMI blk-ctrl, not part of any previous series. > > This one is pretty complex and the documentation in the reference manual > > appears to be inaccurate. I found at least the following issues: > > > > 1. GLOBAL_XTAL24M_CLK_EN does not actually gate the 24MHz reference clock, > > the PHY PLL happily locks and provides correct clocks without this clock being > > ungated. It seems to gate some clock input of the HDMI TX controller instead, > > register access to this controller fails without this clock. > > > > 2. HDMI_RTX_RESET_CTL0 bits 6 and 7 are marked as "reserved" in the manual, > > but they are actually required to be configured. Bit 6 seems to keep something > > in the display pixel clock path in inactive state and Bit 7 seems to reset > > the HDMI TX i2c controller. > > > > The HDMI blk-ctrl part is tested quite extensively with a PoC HDMI support > > patchset, which I will send out in a bit, so I'm pretty sure that the > > support as implemented is working. I've put the patches last in the series > > in case that this turns out to be controversial, so the other patches can > > be applied independent from the HDMI support. > > > > Regards, > > Lucas > > > > [1] https://lore.kernel.org/all/20220330104620.3600159-1-l.stach@pengutronix.de/ > > > > > > Laurent Pinchart (1): > > arm64: dts: imx8mp: Add MEDIAMIX power domains > > > > Lucas Stach (7): > > soc: imx: imx8m-blk-ctrl: set power device name > > soc: imx: add i.MX8MP HSIO blk-ctrl > > dt-bindings: power: imx8mp: add defines for HDMI blk-ctrl domains > > dt-bindings: soc: add binding for i.MX8MP HDMI blk-ctrl > > soc: imx: add i.MX8MP HDMI blk-ctrl > > arm64: dts: imx8mp: add HSIO power-domains > > arm64: dts: imx8mp: add HDMI power-domains > > > > Paul Elder (3): > > dt-bindings: soc: Add i.MX8MP media block control DT bindings > > soc: imx: imx8m-blk-ctrl: Add i.MX8MP media blk-ctrl > > arm64: dts: imx8mp: Add MEDIA_BLK_CTRL > > > > .../soc/imx/fsl,imx8mp-hdmi-blk-ctrl.yaml | 84 +++ > > .../soc/imx/fsl,imx8mp-media-blk-ctrl.yaml | 104 +++ > > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 153 +++- > > drivers/soc/imx/Makefile | 1 + > > drivers/soc/imx/imx8m-blk-ctrl.c | 124 +++- > > drivers/soc/imx/imx8mp-blk-ctrl.c | 696 ++++++++++++++++++ > > include/dt-bindings/power/imx8mp-power.h | 18 + > > 7 files changed, 1172 insertions(+), 8 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/soc/imx/fsl,imx8mp-hdmi-blk-ctrl.yaml > > create mode 100644 Documentation/devicetree/bindings/soc/imx/fsl,imx8mp-media-blk-ctrl.yaml > > create mode 100644 drivers/soc/imx/imx8mp-blk-ctrl.c -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel