[PATCH] u-boot: disable SPL/MLO processing for omapl138 SOCs

[PATCH] u-boot: disable SPL/MLO processing for omapl138 SOCs

[Denys Dmytriyenko]

From: Denys Dmytriyenko <denys@ti.com>

Signed-off-by: Denys Dmytriyenko <denys@ti.com>
---
 recipes-bsp/u-boot/u-boot_2014.07.bb | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/recipes-bsp/u-boot/u-boot_2014.07.bb b/recipes-bsp/u-boot/u-boot_2014.07.bb
index b326aa1..9da6f7d 100644
--- a/recipes-bsp/u-boot/u-boot_2014.07.bb
+++ b/recipes-bsp/u-boot/u-boot_2014.07.bb
@@ -25,3 +25,6 @@ SRCREV = "76b21026ceb5a6a83fc53b0ecdf425f240318022"
 
 SPL_BINARY = "MLO"
 SPL_UART_BINARY = "u-boot-spl.bin"
+
+SPL_BINARY_omapl138 = ""
+SPL_UART_BINARY_omapl138 = ""
-- 
2.0.0

Re: [PATCH] u-boot: disable SPL/MLO processing for omapl138 SOCs

[Tom Rini]

[-- Attachment #1: Type: text/plain, Size: 1026 bytes --]

On Wed, Jun 25, 2014 at 12:10:32PM -0400, Denys Dmytriyenko wrote:

> From: Denys Dmytriyenko <denys@ti.com>
> 
> Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> ---
>  recipes-bsp/u-boot/u-boot_2014.07.bb | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/recipes-bsp/u-boot/u-boot_2014.07.bb b/recipes-bsp/u-boot/u-boot_2014.07.bb
> index b326aa1..9da6f7d 100644
> --- a/recipes-bsp/u-boot/u-boot_2014.07.bb
> +++ b/recipes-bsp/u-boot/u-boot_2014.07.bb
> @@ -25,3 +25,6 @@ SRCREV = "76b21026ceb5a6a83fc53b0ecdf425f240318022"
>  
>  SPL_BINARY = "MLO"
>  SPL_UART_BINARY = "u-boot-spl.bin"
> +
> +SPL_BINARY_omapl138 = ""
> +SPL_UART_BINARY_omapl138 = ""

omapl138 is am18xx right?  SPL supports that.  The only issue is that we
need to ship u-boot.ais which is a combo image.  If I'm looking at the
right branch, the problem(s) right now are we don't use the right config
target (da850_am18xxevm_config is what we want) and UBOOT_SUFFIX to
"ais" (in the omapl138.inc file).

-- 
Tom

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 853 bytes --]

Re: [PATCH] u-boot: disable SPL/MLO processing for omapl138 SOCs

[Denys Dmytriyenko]

On Thu, Jun 26, 2014 at 09:30:45AM -0400, Tom Rini wrote:
> On Wed, Jun 25, 2014 at 12:10:32PM -0400, Denys Dmytriyenko wrote:
> 
> > From: Denys Dmytriyenko <denys@ti.com>
> > 
> > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > ---
> >  recipes-bsp/u-boot/u-boot_2014.07.bb | 3 +++
> >  1 file changed, 3 insertions(+)
> > 
> > diff --git a/recipes-bsp/u-boot/u-boot_2014.07.bb b/recipes-bsp/u-boot/u-boot_2014.07.bb
> > index b326aa1..9da6f7d 100644
> > --- a/recipes-bsp/u-boot/u-boot_2014.07.bb
> > +++ b/recipes-bsp/u-boot/u-boot_2014.07.bb
> > @@ -25,3 +25,6 @@ SRCREV = "76b21026ceb5a6a83fc53b0ecdf425f240318022"
> >  
> >  SPL_BINARY = "MLO"
> >  SPL_UART_BINARY = "u-boot-spl.bin"
> > +
> > +SPL_BINARY_omapl138 = ""
> > +SPL_UART_BINARY_omapl138 = ""
> 
> omapl138 is am18xx right?  SPL supports that.  The only issue is that we
> need to ship u-boot.ais which is a combo image.  If I'm looking at the
> right branch, the problem(s) right now are we don't use the right config
> target (da850_am18xxevm_config is what we want) and UBOOT_SUFFIX to
> "ais" (in the omapl138.inc file).

What's the difference between da850_config and da850_am18xxevm_config? The 
first one is what we used previously. Is the second one set ups the building 
of "ais" image?

As of UBOOT_SUFFIX, with hierarchical inclusion of different .inc files, it 
would be overwritten if defined in the machine config. Usually we end up 
specifying it in the machine-specific u-boot recipe. Let me take a look at it 
and see if we can move that var to the machine config...

-- 
Denys

Re: [PATCH] u-boot: disable SPL/MLO processing for omapl138 SOCs

[Tom Rini]

On Thu, Jun 26, 2014 at 01:31:48PM -0400, Denys Dmytriyenko wrote:
> On Thu, Jun 26, 2014 at 09:30:45AM -0400, Tom Rini wrote:
> > On Wed, Jun 25, 2014 at 12:10:32PM -0400, Denys Dmytriyenko wrote:
> > 
> > > From: Denys Dmytriyenko <denys@ti.com>
> > > 
> > > Signed-off-by: Denys Dmytriyenko <denys@ti.com>
> > > ---
> > >  recipes-bsp/u-boot/u-boot_2014.07.bb | 3 +++
> > >  1 file changed, 3 insertions(+)
> > > 
> > > diff --git a/recipes-bsp/u-boot/u-boot_2014.07.bb b/recipes-bsp/u-boot/u-boot_2014.07.bb
> > > index b326aa1..9da6f7d 100644
> > > --- a/recipes-bsp/u-boot/u-boot_2014.07.bb
> > > +++ b/recipes-bsp/u-boot/u-boot_2014.07.bb
> > > @@ -25,3 +25,6 @@ SRCREV = "76b21026ceb5a6a83fc53b0ecdf425f240318022"
> > >  
> > >  SPL_BINARY = "MLO"
> > >  SPL_UART_BINARY = "u-boot-spl.bin"
> > > +
> > > +SPL_BINARY_omapl138 = ""
> > > +SPL_UART_BINARY_omapl138 = ""
> > 
> > omapl138 is am18xx right?  SPL supports that.  The only issue is that we
> > need to ship u-boot.ais which is a combo image.  If I'm looking at the
> > right branch, the problem(s) right now are we don't use the right config
> > target (da850_am18xxevm_config is what we want) and UBOOT_SUFFIX to
> > "ais" (in the omapl138.inc file).
> 
> What's the difference between da850_config and da850_am18xxevm_config? The 
> first one is what we used previously. Is the second one set ups the building 
> of "ais" image?

The difference between a "da850 evm" and "am18xx evm" is where the MAC
is stored, da850_am18xxevm picks it up from the right place.

> As of UBOOT_SUFFIX, with hierarchical inclusion of different .inc files, it 
> would be overwritten if defined in the machine config. Usually we end up 
> specifying it in the machine-specific u-boot recipe. Let me take a look at it 
> and see if we can move that var to the machine config...

Right, I'm referring to the machine.inc file here since it's a SoC-level
thing rather than a machine-tweakable thing.

-- 
Tom