Re: [PATCH RESEND] ARM: dts: keystone-k2*: Increase SPI Flash partition size for U-Boot

From: Santosh Shilimkar
Date: Mon Oct 10 2016 - 12:02:44 EST


Vignesh,

On 10/10/2016 7:31 AM, Russell King - ARM Linux wrote:
On Mon, Oct 10, 2016 at 07:41:41PM +0530, Vignesh R wrote:
U-Boot SPI Boot image is now more than 512KB for Keystone2 devices and
cannot fit into existing partition. So, increase the SPI Flash partition
for U-Boot to 1MB for all Keystone2 devices.

Signed-off-by: Vignesh R <vigneshr@xxxxxx>
---

This was submitted to v4.9 merge window but was never picked up:
https://patchwork.kernel.org/patch/9135023/

Another point is, if you want me to pick your patch, please copy
me next time :-). AFAIK, am seeing this patch in my inbox first time.


I think you need to explain why it's safe to change the layout of the
flash partitions like this.

- What is this "misc" partition?

- Why is it safe to move the "misc" partition in this way?

- Do users need to do anything with data stored in the "misc" partition
when changing kernels?

If the "misc" partition is simply unused space on the flash device, why
list it in DT?

Thanks Russell. Yes, above clarification would be good to get first.