[U-Boot] [PATCH] OMAP3EVM: Set default bootfile

[U-Boot] [PATCH] OMAP3EVM: Set default bootfile

[Sanjeev Premi]

The current configuration doesn't define default
bootfile; leading to this warning at execution:

OMAP3_EVM # dhcp
...
...
DHCP client bound to address 192.168.1.11
*** Warning: no boot file name; using 'AC18BE16.img'
TFTP from server 0.0.0.0; our IP address is 192.168.1.11;
sending through gateway 192.168.1.1
Filename 'AC18BE16.img'.
Load address: 0x82000000
Loading: *
TFTP error: 'File not found' (1)

Signed-off-by: Sanjeev Premi <premi@ti.com>
---
 include/configs/omap3_evm.h |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/include/configs/omap3_evm.h b/include/configs/omap3_evm.h
index 549cef9..e205c01 100644
--- a/include/configs/omap3_evm.h
+++ b/include/configs/omap3_evm.h
@@ -149,6 +149,8 @@
 /* Environment information */
 #define CONFIG_BOOTDELAY	10
 
+#define CONFIG_BOOTFILE		uImage
+
 #define CONFIG_EXTRA_ENV_SETTINGS \
 	"loadaddr=0x82000000\0" \
 	"console=ttyS2,115200n8\0" \
-- 
1.6.2.2

[U-Boot] [PATCH] OMAP3EVM: Set default bootfile

[Wolfgang Denk]

Dear Sanjeev Premi,

In message <1241515371-24359-1-git-send-email-premi@ti.com> you wrote:
> The current configuration doesn't define default
> bootfile; leading to this warning at execution:

Note that this is not a problem, but a decision of the board
maintainer.

Also, you can always just use setenv + saveenv if you want this
changed on your board.

> OMAP3_EVM # dhcp
> ...
> ...
> DHCP client bound to address 192.168.1.11
> *** Warning: no boot file name; using 'AC18BE16.img'
> TFTP from server 0.0.0.0; our IP address is 192.168.1.11;
> sending through gateway 192.168.1.1
> Filename 'AC18BE16.img'.
> Load address: 0x82000000
> Loading: *
> TFTP error: 'File not found' (1)

This is standard behaviour and not a problem.

> Signed-off-by: Sanjeev Premi <premi@ti.com>
> ---
>  include/configs/omap3_evm.h |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/include/configs/omap3_evm.h b/include/configs/omap3_evm.h
> index 549cef9..e205c01 100644
> --- a/include/configs/omap3_evm.h
> +++ b/include/configs/omap3_evm.h
> @@ -149,6 +149,8 @@
>  /* Environment information */
>  #define CONFIG_BOOTDELAY	10
>  
> +#define CONFIG_BOOTFILE		uImage
> +

You should probably add the board maintainer on Cc: (done here) to get
his ACK.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
The thing is, as you progress in the Craft,  you'll  learn  there  is
another rule... When you break rules, break 'em good and hard.
                                    - Terry Pratchett, _Wyrd Sisters_

[U-Boot] [PATCH] OMAP3EVM: Set default bootfile

[Premi, Sanjeev]

> -----Original Message-----
> From: Wolfgang Denk [mailto:wd at denx.de] 
> Sent: Tuesday, May 05, 2009 3:51 PM
> To: Premi, Sanjeev
> Cc: u-boot at lists.denx.de; Pillai, Manikandan
> Subject: Re: [U-Boot] [PATCH] OMAP3EVM: Set default bootfile
> 
> Dear Sanjeev Premi,
> 
> In message <1241515371-24359-1-git-send-email-premi@ti.com> you wrote:
> > The current configuration doesn't define default
> > bootfile; leading to this warning at execution:
> 
> Note that this is not a problem, but a decision of the board
> maintainer.
> 
> Also, you can always just use setenv + saveenv if you want this
> changed on your board.
> 
> > OMAP3_EVM # dhcp
> > ...
> > ...
> > DHCP client bound to address 192.168.1.11
> > *** Warning: no boot file name; using 'AC18BE16.img'
> > TFTP from server 0.0.0.0; our IP address is 192.168.1.11;
> > sending through gateway 192.168.1.1
> > Filename 'AC18BE16.img'.
> > Load address: 0x82000000
> > Loading: *
> > TFTP error: 'File not found' (1)
> 
> This is standard behaviour and not a problem.
> 
> > Signed-off-by: Sanjeev Premi <premi@ti.com>
> > ---
> >  include/configs/omap3_evm.h |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> > 
> > diff --git a/include/configs/omap3_evm.h 
> b/include/configs/omap3_evm.h
> > index 549cef9..e205c01 100644
> > --- a/include/configs/omap3_evm.h
> > +++ b/include/configs/omap3_evm.h
> > @@ -149,6 +149,8 @@
> >  /* Environment information */
> >  #define CONFIG_BOOTDELAY	10
> >  
> > +#define CONFIG_BOOTFILE		uImage
> > +
> 
> You should probably add the board maintainer on Cc: (done here) to get
> his ACK.

In this case, the board maintainer works with me. He will ACK it.

> 
> Best regards,
> 
> Wolfgang Denk
> 
> -- 
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> The thing is, as you progress in the Craft,  you'll  learn  there  is
> another rule... When you break rules, break 'em good and hard.
>                                     - Terry Pratchett, _Wyrd Sisters_
> 
> 

[U-Boot] [PATCH] OMAP3EVM: Set default bootfile

[Pillai, Manikandan]

