mbox series

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

Message ID 20230503091218.25899-1-vaishnav.a@ti.com (mailing list archive)
Headers show
Series arm64: dts: ti: Describe flash partitions for J7 platforms | expand

Message

Vaishnav Achath May 3, 2023, 9:12 a.m. UTC
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(+)

Comments

Nishanth Menon May 3, 2023, 11:51 a.m. UTC | #1
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?


[1] https://lore.kernel.org/all/f5861b8b-637a-e127-2945-8994a472015a@ti.com/
Vaishnav Achath May 4, 2023, 5:39 a.m. UTC | #2
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@ti.com/
>