[PATCH 0/8] arm64: dts: ti: j7: Add device-tree nodes for MCSPI

Vaishnav Achath posted 8 patches 2 years, 10 months ago
There is a newer version of this series
arch/arm64/boot/dts/ti/k3-j7200-main.dtsi     | 88 +++++++++++++++++++
.../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi      | 33 +++++++
arch/arm64/boot/dts/ti/k3-j721e-main.dtsi     | 88 +++++++++++++++++++
.../boot/dts/ti/k3-j721e-mcu-wakeup.dtsi      | 33 +++++++
arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi    | 88 +++++++++++++++++++
.../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi     | 33 +++++++
arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi    | 88 +++++++++++++++++++
.../boot/dts/ti/k3-j784s4-mcu-wakeup.dtsi     | 33 +++++++
8 files changed, 484 insertions(+)
[PATCH 0/8] arm64: dts: ti: j7: Add device-tree nodes for MCSPI
Posted by Vaishnav Achath 2 years, 10 months ago
This series adds device tree nodes for Multi Channel Serial Peripheral
Interface (MCSPI) on J721E, J7200, J721S2 and J784S4 platforms. All the
MCSPI instances are disabled by default and can be enabled through overlays
as required, the changes were tested using spidev loopback test for all
instances and the data verified only for main_spi4 which is connected
internally as slave to mcu_spi2 for all existing J7 devices.

Vaishnav Achath (8):
  arm64: dts: ti: k3-j721e-main: Add MCSPI nodes
  arm64: dts: ti: k3-j721e-mcu-wakeup: Add MCSPI nodes
  arm64: dts: ti: k3-j7200-main: Add MCSPI nodes
  arm64: dts: ti: k3-j7200-mcu-wakeup: Add MCSPI nodes
  arm64: dts: ti: k3-j721s2-main: Add MCSPI nodes
  arm64: dts: ti: k3-j721s2-mcu-wakeup: Add MCSPI nodes
  arm64: dts: ti: k3-j784s4-main: Add MCSPI nodes
  arm64: dts: ti: k3-j784s4-mcu-wakeup: Add MCSPI nodes

 arch/arm64/boot/dts/ti/k3-j7200-main.dtsi     | 88 +++++++++++++++++++
 .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi      | 33 +++++++
 arch/arm64/boot/dts/ti/k3-j721e-main.dtsi     | 88 +++++++++++++++++++
 .../boot/dts/ti/k3-j721e-mcu-wakeup.dtsi      | 33 +++++++
 arch/arm64/boot/dts/ti/k3-j721s2-main.dtsi    | 88 +++++++++++++++++++
 .../boot/dts/ti/k3-j721s2-mcu-wakeup.dtsi     | 33 +++++++
 arch/arm64/boot/dts/ti/k3-j784s4-main.dtsi    | 88 +++++++++++++++++++
 .../boot/dts/ti/k3-j784s4-mcu-wakeup.dtsi     | 33 +++++++
 8 files changed, 484 insertions(+)

-- 
2.17.1
Re: [PATCH 0/8] arm64: dts: ti: j7: Add device-tree nodes for MCSPI
Posted by Nishanth Menon 2 years, 10 months ago
On 11:00-20230320, Vaishnav Achath wrote:
> This series adds device tree nodes for Multi Channel Serial Peripheral
> Interface (MCSPI) on J721E, J7200, J721S2 and J784S4 platforms. All the
> MCSPI instances are disabled by default and can be enabled through overlays
> as required, the changes were tested using spidev loopback test for all
> instances and the data verified only for main_spi4 which is connected
> internally as slave to mcu_spi2 for all existing J7 devices.
> 
> Vaishnav Achath (8):
>   arm64: dts: ti: k3-j721e-main: Add MCSPI nodes
>   arm64: dts: ti: k3-j721e-mcu-wakeup: Add MCSPI nodes
>   arm64: dts: ti: k3-j7200-main: Add MCSPI nodes
>   arm64: dts: ti: k3-j7200-mcu-wakeup: Add MCSPI nodes
>   arm64: dts: ti: k3-j721s2-main: Add MCSPI nodes
>   arm64: dts: ti: k3-j721s2-mcu-wakeup: Add MCSPI nodes
>   arm64: dts: ti: k3-j784s4-main: Add MCSPI nodes
>   arm64: dts: ti: k3-j784s4-mcu-wakeup: Add MCSPI nodes

Why split the patches per bus? why not club the patches at SoC level?

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D
Re: [PATCH 0/8] arm64: dts: ti: j7: Add device-tree nodes for MCSPI
Posted by Vaishnav Achath 2 years, 10 months ago
Hi Nishanth,

On 20/03/23 20:46, Nishanth Menon wrote:
> On 11:00-20230320, Vaishnav Achath wrote:
>> This series adds device tree nodes for Multi Channel Serial Peripheral
>> Interface (MCSPI) on J721E, J7200, J721S2 and J784S4 platforms. All the
>> MCSPI instances are disabled by default and can be enabled through overlays
>> as required, the changes were tested using spidev loopback test for all
>> instances and the data verified only for main_spi4 which is connected
>> internally as slave to mcu_spi2 for all existing J7 devices.
>>
>> Vaishnav Achath (8):
>>   arm64: dts: ti: k3-j721e-main: Add MCSPI nodes
>>   arm64: dts: ti: k3-j721e-mcu-wakeup: Add MCSPI nodes
>>   arm64: dts: ti: k3-j7200-main: Add MCSPI nodes
>>   arm64: dts: ti: k3-j7200-mcu-wakeup: Add MCSPI nodes
>>   arm64: dts: ti: k3-j721s2-main: Add MCSPI nodes
>>   arm64: dts: ti: k3-j721s2-mcu-wakeup: Add MCSPI nodes
>>   arm64: dts: ti: k3-j784s4-main: Add MCSPI nodes
>>   arm64: dts: ti: k3-j784s4-mcu-wakeup: Add MCSPI nodes
> 
> Why split the patches per bus? why not club the patches at SoC level?
> 

I split them initially since the changes were in two different files, I have
combined the changes at SoC level and send a V2,

https://lore.kernel.org/all/20230321065447.6997-1-vaishnav.a@ti.com/#r

-- 
Regards,
Vaishnav