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 5CC7EC433F5 for ; Tue, 23 Nov 2021 12:17:07 +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=tiCdlJ2rI2Z/QUGN2Hnqs6Q7u0rLXy7A3syGdiL1FzM=; b=nCDtOnkEj4T306 VjWymqC81zY+goHyM/z1fsQ1Im1LoTkaZ80G3ACVNL+mSITlYDVyTtiINzArMjvIOI6W73P2rPo7x uBERFrfnbTjhc+smbJEq8OjPKmDFmoFVXP8jbOjf4NteOVR/IdKdYZaqujuqDIEG/4SZu9CuinGqN mEFC9xi+XCgT6ZjHIm4LFnR7Gs7nhFlxzhQgSKzr1Aj8gfJS3ffzX9ua9CxpcDyLh0jDm3/faUuA1 VVIFqV27rEZh7PH+SSylpQze61GwPhyumxVTD0MGwLFH7ELYJ2VsxwkvLFp4g1D2TLuMQGcqxdMfH MJ9SGnga75c3nG/XYhvQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mpUiL-0023CD-9u; Tue, 23 Nov 2021 12:15:57 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mpUiH-0023BK-JS for linux-arm-kernel@lists.infradead.org; Tue, 23 Nov 2021 12:15:55 +0000 Received: by mail.kernel.org (Postfix) with ESMTPSA id E833060D42; Tue, 23 Nov 2021 12:15:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1637669753; bh=Wth+sVh8/S9qyJ1WmYUOoJConkOt2+OKnDqTUqF3+TM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=taZdQFiKY6D5YoSeRRKtmnUXpRH7zerEchH0JrDI6O8n5Vpdrj2nW3TyDAASMb6R4 cmgtCrR6xT+iBTTBfeGrouf6g60btywqMiweYd/6P2lc+Melxpun5D1GxshE08T86q Ry89FoA6i/xtStHanW65HuDC46UQPSMLRqX98Oujr2J0WiGdJ8eoEMsS42wgEx9H0X wTfTkoCymNwVvrt0JTEdWzF4rPK24Eiw7GV4FnlsKitJOAFIHqGh6cB+0ZJB3UBktj ikH9HlFInEbuqrIzCanDqAUBzW2PU16wWIc+j9Pi8AVUfgnSEXUz1punx97h6JKiRe MviERss5MlAmw== Date: Tue, 23 Nov 2021 20:15:47 +0800 From: Shawn Guo To: Alexander Stein Cc: Rob Herring , Sascha Hauer , Peng Fan , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: (EXT) Re: [PATCH v2 1/1] arm64: dts: imx8qm: Add fsl, imx7ulp-lpuart compatible to lpuart Message-ID: <20211123121547.GC4216@dragon> References: <20211116125814.2674343-1-alexander.stein@ew.tq-group.com> <20211122014314.GV31998@dragon> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211123_041553_714147_5A96F8C9 X-CRM114-Status: GOOD ( 27.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 On Mon, Nov 22, 2021 at 08:38:28AM +0100, Alexander Stein wrote: > Am Montag, dem 22.11.2021 um 09:43 +0800 schrieb Shawn Guo: > > On Tue, Nov 16, 2021 at 01:58:14PM +0100, Alexander Stein wrote: > > > After commit b4b844930f27 ("tty: serial: fsl_lpuart: drop earlycon > > > entry > > > for i.MX8QXP") earlycon support was essentially removed from > > > imx8qm/imx8qxp due to missing compatible. > > > The commit message says "i.MX8QXP lpuart is compatible with > > > i.MX7ULP" so > > > adding a fallback compatible should be fine. > > > With this change earlycon is supported again on imx8qm/imx8qxp. > > > > > > Signed-off-by: Alexander Stein < > > > alexander.stein@ew.tq-group.com > > > > > > > Reviewed-by: Peng Fan < > > > peng.fan@nxp.com > > > > > > > --- > > > Changes in v2: > > > * Fix typo in commit message: 'early' -> 'earlycon' > > > > > > arch/arm64/boot/dts/freescale/imx8qm-ss-dma.dtsi | 8 ++++---- > > > 1 file changed, 4 insertions(+), 4 deletions(-) > > > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8qm-ss-dma.dtsi > > > b/arch/arm64/boot/dts/freescale/imx8qm-ss-dma.dtsi > > > index bbe5f5ecfb92..3486b99ab6eb 100644 > > > --- a/arch/arm64/boot/dts/freescale/imx8qm-ss-dma.dtsi > > > +++ b/arch/arm64/boot/dts/freescale/imx8qm-ss-dma.dtsi > > > @@ -19,19 +19,19 @@ uart4_lpcg: clock-controller@5a4a0000 { > > > }; > > > > > > &lpuart0 { > > > - compatible = "fsl,imx8qm-lpuart", "fsl,imx8qxp-lpuart"; > > > + compatible = "fsl,imx8qm-lpuart", "fsl,imx8qxp-lpuart", > > > "fsl,imx7ulp-lpuart"; > > > > Then fsl,imx8qxp-lpuart should be dropped? > > If you drop fsl,imx8qxp-lpuart in the compatible list, fsl_lpuart.c > needs a new lpuart_soc_data for imx8qm in order to work. "fsl,imx7ulp- > lpuart" is not enough for imx8qm, as (among other things) "baud" clk is > mising in imx7. IMHO there are 2 ways: > > 1st > Add both fsl,imx8qxp-lpuart and fsl,imx7ulp-lpuart to the compatible > list to get both regular serial support (fsl,imx8qxp-lpuart) as well as > earlycon (fsl,imx7ulp-lpuart). In this case the bindings need to get > fixed as well, see [1] > > 2nd > Remove fsl,imx8qxp-lpuart compatible for imx8qm. But in this case > everything done for imx8qxp in fsl_lpuart.c has to be duplicated. > fsl,imx7ulp-lpuart is still required for earlycon support. > > If you want to get rid of fsl,imx7ulp-lpuart compatible for > imx8qm/imx8qxp in either case, you need to revert b4b844930f27 ("tty: > serial: fsl_lpuart: drop earlycon entry for i.MX8QXP"). It looks to me that commit b4b844930f27 caused a regression for imx8qm, so should be reverted. Shawn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel