All the mail mirrored from lore.kernel.org
 help / color / mirror / Atom feed
* [U-Boot-Users] [PATCH] Document 'bootm_low' and 'bootm_size' environment variables
@ 2008-04-09 21:39 Bartlomiej Sieka
  2008-04-09 22:08 ` Kumar Gala
  0 siblings, 1 reply; 4+ messages in thread
From: Bartlomiej Sieka @ 2008-04-09 21:39 UTC (permalink / raw
  To: u-boot

Signed-off-by: Bartlomiej Sieka <tur@semihalf.com>
---
Kumar, this patch pertains to your commit "[new uImage] Provide ability to
restrict region used for boot images", id d3f2fa0d -- could you review
and confirm that the documentation is OK? Thanks.

 README |   19 ++++++++++++++++++-
 1 files changed, 18 insertions(+), 1 deletions(-)

diff --git a/README b/README
index 5d059e7..a0dfe73 100644
--- a/README
+++ b/README
@@ -1995,7 +1995,10 @@ Configuration Settings:
 		Maximum size of memory mapped by the startup code of
 		the Linux kernel; all data that must be processed by
 		the Linux kernel (bd_info, boot arguments, eventually
-		initrd image) must be put below this limit.
+		initrd image) must be put below this limit, unless
+		"bootm_low" enviroment variable is defined and non-zero.
+		In such case all data for the Linux kernel must be
+		between "bootm_low" and "bootm_low" + CFG_BOOTMAPSZ.
 
 - CFG_MAX_FLASH_BANKS:
 		Max number of Flash memory banks
@@ -2733,6 +2736,20 @@ Some configuration options can be set using Environment Variables:
 
   bootfile	- Name of the image to load with TFTP
 
+  bootm_low	- Memory range available for image processing in the bootm
+		  command can be restricted. This variable is given as
+		  a hexadecimal number and defines lowest address allowed
+		  for use by the bootm command. See also "bootm_size"
+		  environment variable. Address defined by "bootm_low" is
+		  also the base of the initial memory mapping for the Linux
+		  kernel -- see the descripton of CFG_BOOTMAPSZ.
+
+  bootm_size	- Memory range available for image processing in the bootm
+		  command can be restricted. This variable is given as
+		  a hexadecimal number and defines the size of the region
+		  allowed for use by the bootm command. See also "bootm_low"
+		  environment variable.
+
   autoload	- if set to "no" (any string beginning with 'n'),
 		  "bootp" will just load perform a lookup of the
 		  configuration from the BOOTP server, but not try to

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot-Users] [PATCH] Document 'bootm_low' and 'bootm_size' environment variables
  2008-04-09 21:39 [U-Boot-Users] [PATCH] Document 'bootm_low' and 'bootm_size' environment variables Bartlomiej Sieka
@ 2008-04-09 22:08 ` Kumar Gala
  2008-04-14 13:44   ` [U-Boot-Users] [PATCH] Boot-related documentation update Bartlomiej Sieka
  0 siblings, 1 reply; 4+ messages in thread
From: Kumar Gala @ 2008-04-09 22:08 UTC (permalink / raw
  To: u-boot


On Apr 9, 2008, at 4:39 PM, Bartlomiej Sieka wrote:
> Signed-off-by: Bartlomiej Sieka <tur@semihalf.com>
> ---
> Kumar, this patch pertains to your commit "[new uImage] Provide  
> ability to
> restrict region used for boot images", id d3f2fa0d -- could you review
> and confirm that the documentation is OK? Thanks.
>
> README |   19 ++++++++++++++++++-
> 1 files changed, 18 insertions(+), 1 deletions(-)
>
> diff --git a/README b/README
> index 5d059e7..a0dfe73 100644
> --- a/README
> +++ b/README
> @@ -1995,7 +1995,10 @@ Configuration Settings:
> 		Maximum size of memory mapped by the startup code of
> 		the Linux kernel; all data that must be processed by
> 		the Linux kernel (bd_info, boot arguments, eventually
> -		initrd image) must be put below this limit.
> +		initrd image) must be put below this limit, unless
> +		"bootm_low" enviroment variable is defined and non-zero.
> +		In such case all data for the Linux kernel must be
> +		between "bootm_low" and "bootm_low" + CFG_BOOTMAPSZ.

all data seems misleading since initrd doesn't have to be within  
CFG_BOOTMAPSZ.

> - CFG_MAX_FLASH_BANKS:
> 		Max number of Flash memory banks
> @@ -2733,6 +2736,20 @@ Some configuration options can be set using  
> Environment Variables:
>
>   bootfile	- Name of the image to load with TFTP
>
> +  bootm_low	- Memory range available for image processing in the  
> bootm
> +		  command can be restricted. This variable is given as
> +		  a hexadecimal number and defines lowest address allowed
> +		  for use by the bootm command. See also "bootm_size"
> +		  environment variable. Address defined by "bootm_low" is
> +		  also the base of the initial memory mapping for the Linux
> +		  kernel -- see the descripton of CFG_BOOTMAPSZ.

>
> +
> +  bootm_size	- Memory range available for image processing in the  
> bootm
> +		  command can be restricted. This variable is given as
> +		  a hexadecimal number and defines the size of the region
> +		  allowed for use by the bootm command. See also "bootm_low"
> +		  environment variable.
> +

these look fine.

- k

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot-Users] [PATCH] Boot-related documentation update
  2008-04-09 22:08 ` Kumar Gala
