public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6
@ 2014-09-01 12:56 Fabio Estevam
  2014-09-01 13:35 ` Nitin Garg
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Fabio Estevam @ 2014-09-01 12:56 UTC (permalink / raw)
  To: u-boot

According to mx53 and mx6 reference manuals:

"The maximum size of the DCD limited to 1768 bytes."

As each DCD entry consists of 8 bytes, we have a total of 1768 / 8 = 221, and
excluding the first entry, which is the header leads to 220 as the maximum
number for DCD size.

Reported-by: Jonas Karlsson <jonas.d.karlsson@gmail.com> 
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 tools/imximage.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/imximage.h b/tools/imximage.h
index 01f861e..5b5ad0e 100644
--- a/tools/imximage.h
+++ b/tools/imximage.h
@@ -8,7 +8,7 @@
 #ifndef _IMXIMAGE_H_
 #define _IMXIMAGE_H_
 
-#define MAX_HW_CFG_SIZE_V2 121 /* Max number of registers imx can set for v2 */
+#define MAX_HW_CFG_SIZE_V2 220 /* Max number of registers imx can set for v2 */
 #define MAX_HW_CFG_SIZE_V1 60  /* Max number of registers imx can set for v1 */
 #define APP_CODE_BARKER	0xB1
 #define DCD_BARKER	0xB17219E9
-- 
1.9.1

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

* [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6
  2014-09-01 12:56 [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6 Fabio Estevam
@ 2014-09-01 13:35 ` Nitin Garg
  2014-09-02 14:15   ` Tom Rini
  2014-09-02 14:57 ` Nitin Garg
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 6+ messages in thread
From: Nitin Garg @ 2014-09-01 13:35 UTC (permalink / raw)
  To: u-boot

Acked!

Regards,
Nitin Garg


