* [PATCH v3 00/50] Add support for sam9x7 SoC family
@ 2023-07-28 10:22 Varshini Rajendran
2023-07-28 11:32 ` Krzysztof Kozlowski
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: Varshini Rajendran @ 2023-07-28 10:22 UTC (permalink / raw)
To: robh+dt, krzysztof.kozlowski+dt, conor+dt, nicolas.ferre,
alexandre.belloni, claudiu.beznea, mturquette, sboyd, herbert,
davem, vkoul, andi.shyti, tglx, maz, lee, ulf.hansson,
tudor.ambarus, miquel.raynal, richard, vigneshr, edumazet, kuba,
pabeni, linus.walleij, sre, p.zabel, olivia, a.zummo,
radu_nicolae.pirea, richard.genoud, gregkh, lgirdwood, broonie,
wim, linux, linux, durai.manickamkr, varshini.rajendran, andrew,
jerry.ray, andre.przywara, mani, alexandre.torgue,
gregory.clement, arnd, rientjes, deller, 42.hyeyoo, vbabka,
mripard, mihai.sain, codrin.ciubotariu, eugen.hristev, devicetree,
linux-arm-kernel, linux-kernel, linux-clk, linux-crypto,
dmaengine, linux-i2c, linux-mmc, linux-mtd, netdev, linux-gpio,
linux-pm, linux-rtc, linux-spi, linux-serial, alsa-devel,
linux-usb, linux-watchdog
This patch series adds support for the new SoC family - sam9x7.
- The device tree, configs and drivers are added
- Clock driver for sam9x7 is added
- Support for basic peripherals is added
- Target board SAM9X75 Curiosity is added
Changes in v3:
--------------
- Fixed the DT documentation errors pointed out in v2.
- Dropped Acked-by tag in tcb DT doc patch as it had to be adapted
according to sam9x7 correctly.
- Picked by the previously missed tags.
- Dropped this patch "dt-bindings: usb: generic-ehci: Document clock-names
property" as the warning was not found while validating DT-schema for
at91-sam9x75_curiosity.dtb.
- Dropped redundant words in the commit message.
- Fixed the CHECK_DTBS warnings validated against
at91-sam9x75_curiosity.dtb.
- Renamed dt nodes according to naming convention.
- Dropped unwanted status property in dts.
- Removed nodes that are not in use from the board dts.
- Removed spi DT doc patch from the series as it was already applied
and a fix patch was applied subsequently. Added a patch to remove the
compatible to adapt sam9x7.
- Added sam9x7 compatibles in usb dt documentation.
Changes in v2:
--------------
- Added sam9x7 specific compatibles in DT with fallbacks
- Documented all the newly added DT compatible strings
- Added device tree for the target board sam9x75 curiosity and
documented the same in the DT bindings documentation
- Removed the dt nodes that are not supported at the moment
- Removed the configs added by previous version that are not supported
at the moment
- Fixed all the corrections in the commit message
- Changed all the instances of copyright year to 2023
- Added sam9x7 flag in PIT64B configuration
- Moved macro definitions to header file
- Added another divider in mck characteristics in the pmc driver
- Fixed the memory leak in the pmc driver
- Dropped patches that are no longer needed
- Picked up Acked-by and Reviewed-by tags
Hari Prasath (1):
irqchip/atmel-aic5: Add support for sam9x7 aic
Varshini Rajendran (49):
dt-bindings: microchip: atmel,at91rm9200-tcb: add sam9x60, sam9x7
compatible
dt-bindings: usb: ehci: Add atmel at91sam9g45-ehci compatible
dt-bindings: net: cdns,macb: add sam9x7 ethernet interface
dt-bindings: clk: at91: add sam9x7
dt-bindings: clk: at91: add sam9x7 clock controller
dt-bindings: reset: atmel,at91sam9260-reset: add sam9x7
dt-bindings: power: reset: atmel,sama5d2-shdwc: add sam9x7
dt-bindings: atmel-sysreg: add sam9x7
dt-bindings: crypto: add sam9x7 in Atmel AES
dt-bindings: crypto: add sam9x7 in Atmel SHA
dt-bindings: crypto: add sam9x7 in Atmel TDES
dt-bindings: dmaengine: at_xdmac: add compatible with microchip,sam9x7
dt-bindings: i2c: at91: Add sam9x7 compatible string
dt-bindings: mfd: at91: Add SAM9X7 compatible string
dt-bindings: atmel-gpbr: add microchip,sam9x7-gpbr
dt-bindings: atmel-matrix: add microchip,sam9x7-matrix
dt-bindings: atmel-smc: add microchip,sam9x7-smc
dt-bindings: atmel-ssc: add microchip,sam9x7-ssc
dt-bindings: sdhci-of-at91: add microchip,sam9x7-sdhci
dt-bindings: atmel-nand: add microchip,sam9x7-pmecc
dt-bindings: pinctrl: at91: add sam9x7
dt-bindings: rng: atmel,at91-trng: add sam9x7 TRNG
dt-bindings: rtc: at91rm9200: add sam9x7 compatible
dt-bindings: rtt: at91rm9260: add sam9x7 compatible
dt-bindings: serial: atmel,at91-usart: add compatible for sam9x7
dt-bindings: atmel-classd: add sam9x7 compatible
dt-bindings: usb: atmel: add sam9x7
dt-bindings: watchdog: sama5d4-wdt: add compatible for sam9x7-wdt
dt-bindings: irqchip/atmel-aic5: Add support for sam9x7 aic
spi: dt-bindings: atmel,at91rm9200-spi: remove 9x60 compatible from
list
ASoC: dt-bindings: microchip: add sam9x7
dt-bindings: usb: ehci: Add sam9x7
dt-bindings: usb: add sam9x7
ARM: at91: pm: add support for sam9x7 SoC family
ARM: at91: pm: add sam9x7 SoC init config
ARM: at91: add support in SoC driver for new sam9x7
clk: at91: clk-sam9x60-pll: re-factor to support individual core freq
outputs
clk: at91: sam9x7: add support for HW PLL freq dividers
clk: at91: sama7g5: move mux table macros to header file
clk: at91: sam9x7: Allow PLLs to be exported and referenced in DT
clk: at91: sam9x7: add sam9x7 pmc driver
power: reset: at91-poweroff: lookup for proper pmc dt node for sam9x7
power: reset: at91-reset: add reset support for sam9x7 SoC
power: reset: at91-reset: add sdhwc support for sam9x7 SoC
ARM: at91: Kconfig: add config flag for SAM9X7 SoC
ARM: configs: at91: enable config flags for sam9x7 SoC family
ARM: dts: at91: sam9x7: add device tree for SoC
dt-bindings: arm: add sam9x75 curiosity board
ARM: dts: at91: sam9x75_curiosity: add sam9x75 curiosity board
.../devicetree/bindings/arm/atmel-at91.yaml | 6 +
.../devicetree/bindings/arm/atmel-sysregs.txt | 7 +-
.../bindings/clock/atmel,at91rm9200-pmc.yaml | 1 +
.../bindings/clock/atmel,at91sam9x5-sckc.yaml | 3 +
.../crypto/atmel,at91sam9g46-aes.yaml | 6 +-
.../crypto/atmel,at91sam9g46-sha.yaml | 6 +-
.../crypto/atmel,at91sam9g46-tdes.yaml | 6 +-
.../devicetree/bindings/dma/atmel-xdma.txt | 3 +-
.../bindings/i2c/atmel,at91sam-i2c.yaml | 4 +
.../interrupt-controller/atmel,aic.txt | 2 +-
.../devicetree/bindings/mfd/atmel-flexcom.txt | 1 +
.../devicetree/bindings/mfd/atmel-gpbr.txt | 1 +
.../devicetree/bindings/mfd/atmel-matrix.txt | 1 +
.../devicetree/bindings/mfd/atmel-smc.txt | 1 +
.../devicetree/bindings/misc/atmel-ssc.txt | 1 +
.../devicetree/bindings/mmc/sdhci-atmel.txt | 4 +-
.../devicetree/bindings/mtd/atmel-nand.txt | 1 +
.../devicetree/bindings/net/cdns,macb.yaml | 6 +
.../bindings/pinctrl/atmel,at91-pinctrl.txt | 2 +
.../power/reset/atmel,sama5d2-shdwc.yaml | 3 +
.../reset/atmel,at91sam9260-reset.yaml | 4 +
.../bindings/rng/atmel,at91-trng.yaml | 4 +
.../bindings/rtc/atmel,at91rm9200-rtc.yaml | 18 +-
.../bindings/rtc/atmel,at91sam9260-rtt.yaml | 3 +
.../bindings/serial/atmel,at91-usart.yaml | 9 +
.../soc/microchip/atmel,at91rm9200-tcb.yaml | 21 +-
.../bindings/sound/atmel,sama5d2-classd.yaml | 7 +-
.../sound/microchip,sama7g5-i2smcc.yaml | 11 +-
.../bindings/spi/atmel,at91rm9200-spi.yaml | 1 -
.../devicetree/bindings/usb/atmel-usb.txt | 11 +-
.../devicetree/bindings/usb/generic-ehci.yaml | 7 +
.../devicetree/bindings/usb/generic-ohci.yaml | 6 +
.../bindings/watchdog/atmel,sama5d4-wdt.yaml | 15 +-
arch/arm/boot/dts/microchip/Makefile | 5 +
.../dts/microchip/at91-sam9x75_curiosity.dts | 311 +++++
arch/arm/boot/dts/microchip/sam9x7.dtsi | 1238 +++++++++++++++++
arch/arm/configs/at91_dt_defconfig | 1 +
arch/arm/mach-at91/Kconfig | 23 +-
arch/arm/mach-at91/Makefile | 1 +
arch/arm/mach-at91/generic.h | 2 +
arch/arm/mach-at91/pm.c | 35 +
arch/arm/mach-at91/sam9x7.c | 34 +
drivers/clk/at91/Makefile | 1 +
drivers/clk/at91/clk-sam9x60-pll.c | 50 +-
drivers/clk/at91/pmc.h | 18 +
drivers/clk/at91/sam9x60.c | 7 +
drivers/clk/at91/sam9x7.c | 946 +++++++++++++
drivers/clk/at91/sama7g5.c | 42 +-
drivers/irqchip/irq-atmel-aic5.c | 10 +
drivers/power/reset/Kconfig | 4 +-
drivers/power/reset/at91-sama5d2_shdwc.c | 1 +
drivers/soc/atmel/soc.c | 23 +
drivers/soc/atmel/soc.h | 9 +
include/dt-bindings/clock/at91.h | 4 +
54 files changed, 2871 insertions(+), 76 deletions(-)
create mode 100644 arch/arm/boot/dts/microchip/at91-sam9x75_curiosity.dts
create mode 100644 arch/arm/boot/dts/microchip/sam9x7.dtsi
create mode 100644 arch/arm/mach-at91/sam9x7.c
create mode 100644 drivers/clk/at91/sam9x7.c
--
2.25.1
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v3 00/50] Add support for sam9x7 SoC family
2023-07-28 10:22 [PATCH v3 00/50] Add support for sam9x7 SoC family Varshini Rajendran
@ 2023-07-28 11:32 ` Krzysztof Kozlowski
2023-07-28 15:50 ` Conor Dooley
2023-07-28 11:32 ` Krzysztof Kozlowski
2023-07-28 14:46 ` Rob Herring
2 siblings, 1 reply; 8+ messages in thread
From: Krzysztof Kozlowski @ 2023-07-28 11:32 UTC (permalink / raw)
To: Varshini Rajendran, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, claudiu.beznea, mturquette,
sboyd, herbert, davem, vkoul, andi.shyti, tglx, maz, lee,
ulf.hansson, tudor.ambarus, miquel.raynal, richard, vigneshr,
edumazet, kuba, pabeni, linus.walleij, sre, p.zabel, olivia,
a.zummo, radu_nicolae.pirea, richard.genoud, gregkh, lgirdwood,
broonie, wim, linux, linux, durai.manickamkr, andrew, jerry.ray,
andre.przywara, mani, alexandre.torgue, gregory.clement, arnd,
rientjes, deller, 42.hyeyoo, vbabka, mripard, mihai.sain,
codrin.ciubotariu, eugen.hristev, devicetree, linux-arm-kernel,
linux-kernel, linux-clk, linux-crypto, dmaengine, linux-i2c,
linux-mmc, linux-mtd, netdev, linux-gpio, linux-pm, linux-rtc,
linux-spi, linux-serial, alsa-devel, linux-usb, linux-watchdog
On 28/07/2023 12:22, Varshini Rajendran wrote:
> This patch series adds support for the new SoC family - sam9x7.
> - The device tree, configs and drivers are added
> - Clock driver for sam9x7 is added
> - Support for basic peripherals is added
> - Target board SAM9X75 Curiosity is added
>
Your threading is absolutely broken making it difficult to review and apply.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v3 00/50] Add support for sam9x7 SoC family
2023-07-28 10:22 [PATCH v3 00/50] Add support for sam9x7 SoC family Varshini Rajendran
2023-07-28 11:32 ` Krzysztof Kozlowski
@ 2023-07-28 11:32 ` Krzysztof Kozlowski
2023-07-28 14:46 ` Rob Herring
2 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2023-07-28 11:32 UTC (permalink / raw)
To: Varshini Rajendran, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, claudiu.beznea, mturquette,
sboyd, herbert, davem, vkoul, andi.shyti, tglx, maz, lee,
ulf.hansson, tudor.ambarus, miquel.raynal, richard, vigneshr,
edumazet, kuba, pabeni, linus.walleij, sre, p.zabel, olivia,
a.zummo, radu_nicolae.pirea, richard.genoud, gregkh, lgirdwood,
broonie, wim, linux, linux, durai.manickamkr, andrew, jerry.ray,
andre.przywara, mani, alexandre.torgue, gregory.clement, arnd,
rientjes, deller, 42.hyeyoo, vbabka, mripard, mihai.sain,
codrin.ciubotariu, eugen.hristev, devicetree, linux-arm-kernel,
linux-kernel, linux-clk, linux-crypto, dmaengine, linux-i2c,
linux-mmc, linux-mtd, netdev, linux-gpio, linux-pm, linux-rtc,
linux-spi, linux-serial, alsa-devel, linux-usb, linux-watchdog
On 28/07/2023 12:22, Varshini Rajendran wrote:
> This patch series adds support for the new SoC family - sam9x7.
> - The device tree, configs and drivers are added
> - Clock driver for sam9x7 is added
> - Support for basic peripherals is added
> - Target board SAM9X75 Curiosity is added
>
> Changes in v3:
> --------------
>
> - Fixed the DT documentation errors pointed out in v2.
That's too generic. What changed?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v3 00/50] Add support for sam9x7 SoC family
2023-07-28 10:22 [PATCH v3 00/50] Add support for sam9x7 SoC family Varshini Rajendran
2023-07-28 11:32 ` Krzysztof Kozlowski
2023-07-28 11:32 ` Krzysztof Kozlowski
@ 2023-07-28 14:46 ` Rob Herring
2 siblings, 0 replies; 8+ messages in thread
From: Rob Herring @ 2023-07-28 14:46 UTC (permalink / raw)
To: Varshini Rajendran
Cc: krzysztof.kozlowski+dt, conor+dt, nicolas.ferre,
alexandre.belloni, claudiu.beznea, mturquette, sboyd, herbert,
davem, vkoul, andi.shyti, tglx, maz, lee, ulf.hansson,
tudor.ambarus, miquel.raynal, richard, vigneshr, edumazet, kuba,
pabeni, linus.walleij, sre, p.zabel, olivia, a.zummo,
radu_nicolae.pirea, richard.genoud, gregkh, lgirdwood, broonie,
wim, linux, linux, durai.manickamkr, andrew, jerry.ray,
andre.przywara, mani, alexandre.torgue, gregory.clement, arnd,
rientjes, deller, 42.hyeyoo, vbabka, mripard, mihai.sain,
codrin.ciubotariu, eugen.hristev, devicetree, linux-arm-kernel,
linux-kernel, linux-clk, linux-crypto, dmaengine, linux-i2c,
linux-mmc, linux-mtd, netdev, linux-gpio, linux-pm, linux-rtc,
linux-spi, linux-serial, alsa-devel, linux-usb, linux-watchdog
On Fri, Jul 28, 2023 at 4:23 AM Varshini Rajendran
<varshini.rajendran@microchip.com> wrote:
>
> This patch series adds support for the new SoC family - sam9x7.
> - The device tree, configs and drivers are added
> - Clock driver for sam9x7 is added
> - Support for basic peripherals is added
> - Target board SAM9X75 Curiosity is added
>
> Changes in v3:
> --------------
>
> - Fixed the DT documentation errors pointed out in v2.
> - Dropped Acked-by tag in tcb DT doc patch as it had to be adapted
> according to sam9x7 correctly.
> - Picked by the previously missed tags.
> - Dropped this patch "dt-bindings: usb: generic-ehci: Document clock-names
> property" as the warning was not found while validating DT-schema for
> at91-sam9x75_curiosity.dtb.
> - Dropped redundant words in the commit message.
> - Fixed the CHECK_DTBS warnings validated against
> at91-sam9x75_curiosity.dtb.
> - Renamed dt nodes according to naming convention.
> - Dropped unwanted status property in dts.
> - Removed nodes that are not in use from the board dts.
> - Removed spi DT doc patch from the series as it was already applied
> and a fix patch was applied subsequently. Added a patch to remove the
> compatible to adapt sam9x7.
> - Added sam9x7 compatibles in usb dt documentation.
>
>
> Changes in v2:
> --------------
>
> - Added sam9x7 specific compatibles in DT with fallbacks
> - Documented all the newly added DT compatible strings
> - Added device tree for the target board sam9x75 curiosity and
> documented the same in the DT bindings documentation
> - Removed the dt nodes that are not supported at the moment
> - Removed the configs added by previous version that are not supported
> at the moment
> - Fixed all the corrections in the commit message
> - Changed all the instances of copyright year to 2023
> - Added sam9x7 flag in PIT64B configuration
> - Moved macro definitions to header file
> - Added another divider in mck characteristics in the pmc driver
> - Fixed the memory leak in the pmc driver
> - Dropped patches that are no longer needed
> - Picked up Acked-by and Reviewed-by tags
>
>
> Hari Prasath (1):
> irqchip/atmel-aic5: Add support for sam9x7 aic
>
> Varshini Rajendran (49):
> dt-bindings: microchip: atmel,at91rm9200-tcb: add sam9x60, sam9x7
> compatible
> dt-bindings: usb: ehci: Add atmel at91sam9g45-ehci compatible
> dt-bindings: net: cdns,macb: add sam9x7 ethernet interface
> dt-bindings: clk: at91: add sam9x7
> dt-bindings: clk: at91: add sam9x7 clock controller
> dt-bindings: reset: atmel,at91sam9260-reset: add sam9x7
> dt-bindings: power: reset: atmel,sama5d2-shdwc: add sam9x7
> dt-bindings: atmel-sysreg: add sam9x7
> dt-bindings: crypto: add sam9x7 in Atmel AES
> dt-bindings: crypto: add sam9x7 in Atmel SHA
> dt-bindings: crypto: add sam9x7 in Atmel TDES
> dt-bindings: dmaengine: at_xdmac: add compatible with microchip,sam9x7
> dt-bindings: i2c: at91: Add sam9x7 compatible string
> dt-bindings: mfd: at91: Add SAM9X7 compatible string
> dt-bindings: atmel-gpbr: add microchip,sam9x7-gpbr
> dt-bindings: atmel-matrix: add microchip,sam9x7-matrix
> dt-bindings: atmel-smc: add microchip,sam9x7-smc
> dt-bindings: atmel-ssc: add microchip,sam9x7-ssc
> dt-bindings: sdhci-of-at91: add microchip,sam9x7-sdhci
> dt-bindings: atmel-nand: add microchip,sam9x7-pmecc
> dt-bindings: pinctrl: at91: add sam9x7
> dt-bindings: rng: atmel,at91-trng: add sam9x7 TRNG
> dt-bindings: rtc: at91rm9200: add sam9x7 compatible
> dt-bindings: rtt: at91rm9260: add sam9x7 compatible
> dt-bindings: serial: atmel,at91-usart: add compatible for sam9x7
> dt-bindings: atmel-classd: add sam9x7 compatible
> dt-bindings: usb: atmel: add sam9x7
> dt-bindings: watchdog: sama5d4-wdt: add compatible for sam9x7-wdt
> dt-bindings: irqchip/atmel-aic5: Add support for sam9x7 aic
> spi: dt-bindings: atmel,at91rm9200-spi: remove 9x60 compatible from
> list
> ASoC: dt-bindings: microchip: add sam9x7
> dt-bindings: usb: ehci: Add sam9x7
> dt-bindings: usb: add sam9x7
> ARM: at91: pm: add support for sam9x7 SoC family
> ARM: at91: pm: add sam9x7 SoC init config
> ARM: at91: add support in SoC driver for new sam9x7
> clk: at91: clk-sam9x60-pll: re-factor to support individual core freq
> outputs
> clk: at91: sam9x7: add support for HW PLL freq dividers
> clk: at91: sama7g5: move mux table macros to header file
> clk: at91: sam9x7: Allow PLLs to be exported and referenced in DT
> clk: at91: sam9x7: add sam9x7 pmc driver
> power: reset: at91-poweroff: lookup for proper pmc dt node for sam9x7
> power: reset: at91-reset: add reset support for sam9x7 SoC
> power: reset: at91-reset: add sdhwc support for sam9x7 SoC
> ARM: at91: Kconfig: add config flag for SAM9X7 SoC
> ARM: configs: at91: enable config flags for sam9x7 SoC family
> ARM: dts: at91: sam9x7: add device tree for SoC
> dt-bindings: arm: add sam9x75 curiosity board
> ARM: dts: at91: sam9x75_curiosity: add sam9x75 curiosity board
>
> .../devicetree/bindings/arm/atmel-sysregs.txt | 7 +-
> .../devicetree/bindings/dma/atmel-xdma.txt | 3 +-
> .../interrupt-controller/atmel,aic.txt | 2 +-
> .../devicetree/bindings/mfd/atmel-flexcom.txt | 1 +
> .../devicetree/bindings/mfd/atmel-gpbr.txt | 1 +
> .../devicetree/bindings/mfd/atmel-matrix.txt | 1 +
> .../devicetree/bindings/mfd/atmel-smc.txt | 1 +
> .../devicetree/bindings/misc/atmel-ssc.txt | 1 +
> .../devicetree/bindings/mmc/sdhci-atmel.txt | 4 +-
> .../devicetree/bindings/mtd/atmel-nand.txt | 1 +
> .../bindings/pinctrl/atmel,at91-pinctrl.txt | 2 +
> .../devicetree/bindings/usb/atmel-usb.txt | 11 +-
Still quite a few .txt bindings. When are Microchip folks going to get
around to converting the rest of their bindings to schema?
Be warned that at some point we will simply not accept any .txt binding changes.
> .../dts/microchip/at91-sam9x75_curiosity.dts | 311 +++++
> arch/arm/boot/dts/microchip/sam9x7.dtsi | 1238 +++++++++++++++++
Don't you want to do that before adding new chips so warnings can be
fixed up front?
Overall, the Atmel/Microchip stuff is not in great shape. 1368 unique
warnings and 189 undocumented (by schema) compatibles. Note lots of
OHCI/EHCI related warnings already. I'm sure you didn't add more with
this new chip, right?
(All this data is available here:
https://gitlab.com/robherring/linux-dt/-/jobs )
arch/arm/boot/dts/microchip:4655:1368
71 nand-controller: #size-cells:0:0: 0 was expected
71 nand-controller: #address-cells:0:0: 1 was expected
53 /: compatible: 'oneOf' conditional failed, one must be fixed:
49 serial@200: $nodename:0: 'serial@200' does not match
'^spi(@.*|-([0-9]|[1-9][0-9]+))?$'
39 pinctrl@fffff400: $nodename:0: 'pinctrl@fffff400' does not
match '^([a-z][a-z0-9\\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
31 ohci@600000: compatible: 'oneOf' conditional failed, one must be fixed:
31 ohci@600000: $nodename:0: 'ohci@600000' does not match '^usb(@.*)?'
31 ehci@700000: compatible: 'oneOf' conditional failed, one must be fixed:
31 ehci@700000: $nodename:0: 'ehci@700000' does not match '^usb(@.*)?'
27 ehci@700000: Unevaluated properties are not allowed
('clock-names' was unexpected)
25 serial@200: atmel,fifo-size: False schema does not allow [[32]]
24 serial@200: atmel,use-dma-tx: False schema does not allow True
24 serial@200: atmel,use-dma-rx: False schema does not allow True
24 serial@200: atmel,fifo-size: False schema does not allow [[16]]
24 ohci@500000: compatible: 'oneOf' conditional failed, one must be fixed:
24 ohci@500000: $nodename:0: 'ohci@500000' does not match '^usb(@.*)?'
20 pinctrl@fffff200: $nodename:0: 'pinctrl@fffff200' does not
match '^([a-z][a-z0-9\\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
18 pinctrl@fffff400: tcb0: {'tcb0_tclk0-0': {'atmel,pins': [[0,
25, 1, 0]]}, 'tcb0_tclk1-0': {'atmel,pins': [[1, 6, 2, 0]]},
'tcb0_tclk2-0': {'atmel,pins': [[1, 7, 2, 0]]}, 'tcb0_tioa0-0':
{'atmel,pins': [[0, 26, 1, 0]]}, 'tcb0_tioa1-0': {'atmel,pins': [[0,
27, 1, 0]]}, 'tcb0_tioa2-0': {'atmel,pins': [[0, 28, 1, 0]]},
'tcb0_tiob0-0': {'atmel,pins': [[2, 9, 2, 0]]}, 'tcb0_tiob1-0':
{'atmel,pins': [[2, 7, 1, 0]]}, 'tcb0_tiob2-0': {'atmel,pins': [[2, 6,
1, 0]]}} should not be valid under {'type': 'object'}
17 pinctrl@fffff400: tcb1: {'tcb1_tclk0-0': {'atmel,pins': [[1,
16, 2, 0]]}, 'tcb1_tclk1-0': {'atmel,pins': [[1, 17, 2, 0]]},
'tcb1_tclk2-0': {'atmel,pins': [[2, 22, 2, 0]]}, 'tcb1_tioa0-0':
{'atmel,pins': [[1, 0, 2, 0]]}, 'tcb1_tioa1-0': {'atmel,pins': [[1, 2,
2, 0]]}, 'tcb1_tioa2-0': {'atmel,pins': [[1, 3, 2, 0]]},
'tcb1_tiob0-0': {'atmel,pins': [[1, 1, 2, 0]]}, 'tcb1_tiob1-0':
{'atmel,pins': [[1, 18, 2, 0]]}, 'tcb1_tiob2-0': {'atmel,pins': [[1,
19, 2, 0]]}} should not be valid under {'type': 'object'}
17 pinctrl@fffff400: dbgu: {'dbgu-0': {'atmel,pins': [[1, 14, 1,
1, 1, 15, 1, 0]], 'phandle': [[5]]}} should not be valid under
{'type': 'object'}
17 ohci@600000: Unevaluated properties are not allowed
('atmel,vbus-gpio', 'clock-names', 'compatible' were unexpected)
17 i2c-gpio-0: 'sda-gpios' is a required property
17 i2c-gpio-0: 'scl-gpios' is a required property
16 ohci@500000: Unevaluated properties are not allowed
('clock-names', 'compatible' were unexpected)
15 pinctrl@fffff400: tcb1: {'tcb1_tclk0-0': {'atmel,pins': [[2,
4, 3, 0]]}, 'tcb1_tclk1-0': {'atmel,pins': [[2, 7, 3, 0]]},
'tcb1_tclk2-0': {'atmel,pins': [[2, 14, 3, 0]]}, 'tcb1_tioa0-0':
{'atmel,pins': [[2, 2, 3, 0]]}, 'tcb1_tioa1-0': {'atmel,pins': [[2, 5,
3, 0]]}, 'tcb1_tioa2-0': {'atmel,pins': [[2, 12, 3, 0]]},
'tcb1_tiob0-0': {'atmel,pins': [[2, 3, 3, 0]]}, 'tcb1_tiob1-0':
{'atmel,pins': [[2, 6, 3, 0]]}, 'tcb1_tiob2-0': {'atmel,pins': [[2,
13, 3, 0]]}} should not be valid under {'type': 'object'}
15 pinctrl@fffff400: tcb0: {'tcb0_tclk0-0': {'atmel,pins': [[0,
24, 1, 0]]}, 'tcb0_tclk1-0': {'atmel,pins': [[0, 25, 1, 0]]},
'tcb0_tclk2-0': {'atmel,pins': [[0, 26, 1, 0]]}, 'tcb0_tioa0-0':
{'atmel,pins': [[0, 21, 1, 0]]}, 'tcb0_tioa1-0': {'atmel,pins': [[0,
22, 1, 0]]}, 'tcb0_tioa2-0': {'atmel,pins': [[0, 23, 1, 0]]},
'tcb0_tiob0-0': {'atmel,pins': [[0, 27, 1, 0]]}, 'tcb0_tiob1-0':
{'atmel,pins': [[0, 28, 1, 0]]}, 'tcb0_tiob2-0': {'atmel,pins': [[0,
29, 1, 0]]}} should not be valid under {'type': 'object'}
14 pinctrl@fffff400: ssc0: {'ssc0_tx-0': {'atmel,pins': [[0, 24,
2, 0, 0, 25, 2, 0, 0, 26, 2, 0]], 'phandle': [[7]]}, 'ssc0_rx-0':
{'atmel,pins': [[0, 27, 2, 0, 0, 28, 2, 0, 0, 29, 2, 0]], 'phandle':
[[8]]}} should not be valid under {'type': 'object'}
13 pinctrl@fffff200: nand0: {'nand0_ale_cle-0': {'atmel,pins':
[[4, 21, 1, 1, 4, 22, 1, 1]]}} should not be valid under {'type':
'object'}
11 pinctrl@fffff400: mmc0: {'mmc0_clk-0': {'atmel,pins': [[0, 8,
1, 0]]}, 'mmc0_slot0_cmd_dat0-0': {'atmel,pins': [[0, 7, 1, 1, 0, 6,
1, 1]]}, 'mmc0_slot0_dat1_3-0': {'atmel,pins': [[0, 9, 1, 1, 0, 10, 1,
1, 0, 11, 1, 1]]}, 'mmc0_slot1_cmd_dat0-0': {'atmel,pins': [[0, 1, 2,
1, 0, 0, 2, 1]]}, 'mmc0_slot1_dat1_3-0': {'atmel,pins': [[0, 5, 2, 1,
0, 4, 2, 1, 0, 3, 2, 1]]}} should not be valid under {'type':
'object'}
11 pinctrl@fffff200: pwm0: {'pwm0_pwmh0-0': {'atmel,pins': [[0,
20, 2, 0]]}, 'pwm0_pwmh0-1': {'atmel,pins': [[1, 0, 2, 0]]},
'pwm0_pwml0-0': {'atmel,pins': [[0, 21, 2, 0]]}, 'pwm0_pwml0-1':
{'atmel,pins': [[1, 1, 2, 0]]}, 'pwm0_pwmh1-0': {'atmel,pins': [[0,
22, 2, 0]]}, 'pwm0_pwmh1-1': {'atmel,pins': [[1, 4, 2, 0]]},
'pwm0_pwmh1-2': {'atmel,pins': [[1, 27, 3, 0]]}, 'pwm0_pwml1-0':
{'atmel,pins': [[0, 23, 2, 0]]}, 'pwm0_pwml1-1': {'atmel,pins': [[1,
5, 2, 0]]}, 'pwm0_pwml1-2': {'atmel,pins': [[4, 31, 2, 0]]},
'pwm0_pwmh2-0': {'atmel,pins': [[1, 8, 2, 0]]}, 'pwm0_pwmh2-1':
{'atmel,pins': [[3, 5, 3, 0]]}, 'pwm0_pwml2-0': {'atmel,pins': [[1, 9,
2, 0]]}, 'pwm0_pwml2-1': {'atmel,pins': [[3, 6, 3, 0]]},
'pwm0_pwmh3-0': {'atmel,pins': [[1, 12, 2, 0]]}, 'pwm0_pwmh3-1':
{'atmel,pins': [[3, 7, 3, 0]]}, 'pwm0_pwml3-0': {'atmel,pins': [[1,
13, 2, 0]]}, 'pwm0_pwml3-1': {'atmel,pins': [[3, 8, 3, 0]]}} should
not be valid under {'type': 'object'}
arch/arm/boot/dts/microchip:189
['acme,ariag25', 'atmel,at91sam9x5ek', 'atmel,at91sam9x5', 'atmel,at91sam9']
['acme,ariettag25', 'atmel,at91sam9x5', 'atmel,at91sam9']
['acme,foxg20', 'atmel,at91sam9g20', 'atmel,at91sam9']
['aries,ma5d4evk', 'denx,ma5d4evk', 'atmel,sama5d4', 'atmel,sama5']
['atheros,ath6kl']
['atmel,asoc-wm8904']
['atmel,at45', 'atmel,dataflash']
['atmel,at91rm9200-aic']
['atmel,at91rm9200-gpio']
['atmel,at91rm9200-nand']
['atmel,at91rm9200-ohci', 'usb-ohci']
['atmel,at91rm9200-pinctrl', 'simple-bus']
['atmel,at91rm9200-sdramc', 'syscon']
['atmel,at91rm9200-ssc']
['atmel,at91rm9200-st', 'syscon', 'simple-mfd']
['atmel,at91rm9200-udc']
['atmel,at91rm9200-wdt']
['atmel,at91rm9200ek', 'atmel,at91rm9200']
['atmel,at91sam9260-ebi']
['atmel,at91sam9260-gpbr', 'syscon']
['atmel,at91sam9260-matrix', 'syscon']
['atmel,at91sam9260-nand-controller']
['atmel,at91sam9260-pit']
['atmel,at91sam9260-sdramc']
['atmel,at91sam9260-smc', 'syscon']
['atmel,at91sam9260-udc']
['atmel,at91sam9260ek', 'atmel,at91sam9260', 'atmel,at91sam9']
['atmel,at91sam9261-ebi']
['atmel,at91sam9261-lcdc']
['atmel,at91sam9261-matrix', 'syscon']
['atmel,at91sam9261-nand-controller']
['atmel,at91sam9261-pmc', 'syscon']
['atmel,at91sam9261-udc']
['atmel,at91sam9261ek', 'atmel,at91sam9261', 'atmel,at91sam9']
['atmel,at91sam9263-ac97c']
['atmel,at91sam9263-can']
['atmel,at91sam9263-ebi0']
['atmel,at91sam9263-ebi1']
['atmel,at91sam9263-lcdc']
['atmel,at91sam9263-matrix', 'syscon']
['atmel,at91sam9263-pmc', 'syscon']
['atmel,at91sam9263-udc']
['atmel,at91sam9263ek', 'atmel,at91sam9263', 'atmel,at91sam9']
['atmel,at91sam9g15ek', 'atmel,at91sam9x5ek', 'atmel,at91sam9x5',
'atmel,at91sam9']
['atmel,at91sam9g20ek', 'atmel,at91sam9g20', 'atmel,at91sam9']
['atmel,at91sam9g20ek-wm8731-audio']
['atmel,at91sam9g20ek_2mmc', 'atmel,at91sam9g20', 'atmel,at91sam9']
['atmel,at91sam9g25ek', 'atmel,at91sam9x5ek', 'atmel,at91sam9x5',
'atmel,at91sam9']
['atmel,at91sam9g35ek', 'atmel,at91sam9x5ek', 'atmel,at91sam9x5',
'atmel,at91sam9']
['atmel,at91sam9g45-ddramc']
['atmel,at91sam9g45-dma']
['atmel,at91sam9g45-ebi']
['atmel,at91sam9g45-ehci', 'usb-ehci']
['atmel,at91sam9g45-isi']
['atmel,at91sam9g45-lcdc']
['atmel,at91sam9g45-matrix', 'syscon']
['atmel,at91sam9g45-nand-controller']
['atmel,at91sam9g45-pmecc']
['atmel,at91sam9g45-ssc']
['atmel,at91sam9g45-udc']
['atmel,at91sam9m10g45ek', 'atmel,at91sam9g45', 'atmel,at91sam9']
['atmel,at91sam9n12-hlcdc']
['atmel,at91sam9n12-matrix', 'syscon']
['atmel,at91sam9n12ek', 'atmel,at91sam9n12', 'atmel,at91sam9']
['atmel,at91sam9rl-dma']
['atmel,at91sam9rl-ebi']
['atmel,at91sam9rl-lcdc']
['atmel,at91sam9rl-matrix', 'syscon']
['atmel,at91sam9rl-ssc']
['atmel,at91sam9rl-udc']
['atmel,at91sam9rlek', 'atmel,at91sam9rl', 'atmel,at91sam9']
['atmel,at91sam9x25ek', 'atmel,at91sam9x5ek', 'atmel,at91sam9x5',
'atmel,at91sam9']
['atmel,at91sam9x35ek', 'atmel,at91sam9x5ek', 'atmel,at91sam9x5',
'atmel,at91sam9']
['atmel,at91sam9x5-can']
['atmel,at91sam9x5-ebi']
['atmel,at91sam9x5-gpio', 'atmel,at91rm9200-gpio']
['atmel,at91sam9x5-hlcdc']
['atmel,at91sam9x5-matrix', 'syscon']
['atmel,at91sam9x5-pinctrl', 'atmel,at91rm9200-pinctrl', 'simple-bus']
['atmel,hlcdc-display-controller']
['atmel,hlcdc-pwm']
['atmel,hsmci']
['atmel,osc', 'fixed-clock']
['atmel,sam9x5-wm8731-audio']
['atmel,sama5d2-aic']
['atmel,sama5d2-chipid']
['atmel,sama5d2-flexcom']
['atmel,sama5d2-hlcdc']
['atmel,sama5d2-pinctrl']
['atmel,sama5d2-pmecc']
['atmel,sama5d2-ptc_ek', 'atmel,sama5d2', 'atmel,sama5']
['atmel,sama5d2-sdhci']
['atmel,sama5d2-secumod', 'syscon']
['atmel,sama5d2-sfr', 'syscon']
['atmel,sama5d2-sfrbu', 'syscon']
['atmel,sama5d2-smc', 'syscon', 'simple-mfd']
['atmel,sama5d2-xplained', 'atmel,sama5d2', 'atmel,sama5']
['atmel,sama5d27-som1-ek', 'atmel,sama5d27-som1', 'atmel,sama5d27',
'atmel,sama5d2', 'atmel,sama5']
['atmel,sama5d3-adc']
['atmel,sama5d3-aic']
['atmel,sama5d3-ddramc']
['atmel,sama5d3-ebi']
['atmel,sama5d3-hlcdc']
['atmel,sama5d3-nand-controller']
['atmel,sama5d3-nfc-io', 'syscon']
['atmel,sama5d3-pinctrl', 'atmel,at91sam9x5-pinctrl', 'simple-bus']
['atmel,sama5d3-sfr', 'syscon']
['atmel,sama5d3-smc', 'syscon', 'simple-mfd']
['atmel,sama5d3-udc']
['atmel,sama5d3-xplained', 'atmel,sama5d3', 'atmel,sama5']
['atmel,sama5d31ek', 'atmel,sama5d3xmb', 'atmel,sama5d3xcm',
'atmel,sama5d31', 'atmel,sama5d3', 'atmel,sama5']
['atmel,sama5d33ek', 'atmel,sama5d3xmb', 'atmel,sama5d3xcm',
'atmel,sama5d33', 'atmel,sama5d3', 'atmel,sama5']
['atmel,sama5d34ek', 'atmel,sama5d3xmb', 'atmel,sama5d3xcm',
'atmel,sama5d34', 'atmel,sama5d3', 'atmel,sama5']
['atmel,sama5d35ek', 'atmel,sama5d3xmb', 'atmel,sama5d3xcm',
'atmel,sama5d35', 'atmel,sama5d3', 'atmel,sama5']
['atmel,sama5d36ek', 'atmel,sama5d3xmb', 'atmel,sama5d3xcm',
'atmel,sama5d36', 'atmel,sama5d3', 'atmel,sama5']
['atmel,sama5d36ek-cmp', 'atmel,sama5d3xmb-cmp',
'atmel,sama5d3xcm-cmp', 'atmel,sama5d36', 'atmel,sama5d3',
'atmel,sama5']
['atmel,sama5d4-aic']
['atmel,sama5d4-dma']
['atmel,sama5d4-hlcdc']
['atmel,sama5d4-pmecc']
['atmel,sama5d4-sfr', 'syscon']
['atmel,sama5d4-xplained', 'atmel,sama5d4', 'atmel,sama5']
['atmel,sama5d4ek', 'atmel,sama5d4', 'atmel,sama5']
['atmel,tny-a9263', 'atmel,at91sam9263', 'atmel,at91sam9']
['atmel,usb-a9263', 'atmel,at91sam9263', 'atmel,at91sam9']
['axentia,tse850-pcm5142']
['calao,qil-a9260', 'atmel,at91sam9260', 'atmel,at91sam9']
['calao,tny-a9260', 'atmel,at91sam9260', 'atmel,at91sam9']
['calao,tny-a9g20', 'atmel,at91sam9g20', 'atmel,at91sam9']
['calao,usb-a9260', 'atmel,at91sam9260', 'atmel,at91sam9']
['calao,usb-a9g20', 'atmel,at91sam9g20', 'atmel,at91sam9']
['calao,usb-a9g20-lpw', 'calao,usb-a9g20', 'atmel,at91sam9g20',
'atmel,at91sam9']
['egnite,ethernut5', 'atmel,at91sam9260', 'atmel,at91sam9']
['focaltech,ft5426', 'edt,edt-ft5406']
['hce,cosino_mega2560', 'atmel,at91sam9x5', 'atmel,at91sam9']
['l+g,vinco', 'atmel,sama5d4', 'atmel,sama5']
['laird,dvk-som60', 'laird,som60', 'atmel,sama5d36', 'atmel,sama5d3',
'atmel,sama5']
['laird,gatwick', 'laird,wb50n', 'atmel,sama5d31', 'atmel,sama5d3',
'atmel,sama5']
['laird,wb45n', 'laird,wbxx', 'atmel,at91sam9x5', 'atmel,at91sam9']
['laird,wb50n', 'atmel,sama5d31', 'atmel,sama5d3', 'atmel,sama5']
['linux,spdif-dir']
['microchip,lan9662-udc', 'atmel,sama5d3-udc']
['microchip,mcp16502']
['microchip,mcp23008']
['microchip,mcp2515']
['microchip,sam9x60-aic']
['microchip,sam9x60-can', 'atmel,at91sam9x5-can']
['microchip,sam9x60-ddramc', 'atmel,sama5d3-ddramc']
['microchip,sam9x60-dma', 'atmel,sama5d4-dma']
['microchip,sam9x60-ebi']
['microchip,sam9x60-gpbr', 'atmel,at91sam9260-gpbr', 'syscon']
['microchip,sam9x60-gpio', 'atmel,at91sam9x5-gpio', 'atmel,at91rm9200-gpio']
['microchip,sam9x60-hlcdc']
['microchip,sam9x60-isi', 'atmel,at91sam9g45-isi']
['microchip,sam9x60-matrix', 'atmel,at91sam9x5-matrix', 'syscon']
['microchip,sam9x60-nand-controller']
['microchip,sam9x60-pinctrl', 'atmel,at91sam9x5-pinctrl',
'atmel,at91rm9200-pinctrl', 'simple-bus']
['microchip,sam9x60-pit64b']
['microchip,sam9x60-pmecc', 'atmel,at91sam9g45-pmecc']
['microchip,sam9x60-sdhci']
['microchip,sam9x60-sfr', 'syscon']
['microchip,sam9x60-smc', 'atmel,at91sam9260-smc', 'syscon']
['microchip,sam9x60-tcb', 'atmel,at91sam9x5-tcb', 'simple-mfd', 'syscon']
['microchip,sam9x60-udc']
['microchip,sama5d3-ksz9477-evb', 'atmel,sama5d36', 'atmel,sama5d3',
'atmel,sama5']
['microchip,sama7g5-chipid']
['microchip,sama7g5-ddr3phy']
['microchip,sama7g5-dma']
['microchip,sama7g5-gpbr', 'syscon']
['microchip,sama7g5-pinctrl']
['microchip,sama7g5-pit64b', 'microchip,sam9x60-pit64b']
['microchip,sama7g5-sdhci', 'microchip,sam9x60-sdhci']
['microchip,sama7g5-secumod', 'atmel,sama5d2-secumod', 'syscon']
['microchip,sama7g5-securam', 'atmel,sama5d2-securam', 'mmio-sram']
['microchip,sama7g5-sfrbu', 'atmel,sama5d2-sfrbu', 'syscon']
['microchip,sama7g5-uddrc']
['nxp,tfa9879']
['overkiz,kizbox', 'atmel,at91sam9g20', 'atmel,at91sam9']
['ovti,ov2640']
['phontech,mpa1600', 'atmel,at91rm9200']
['qt1070']
['ronetix,pm9g45', 'atmel,at91sam9g45', 'atmel,at91sam9']
['somfy,animeo-ip', 'atmel,at91sam9260', 'atmel,at91sam9']
['st,m41t94']
['telit,evk-pro3', 'atmel,at91sam9260', 'atmel,at91sam9']
['ti,ads7843']
['ti,pcm5142']
['winstar,wf70gtiagdng0', 'innolux,at070tn92']
['wm8731']
Rob
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v3 00/50] Add support for sam9x7 SoC family
2023-07-28 11:32 ` Krzysztof Kozlowski
@ 2023-07-28 15:50 ` Conor Dooley
2023-07-28 16:04 ` Miquel Raynal
0 siblings, 1 reply; 8+ messages in thread
From: Conor Dooley @ 2023-07-28 15:50 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: Varshini Rajendran, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, claudiu.beznea, mturquette,
sboyd, herbert, davem, vkoul, andi.shyti, tglx, maz, lee,
ulf.hansson, tudor.ambarus, miquel.raynal, richard, vigneshr,
edumazet, kuba, pabeni, linus.walleij, sre, p.zabel, olivia,
a.zummo, radu_nicolae.pirea, richard.genoud, gregkh, lgirdwood,
broonie, wim, linux, linux, durai.manickamkr, andrew, jerry.ray,
andre.przywara, mani, alexandre.torgue, gregory.clement, arnd,
rientjes, deller, 42.hyeyoo, vbabka, mripard, mihai.sain,
codrin.ciubotariu, eugen.hristev, devicetree, linux-arm-kernel,
linux-kernel, linux-clk, linux-crypto, dmaengine, linux-i2c,
linux-mmc, linux-mtd, netdev, linux-gpio, linux-pm, linux-rtc,
linux-spi, linux-serial, alsa-devel, linux-usb, linux-watchdog
[-- Attachment #1: Type: text/plain, Size: 877 bytes --]
On Fri, Jul 28, 2023 at 01:32:12PM +0200, Krzysztof Kozlowski wrote:
> On 28/07/2023 12:22, Varshini Rajendran wrote:
> > This patch series adds support for the new SoC family - sam9x7.
> > - The device tree, configs and drivers are added
> > - Clock driver for sam9x7 is added
> > - Support for basic peripherals is added
> > - Target board SAM9X75 Curiosity is added
> >
>
> Your threading is absolutely broken making it difficult to review and apply.
I had a chat with Varshini today, they were trying to avoid sending the
patches to a massive CC list, but didn't set any in-reply-to header.
For the next submission whole series could be sent to the binding &
platform maintainers and the individual patches additionally to their
respective lists/maintainers. Does that sound okay to you, or do you
think it should be broken up?
Cheers,
Conor.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v3 00/50] Add support for sam9x7 SoC family
2023-07-28 15:50 ` Conor Dooley
@ 2023-07-28 16:04 ` Miquel Raynal
2023-07-28 16:10 ` Conor Dooley
0 siblings, 1 reply; 8+ messages in thread
From: Miquel Raynal @ 2023-07-28 16:04 UTC (permalink / raw)
To: Conor Dooley
Cc: Krzysztof Kozlowski, Varshini Rajendran, robh+dt,
krzysztof.kozlowski+dt, conor+dt, nicolas.ferre,
alexandre.belloni, claudiu.beznea, mturquette, sboyd, herbert,
davem, vkoul, andi.shyti, tglx, maz, lee, ulf.hansson,
tudor.ambarus, richard, vigneshr, edumazet, kuba, pabeni,
linus.walleij, sre, p.zabel, olivia, a.zummo, radu_nicolae.pirea,
richard.genoud, gregkh, lgirdwood, broonie, wim, linux, linux,
durai.manickamkr, andrew, jerry.ray, andre.przywara, mani,
alexandre.torgue, gregory.clement, arnd, rientjes, deller,
42.hyeyoo, vbabka, mripard, mihai.sain, codrin.ciubotariu,
eugen.hristev, devicetree, linux-arm-kernel, linux-kernel,
linux-clk, linux-crypto, dmaengine, linux-i2c, linux-mmc,
linux-mtd, netdev, linux-gpio, linux-pm, linux-rtc, linux-spi,
linux-serial, alsa-devel, linux-usb, linux-watchdog
Hi Conor,
conor@kernel.org wrote on Fri, 28 Jul 2023 16:50:24 +0100:
> On Fri, Jul 28, 2023 at 01:32:12PM +0200, Krzysztof Kozlowski wrote:
> > On 28/07/2023 12:22, Varshini Rajendran wrote:
> > > This patch series adds support for the new SoC family - sam9x7.
> > > - The device tree, configs and drivers are added
> > > - Clock driver for sam9x7 is added
> > > - Support for basic peripherals is added
> > > - Target board SAM9X75 Curiosity is added
> > >
> >
> > Your threading is absolutely broken making it difficult to review and apply.
>
> I had a chat with Varshini today, they were trying to avoid sending the
> patches to a massive CC list, but didn't set any in-reply-to header.
> For the next submission whole series could be sent to the binding &
> platform maintainers and the individual patches additionally to their
> respective lists/maintainers. Does that sound okay to you, or do you
> think it should be broken up?
I usually prefer receiving the dt-bindings *and* the driver changes, so
I can give my feedback on the description side, as well as looking at
the implementation and see if that really matches what was discussed
with you :)
Thanks,
Miquèl
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v3 00/50] Add support for sam9x7 SoC family
2023-07-28 16:04 ` Miquel Raynal
@ 2023-07-28 16:10 ` Conor Dooley
2023-07-28 16:36 ` Krzysztof Kozlowski
0 siblings, 1 reply; 8+ messages in thread
From: Conor Dooley @ 2023-07-28 16:10 UTC (permalink / raw)
To: Miquel Raynal
Cc: Krzysztof Kozlowski, Varshini Rajendran, robh+dt,
krzysztof.kozlowski+dt, conor+dt, nicolas.ferre,
alexandre.belloni, claudiu.beznea, mturquette, sboyd, herbert,
davem, vkoul, andi.shyti, tglx, maz, lee, ulf.hansson,
tudor.ambarus, richard, vigneshr, edumazet, kuba, pabeni,
linus.walleij, sre, p.zabel, olivia, a.zummo, radu_nicolae.pirea,
richard.genoud, gregkh, lgirdwood, broonie, wim, linux, linux,
durai.manickamkr, andrew, jerry.ray, andre.przywara, mani,
alexandre.torgue, gregory.clement, arnd, rientjes, deller,
42.hyeyoo, vbabka, mripard, mihai.sain, codrin.ciubotariu,
eugen.hristev, devicetree, linux-arm-kernel, linux-kernel,
linux-clk, linux-crypto, dmaengine, linux-i2c, linux-mmc,
linux-mtd, netdev, linux-gpio, linux-pm, linux-rtc, linux-spi,
linux-serial, alsa-devel, linux-usb, linux-watchdog
[-- Attachment #1: Type: text/plain, Size: 1508 bytes --]
On Fri, Jul 28, 2023 at 06:04:43PM +0200, Miquel Raynal wrote:
> Hi Conor,
>
> conor@kernel.org wrote on Fri, 28 Jul 2023 16:50:24 +0100:
>
> > On Fri, Jul 28, 2023 at 01:32:12PM +0200, Krzysztof Kozlowski wrote:
> > > On 28/07/2023 12:22, Varshini Rajendran wrote:
> > > > This patch series adds support for the new SoC family - sam9x7.
> > > > - The device tree, configs and drivers are added
> > > > - Clock driver for sam9x7 is added
> > > > - Support for basic peripherals is added
> > > > - Target board SAM9X75 Curiosity is added
> > > >
> > >
> > > Your threading is absolutely broken making it difficult to review and apply.
> >
> > I had a chat with Varshini today, they were trying to avoid sending the
> > patches to a massive CC list, but didn't set any in-reply-to header.
> > For the next submission whole series could be sent to the binding &
> > platform maintainers and the individual patches additionally to their
> > respective lists/maintainers. Does that sound okay to you, or do you
> > think it should be broken up?
>
> I usually prefer receiving the dt-bindings *and* the driver changes, so
> I can give my feedback on the description side, as well as looking at
> the implementation and see if that really matches what was discussed
> with you :)
Right, that is what I was suggesting. Respective maintainers would get
the drivers *and* bindings for their subsystems - IOW, each patch is
sent to what get_maintainer.pl outputs for it.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v3 00/50] Add support for sam9x7 SoC family
2023-07-28 16:10 ` Conor Dooley
@ 2023-07-28 16:36 ` Krzysztof Kozlowski
0 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2023-07-28 16:36 UTC (permalink / raw)
To: Conor Dooley, Miquel Raynal
Cc: Varshini Rajendran, robh+dt, krzysztof.kozlowski+dt, conor+dt,
nicolas.ferre, alexandre.belloni, claudiu.beznea, mturquette,
sboyd, herbert, davem, vkoul, andi.shyti, tglx, maz, lee,
ulf.hansson, tudor.ambarus, richard, vigneshr, edumazet, kuba,
pabeni, linus.walleij, sre, p.zabel, olivia, a.zummo,
radu_nicolae.pirea, richard.genoud, gregkh, lgirdwood, broonie,
wim, linux, linux, durai.manickamkr, andrew, jerry.ray,
andre.przywara, mani, alexandre.torgue, gregory.clement, arnd,
rientjes, deller, 42.hyeyoo, vbabka, mripard, mihai.sain,
codrin.ciubotariu, eugen.hristev, devicetree, linux-arm-kernel,
linux-kernel, linux-clk, linux-crypto, dmaengine, linux-i2c,
linux-mmc, linux-mtd, netdev, linux-gpio, linux-pm, linux-rtc,
linux-spi, linux-serial, alsa-devel, linux-usb, linux-watchdog
On 28/07/2023 18:10, Conor Dooley wrote:
> On Fri, Jul 28, 2023 at 06:04:43PM +0200, Miquel Raynal wrote:
>> Hi Conor,
>>
>> conor@kernel.org wrote on Fri, 28 Jul 2023 16:50:24 +0100:
>>
>>> On Fri, Jul 28, 2023 at 01:32:12PM +0200, Krzysztof Kozlowski wrote:
>>>> On 28/07/2023 12:22, Varshini Rajendran wrote:
>>>>> This patch series adds support for the new SoC family - sam9x7.
>>>>> - The device tree, configs and drivers are added
>>>>> - Clock driver for sam9x7 is added
>>>>> - Support for basic peripherals is added
>>>>> - Target board SAM9X75 Curiosity is added
>>>>>
>>>>
>>>> Your threading is absolutely broken making it difficult to review and apply.
>>>
>>> I had a chat with Varshini today, they were trying to avoid sending the
>>> patches to a massive CC list, but didn't set any in-reply-to header.
>>> For the next submission whole series could be sent to the binding &
>>> platform maintainers and the individual patches additionally to their
>>> respective lists/maintainers. Does that sound okay to you, or do you
>>> think it should be broken up?
>>
>> I usually prefer receiving the dt-bindings *and* the driver changes, so
>> I can give my feedback on the description side, as well as looking at
>> the implementation and see if that really matches what was discussed
>> with you :)
>
> Right, that is what I was suggesting. Respective maintainers would get
> the drivers *and* bindings for their subsystems - IOW, each patch is
> sent to what get_maintainer.pl outputs for it.
For reviewers I find the easiest if this is mostly split per subsystem.
There were here few patches for USB, few clk etc, so these easily can be
separate patchsets. All the rest one-liners or one-patch-per-subsystem
could be grouped and set in one patchset, after fixing the threading.
But the moment the patchset grows to 50 it's time to re-think it whether
this grouping is necessary or even beneficial.
This is not a conversion of mach to DT (like ep93xx) which benefits of
doing everything in one step. Therefore my recommendation for this work
is to split it entirely per each subsystem.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-07-28 16:37 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-28 10:22 [PATCH v3 00/50] Add support for sam9x7 SoC family Varshini Rajendran
2023-07-28 11:32 ` Krzysztof Kozlowski
2023-07-28 15:50 ` Conor Dooley
2023-07-28 16:04 ` Miquel Raynal
2023-07-28 16:10 ` Conor Dooley
2023-07-28 16:36 ` Krzysztof Kozlowski
2023-07-28 11:32 ` Krzysztof Kozlowski
2023-07-28 14:46 ` Rob Herring
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).