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 4471DC00528 for ; Mon, 31 Jul 2023 13:36:59 +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:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To: References:Message-ID:Subject: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=bGXc+YP8Jq9iM6xjEiiKr3Q29z2EEXm7CsTVypENFBI=; b=xIelTjBFsSzi5gUdfH85PJeb73 3BajMzv3ZeI7NIu48CJ2vRNccVXVjBT0xV2PQOe7QlBuRZyogBy4mOfGOrHxrzEc39RT00+Rj3vOV rMbjihzWsEvTNihw0AkHDMZe5HQhF0kr4k9za13exowtYmmbjt3TAQXZ/9ZKFi0OezrYldJGAZjDu Fz4FgKalh5wRTjG256rN6tYtLjZ1wCII/3sK1b3ugtQVKgTnHriCdRRasw9Z9hGRyFdJpXkNuolTm v1lbafVcOSV9lMOLXYSsF6vhcL4AkJol2tzNqAiTeOl0y+NW7vsuEv/MG9NX2DetYS8QG2WLMzm9N bfHb6u4Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qQT4c-00FpSV-0X; Mon, 31 Jul 2023 13:36:34 +0000 Received: from mail.thorsis.com ([92.198.35.195]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qQT4Y-00FpOh-1l for linux-arm-kernel@lists.infradead.org; Mon, 31 Jul 2023 13:36:32 +0000 Date: Mon, 31 Jul 2023 15:35:54 +0200 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=thorsis.com; s=default; t=1690810576; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:content-type:content-type:in-reply-to:in-reply-to: references:references; bh=D0C9W4jLWSTvnH8GOqVr0jR88Cd4UebzZne1ky7oQew=; b=MgCaJUUhtnwJoabwcOv/O2xDIrYyU6e2SrIdTvpRF6hQCBOUwM43hoGiBDHxfeOvLgytKm E/s1em5jauL5/eDip6Z0a0Q4wmHaM0lHd/YUnaNq57TZAb+jQ76miKtoulqJTKefuzPPMh YsK0EKUMHQKC6NxppBGQIvaQoi+KFwz6yBuWKu1lEg/FGUS2KWohgybOvHx031wiEV+pY1 w1sU6nGM0DHfDX4/FjJ/MiD69uxnqc+4j2tz8JpfD0MPUCv4NevSDVAmibrOwwJUaqP3p7 WL8TbB+VpJoKmCndRZDqIf+PGy3q9LgX0Ql1PsqQuunp4h+33rjJNlAH2JGdyw== From: Alexander Dahl To: Durai Manickam KR Subject: Re: [PATCHv3 2/2] ARM: dts: at91: sam9x60: fix the SOC detection Message-ID: <20230731-confess-prison-0ce898c937ba@ifak-system.com> Mail-Followup-To: Durai Manickam KR , Hari.PrasathGE@microchip.com, balamanikandan.gunasundar@microchip.com, manikandan.m@microchip.com, varshini.rajendran@microchip.com, dharma.b@microchip.com, nayabbasha.sayed@microchip.com, balakrishnan.s@microchip.com, cristian.birsan@microchip.com, nicolas.ferre@microchip.com, krzysztof.kozlowski@linaro.org, alexandre.belloni@bootlin.com, davem@davemloft.net, arnd@arndb.de, olof@lixom.net, soc@kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Kavyasree.Kotagiri@microchip.com, Horatiu.Vultur@microchip.com, robh+dt@kernel.org, andrew@lunn.ch, michael@walle.cc, jerry.ray@microchip.com, conor+dt@kernel.org, jesper.nilsson@axis.com, andre.przywara@arm.com, radu_nicolae.pirea@upb.ro, richard.genoud@gmail.com, gregkh@linuxfoundation.org, linux-spi@vger.kernel.org, linux-serial@vger.kernel.org References: <20230718065735.10187-1-durai.manickamkr@microchip.com> <20230718065735.10187-3-durai.manickamkr@microchip.com> Content-Disposition: inline In-Reply-To: <20230718065735.10187-3-durai.manickamkr@microchip.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230731_063630_816923_DD4756C2 X-CRM114-Status: GOOD ( 16.37 ) 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: , Cc: andrew@lunn.ch, alexandre.belloni@bootlin.com, devicetree@vger.kernel.org, jerry.ray@microchip.com, Hari.PrasathGE@microchip.com, varshini.rajendran@microchip.com, Horatiu.Vultur@microchip.com, manikandan.m@microchip.com, jesper.nilsson@axis.com, balamanikandan.gunasundar@microchip.com, michael@walle.cc, dharma.b@microchip.com, linux-serial@vger.kernel.org, cristian.birsan@microchip.com, nayabbasha.sayed@microchip.com, conor+dt@kernel.org, arnd@arndb.de, andre.przywara@arm.com, radu_nicolae.pirea@upb.ro, soc@kernel.org, robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org, balakrishnan.s@microchip.com, ada@thorsis.com, gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, linux-spi@vger.kernel.org, richard.genoud@gmail.com, krzysztof.kozlowski@linaro.org, Kavyasree.Kotagiri@microchip.com, olof@lixom.net, davem@davemloft.net MIME-Version: 1.0 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 Hello Durai, Am Tue, Jul 18, 2023 at 12:27:35PM +0530 schrieb Durai Manickam KR: > Defining the dbgu compatible strings in the UART submodule of the > flexcom gives the below error log, > AT91: Could not find matching SoC description > This error arises due to defining the dbgu compatible strings to > the flexcom usart which is not valid because there is only one debug unit > in the sam9x60 SOC and it has the chipid register. The dbgu compatible > strings are valid only for debug usart and not valid for flexcom usart. > Thus removing the dbgu compatible strings in the UART submodule of the > flexcom for the proper SOC detection. > > Fixes: 99c808335877 (ARM: dts: at91: sam9x60: Add missing flexcom definitions) > Signed-off-by: Durai Manickam KR Acked-by: Alexander Dahl (The diff of the patch is identical to the diff of a quick hacked patch I made weeks ago for my BSP which fixed the issue. SoC variant was reported correctly on serial console on boot again. However I did not actually test *your* patch on my hardware (sam9x60 curiosity), thus Acked-by and not Tested-by.) Greets Alex > --- > arch/arm/boot/dts/microchip/sam9x60.dtsi | 26 ++++++++++++------------ > 1 file changed, 13 insertions(+), 13 deletions(-) > > diff --git a/arch/arm/boot/dts/microchip/sam9x60.dtsi b/arch/arm/boot/dts/microchip/sam9x60.dtsi > index 8b53997675e7..73d570a17269 100644 > --- a/arch/arm/boot/dts/microchip/sam9x60.dtsi > +++ b/arch/arm/boot/dts/microchip/sam9x60.dtsi > @@ -172,7 +172,7 @@ flx4: flexcom@f0000000 { > status = "disabled"; > > uart4: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <13 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -240,7 +240,7 @@ flx5: flexcom@f0004000 { > status = "disabled"; > > uart5: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > atmel,usart-mode = ; > interrupts = <14 IRQ_TYPE_LEVEL_HIGH 7>; > @@ -370,7 +370,7 @@ flx11: flexcom@f0020000 { > status = "disabled"; > > uart11: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <32 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -419,7 +419,7 @@ flx12: flexcom@f0024000 { > status = "disabled"; > > uart12: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <33 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -576,7 +576,7 @@ flx6: flexcom@f8010000 { > status = "disabled"; > > uart6: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <9 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -625,7 +625,7 @@ flx7: flexcom@f8014000 { > status = "disabled"; > > uart7: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <10 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -674,7 +674,7 @@ flx8: flexcom@f8018000 { > status = "disabled"; > > uart8: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <11 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -723,7 +723,7 @@ flx0: flexcom@f801c000 { > status = "disabled"; > > uart0: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <5 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -791,7 +791,7 @@ flx1: flexcom@f8020000 { > status = "disabled"; > > uart1: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <6 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -859,7 +859,7 @@ flx2: flexcom@f8024000 { > status = "disabled"; > > uart2: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <7 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -927,7 +927,7 @@ flx3: flexcom@f8028000 { > status = "disabled"; > > uart3: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <8 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -1050,7 +1050,7 @@ flx9: flexcom@f8040000 { > status = "disabled"; > > uart9: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <15 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > @@ -1099,7 +1099,7 @@ flx10: flexcom@f8044000 { > status = "disabled"; > > uart10: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + compatible = "microchip,sam9x60-usart", "atmel,at91sam9260-usart"; > reg = <0x200 0x200>; > interrupts = <16 IRQ_TYPE_LEVEL_HIGH 7>; > dmas = <&dma0 > -- > 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel