Re: [PATCH v2 02/18] arm64: dts: ti: k3-j721e-main: Add multiple channels for CSI2RX DMA

Hi Udit,
Thanks for the review.

On 27/07/26 09:43, Kumar, Udit wrote:
> 
> 
> On 7/13/2026 5:04 PM, Yemike Abhilash Chandra wrote:
>> From: Vaishnav Achath <[email protected]>
>>
>> J721E CSI2RX SHIM layer can support up to 32 DMA channel contexts. Add
>> additional DMA channels to enable multistream support for CSI2RX.
>>
> 
> If hardware supports 32 channel, then why you are limiting to 16
> channels only ?

While the J721E CSI2RX SHIM IP supports up to 32 DMA channel contexts, in
the current RM configuration, only 16[1] BCDMA RX channels are allocated
to the Linux host for CSI2RX.

Hence added channels accordingly.

Thanks and Regards,
Yemike Abhilash Chandra

[1]: 
https://github.com/u-boot/u-boot/blob/main/board/ti/j721e/rm-cfg.yaml#L1205

>> Signed-off-by: Vaishnav Achath <[email protected]>
>> Signed-off-by: Yemike Abhilash Chandra <[email protected]>
>> ---
>>   arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 20 ++++++++++++++++----
>>   1 file changed, 16 insertions(+), 4 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
>> index 5a8414fc5751..b1988437f52d 100644
>> --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
>> +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi
>> @@ -600,8 +600,14 @@ ti_csi2rx0: ticsi2rx@4500000 {
>>   		ranges;
>>   		#address-cells = <2>;
>>   		#size-cells = <2>;
>> -		dmas = <&main_udmap 0x4940>;
>> -		dma-names = "rx0";
>> +		dmas = <&main_udmap 0x4940>, <&main_udmap 0x4941>, <&main_udmap 0x4942>,
>> +		       <&main_udmap 0x4943>, <&main_udmap 0x4944>, <&main_udmap 0x4945>,
>> +		       <&main_udmap 0x4946>, <&main_udmap 0x4947>, <&main_udmap 0x4948>,
>> +		       <&main_udmap 0x4949>, <&main_udmap 0x494a>, <&main_udmap 0x494b>,
>> +		       <&main_udmap 0x494c>, <&main_udmap 0x494d>, <&main_udmap 0x494e>,
>> +		       <&main_udmap 0x494f>;
>> +		dma-names = "rx0", "rx1", "rx2", "rx3", "rx4", "rx5", "rx6", "rx7",
>> +			    "rx8", "rx9", "rx10", "rx11", "rx12", "rx13", "rx14", "rx15";
>>   		power-domains = <&k3_pds 26 TI_SCI_PD_EXCLUSIVE>;
>>   		status = "disabled";
>>   
>> @@ -656,8 +662,14 @@ ti_csi2rx1: ticsi2rx@4510000 {
>>   		ranges;
>>   		#address-cells = <2>;
>>   		#size-cells = <2>;
>> -		dmas = <&main_udmap 0x4960>;
>> -		dma-names = "rx0";
>> +		dmas = <&main_udmap 0x4960>, <&main_udmap 0x4961>, <&main_udmap 0x4962>,
>> +		       <&main_udmap 0x4963>, <&main_udmap 0x4964>, <&main_udmap 0x4965>,
>> +		       <&main_udmap 0x4966>, <&main_udmap 0x4967>, <&main_udmap 0x4968>,
>> +		       <&main_udmap 0x4969>, <&main_udmap 0x496a>, <&main_udmap 0x496b>,
>> +		       <&main_udmap 0x496c>, <&main_udmap 0x496d>, <&main_udmap 0x496e>,
>> +		       <&main_udmap 0x496f>;
>> +		dma-names = "rx0", "rx1", "rx2", "rx3", "rx4", "rx5", "rx6", "rx7",
>> +			    "rx8", "rx9", "rx10", "rx11", "rx12", "rx13", "rx14", "rx15";
>>   		power-domains = <&k3_pds 27 TI_SCI_PD_EXCLUSIVE>;
>>   		status = "disabled";
>>   
>