@ 2008-04-14 13:44   ` Bartlomiej Sieka
  2008-04-18  7:54     ` Wolfgang Denk
  0 siblings, 1 reply; 4+ messages in thread
From: Bartlomiej Sieka @ 2008-04-14 13:44 UTC (permalink / raw
  To: u-boot

- document 'bootm_low' and 'bootm_size' environment variables
- update inaccurate CFG_BOOTMAPSZ entry

Signed-off-by: Bartlomiej Sieka <tur@semihalf.com>
---
Addressed comment by Kumar, changed patch description accordingly.

 README |   21 +++++++++++++++++++--
 1 files changed, 19 insertions(+), 2 deletions(-)

diff --git a/README b/README
index 5d059e7..63b1417 100644
--- a/README
+++ b/README
@@ -1994,8 +1994,11 @@ Configuration Settings:
 - CFG_BOOTMAPSZ:
 		Maximum size of memory mapped by the startup code of
 		the Linux kernel; all data that must be processed by
-		the Linux kernel (bd_info, boot arguments, eventually
-		initrd image) must be put below this limit.
+		the Linux kernel (bd_info, boot arguments, FDT blob if
+		used) must be put below this limit, unless "bootm_low"
+		enviroment variable is defined and non-zero. In such case
+		all data for the Linux kernel must be between "bootm_low"
+		and "bootm_low" + CFG_BOOTMAPSZ.
 
 - CFG_MAX_FLASH_BANKS:
 		Max number of Flash memory banks
@@ -2733,6 +2736,20 @@ Some configuration options can be set using Environment Variables:
 
   bootfile	- Name of the image to load with TFTP
 
+  bootm_low	- Memory range available for image processing in the bootm
+		  command can be restricted. This variable is given as
+		  a hexadecimal number and defines lowest address allowed
+		  for use by the bootm command. See also "bootm_size"
+		  environment variable. Address defined by "bootm_low" is
+		  also the base of the initial memory mapping for the Linux
+		  kernel -- see the descripton of CFG_BOOTMAPSZ.
+
+  bootm_size	- Memory range available for image processing in the bootm
+		  command can be restricted. This variable is given as
+		  a hexadecimal number and defines the size of the region
+		  allowed for use by the bootm command. See also "bootm_low"
+		  environment variable.
+
   autoload	- if set to "no" (any string beginning with 'n'),
 		  "bootp" will just load perform a lookup of the
 		  configuration from the BOOTP server, but not try to

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot-Users] [PATCH] Boot-related documentation update
  2008-04-14 13:44   ` [U-Boot-Users] [PATCH] Boot-related documentation update Bartlomiej Sieka
@ 2008-04-18  7:54     ` Wolfgang Denk
  0 siblings, 0 replies; 4+ messages in thread
From: Wolfgang Denk @ 2008-04-18  7:54 UTC (permalink / raw
  To: u-boot

In message <20080414134306.15603.84966.stgit@pollux.denx.de> you wrote:
> - document 'bootm_low' and 'bootm_size' environment variables
> - update inaccurate CFG_BOOTMAPSZ entry
> 
> Signed-off-by: Bartlomiej Sieka <tur@semihalf.com>
> ---
> Addressed comment by Kumar, changed patch description accordingly.
> 
>  README |   21 +++++++++++++++++++--
>  1 files changed, 19 insertions(+), 2 deletions(-)

Applied, thanks.

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
It [being a Vulcan] means to adopt a philosophy, a way of life  which
is logical and beneficial. We cannot disregard that philosophy merely
for personal gain, no matter how important that gain might be.
	-- Spock, "Journey to Babel", stardate 3842.4

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2008-04-18  7:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-04-09 21:39 [U-Boot-Users] [PATCH] Document 'bootm_low' and 'bootm_size' environment variables Bartlomiej Sieka
2008-04-09 22:08 ` Kumar Gala
2008-04-14 13:44   ` [U-Boot-Users] [PATCH] Boot-related documentation update Bartlomiej Sieka
2008-04-18  7:54     ` Wolfgang Denk

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.