Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-13 Thread Derald D. Woods
On Wed, Dec 13, 2017 at 04:05:06PM +0100, Ladislav Michl wrote: > On Wed, Dec 13, 2017 at 07:44:25AM -0600, Derald D. Woods wrote: > > On Tue, Dec 12, 2017 at 06:13:51PM -0600, Adam Ford wrote: > > > On Dec 12, 2017 12:41 PM, "Ladislav Michl" wrote: > > > > > > On Tue, Dec 12, 2017 at 12:24:02PM

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-13 Thread Ladislav Michl
On Wed, Dec 13, 2017 at 07:44:25AM -0600, Derald D. Woods wrote: > On Tue, Dec 12, 2017 at 06:13:51PM -0600, Adam Ford wrote: > > On Dec 12, 2017 12:41 PM, "Ladislav Michl" wrote: > > > > On Tue, Dec 12, 2017 at 12:24:02PM -0600, Derald D. Woods wrote: > > > On Tue, Dec 12, 2017 at 10:15:03AM -08

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-13 Thread Derald D. Woods
On Tue, Dec 12, 2017 at 06:13:51PM -0600, Adam Ford wrote: > On Dec 12, 2017 12:41 PM, "Ladislav Michl" wrote: > > 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

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-12 Thread Ladislav Michl
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.

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-12 Thread Derald D. Woods
On Tue, Dec 12, 2017 at 10:15:03AM -0800, Tony Lindgren wrote: > * Derald D. Woods [171212 18:11]: > > On Tue, Dec 12, 2017 at 06:50:54PM +0100, Ladislav Michl wrote: > > > On Tue, Dec 12, 2017 at 08:55:42AM -0800, Tony Lindgren wrote: > > > > * Derald D. Woods [171212 16:34]: > > > > > I am test

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-12 Thread Tony Lindgren
* Derald D. Woods [171212 18:11]: > On Tue, Dec 12, 2017 at 06:50:54PM +0100, Ladislav Michl wrote: > > On Tue, Dec 12, 2017 at 08:55:42AM -0800, Tony Lindgren wrote: > > > * Derald D. Woods [171212 16:34]: > > > > I am testing using an appended device-tree. This has been the most > > > > reliabl

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-12 Thread Derald D. Woods
On Tue, Dec 12, 2017 at 06:50:54PM +0100, Ladislav Michl wrote: > On Tue, Dec 12, 2017 at 08:55:42AM -0800, Tony Lindgren wrote: > > * Derald D. Woods [171212 16:34]: > > > I am testing using an appended device-tree. This has been the most > > > reliable method for the OMAP34XX boards that I have.

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-12 Thread Ladislav Michl
On Tue, Dec 12, 2017 at 08:55:42AM -0800, Tony Lindgren wrote: > * Derald D. Woods [171212 16:34]: > > I am testing using an appended device-tree. This has been the most > > reliable method for the OMAP34XX boards that I have. If you have an > > example config, with working command line MTDPARTS,

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-12 Thread Ladislav Michl
On Tue, Dec 12, 2017 at 10:31:26AM -0600, Derald D. Woods wrote: > > What about passing U-Boot partitions information to kernel instead? > > I am testing using an appended device-tree. This has been the most > reliable method for the OMAP34XX boards that I have. If you have an > example config, wit

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-12 Thread Tony Lindgren
* Derald D. Woods [171212 16:34]: > On Tue, Dec 12, 2017 at 07:39:30AM +0100, Ladislav Michl wrote: > > On Mon, Dec 11, 2017 at 10:12:13PM -0600, Derald D. Woods wrote: > > > The partition information was omitted during the conversion to OMAP3430 > > > specific data. > > > > That could be intenti

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-12 Thread Derald D. Woods
On Tue, Dec 12, 2017 at 07:39:30AM +0100, Ladislav Michl wrote: > On Mon, Dec 11, 2017 at 10:12:13PM -0600, Derald D. Woods wrote: > > The partition information was omitted during the conversion to OMAP3430 > > specific data. > > That could be intentional... I am fixing an addition that I created

Re: [PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-11 Thread Ladislav Michl
On Mon, Dec 11, 2017 at 10:12:13PM -0600, Derald D. Woods wrote: > The partition information was omitted during the conversion to OMAP3430 > specific data. That could be intentional... > The data added by this commit is consistent with current U-Boot default > definitions. What about passing U-B

[PATCH] ARM: dts: omap3-evm: Fix missing NAND partition information

2017-12-11 Thread Derald D. Woods
The partition information was omitted during the conversion to OMAP3430 specific data. The data added by this commit is consistent with current U-Boot default definitions. Fixes: 62fe1d337461 ("ARM: dts: omap3-evm: Add OMAP3530 specific device tree processor data") Signed-off-by: Derald D. Woods