Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752386AbdLLSlN (ORCPT ); Tue, 12 Dec 2017 13:41:13 -0500 Received: from eddie.linux-mips.org ([148.251.95.138]:41464 "EHLO cvs.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751643AbdLLSlI (ORCPT ); Tue, 12 Dec 2017 13:41:08 -0500 Date: Tue, 12 Dec 2017 19:40:52 +0100 From: Ladislav Michl To: "Derald D. Woods" Cc: Tony Lindgren , linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information Message-ID: <20171212184052.GA11423@lenoch> References: <20171212041213.27436-1-woods.technical@gmail.com> <20171212063930.GA2791@lenoch> <20171212163125.GA8032@DeraldWoods-PC.wicab.com> <20171212165542.GC14441@atomide.com> <20171212175054.GA10337@lenoch> <20171212180827.GA7404@DeraldWoods-PC.wicab.com> <20171212181503.GE14441@atomide.com> <20171212182401.GB7404@DeraldWoods-PC.wicab.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20171212182401.GB7404@DeraldWoods-PC.wicab.com> User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 953 Lines: 19 On Tue, Dec 12, 2017 at 12:24:02PM -0600, Derald D. Woods wrote: > On Tue, Dec 12, 2017 at 10:15:03AM -0800, Tony Lindgren wrote: > > Well that's good to hear :) My only concern with your patch is what > > happens if somebody boots with older u-boot with different partition > > sizes? > > I agree. The 'bootargs' mechanisms have seen some recent changes that > may be a factor in what I am seeing. I had to include the command line > in my config to test some NAND partitioning schemes and UBI. I am > learning and Hopefully fixing some things as I go. u-boot/board/isee/igep00x0/igep00x0.c is using only two MTD partitions, runtime generated. As OMAP's BootROM tries to read from atmost first four sectors before it gives up, size of SPL partiton is runtime generated according to NAND sector size. The rest of NAND is UBI managed. This of course breaks backward compatibility, but is unlikely to break in future. Someone needs to decide :) ladis