Hi Nishanth, On 03/05/23 17:21, Nishanth Menon wrote: > On 14:42-20230503, Vaishnav Achath wrote: >> This series adds flash partition description through DT, currently >> the flash partitions for J7 platforms is described through the mtdparts >> commandline parameter passed from bootloader, this requires maintenance >> of the partition information in a mtdparts string which is error prone. >> >> Once the flash partitions are described through DT, the support for >> mtdparts can be removed for these platforms from u-boot also. >> >> Vaishnav Achath (4): >> arm64: dts: ti: k3-j721e-som: Describe OSPI flash partition info >> arm64: dts: ti: k3-j721e-som: Describe QSPI flash partition info >> arm64: dts: ti: k3-j721e-sk: Describe OSPI flash partition info >> arm64: dts: ti: k3-j7200-som: Describe OSPI and Hyperflash partition >> info >> >> arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi | 72 +++++++++++++++++++ >> .../dts/ti/k3-j721e-common-proc-board.dts | 46 ++++++++++++ >> arch/arm64/boot/dts/ti/k3-j721e-sk.dts | 46 ++++++++++++ >> arch/arm64/boot/dts/ti/k3-j721e-som-p0.dtsi | 46 ++++++++++++ >> 4 files changed, 210 insertions(+) > > Please address the feedback in [1] prior to adding the partitions? and > could you ensure it is done for all the impacted k3 SoCs? > I will do both of these in corresponding next revisions. Thanks and Regards, Vaishnav > > [1] https://lore.kernel.org/all/f5861b8b-637a-e127-2945-8994a472015a@xxxxxx/ >