* [U-Boot] [PATCH 0/2] cm-t3x: minor config updates and cleanups
@ 2012-10-07 11:17 Igor Grinberg
2012-10-07 11:17 ` [U-Boot] [PATCH 1/2] cm-t35: clean unused defines from config Igor Grinberg
2012-10-07 11:17 ` [U-Boot] [PATCH 2/2] cm-t35: increase the MAXARGS Igor Grinberg
0 siblings, 2 replies; 6+ messages in thread
From: Igor Grinberg @ 2012-10-07 11:17 UTC (permalink / raw)
To: u-boot
Remove OneNAND and flash related defines.
Increase the MAXARGS value.
Igor Grinberg (2):
cm-t35: clean unused defines from config
cm-t35: increase the MAXARGS
include/configs/cm_t35.h | 14 ++------------
1 files changed, 2 insertions(+), 12 deletions(-)
--
1.7.3.4
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 1/2] cm-t35: clean unused defines from config
2012-10-07 11:17 [U-Boot] [PATCH 0/2] cm-t3x: minor config updates and cleanups Igor Grinberg
@ 2012-10-07 11:17 ` Igor Grinberg
2012-10-17 15:40 ` Tom Rini
2012-10-07 11:17 ` [U-Boot] [PATCH 2/2] cm-t35: increase the MAXARGS Igor Grinberg
1 sibling, 1 reply; 6+ messages in thread
From: Igor Grinberg @ 2012-10-07 11:17 UTC (permalink / raw)
To: u-boot
Neither cm-t35, nor cm-t3730 is using OneNAND or flash.
Remove the related defines from config file.
Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
---
include/configs/cm_t35.h | 12 +-----------
1 files changed, 1 insertions(+), 11 deletions(-)
diff --git a/include/configs/cm_t35.h b/include/configs/cm_t35.h
index 46c556d..8e28fbc 100644
--- a/include/configs/cm_t35.h
+++ b/include/configs/cm_t35.h
@@ -289,25 +289,15 @@
*/
/* **** PISMO SUPPORT *** */
-
/* Configure the PISMO */
#define PISMO1_NAND_SIZE GPMC_SIZE_128M
-#define PISMO1_ONEN_SIZE GPMC_SIZE_128M
-
-#define CONFIG_SYS_MONITOR_LEN (256 << 10) /* Reserve 2 sectors */
-
-#if defined(CONFIG_CMD_NAND)
-#define CONFIG_SYS_FLASH_BASE PISMO1_NAND_BASE
-#endif
/* Monitor@start of flash */
#define CONFIG_SYS_MONITOR_BASE CONFIG_SYS_FLASH_BASE
-#define CONFIG_SYS_ONENAND_BASE ONENAND_MAP
+#define CONFIG_SYS_MONITOR_LEN (256 << 10) /* Reserve 2 sectors */
#define CONFIG_ENV_IS_IN_NAND
-#define ONENAND_ENV_OFFSET 0x260000 /* environment starts here */
#define SMNAND_ENV_OFFSET 0x260000 /* environment starts here */
-
#define CONFIG_ENV_OFFSET SMNAND_ENV_OFFSET
#define CONFIG_ENV_ADDR SMNAND_ENV_OFFSET
--
1.7.3.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 2/2] cm-t35: increase the MAXARGS
2012-10-07 11:17 [U-Boot] [PATCH 0/2] cm-t3x: minor config updates and cleanups Igor Grinberg
2012-10-07 11:17 ` [U-Boot] [PATCH 1/2] cm-t35: clean unused defines from config Igor Grinberg
@ 2012-10-07 11:17 ` Igor Grinberg
2012-10-08 15:39 ` Tom Rini
1 sibling, 1 reply; 6+ messages in thread
From: Igor Grinberg @ 2012-10-07 11:17 UTC (permalink / raw)
To: u-boot
Increase the MAXARGS to support more than 16 arguments.
Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
---
include/configs/cm_t35.h | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/include/configs/cm_t35.h b/include/configs/cm_t35.h
index 8e28fbc..e18fb5a 100644
--- a/include/configs/cm_t35.h
+++ b/include/configs/cm_t35.h
@@ -256,7 +256,7 @@
/* Print Buffer Size */
#define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE + \
sizeof(CONFIG_SYS_PROMPT) + 16)
-#define CONFIG_SYS_MAXARGS 16 /* max number of command args */
+#define CONFIG_SYS_MAXARGS 32 /* max number of command args */
/* Boot Argument Buffer Size */
#define CONFIG_SYS_BARGSIZE (CONFIG_SYS_CBSIZE)
--
1.7.3.4
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 2/2] cm-t35: increase the MAXARGS
2012-10-07 11:17 ` [U-Boot] [PATCH 2/2] cm-t35: increase the MAXARGS Igor Grinberg
@ 2012-10-08 15:39 ` Tom Rini
2012-10-09 12:59 ` Igor Grinberg
0 siblings, 1 reply; 6+ messages in thread
From: Tom Rini @ 2012-10-08 15:39 UTC (permalink / raw)
To: u-boot
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 10/07/12 04:17, Igor Grinberg wrote:
> Increase the MAXARGS to support more than 16 arguments.
>
> Signed-off-by: Igor Grinberg <grinberg@compulab.co.il> ---
> include/configs/cm_t35.h | 2 +- 1 files changed, 1
> insertions(+), 1 deletions(-)
>
> diff --git a/include/configs/cm_t35.h b/include/configs/cm_t35.h
> index 8e28fbc..e18fb5a 100644 --- a/include/configs/cm_t35.h +++
> b/include/configs/cm_t35.h @@ -256,7 +256,7 @@ /* Print Buffer Size
> */ #define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE + \
> sizeof(CONFIG_SYS_PROMPT) + 16) -#define CONFIG_SYS_MAXARGS 16 /*
> max number of command args */ +#define CONFIG_SYS_MAXARGS 32 /*
> max number of command args */ /* Boot Argument Buffer Size */
> #define CONFIG_SYS_BARGSIZE (CONFIG_SYS_CBSIZE)
Why? This is the max number of arguments to a command in U-Boot not
the size of the command line arguments.
- --
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://www.enigmail.net/
iQIcBAEBAgAGBQJQcvOuAAoJENk4IS6UOR1W400P/1DjgD7n/1O19rZvZ0ET4wW1
b8hrNJXdWjuvQhB/suDurRHAaSicnBzavvAtg4ZGwMhqMfo4eg6fRfxNlB3fxMjE
HO5F188x07ycnwma4uDDhAnKOVve/iAYZbZTvMXK6QYQCnfA2FKtdliwRsQ4rN3r
a8nlkUc00mZrzKalk40KPP/8vWsaIc/EwgGaytnvpMRKqIGkZB/e1Ay028U678TH
k+P12xSHoE8WqiidsIJQAO+fCK5RUkHo7uJWg9q1eHAw/0CeEGGlpj2p2vVSgEuh
RxrsQn1/Bv6My5HuEhemcemgIbhHFCNFQ3LiY5cRDbDg49EJKkWkTq/zHhcxPJV0
zXAPINlTHUZ2NtWMHmk/VN7VdQL9at+IZvplGJ4oPSWLxi74ResdA64q17ICSiPN
cZEOk0vTkr1ib37Bboi/NNGmHpuWi0XY9lLYrJJSsqtOg5wqJw7uRo89MOPn2EXR
/pMHgqbaavYZzVw6wQ9Y1rxhhf+U+fo0x5nUbT9lGyXM9+FbXwsoXrf6eDmUNAPV
9t6/oVNh4FumZ64RHCucYUiluzXiV/eeKR6oeNfMhBzLqjec7gz0G/lMENoQada7
Du3RzAdsqbVGFL/4NKY2w1qZb+XDpQM+to4bJpRWeYq34potCwshmZJ4Ev7FSfpz
eTnaIqbfCMk1nnAuDsYo
=LfT0
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 2/2] cm-t35: increase the MAXARGS
2012-10-08 15:39 ` Tom Rini
@ 2012-10-09 12:59 ` Igor Grinberg
0 siblings, 0 replies; 6+ messages in thread
From: Igor Grinberg @ 2012-10-09 12:59 UTC (permalink / raw)
To: u-boot
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 10/08/12 17:39, Tom Rini wrote:
> On 10/07/12 04:17, Igor Grinberg wrote:
>> Increase the MAXARGS to support more than 16 arguments.
>
>> Signed-off-by: Igor Grinberg <grinberg@compulab.co.il> ---
>> include/configs/cm_t35.h | 2 +- 1 files changed, 1
>> insertions(+), 1 deletions(-)
>
>> diff --git a/include/configs/cm_t35.h b/include/configs/cm_t35.h
>> index 8e28fbc..e18fb5a 100644 --- a/include/configs/cm_t35.h +++
>> b/include/configs/cm_t35.h @@ -256,7 +256,7 @@ /* Print Buffer Size
>> */ #define CONFIG_SYS_PBSIZE (CONFIG_SYS_CBSIZE + \
>> sizeof(CONFIG_SYS_PROMPT) + 16) -#define CONFIG_SYS_MAXARGS 16 /*
>> max number of command args */ +#define CONFIG_SYS_MAXARGS 32 /*
>> max number of command args */ /* Boot Argument Buffer Size */
>> #define CONFIG_SYS_BARGSIZE (CONFIG_SYS_CBSIZE)
>
> Why? This is the max number of arguments to a command in U-Boot not
> the size of the command line arguments.
Yes, I know that, but for some reason I was mislead by one of our engineers
that we need to increase it.
After checking the issue, I've came to a conclusion, that there was a missing
quote(s) and we don't really need to increase the MAXARGS.
Thanks for bringing my attention to this.
You can drop this patch.
- --
Regards,
Igor.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.17 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
iQIcBAEBAgAGBQJQdB/GAAoJEBDE8YO64Efa08oP/AkwuOWXy5h/P2LwYCBSc+ZC
s9nEb+ZKdkICLrOaxaV6lCj9PEce3/PZIcPTYPXeFx3O3dW5ZAXSGiyLHTdB2Nut
N+9Tfmi9lsOywWn7rmWbEsC67+ZVG28kq4IzCpPccV7HTjEHT8BTUVE55qGRJgDj
jM1EcTYJ/MwoYx2ymel0XHDLqkUMzLChk47T8+OwiBj5SDAWXBcp4Iiufy5cBKB8
mqKbUkHklfzGt4PZrLB47MFJIl9B0y2864fyCrAz3cCS3d2W0Lf6MbUA1CguxsHY
DCs3prineQFwatkgiNMTLzk0vCqpakLfzLEIVtOf4EI+IVdbyhNnTAGkacr0HMbt
PqLnC8evfdYbaGNEeU1h4WCL6Fjzs0vD/KW29mGxNbj0tTE8TypXto8LbnF1MLJ6
vE0JKQMCu9oMA9ph4cQ00ZX6GMgbyBxnhpVQ02MyUanH+dylWtdkosoodcqdHf7p
RhpkDnOJV8//dmTS32bMp42k8hNb7tXbU1ebFU6ZMwpzY+jEmqrl15E7mYmki/ZE
F488LJQektTzEGZbMqpBtlIomLQ5OMRG2vhKz9gwY/FNvJsjKRw4+Owy+HGIOKwU
IZzNaOhBp8U0yx+/70G5N2i2mnh6+on30vWfMC3x6x2yAy/Uu05ayG5NyK/sIsDP
KydW6Ibm5CvE0fFwds4l
=DXK1
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 6+ messages in thread
* [U-Boot] [PATCH 1/2] cm-t35: clean unused defines from config
2012-10-07 11:17 ` [U-Boot] [PATCH 1/2] cm-t35: clean unused defines from config Igor Grinberg
@ 2012-10-17 15:40 ` Tom Rini
0 siblings, 0 replies; 6+ messages in thread
From: Tom Rini @ 2012-10-17 15:40 UTC (permalink / raw)
To: u-boot
On Sun, Oct 07, 2012 at 01:17:34PM +0200, Igor Grinberg wrote:
> Neither cm-t35, nor cm-t3730 is using OneNAND or flash.
> Remove the related defines from config file.
>
> Signed-off-by: Igor Grinberg <grinberg@compulab.co.il>
Applied to u-boot-ti/master, thanks!
--
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/20121017/2429b81e/attachment.pgp>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-10-17 15:40 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-10-07 11:17 [U-Boot] [PATCH 0/2] cm-t3x: minor config updates and cleanups Igor Grinberg
2012-10-07 11:17 ` [U-Boot] [PATCH 1/2] cm-t35: clean unused defines from config Igor Grinberg
2012-10-17 15:40 ` Tom Rini
2012-10-07 11:17 ` [U-Boot] [PATCH 2/2] cm-t35: increase the MAXARGS Igor Grinberg
2012-10-08 15:39 ` Tom Rini
2012-10-09 12:59 ` Igor Grinberg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox