Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753253AbcJJQCm (ORCPT ); Mon, 10 Oct 2016 12:02:42 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:26860 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752286AbcJJQCk (ORCPT ); Mon, 10 Oct 2016 12:02:40 -0400 Subject: Re: [PATCH RESEND] ARM: dts: keystone-k2*: Increase SPI Flash partition size for U-Boot To: Vignesh R References: <20161010141141.21333-1-vigneshr@ti.com> <20161010143144.GB1041@n2100.armlinux.org.uk> Cc: Russell King - ARM Linux , Santosh Shilimkar , devicetree@vger.kernel.org, Rob Herring , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org From: Santosh Shilimkar Organization: Oracle Corporation Message-ID: Date: Mon, 10 Oct 2016 09:01:58 -0700 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <20161010143144.GB1041@n2100.armlinux.org.uk> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-Source-IP: aserv0022.oracle.com [141.146.126.234] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1106 Lines: 32 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. > > 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.