-----Original Message-----
From: Fabio Estevam [mailto:fabio.estevam at freescale.com] 
Sent: Monday, September 01, 2014 7:56 AM
To: sbabic at denx.de
Cc: u-boot at lists.denx.de; jonas.d.karlsson at gmail.com; Li Ye-B37916; Garg Nitin-B37173; Estevam Fabio-R49496
Subject: [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6

According to mx53 and mx6 reference manuals:

"The maximum size of the DCD limited to 1768 bytes."

As each DCD entry consists of 8 bytes, we have a total of 1768 / 8 = 221, and excluding the first entry, which is the header leads to 220 as the maximum number for DCD size.

Reported-by: Jonas Karlsson <jonas.d.karlsson@gmail.com>
Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 tools/imximage.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/imximage.h b/tools/imximage.h index 01f861e..5b5ad0e 100644
--- a/tools/imximage.h
+++ b/tools/imximage.h
@@ -8,7 +8,7 @@
 #ifndef _IMXIMAGE_H_
 #define _IMXIMAGE_H_
 
-#define MAX_HW_CFG_SIZE_V2 121 /* Max number of registers imx can set for v2 */
+#define MAX_HW_CFG_SIZE_V2 220 /* Max number of registers imx can set 
+for v2 */
 #define MAX_HW_CFG_SIZE_V1 60  /* Max number of registers imx can set for v1 */
 #define APP_CODE_BARKER	0xB1
 #define DCD_BARKER	0xB17219E9
--
1.9.1

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

* [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6
  2014-09-01 13:35 ` Nitin Garg
@ 2014-09-02 14:15   ` Tom Rini
  0 siblings, 0 replies; 6+ messages in thread
From: Tom Rini @ 2014-09-02 14:15 UTC (permalink / raw)
  To: u-boot

On Mon, Sep 01, 2014 at 01:35:22PM +0000, Nitin Garg wrote:

> Acked!

Please note that patchwork won't pick that up, only a properly
formatted acked-by line, thanks!

> 
> Regards,
> Nitin Garg
> 
> 
> -----Original Message-----
> From: Fabio Estevam [mailto:fabio.estevam at freescale.com] 
> Sent: Monday, September 01, 2014 7:56 AM
> To: sbabic at denx.de
> Cc: u-boot at lists.denx.de; jonas.d.karlsson at gmail.com; Li Ye-B37916; Garg Nitin-B37173; Estevam Fabio-R49496
> Subject: [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6
> 
> According to mx53 and mx6 reference manuals:
> 
> "The maximum size of the DCD limited to 1768 bytes."
> 
> As each DCD entry consists of 8 bytes, we have a total of 1768 / 8 = 221, and excluding the first entry, which is the header leads to 220 as the maximum number for DCD size.
> 
> Reported-by: Jonas Karlsson <jonas.d.karlsson@gmail.com>
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  tools/imximage.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/imximage.h b/tools/imximage.h index 01f861e..5b5ad0e 100644
> --- a/tools/imximage.h
> +++ b/tools/imximage.h
> @@ -8,7 +8,7 @@
>  #ifndef _IMXIMAGE_H_
>  #define _IMXIMAGE_H_
>  
> -#define MAX_HW_CFG_SIZE_V2 121 /* Max number of registers imx can set for v2 */
> +#define MAX_HW_CFG_SIZE_V2 220 /* Max number of registers imx can set 
> +for v2 */
>  #define MAX_HW_CFG_SIZE_V1 60  /* Max number of registers imx can set for v1 */
>  #define APP_CODE_BARKER	0xB1
>  #define DCD_BARKER	0xB17219E9
> --
> 1.9.1
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140902/fc89b0fe/attachment.pgp>

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

* [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6
  2014-09-01 12:56 [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6 Fabio Estevam
  2014-09-01 13:35 ` Nitin Garg
@ 2014-09-02 14:57 ` Nitin Garg
  2014-09-02 15:11 ` Nitin Garg
  2014-09-09 14:24 ` Stefano Babic
  3 siblings, 0 replies; 6+ messages in thread
From: Nitin Garg @ 2014-09-02 14:57 UTC (permalink / raw)
  To: u-boot

On 09/01/2014 07:56 AM, Fabio Estevam wrote:
> According to mx53 and mx6 reference manuals:
> 
> "The maximum size of the DCD limited to 1768 bytes."
> 
> As each DCD entry consists of 8 bytes, we have a total of 1768 / 8 = 221, and
> excluding the first entry, which is the header leads to 220 as the maximum
> number for DCD size.
> 
> Reported-by: Jonas Karlsson <jonas.d.karlsson@gmail.com> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  tools/imximage.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/imximage.h b/tools/imximage.h
> index 01f861e..5b5ad0e 100644
> --- a/tools/imximage.h
> +++ b/tools/imximage.h
> @@ -8,7 +8,7 @@
>  #ifndef _IMXIMAGE_H_
>  #define _IMXIMAGE_H_
>  
> -#define MAX_HW_CFG_SIZE_V2 121 /* Max number of registers imx can set for v2 */
> +#define MAX_HW_CFG_SIZE_V2 220 /* Max number of registers imx can set for v2 */
>  #define MAX_HW_CFG_SIZE_V1 60  /* Max number of registers imx can set for v1 */
>  #define APP_CODE_BARKER	0xB1
>  #define DCD_BARKER	0xB17219E9
> 

Acked-by: Nitin Garg <nitin.garg@freescale.com>

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

* [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6
  2014-09-01 12:56 [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6 Fabio Estevam
  2014-09-01 13:35 ` Nitin Garg
  2014-09-02 14:57 ` Nitin Garg
@ 2014-09-02 15:11 ` Nitin Garg
  2014-09-09 14:24 ` Stefano Babic
  3 siblings, 0 replies; 6+ messages in thread
From: Nitin Garg @ 2014-09-02 15:11 UTC (permalink / raw)
  To: u-boot

On 09/01/2014 07:56 AM, Fabio Estevam wrote:
> According to mx53 and mx6 reference manuals:
> 
> "The maximum size of the DCD limited to 1768 bytes."
> 
> As each DCD entry consists of 8 bytes, we have a total of 1768 / 8 = 221, and
> excluding the first entry, which is the header leads to 220 as the maximum
> number for DCD size.
> 
> Reported-by: Jonas Karlsson <jonas.d.karlsson@gmail.com> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  tools/imximage.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/imximage.h b/tools/imximage.h
> index 01f861e..5b5ad0e 100644
> --- a/tools/imximage.h
> +++ b/tools/imximage.h
> @@ -8,7 +8,7 @@
>  #ifndef _IMXIMAGE_H_
>  #define _IMXIMAGE_H_
>  
> -#define MAX_HW_CFG_SIZE_V2 121 /* Max number of registers imx can set for v2 */
> +#define MAX_HW_CFG_SIZE_V2 220 /* Max number of registers imx can set for v2 */
>  #define MAX_HW_CFG_SIZE_V1 60  /* Max number of registers imx can set for v1 */
>  #define APP_CODE_BARKER	0xB1
>  #define DCD_BARKER	0xB17219E9
> 

Acked-by: Nitin Garg <nitin.garg@freescale.com>

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

* [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6
  2014-09-01 12:56 [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6 Fabio Estevam
                   ` (2 preceding siblings ...)
  2014-09-02 15:11 ` Nitin Garg
@ 2014-09-09 14:24 ` Stefano Babic
  3 siblings, 0 replies; 6+ messages in thread
From: Stefano Babic @ 2014-09-09 14:24 UTC (permalink / raw)
  To: u-boot

On 01/09/2014 14:56, Fabio Estevam wrote:
> According to mx53 and mx6 reference manuals:
> 
> "The maximum size of the DCD limited to 1768 bytes."
> 
> As each DCD entry consists of 8 bytes, we have a total of 1768 / 8 = 221, and
> excluding the first entry, which is the header leads to 220 as the maximum
> number for DCD size.
> 
> Reported-by: Jonas Karlsson <jonas.d.karlsson@gmail.com> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

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

end of thread, other threads:[~2014-09-09 14:24 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-01 12:56 [U-Boot] [PATCH] tools: imximage: Fix the maximum DCD size for mx53/mx6 Fabio Estevam
2014-09-01 13:35 ` Nitin Garg
2014-09-02 14:15   ` Tom Rini
2014-09-02 14:57 ` Nitin Garg
2014-09-02 15:11 ` Nitin Garg
2014-09-09 14:24 ` Stefano Babic

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox