* [PATCHv2 0/2] update at91 usart compatible for sam9x60 @ 2023-07-13 9:51 Durai Manickam KR 2023-07-13 9:51 ` [PATCHv2 1/2] dt-bindings: serial: atmel,at91-usart: add " Durai Manickam KR 2023-07-13 9:51 ` [PATCHv2 2/2] ARM: dts: at91: sam9x60: fix the SOC detection Durai Manickam KR 0 siblings, 2 replies; 7+ messages in thread From: Durai Manickam KR @ 2023-07-13 9:51 UTC (permalink / raw) To: Hari.PrasathGE, balamanikandan.gunasundar, manikandan.m, varshini.rajendran, dharma.b, nayabbasha.sayed, balakrishnan.s, cristian.birsan, nicolas.ferre, krzysztof.kozlowski, alexandre.belloni, davem, arnd, olof, soc, devicetree, linux-arm-kernel, linux-kernel, Kavyasree.Kotagiri, Horatiu.Vultur, robh+dt, andrew, michael, jerry.ray, conor+dt, jesper.nilsson, andre.przywara, ada, radu_nicolae.pirea, richard.genoud, gregkh, linux-spi, linux-serial Cc: Durai Manickam KR There is only one debug unit in the sam9x60 SOC and it has the chipid register. So defining these dbgu compatible strings to the flexcom usart is not correct and causes the SOC detection failure. So removing these dbgu compatible strings defined under the flexcom usart and updating the yaml file accordingly. Durai Manickam KR (2): dt-bindings: serial: atmel,at91-usart: add compatible for sam9x60 ARM: dts: at91: sam9x60: fix the SOC detection .../bindings/serial/atmel,at91-usart.yaml | 4 ++- arch/arm/boot/dts/microchip/sam9x60.dtsi | 26 +++++++++---------- 2 files changed, 16 insertions(+), 14 deletions(-) -- 2.25.1 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCHv2 1/2] dt-bindings: serial: atmel,at91-usart: add compatible for sam9x60 2023-07-13 9:51 [PATCHv2 0/2] update at91 usart compatible for sam9x60 Durai Manickam KR @ 2023-07-13 9:51 ` Durai Manickam KR 2023-07-13 10:02 ` Krzysztof Kozlowski 2023-07-13 9:51 ` [PATCHv2 2/2] ARM: dts: at91: sam9x60: fix the SOC detection Durai Manickam KR 1 sibling, 1 reply; 7+ messages in thread From: Durai Manickam KR @ 2023-07-13 9:51 UTC (permalink / raw) To: Hari.PrasathGE, balamanikandan.gunasundar, manikandan.m, varshini.rajendran, dharma.b, nayabbasha.sayed, balakrishnan.s, cristian.birsan, nicolas.ferre, krzysztof.kozlowski, alexandre.belloni, davem, arnd, olof, soc, devicetree, linux-arm-kernel, linux-kernel, Kavyasree.Kotagiri, Horatiu.Vultur, robh+dt, andrew, michael, jerry.ray, conor+dt, jesper.nilsson, andre.przywara, ada, radu_nicolae.pirea, richard.genoud, gregkh, linux-spi, linux-serial Cc: Durai Manickam KR Add sam9x60 compatible to DT bindings documentation. Signed-off-by: Durai Manickam KR <durai.manickamkr@microchip.com> --- .../devicetree/bindings/serial/atmel,at91-usart.yaml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml b/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml index 30b2131b5860..65cb2e5c5eee 100644 --- a/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml +++ b/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml @@ -16,13 +16,15 @@ properties: - enum: - atmel,at91rm9200-usart - atmel,at91sam9260-usart - - microchip,sam9x60-usart - items: - const: atmel,at91rm9200-dbgu - const: atmel,at91rm9200-usart - items: - const: atmel,at91sam9260-dbgu - const: atmel,at91sam9260-usart + - items: + - const: microchip,sam9x60-usart + - const: atmel,at91sam9260-usart - items: - const: microchip,sam9x60-dbgu - const: microchip,sam9x60-usart -- 2.25.1 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCHv2 1/2] dt-bindings: serial: atmel,at91-usart: add compatible for sam9x60 2023-07-13 9:51 ` [PATCHv2 1/2] dt-bindings: serial: atmel,at91-usart: add " Durai Manickam KR @ 2023-07-13 10:02 ` Krzysztof Kozlowski 0 siblings, 0 replies; 7+ messages in thread From: Krzysztof Kozlowski @ 2023-07-13 10:02 UTC (permalink / raw) To: Durai Manickam KR, Hari.PrasathGE, balamanikandan.gunasundar, manikandan.m, varshini.rajendran, dharma.b, nayabbasha.sayed, balakrishnan.s, cristian.birsan, nicolas.ferre, alexandre.belloni, davem, arnd, olof, soc, devicetree, linux-arm-kernel, linux-kernel, Kavyasree.Kotagiri, Horatiu.Vultur, robh+dt, andrew, michael, jerry.ray, conor+dt, jesper.nilsson, andre.przywara, ada, radu_nicolae.pirea, richard.genoud, gregkh, linux-spi, linux-serial On 13/07/2023 11:51, Durai Manickam KR wrote: > Add sam9x60 compatible to DT bindings documentation. Unfortunately, I don't understand this commit msg with patch. You say you added a compatible, but the compatible was there before and your diff moves it to a list. Explain in the commit msg what is the real problem. > > Signed-off-by: Durai Manickam KR <durai.manickamkr@microchip.com> > --- > .../devicetree/bindings/serial/atmel,at91-usart.yaml | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml b/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml > index 30b2131b5860..65cb2e5c5eee 100644 > --- a/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml > +++ b/Documentation/devicetree/bindings/serial/atmel,at91-usart.yaml > @@ -16,13 +16,15 @@ properties: > - enum: > - atmel,at91rm9200-usart > - atmel,at91sam9260-usart > - - microchip,sam9x60-usart It's here, look ^^. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCHv2 2/2] ARM: dts: at91: sam9x60: fix the SOC detection 2023-07-13 9:51 [PATCHv2 0/2] update at91 usart compatible for sam9x60 Durai Manickam KR 2023-07-13 9:51 ` [PATCHv2 1/2] dt-bindings: serial: atmel,at91-usart: add " Durai Manickam KR @ 2023-07-13 9:51 ` Durai Manickam KR 2023-07-13 10:03 ` Krzysztof Kozlowski 1 sibling, 1 reply; 7+ messages in thread From: Durai Manickam KR @ 2023-07-13 9:51 UTC (permalink / raw) To: Hari.PrasathGE, balamanikandan.gunasundar, manikandan.m, varshini.rajendran, dharma.b, nayabbasha.sayed, balakrishnan.s, cristian.birsan, nicolas.ferre, krzysztof.kozlowski, alexandre.belloni, davem, arnd, olof, soc, devicetree, linux-arm-kernel, linux-kernel, Kavyasree.Kotagiri, Horatiu.Vultur, robh+dt, andrew, michael, jerry.ray, conor+dt, jesper.nilsson, andre.przywara, ada, radu_nicolae.pirea, richard.genoud, gregkh, linux-spi, linux-serial Cc: Durai Manickam KR Remove 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 <durai.manickamkr@microchip.com> --- 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 = <AT91_USART_MODE_SERIAL>; 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 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCHv2 2/2] ARM: dts: at91: sam9x60: fix the SOC detection 2023-07-13 9:51 ` [PATCHv2 2/2] ARM: dts: at91: sam9x60: fix the SOC detection Durai Manickam KR @ 2023-07-13 10:03 ` Krzysztof Kozlowski 2023-07-13 10:28 ` Durai.ManickamKR 0 siblings, 1 reply; 7+ messages in thread From: Krzysztof Kozlowski @ 2023-07-13 10:03 UTC (permalink / raw) To: Durai Manickam KR, Hari.PrasathGE, balamanikandan.gunasundar, manikandan.m, varshini.rajendran, dharma.b, nayabbasha.sayed, balakrishnan.s, cristian.birsan, nicolas.ferre, alexandre.belloni, davem, arnd, olof, soc, devicetree, linux-arm-kernel, linux-kernel, Kavyasree.Kotagiri, Horatiu.Vultur, robh+dt, andrew, michael, jerry.ray, conor+dt, jesper.nilsson, andre.przywara, ada, radu_nicolae.pirea, richard.genoud, gregkh, linux-spi, linux-serial On 13/07/2023 11:51, Durai Manickam KR wrote: > Remove the dbgu compatible strings in the UART submodule of the > flexcom for the proper SOC detection. Why? > > Fixes: 99c808335877 (ARM: dts: at91: sam9x60: Add missing flexcom definitions) If this is a bugfix, then please clearly describe the bug and its impact. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCHv2 2/2] ARM: dts: at91: sam9x60: fix the SOC detection 2023-07-13 10:03 ` Krzysztof Kozlowski @ 2023-07-13 10:28 ` Durai.ManickamKR 2023-07-13 10:43 ` Krzysztof Kozlowski 0 siblings, 1 reply; 7+ messages in thread From: Durai.ManickamKR @ 2023-07-13 10:28 UTC (permalink / raw) To: krzysztof.kozlowski, Hari.PrasathGE, Balamanikandan.Gunasundar, Manikandan.M, Varshini.Rajendran, Dharma.B, Nayabbasha.Sayed, Balakrishnan.S, Cristian.Birsan, Nicolas.Ferre, alexandre.belloni, davem, arnd, olof, soc, devicetree, linux-arm-kernel, linux-kernel, Kavyasree.Kotagiri, Horatiu.Vultur, robh+dt, andrew, michael, Jerry.Ray, conor+dt, jesper.nilsson, andre.przywara, ada, radu_nicolae.pirea, richard.genoud, gregkh, linux-spi, linux-serial On 13/07/23 15:33, Krzysztof Kozlowski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On 13/07/2023 11:51, Durai Manickam KR wrote: >> Remove the dbgu compatible strings in the UART submodule of the >> flexcom for the proper SOC detection. > Why? > >> Fixes: 99c808335877 (ARM: dts: at91: sam9x60: Add missing flexcom definitions) > If this is a bugfix, then please clearly describe the bug and its impact. Please check the coverletter (PATCHv2 0/2) for the real problem. But will try to explain the bug a bit more in the next set of patches. > > Best regards, > Krzysztof > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCHv2 2/2] ARM: dts: at91: sam9x60: fix the SOC detection 2023-07-13 10:28 ` Durai.ManickamKR @ 2023-07-13 10:43 ` Krzysztof Kozlowski 0 siblings, 0 replies; 7+ messages in thread From: Krzysztof Kozlowski @ 2023-07-13 10:43 UTC (permalink / raw) To: Durai.ManickamKR, Hari.PrasathGE, Balamanikandan.Gunasundar, Manikandan.M, Varshini.Rajendran, Dharma.B, Nayabbasha.Sayed, Balakrishnan.S, Cristian.Birsan, Nicolas.Ferre, alexandre.belloni, davem, arnd, olof, soc, devicetree, linux-arm-kernel, linux-kernel, Kavyasree.Kotagiri, Horatiu.Vultur, robh+dt, andrew, michael, Jerry.Ray, conor+dt, jesper.nilsson, andre.przywara, ada, radu_nicolae.pirea, richard.genoud, gregkh, linux-spi, linux-serial On 13/07/2023 12:28, Durai.ManickamKR@microchip.com wrote: > > On 13/07/23 15:33, Krzysztof Kozlowski wrote: >> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe >> >> On 13/07/2023 11:51, Durai Manickam KR wrote: >>> Remove the dbgu compatible strings in the UART submodule of the >>> flexcom for the proper SOC detection. >> Why? >> >>> Fixes: 99c808335877 (ARM: dts: at91: sam9x60: Add missing flexcom definitions) >> If this is a bugfix, then please clearly describe the bug and its impact. > Please check the coverletter (PATCHv2 0/2) for the real problem. But > will try to explain the bug a bit more in the next set of patches. Commit should stand on its own. Cover letters are usually not part of the commits. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2023-07-13 10:43 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-07-13 9:51 [PATCHv2 0/2] update at91 usart compatible for sam9x60 Durai Manickam KR 2023-07-13 9:51 ` [PATCHv2 1/2] dt-bindings: serial: atmel,at91-usart: add " Durai Manickam KR 2023-07-13 10:02 ` Krzysztof Kozlowski 2023-07-13 9:51 ` [PATCHv2 2/2] ARM: dts: at91: sam9x60: fix the SOC detection Durai Manickam KR 2023-07-13 10:03 ` Krzysztof Kozlowski 2023-07-13 10:28 ` Durai.ManickamKR 2023-07-13 10:43 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).