From: <Claudiu.Beznea@microchip.com> To: <Durai.ManickamKR@microchip.com>, <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> Subject: Re: [PATCH v5 2/8] ARM: dts: at91: sam9x60: move flexcom definitions Date: Fri, 3 Feb 2023 12:08:04 +0000 [thread overview] Message-ID: <c714d161-7cda-8a9c-de05-692416e4d7af@microchip.com> (raw) In-Reply-To: <20230203034833.451461-3-durai.manickamkr@microchip.com> On 03.02.2023 05:48, Durai Manickam KR wrote: > From: Manikandan Muralidharan <manikandan.m@microchip.com> > > Move the flexcom definitions from board specific DTS file > to the SoC specific DTSI file for sam9x60ek. > > Signed-off-by: Manikandan Muralidharan <manikandan.m@microchip.com> > Signed-off-by: Hari Prasath Gujulan Elango <Hari.PrasathGE@microchip.com> > [durai.manickamkr@microchip.com: Logical split-up of this patch and added > missing UART5 compatibles] > Signed-off-by: Durai Manickam KR <durai.manickamkr@microchip.com> Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com> Also, keep tags on next versions you will post. > --- > arch/arm/boot/dts/at91-sam9x60ek.dts | 33 +------------------ > arch/arm/boot/dts/sam9x60.dtsi | 49 ++++++++++++++++++++++++++++ > 2 files changed, 50 insertions(+), 32 deletions(-) > > diff --git a/arch/arm/boot/dts/at91-sam9x60ek.dts b/arch/arm/boot/dts/at91-sam9x60ek.dts > index cf5d786531f2..4ff84633dd43 100644 > --- a/arch/arm/boot/dts/at91-sam9x60ek.dts > +++ b/arch/arm/boot/dts/at91-sam9x60ek.dts > @@ -207,15 +207,10 @@ &flx0 { > status = "okay"; > > i2c0: i2c@600 { > - compatible = "microchip,sam9x60-i2c"; > - reg = <0x600 0x200>; > - interrupts = <5 IRQ_TYPE_LEVEL_HIGH 7>; > #address-cells = <1>; > #size-cells = <0>; > - clocks = <&pmc PMC_TYPE_PERIPHERAL 5>; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_flx0_default>; > - atmel,fifo-size = <16>; > i2c-analog-filter; > i2c-digital-filter; > i2c-digital-filter-width-ns = <35>; > @@ -235,14 +230,8 @@ &flx4 { > status = "disabled"; > > spi4: spi@400 { > - compatible = "microchip,sam9x60-spi", "atmel,at91rm9200-spi"; > - reg = <0x400 0x200>; > - interrupts = <13 IRQ_TYPE_LEVEL_HIGH 7>; > - clocks = <&pmc PMC_TYPE_PERIPHERAL 13>; > - clock-names = "spi_clk"; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_flx4_default>; > - atmel,fifo-size = <16>; > #address-cells = <1>; > #size-cells = <0>; > status = "disabled"; > @@ -254,23 +243,8 @@ &flx5 { > status = "okay"; > > uart5: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > - reg = <0x200 0x200>; > - atmel,usart-mode = <AT91_USART_MODE_SERIAL>; > - interrupts = <14 IRQ_TYPE_LEVEL_HIGH 7>; > - dmas = <&dma0 > - (AT91_XDMAC_DT_MEM_IF(0) | AT91_XDMAC_DT_PER_IF(1) | > - AT91_XDMAC_DT_PERID(10))>, > - <&dma0 > - (AT91_XDMAC_DT_MEM_IF(0) | AT91_XDMAC_DT_PER_IF(1) | > - AT91_XDMAC_DT_PERID(11))>; > - dma-names = "tx", "rx"; > - clocks = <&pmc PMC_TYPE_PERIPHERAL 14>; > - clock-names = "usart"; > - pinctrl-0 = <&pinctrl_flx5_default>; > pinctrl-names = "default"; > - atmel,use-dma-rx; > - atmel,use-dma-tx; > + pinctrl-0 = <&pinctrl_flx5_default>; > status = "okay"; > }; > }; > @@ -280,15 +254,10 @@ &flx6 { > status = "okay"; > > i2c6: i2c@600 { > - compatible = "microchip,sam9x60-i2c"; > - reg = <0x600 0x200>; > - interrupts = <9 IRQ_TYPE_LEVEL_HIGH 7>; > #address-cells = <1>; > #size-cells = <0>; > - clocks = <&pmc PMC_TYPE_PERIPHERAL 9>; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_flx6_default>; > - atmel,fifo-size = <16>; > i2c-analog-filter; > i2c-digital-filter; > i2c-digital-filter-width-ns = <35>; > diff --git a/arch/arm/boot/dts/sam9x60.dtsi b/arch/arm/boot/dts/sam9x60.dtsi > index 8f5477e307dd..ee6cc4329ae4 100644 > --- a/arch/arm/boot/dts/sam9x60.dtsi > +++ b/arch/arm/boot/dts/sam9x60.dtsi > @@ -170,6 +170,16 @@ flx4: flexcom@f0000000 { > #size-cells = <1>; > ranges = <0x0 0xf0000000 0x800>; > status = "disabled"; > + > + spi4: spi@400 { > + compatible = "microchip,sam9x60-spi", "atmel,at91rm9200-spi"; > + reg = <0x400 0x200>; > + interrupts = <13 IRQ_TYPE_LEVEL_HIGH 7>; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 13>; > + clock-names = "spi_clk"; > + atmel,fifo-size = <16>; > + status = "disabled"; > + }; > }; > > flx5: flexcom@f0004000 { > @@ -180,6 +190,27 @@ flx5: flexcom@f0004000 { > #size-cells = <1>; > ranges = <0x0 0xf0004000 0x800>; > status = "disabled"; > + > + uart5: serial@200 { > + compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + reg = <0x200 0x200>; > + atmel,usart-mode = <AT91_USART_MODE_SERIAL>; > + interrupts = <14 IRQ_TYPE_LEVEL_HIGH 7>; > + dmas = <&dma0 > + (AT91_XDMAC_DT_MEM_IF(0) | > + AT91_XDMAC_DT_PER_IF(1) | > + AT91_XDMAC_DT_PERID(10))>, > + <&dma0 > + (AT91_XDMAC_DT_MEM_IF(0) | > + AT91_XDMAC_DT_PER_IF(1) | > + AT91_XDMAC_DT_PERID(11))>; > + dma-names = "tx", "rx"; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 14>; > + clock-names = "usart"; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "disabled"; > + }; > }; > > dma0: dma-controller@f0008000 { > @@ -379,6 +410,15 @@ flx6: flexcom@f8010000 { > #size-cells = <1>; > ranges = <0x0 0xf8010000 0x800>; > status = "disabled"; > + > + i2c6: i2c@600 { > + compatible = "microchip,sam9x60-i2c"; > + reg = <0x600 0x200>; > + interrupts = <9 IRQ_TYPE_LEVEL_HIGH 7>; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 9>; > + atmel,fifo-size = <16>; > + status = "disabled"; > + }; > }; > > flx7: flexcom@f8014000 { > @@ -409,6 +449,15 @@ flx0: flexcom@f801c000 { > #size-cells = <1>; > ranges = <0x0 0xf801c000 0x800>; > status = "disabled"; > + > + i2c0: i2c@600 { > + compatible = "microchip,sam9x60-i2c"; > + reg = <0x600 0x200>; > + interrupts = <5 IRQ_TYPE_LEVEL_HIGH 7>; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 5>; > + atmel,fifo-size = <16>; > + status = "disabled"; > + }; > }; > > flx1: flexcom@f8020000 {
WARNING: multiple messages have this Message-ID (diff)
From: <Claudiu.Beznea@microchip.com> To: <Durai.ManickamKR@microchip.com>, <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> Subject: Re: [PATCH v5 2/8] ARM: dts: at91: sam9x60: move flexcom definitions Date: Fri, 3 Feb 2023 12:08:04 +0000 [thread overview] Message-ID: <c714d161-7cda-8a9c-de05-692416e4d7af@microchip.com> (raw) In-Reply-To: <20230203034833.451461-3-durai.manickamkr@microchip.com> On 03.02.2023 05:48, Durai Manickam KR wrote: > From: Manikandan Muralidharan <manikandan.m@microchip.com> > > Move the flexcom definitions from board specific DTS file > to the SoC specific DTSI file for sam9x60ek. > > Signed-off-by: Manikandan Muralidharan <manikandan.m@microchip.com> > Signed-off-by: Hari Prasath Gujulan Elango <Hari.PrasathGE@microchip.com> > [durai.manickamkr@microchip.com: Logical split-up of this patch and added > missing UART5 compatibles] > Signed-off-by: Durai Manickam KR <durai.manickamkr@microchip.com> Reviewed-by: Claudiu Beznea <claudiu.beznea@microchip.com> Also, keep tags on next versions you will post. > --- > arch/arm/boot/dts/at91-sam9x60ek.dts | 33 +------------------ > arch/arm/boot/dts/sam9x60.dtsi | 49 ++++++++++++++++++++++++++++ > 2 files changed, 50 insertions(+), 32 deletions(-) > > diff --git a/arch/arm/boot/dts/at91-sam9x60ek.dts b/arch/arm/boot/dts/at91-sam9x60ek.dts > index cf5d786531f2..4ff84633dd43 100644 > --- a/arch/arm/boot/dts/at91-sam9x60ek.dts > +++ b/arch/arm/boot/dts/at91-sam9x60ek.dts > @@ -207,15 +207,10 @@ &flx0 { > status = "okay"; > > i2c0: i2c@600 { > - compatible = "microchip,sam9x60-i2c"; > - reg = <0x600 0x200>; > - interrupts = <5 IRQ_TYPE_LEVEL_HIGH 7>; > #address-cells = <1>; > #size-cells = <0>; > - clocks = <&pmc PMC_TYPE_PERIPHERAL 5>; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_flx0_default>; > - atmel,fifo-size = <16>; > i2c-analog-filter; > i2c-digital-filter; > i2c-digital-filter-width-ns = <35>; > @@ -235,14 +230,8 @@ &flx4 { > status = "disabled"; > > spi4: spi@400 { > - compatible = "microchip,sam9x60-spi", "atmel,at91rm9200-spi"; > - reg = <0x400 0x200>; > - interrupts = <13 IRQ_TYPE_LEVEL_HIGH 7>; > - clocks = <&pmc PMC_TYPE_PERIPHERAL 13>; > - clock-names = "spi_clk"; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_flx4_default>; > - atmel,fifo-size = <16>; > #address-cells = <1>; > #size-cells = <0>; > status = "disabled"; > @@ -254,23 +243,8 @@ &flx5 { > status = "okay"; > > uart5: serial@200 { > - compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > - reg = <0x200 0x200>; > - atmel,usart-mode = <AT91_USART_MODE_SERIAL>; > - interrupts = <14 IRQ_TYPE_LEVEL_HIGH 7>; > - dmas = <&dma0 > - (AT91_XDMAC_DT_MEM_IF(0) | AT91_XDMAC_DT_PER_IF(1) | > - AT91_XDMAC_DT_PERID(10))>, > - <&dma0 > - (AT91_XDMAC_DT_MEM_IF(0) | AT91_XDMAC_DT_PER_IF(1) | > - AT91_XDMAC_DT_PERID(11))>; > - dma-names = "tx", "rx"; > - clocks = <&pmc PMC_TYPE_PERIPHERAL 14>; > - clock-names = "usart"; > - pinctrl-0 = <&pinctrl_flx5_default>; > pinctrl-names = "default"; > - atmel,use-dma-rx; > - atmel,use-dma-tx; > + pinctrl-0 = <&pinctrl_flx5_default>; > status = "okay"; > }; > }; > @@ -280,15 +254,10 @@ &flx6 { > status = "okay"; > > i2c6: i2c@600 { > - compatible = "microchip,sam9x60-i2c"; > - reg = <0x600 0x200>; > - interrupts = <9 IRQ_TYPE_LEVEL_HIGH 7>; > #address-cells = <1>; > #size-cells = <0>; > - clocks = <&pmc PMC_TYPE_PERIPHERAL 9>; > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_flx6_default>; > - atmel,fifo-size = <16>; > i2c-analog-filter; > i2c-digital-filter; > i2c-digital-filter-width-ns = <35>; > diff --git a/arch/arm/boot/dts/sam9x60.dtsi b/arch/arm/boot/dts/sam9x60.dtsi > index 8f5477e307dd..ee6cc4329ae4 100644 > --- a/arch/arm/boot/dts/sam9x60.dtsi > +++ b/arch/arm/boot/dts/sam9x60.dtsi > @@ -170,6 +170,16 @@ flx4: flexcom@f0000000 { > #size-cells = <1>; > ranges = <0x0 0xf0000000 0x800>; > status = "disabled"; > + > + spi4: spi@400 { > + compatible = "microchip,sam9x60-spi", "atmel,at91rm9200-spi"; > + reg = <0x400 0x200>; > + interrupts = <13 IRQ_TYPE_LEVEL_HIGH 7>; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 13>; > + clock-names = "spi_clk"; > + atmel,fifo-size = <16>; > + status = "disabled"; > + }; > }; > > flx5: flexcom@f0004000 { > @@ -180,6 +190,27 @@ flx5: flexcom@f0004000 { > #size-cells = <1>; > ranges = <0x0 0xf0004000 0x800>; > status = "disabled"; > + > + uart5: serial@200 { > + compatible = "microchip,sam9x60-dbgu", "microchip,sam9x60-usart", "atmel,at91sam9260-dbgu", "atmel,at91sam9260-usart"; > + reg = <0x200 0x200>; > + atmel,usart-mode = <AT91_USART_MODE_SERIAL>; > + interrupts = <14 IRQ_TYPE_LEVEL_HIGH 7>; > + dmas = <&dma0 > + (AT91_XDMAC_DT_MEM_IF(0) | > + AT91_XDMAC_DT_PER_IF(1) | > + AT91_XDMAC_DT_PERID(10))>, > + <&dma0 > + (AT91_XDMAC_DT_MEM_IF(0) | > + AT91_XDMAC_DT_PER_IF(1) | > + AT91_XDMAC_DT_PERID(11))>; > + dma-names = "tx", "rx"; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 14>; > + clock-names = "usart"; > + atmel,use-dma-rx; > + atmel,use-dma-tx; > + status = "disabled"; > + }; > }; > > dma0: dma-controller@f0008000 { > @@ -379,6 +410,15 @@ flx6: flexcom@f8010000 { > #size-cells = <1>; > ranges = <0x0 0xf8010000 0x800>; > status = "disabled"; > + > + i2c6: i2c@600 { > + compatible = "microchip,sam9x60-i2c"; > + reg = <0x600 0x200>; > + interrupts = <9 IRQ_TYPE_LEVEL_HIGH 7>; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 9>; > + atmel,fifo-size = <16>; > + status = "disabled"; > + }; > }; > > flx7: flexcom@f8014000 { > @@ -409,6 +449,15 @@ flx0: flexcom@f801c000 { > #size-cells = <1>; > ranges = <0x0 0xf801c000 0x800>; > status = "disabled"; > + > + i2c0: i2c@600 { > + compatible = "microchip,sam9x60-i2c"; > + reg = <0x600 0x200>; > + interrupts = <5 IRQ_TYPE_LEVEL_HIGH 7>; > + clocks = <&pmc PMC_TYPE_PERIPHERAL 5>; > + atmel,fifo-size = <16>; > + status = "disabled"; > + }; > }; > > flx1: flexcom@f8020000 { _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-02-03 12:08 UTC|newest] Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-02-03 3:48 [PATCH v5 0/8] Add support for sam9x60 curiosity board Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 3:48 ` [PATCH v5 1/8] ARM: dts: at91: sam9x60: Fix the label numbering for the flexcom functions Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 12:07 ` Claudiu.Beznea 2023-02-03 12:07 ` Claudiu.Beznea 2023-02-03 3:48 ` [PATCH v5 2/8] ARM: dts: at91: sam9x60: move flexcom definitions Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 12:08 ` Claudiu.Beznea [this message] 2023-02-03 12:08 ` Claudiu.Beznea 2023-02-03 3:48 ` [PATCH v5 3/8] ARM: dts: at91: sam9x60: fix spi4 node Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 12:08 ` Claudiu.Beznea 2023-02-03 12:08 ` Claudiu.Beznea 2023-02-03 3:48 ` [PATCH v5 4/8] ARM: dts: at91: sam9x60: Specify the FIFO size for the Flexcom UART Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 12:08 ` Claudiu.Beznea 2023-02-03 12:08 ` Claudiu.Beznea 2023-02-03 3:48 ` [PATCH v5 5/8] ARM: dts: at91: sam9x60: Add DMA bindings for the flexcom nodes Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 12:08 ` Claudiu.Beznea 2023-02-03 12:08 ` Claudiu.Beznea 2023-02-03 3:48 ` [PATCH v5 6/8] ARM: dts: at91: sam9x60: Add missing flexcom definitions Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 12:08 ` Claudiu.Beznea 2023-02-03 12:08 ` Claudiu.Beznea 2023-07-04 15:42 ` Alexander Dahl 2023-07-04 15:42 ` Alexander Dahl 2023-02-03 3:48 ` [PATCH v5 7/8] dt-bindings: arm: at91: Add info on sam9x60 curiosity Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 8:12 ` Krzysztof Kozlowski 2023-02-03 8:12 ` Krzysztof Kozlowski 2023-02-03 12:10 ` Claudiu.Beznea 2023-02-03 12:10 ` Claudiu.Beznea 2023-02-03 3:48 ` [PATCH v5 8/8] ARM: dts: at91: sam9x60_curiosity: Add device tree for sam9x60 curiosity board Durai Manickam KR 2023-02-03 3:48 ` Durai Manickam KR 2023-02-03 12:06 ` Claudiu.Beznea 2023-02-03 12:06 ` Claudiu.Beznea 2023-02-03 12:11 ` Claudiu.Beznea 2023-02-03 12:11 ` Claudiu.Beznea
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=c714d161-7cda-8a9c-de05-692416e4d7af@microchip.com \ --to=claudiu.beznea@microchip.com \ --cc=Balakrishnan.S@microchip.com \ --cc=Balamanikandan.Gunasundar@microchip.com \ --cc=Cristian.Birsan@microchip.com \ --cc=Dharma.B@microchip.com \ --cc=Durai.ManickamKR@microchip.com \ --cc=Hari.PrasathGE@microchip.com \ --cc=Horatiu.Vultur@microchip.com \ --cc=Jerry.Ray@microchip.com \ --cc=Kavyasree.Kotagiri@microchip.com \ --cc=Manikandan.M@microchip.com \ --cc=Nayabbasha.Sayed@microchip.com \ --cc=Nicolas.Ferre@microchip.com \ --cc=Varshini.Rajendran@microchip.com \ --cc=alexandre.belloni@bootlin.com \ --cc=andrew@lunn.ch \ --cc=arnd@arndb.de \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=krzysztof.kozlowski@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=michael@walle.cc \ --cc=olof@lixom.net \ --cc=robh+dt@kernel.org \ --cc=soc@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.