Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753154AbcJLLLD (ORCPT ); Wed, 12 Oct 2016 07:11:03 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:43263 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752060AbcJLLLB (ORCPT ); Wed, 12 Oct 2016 07:11:01 -0400 Subject: Re: [PATCH RESEND] ARM: dts: keystone-k2*: Increase SPI Flash partition size for U-Boot To: Santosh Shilimkar References: <20161010141141.21333-1-vigneshr@ti.com> <20161010143144.GB1041@n2100.armlinux.org.uk> CC: Russell King - ARM Linux , Santosh Shilimkar , , Rob Herring , , From: Vignesh R Message-ID: <71299871-965f-aef3-7301-4d587430a1a5@ti.com> Date: Wed, 12 Oct 2016 16:40:20 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1389 Lines: 46 Hi, On Monday 10 October 2016 09:31 PM, Santosh Shilimkar wrote: > 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 >>> --- >>> >>> 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. > Sorry, I did address the previous patch to you. Not sure what happened :( >> >> 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. Ok, will send v2 with updated commit message as per my reply in other thread. -- Regards Vignesh