> -----Original Message-----
> From: Wolfgang Denk [mailto:wd at denx.de]
> Sent: Tuesday, May 05, 2009 3:51 PM
> To: Premi, Sanjeev
> Cc: u-boot at lists.denx.de; Pillai, Manikandan
> Subject: Re: [U-Boot] [PATCH] OMAP3EVM: Set default bootfile
> 
> Dear Sanjeev Premi,
> 
> In message <1241515371-24359-1-git-send-email-premi@ti.com> you wrote:
> > The current configuration doesn't define default
> > bootfile; leading to this warning at execution:
> 
> Note that this is not a problem, but a decision of the board
> maintainer.
> 
> Also, you can always just use setenv + saveenv if you want this
> changed on your board.
> 
> > OMAP3_EVM # dhcp
> > ...
> > ...
> > DHCP client bound to address 192.168.1.11
> > *** Warning: no boot file name; using 'AC18BE16.img'
> > TFTP from server 0.0.0.0; our IP address is 192.168.1.11;
> > sending through gateway 192.168.1.1
> > Filename 'AC18BE16.img'.
> > Load address: 0x82000000
> > Loading: *
> > TFTP error: 'File not found' (1)
> 
> This is standard behaviour and not a problem.
> 
> > Signed-off-by: Sanjeev Premi <premi@ti.com>
> > ---
> >  include/configs/omap3_evm.h |    2 ++
> >  1 files changed, 2 insertions(+), 0 deletions(-)
> >
> > diff --git a/include/configs/omap3_evm.h b/include/configs/omap3_evm.h
> > index 549cef9..e205c01 100644
> > --- a/include/configs/omap3_evm.h
> > +++ b/include/configs/omap3_evm.h
> > @@ -149,6 +149,8 @@
> >  /* Environment information */
> >  #define CONFIG_BOOTDELAY	10
> >
> > +#define CONFIG_BOOTFILE		uImage
> > +
> 
> You should probably add the board maintainer on Cc: (done here) to get
> his ACK.

[Pillai, Manikandan] 

Acked

> 
> Best regards,
> 
> Wolfgang Denk
> 
> --
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> The thing is, as you progress in the Craft,  you'll  learn  there  is
> another rule... When you break rules, break 'em good and hard.
>                                     - Terry Pratchett, _Wyrd Sisters_

[U-Boot] [PATCH] OMAP3EVM: Set default bootfile

[Dirk Behme]

Sanjeev Premi wrote:
> The current configuration doesn't define default
> bootfile; leading to this warning at execution:
> 
> OMAP3_EVM # dhcp
> ...
> ...
> DHCP client bound to address 192.168.1.11
> *** Warning: no boot file name; using 'AC18BE16.img'
> TFTP from server 0.0.0.0; our IP address is 192.168.1.11;
> sending through gateway 192.168.1.1
> Filename 'AC18BE16.img'.
> Load address: 0x82000000
> Loading: *
> TFTP error: 'File not found' (1)
> 
> Signed-off-by: Sanjeev Premi <premi@ti.com>

Acked-by: Dirk Behme <dirk.behme@googlemail.com>

> ---
>  include/configs/omap3_evm.h |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
> 
> diff --git a/include/configs/omap3_evm.h b/include/configs/omap3_evm.h
> index 549cef9..e205c01 100644
> --- a/include/configs/omap3_evm.h
> +++ b/include/configs/omap3_evm.h
> @@ -149,6 +149,8 @@
>  /* Environment information */
>  #define CONFIG_BOOTDELAY	10
>  
> +#define CONFIG_BOOTFILE		uImage
> +
>  #define CONFIG_EXTRA_ENV_SETTINGS \
>  	"loadaddr=0x82000000\0" \
>  	"console=ttyS2,115200n8\0" \

[U-Boot] [PATCH] OMAP3EVM: Set default bootfile

[Jean-Christophe PLAGNIOL-VILLARD]

On 14:52 Tue 05 May     , Sanjeev Premi wrote:
> The current configuration doesn't define default
> bootfile; leading to this warning at execution:
> 
> OMAP3_EVM # dhcp
> ...
> ...
> DHCP client bound to address 192.168.1.11
> *** Warning: no boot file name; using 'AC18BE16.img'
> TFTP from server 0.0.0.0; our IP address is 192.168.1.11;
> sending through gateway 192.168.1.1
> Filename 'AC18BE16.img'.
> Load address: 0x82000000
> Loading: *
> TFTP error: 'File not found' (1)
IMHO it's the correct way as you will be allow to have only one tftp server
with one file per mac address

but as the Board Maintainer Ack it ok

as it's not really a fix but a preference it will be for the next
expect you have a specific reason for this release

Best Regards,
J.

[U-Boot] [PATCH] OMAP3EVM: Set default bootfile

[Jean-Christophe PLAGNIOL-VILLARD]

On 14:52 Tue 05 May     , Sanjeev Premi wrote:
> The current configuration doesn't define default
> bootfile; leading to this warning at execution:
> 
> OMAP3_EVM # dhcp
> ...
> ...
> DHCP client bound to address 192.168.1.11
> *** Warning: no boot file name; using 'AC18BE16.img'
> TFTP from server 0.0.0.0; our IP address is 192.168.1.11;
> sending through gateway 192.168.1.1
> Filename 'AC18BE16.img'.
> Load address: 0x82000000
> Loading: *
> TFTP error: 'File not found' (1)
> 
> Signed-off-by: Sanjeev Premi <premi@ti.com>
> ---
applied to arm/next

Best Regards,
J.