mbox series

[v2,0/2] arm64: dts: ti: k3-am62: Add MCU MCAN

Message ID 20230802073635.11290-1-francesco@dolcini.it (mailing list archive)
Headers show
Series arm64: dts: ti: k3-am62: Add MCU MCAN | expand

Message

Francesco Dolcini Aug. 2, 2023, 7:36 a.m. UTC
From: Francesco Dolcini <francesco.dolcini@toradex.com>

On AM62x there are no hardware interrupts routed to A53 GIC
interrupt controller for MCU MCAN IPs, so MCU MCAN nodes were
omitted from MCU dtsi.

Timer polling was introduced in commits [1][2] so now add MCU MCAN nodes
to the MCU dtsi for Cortex A53.

[1] commit b382380c0d2d ("can: m_can: Add hrtimer to generate software interrupt")
[2] commit bb410c03b999 ("dt-bindings: net: can: Remove interrupt properties for MCAN")

Once the MCU MCANs are added to the SOC dtsi, enable the Verdin CAN2.

v1..v2:
 - fixed can node name
 - added commit prefix before sha in commit message

Hiago De Franco (1):
  arm64: dts: ti: k3-am625-verdin: enable CAN_2

Judith Mendez (1):
  arm64: dts: ti: k3-am62: Add MCU MCAN nodes

 arch/arm64/boot/dts/ti/k3-am62-mcu.dtsi       | 24 +++++++++++++++++++
 .../boot/dts/ti/k3-am62-verdin-dahlia.dtsi    |  5 ++++
 .../arm64/boot/dts/ti/k3-am62-verdin-dev.dtsi |  5 ++++
 .../boot/dts/ti/k3-am62-verdin-yavia.dtsi     |  5 ++++
 arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi    | 19 +++++++++++++--
 5 files changed, 56 insertions(+), 2 deletions(-)

Comments

Nishanth Menon Aug. 5, 2023, 7 p.m. UTC | #1
Hi Francesco Dolcini,

On Wed, 02 Aug 2023 09:36:33 +0200, Francesco Dolcini wrote:
> From: Francesco Dolcini <francesco.dolcini@toradex.com>
> 
> On AM62x there are no hardware interrupts routed to A53 GIC
> interrupt controller for MCU MCAN IPs, so MCU MCAN nodes were
> omitted from MCU dtsi.
> 
> Timer polling was introduced in commits [1][2] so now add MCU MCAN nodes
> to the MCU dtsi for Cortex A53.

I understand the dependency, it does create a dtbs_check warnings on
rc1, but I am picking this up since the yaml fixup is in next already
and not a new compatible addition. if the fixups does'nt end up in
linus-master for 6.6-rc1, we might need to look at what we need to do.
Please let me know if there is any concerns and I can drop this series
from my tree.

> 
> [...]

I have applied the following to branch ti-k3-dts-next on [1].
Thank you!

[1/2] arm64: dts: ti: k3-am62: Add MCU MCAN nodes
      commit: 108f61e03963a3a019486fd80cc84d5db5c204d0
[2/2] arm64: dts: ti: k3-am625-verdin: enable CAN_2
      commit: 7480cea33b4c36aeffe592eb6f151bc9974fee70

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git