Re: [PATCH 0/4] arm64: dts: ti: Describe flash partitions for J7 platforms

[Vaishnav Achath]

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