Re: [U-Boot] config: enable CONFIG_API in distro config

On 04/21/2014 11:57 AM, Simon Glass wrote: > Hi Dennis, > > On 21 April 2014 08:49, Dennis Gilmore > wrote: > > On Sun, 20 Apr 2014 09:55:53 -0700 > Simon Glass mailto:s...@chromium.org>> wrote: > > [snip] > > > Can you suggest a starting point for trying th

Re: [U-Boot] config: enable CONFIG_API in distro config

Hi Dennis, On 21 April 2014 08:49, Dennis Gilmore wrote: > On Sun, 20 Apr 2014 09:55:53 -0700 > Simon Glass wrote: > > [snip] > Can you suggest a starting point for trying this out on am ARM > > platform? (e.g. OMAP, Tegra, Exynos) I'd like to see how it works. > > as of 2014.04 it will work o

Re: [U-Boot] config: enable CONFIG_API in distro config

On Sun, 20 Apr 2014 09:55:53 -0700 Simon Glass wrote: > Hi Dennis, > > On 19 April 2014 08:42, Dennis Gilmore wrote: > > > -BEGIN PGP SIGNED MESSAGE- > > Hash: SHA1 > > > > On Sat, 19 Apr 2014 15:01:40 +0100 > > Ian Campbell wrote: > > > > > On Fri, 2014-04-18 at 11:43 -0600, Simon Gl

Re: [U-Boot] config: enable CONFIG_API in distro config

Hi Dennis, On 19 April 2014 08:42, Dennis Gilmore wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On Sat, 19 Apr 2014 15:01:40 +0100 > Ian Campbell wrote: > > > On Fri, 2014-04-18 at 11:43 -0600, Simon Glass wrote: > > > Hi Stephen, > > > > > > On 18 April 2014 11:23, Stephen Warre

Re: [U-Boot] config: enable CONFIG_API in distro config

On Fri, 2014-04-18 at 11:23 -0600, Stephen Warren wrote: > That said, I still conceptually object to config_distro_defaults.h > enabling API support in order to support grub; A worse problem is that I'm far from convinced that the API is suitable to be used in production. The mechanism for determ

Re: [U-Boot] config: enable CONFIG_API in distro config

On Sat, 2014-04-19 at 02:05 +0100, Leif Lindholm wrote: > Ian Campbell wrote some neat patches for patching the (grub) kernel > link addres at grub-install time, which would be an improvement, but is > also quite invasive over several ports: > http://lists.gnu.org/archive/html/grub-devel/2013-12/ms

Re: [U-Boot] config: enable CONFIG_API in distro config

On Fri, 2014-04-18 at 11:43 -0600, Simon Glass wrote: > Hi Stephen, > > On 18 April 2014 11:23, Stephen Warren wrote: > > On 04/18/2014 09:35 AM, Tom Rini wrote: > >> On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: > >> > >>> From: Rob Herring > >>> > >>> CONFIG_API is needed for u-

Re: [U-Boot] config: enable CONFIG_API in distro config

On Fri, Apr 18, 2014 at 05:54:51PM -0500, Rob Herring wrote: > > a) It was excruciatingly slow, to the point of not being useful. > > > > b) The binary had hard-coded memory layout, and hence couldn't be > > generic across multiple boards, since SoCs put their RAM in different > > places, boards ha

Re: [U-Boot] config: enable CONFIG_API in distro config

On Fri, Apr 18, 2014 at 12:23 PM, Stephen Warren wrote: > On 04/18/2014 09:35 AM, Tom Rini wrote: >> On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: >> >>> From: Rob Herring >>> >>> CONFIG_API is needed for u-boot apps such as grub2, so enable it for >>> distro config. >>> >>> Cc: De

Re: [U-Boot] config: enable CONFIG_API in distro config

Hi Stephen, On 18 April 2014 11:23, Stephen Warren wrote: > On 04/18/2014 09:35 AM, Tom Rini wrote: >> On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: >> >>> From: Rob Herring >>> >>> CONFIG_API is needed for u-boot apps such as grub2, so enable it for >>> distro config. >>> >>> Cc:

Re: [U-Boot] config: enable CONFIG_API in distro config

On 04/18/2014 09:35 AM, Tom Rini wrote: > On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: > >> From: Rob Herring >> >> CONFIG_API is needed for u-boot apps such as grub2, so enable it for >> distro config. >> >> Cc: Dennis Gilmore >> Signed-off-by: Rob Herring > > This breaks a nu

Re: [U-Boot] config: enable CONFIG_API in distro config

On Fri, Apr 18, 2014 at 10:56 AM, Tom Rini wrote: > On Fri, Apr 18, 2014 at 08:44:49AM -0700, Tom Warren wrote: > >> If Rob broke it, Rob should fix it. > > Well, it's non-obvious system details, in some cases. I can guess at > CONFIG_SYS_MMC_MAX_DEVICE for Tegra for example, but there was some >

Re: [U-Boot] config: enable CONFIG_API in distro config

On 04/18/2014 09:55 AM, Tom Rini wrote: > On Fri, Apr 18, 2014 at 09:44:44AM -0600, Stephen Warren wrote: >> On 04/18/2014 09:35 AM, Tom Rini wrote: >>> On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: >>> From: Rob Herring CONFIG_API is needed for u-boot apps such as gr

Re: [U-Boot] config: enable CONFIG_API in distro config

On Fri, Apr 18, 2014 at 10:44 AM, Stephen Warren wrote: > On 04/18/2014 09:35 AM, Tom Rini wrote: >> On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: >> >>> From: Rob Herring >>> >>> CONFIG_API is needed for u-boot apps such as grub2, so enable it for >>> distro config. >>> >>> Cc: De

Re: [U-Boot] config: enable CONFIG_API in distro config

On Fri, Apr 18, 2014 at 08:44:49AM -0700, Tom Warren wrote: > If Rob broke it, Rob should fix it. Well, it's non-obvious system details, in some cases. I can guess at CONFIG_SYS_MMC_MAX_DEVICE for Tegra for example, but there was some other breakage wrt lcd things. -- Tom signature.asc Descr

Re: [U-Boot] config: enable CONFIG_API in distro config

On Fri, Apr 18, 2014 at 09:44:44AM -0600, Stephen Warren wrote: > On 04/18/2014 09:35 AM, Tom Rini wrote: > > On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: > > > >> From: Rob Herring > >> > >> CONFIG_API is needed for u-boot apps such as grub2, so enable it for > >> distro config.

Re: [U-Boot] config: enable CONFIG_API in distro config

is Gilmore; Rob Herring > Subject: Re: [U-Boot] config: enable CONFIG_API in distro config > > * PGP Signed by an unknown key > > On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: > > > From: Rob Herring > > > > CONFIG_API is needed for u-boot app

Re: [U-Boot] config: enable CONFIG_API in distro config

On 04/18/2014 09:35 AM, Tom Rini wrote: > On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: > >> From: Rob Herring >> >> CONFIG_API is needed for u-boot apps such as grub2, so enable it for >> distro config. >> >> Cc: Dennis Gilmore >> Signed-off-by: Rob Herring > > This breaks a nu

Re: [U-Boot] config: enable CONFIG_API in distro config

On Thu, Apr 10, 2014 at 03:55:48PM -0500, Rob Herring wrote: > From: Rob Herring > > CONFIG_API is needed for u-boot apps such as grub2, so enable it for > distro config. > > Cc: Dennis Gilmore > Signed-off-by: Rob Herring This breaks a number of boards that have opted in to the distro confi