* [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