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@xxxxxx/#r -- Regards, Vaishnav