mbox series

[v2,0/3] Fix reg ranges for main_pktdma dma-controller node

Message ID 20240430105253.203750-1-j-choudhary@ti.com (mailing list archive)
Headers show
Series Fix reg ranges for main_pktdma dma-controller node | expand

Message

Jayesh Choudhary April 30, 2024, 10:52 a.m. UTC
The dma-controller node 'main_pktdma' has few memory regions with
wrong sizes.

DMASS0_PKTDMA_RINGRT is marked as 4MB region when it is actually a 2MB
region. Similarly, DMASS0_PKTDMA_TCHANRT is marked as 256KB region but
the actual size is 128KB as shown in TRM in the section for Main Memory
Map (Table 2-1)

Fix these region across AM62, AM62A and AM62P (which is also used in
J722S)

TRM:

AM625: <https://www.ti.com/lit/pdf/spruiv7>
AM62A7: <https://www.ti.com/lit/pdf/spruj16>
AM62P: <https://www.ti.com/lit/pdf/spruj83>
J722S: <https://www.ti.com/lit/zip/sprujb3>

Changelog v1->v2:
- Add main_pktdma node name in commit message for more clarity about the
  dma-controller and mention the table for memory map in TRM in each patch.

v1 patch:
<https://lore.kernel.org/all/20240405085208.32227-1-j-choudhary@ti.com/>

Jayesh Choudhary (3):
  arm64: dts: ti: k3-am62-main: Fix the reg-range for main_pktdma
  arm64: dts: ti: k3-am62a-main: Fix the reg-range for main_pktdma
  arm64: dts: ti: k3-am62p-main: Fix the reg-range for main_pktdma

 arch/arm64/boot/dts/ti/k3-am62-main.dtsi  | 4 ++--
 arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 4 ++--
 arch/arm64/boot/dts/ti/k3-am62p-main.dtsi | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

Comments

Kumar, Udit May 2, 2024, 10:20 a.m. UTC | #1
Thanks Jayesh

On 4/30/2024 4:22 PM, Jayesh Choudhary wrote:
> The dma-controller node 'main_pktdma' has few memory regions with
> wrong sizes.
>
> DMASS0_PKTDMA_RINGRT is marked as 4MB region when it is actually a 2MB
> region. Similarly, DMASS0_PKTDMA_TCHANRT is marked as 256KB region but
> the actual size is 128KB as shown in TRM in the section for Main Memory
> Map (Table 2-1)
>
> Fix these region across AM62, AM62A and AM62P (which is also used in
> J722S)
>
> TRM:


For series

Reviewed-by: Udit Kumar <u-kumar1@ti.com>


>
> AM625: <https://www.ti.com/lit/pdf/spruiv7>
> AM62A7: <https://www.ti.com/lit/pdf/spruj16>
> AM62P: <https://www.ti.com/lit/pdf/spruj83>
> J722S: <https://www.ti.com/lit/zip/sprujb3>
>
> Changelog v1->v2:
> - Add main_pktdma node name in commit message for more clarity about the
>    dma-controller and mention the table for memory map in TRM in each patch.
>
> v1 patch:
> <https://lore.kernel.org/all/20240405085208.32227-1-j-choudhary@ti.com/>
>
> Jayesh Choudhary (3):
>    arm64: dts: ti: k3-am62-main: Fix the reg-range for main_pktdma
>    arm64: dts: ti: k3-am62a-main: Fix the reg-range for main_pktdma
>    arm64: dts: ti: k3-am62p-main: Fix the reg-range for main_pktdma
>
>   arch/arm64/boot/dts/ti/k3-am62-main.dtsi  | 4 ++--
>   arch/arm64/boot/dts/ti/k3-am62a-main.dtsi | 4 ++--
>   arch/arm64/boot/dts/ti/k3-am62p-main.dtsi | 4 ++--
>   3 files changed, 6 insertions(+), 6 deletions(-)
>
Vignesh Raghavendra June 13, 2024, 6:31 a.m. UTC | #2
Hi Jayesh Choudhary,

On Tue, 30 Apr 2024 16:22:50 +0530, Jayesh Choudhary wrote:
> The dma-controller node 'main_pktdma' has few memory regions with
> wrong sizes.
> 
> DMASS0_PKTDMA_RINGRT is marked as 4MB region when it is actually a 2MB
> region. Similarly, DMASS0_PKTDMA_TCHANRT is marked as 256KB region but
> the actual size is 128KB as shown in TRM in the section for Main Memory
> Map (Table 2-1)
> 
> [...]

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

[1/3] arm64: dts: ti: k3-am62-main: Fix the reg-range for main_pktdma
      commit: 6edad223553c7f1680fcaca25ded59eba7c6d82d
[2/3] arm64: dts: ti: k3-am62a-main: Fix the reg-range for main_pktdma
      commit: d007a883a61f55b9b195c4c18bbe29de5b802822
[3/3] arm64: dts: ti: k3-am62p-main: Fix the reg-range for main_pktdma
      commit: a6e6604c600aeedf9700de4a55255850391bc3fc

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
--
Vignesh