mbox series

[0/8] Add System Components for Microchip SAMA7D65 SoC

Message ID cover.1739555984.git.Ryan.Wanner@microchip.com (mailing list archive)
Headers show
Series Add System Components for Microchip SAMA7D65 SoC | expand

Message

Ryan Wanner Feb. 14, 2025, 6:08 p.m. UTC
From: Ryan Wanner <Ryan.Wanner@microchip.com>

This patch set adds support for the following systems in the SAMA7D65
SoC:
- DMAs
- Chip ID
- Dual watchdog timer.

Ryan Wanner (8):
  dt-bindings: atmel-sysreg: Add SAMA7D65 Chip ID
  dt-bindings: watchdog: sama5d4-wdt: Add sama7d65-wdt
  dt-bindings: dma: atmel: add microchip,sama7d65-dma
  ARM: at91: Add Support in SoC driver for SAMA7D65
  ARM: dts: microchip: sama7d65: Add chipID for sama7d65
  ARM: dts: microchip: sama7d65: Add watchdog for sama7d65
  ARM: dts: microchip: sama7d65: Add DMAs to sama7d65 SoC
  ARM: dts: microchip: sama7d65: Enable DMAs

 .../devicetree/bindings/arm/atmel-sysregs.txt |  1 +
 .../bindings/dma/atmel,sama5d4-dma.yaml       |  3 ++
 .../bindings/watchdog/atmel,sama5d4-wdt.yaml  |  3 ++
 .../dts/microchip/at91-sama7d65_curiosity.dts | 12 +++++
 arch/arm/boot/dts/microchip/sama7d65.dtsi     | 44 +++++++++++++++++++
 drivers/soc/atmel/soc.c                       |  5 +++
 drivers/soc/atmel/soc.h                       |  3 ++
 7 files changed, 71 insertions(+)

Comments

claudiu beznea (tuxon) Feb. 18, 2025, 8:45 a.m. UTC | #1
Hi, Ryan,

On 14.02.2025 20:08, Ryan.Wanner@microchip.com wrote:
> From: Ryan Wanner <Ryan.Wanner@microchip.com>
> 
> This patch set adds support for the following systems in the SAMA7D65
> SoC:
> - DMAs
> - Chip ID
> - Dual watchdog timer.
> 
> Ryan Wanner (8):
>   dt-bindings: atmel-sysreg: Add SAMA7D65 Chip ID
>   dt-bindings: watchdog: sama5d4-wdt: Add sama7d65-wdt
>   dt-bindings: dma: atmel: add microchip,sama7d65-dma
>   ARM: at91: Add Support in SoC driver for SAMA7D65
>   ARM: dts: microchip: sama7d65: Add chipID for sama7d65
>   ARM: dts: microchip: sama7d65: Add watchdog for sama7d65
>   ARM: dts: microchip: sama7d65: Add DMAs to sama7d65 SoC
>   ARM: dts: microchip: sama7d65: Enable DMAs

Series looks good to me. I'm waiting to see if there are comments on
bindings before applying the DT part.

Thank you,
Claudiu
Rob Herring (Arm) Feb. 19, 2025, 12:42 a.m. UTC | #2
On Fri, 14 Feb 2025 11:08:13 -0700, Ryan.Wanner@microchip.com wrote:
> From: Ryan Wanner <Ryan.Wanner@microchip.com>
> 
> This patch set adds support for the following systems in the SAMA7D65
> SoC:
> - DMAs
> - Chip ID
> - Dual watchdog timer.
> 
> Ryan Wanner (8):
>   dt-bindings: atmel-sysreg: Add SAMA7D65 Chip ID
>   dt-bindings: watchdog: sama5d4-wdt: Add sama7d65-wdt
>   dt-bindings: dma: atmel: add microchip,sama7d65-dma
>   ARM: at91: Add Support in SoC driver for SAMA7D65
>   ARM: dts: microchip: sama7d65: Add chipID for sama7d65
>   ARM: dts: microchip: sama7d65: Add watchdog for sama7d65
>   ARM: dts: microchip: sama7d65: Add DMAs to sama7d65 SoC
>   ARM: dts: microchip: sama7d65: Enable DMAs
> 
>  .../devicetree/bindings/arm/atmel-sysregs.txt |  1 +
>  .../bindings/dma/atmel,sama5d4-dma.yaml       |  3 ++
>  .../bindings/watchdog/atmel,sama5d4-wdt.yaml  |  3 ++
>  .../dts/microchip/at91-sama7d65_curiosity.dts | 12 +++++
>  arch/arm/boot/dts/microchip/sama7d65.dtsi     | 44 +++++++++++++++++++
>  drivers/soc/atmel/soc.c                       |  5 +++
>  drivers/soc/atmel/soc.h                       |  3 ++
>  7 files changed, 71 insertions(+)
> 
> --
> 2.43.0
> 
> 
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y for arch/arm/boot/dts/microchip/' for cover.1739555984.git.Ryan.Wanner@microchip.com:

arch/arm/boot/dts/microchip/at91-sama7d65_curiosity.dtb: watchdog@e001d180: Unevaluated properties are not allowed ('clocks' was unexpected)
	from schema $id: http://devicetree.org/schemas/watchdog/atmel,sama5d4-wdt.yaml#
arch/arm/boot/dts/microchip/at91-sama7d65_curiosity.dtb: /soc/chipid@e0020000: failed to match any schema with compatible: ['microchip,sama7d65-chipid']