[U-Boot] [PATCH] CM41xx: fix signedness of env bootargs string pointer

The pointer to the flash based bootargs should be a "char *", not unsigned. Fixes:
cm41xx.c: In function ‘env_flash_cmdline’: cm41xx.c:67: warning: pointer targets in passing argument 2 of ‘setenv’ differ in signedness
Signed-off-by: Greg Ungerer greg.ungerer@opengear.com --- board/cm41xx/cm41xx.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/board/cm41xx/cm41xx.c b/board/cm41xx/cm41xx.c index 02d05af..9a6d89f 100644 --- a/board/cm41xx/cm41xx.c +++ b/board/cm41xx/cm41xx.c @@ -46,8 +46,8 @@ DECLARE_GLOBAL_DATA_PTR; */ int env_flash_cmdline (void) { - unsigned char *sp = (unsigned char *) 0x0201c020; - unsigned char *ep; + char *sp = (char *) 0x0201c020; + char *ep; int len;
/* Check if "erase" push button is depressed */

Dear Greg Ungerer,
In message 1315643841-10616-1-git-send-email-greg.ungerer@opengear.com you wrote:
The pointer to the flash based bootargs should be a "char *", not unsigned. Fixes:
cm41xx.c: In function âenv_flash_cmdlineâ: cm41xx.c:67: warning: pointer targets in passing argument 2 of âsetenvâ differ in signedness
Signed-off-by: Greg Ungerer greg.ungerer@opengear.com
board/cm41xx/cm41xx.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)
Applied, thanks.
Best regards,
Wolfgang Denk
participants (2)
-
Greg Ungerer
-
Wolfgang Denk