fix printf related compiler warnings

fix printf related compiler warnings

[Sascha Hauer]

The printf format checking introduced with d66ae58cd87c78b76652ecf750371b1e005d44de
introduced a bunch of new compiler warnings. These patches fix
the warnings visible when compiling all defconfigs.

Sascha

The following changes since commit d4b8e67fcd9eccdc908cbd851ffe4d4a8de0a60a:

  nand: Fix warnings due to incompatible format strings (2011-01-07 10:34:27 +0100)

are available in the git repository at:
  git://git.pengutronix.de/git/barebox.git compiler-warnings

Sascha Hauer (24):
      i.MX: fix printf compiler warnings
      net: fix printf compiler warnings
      fec_imx: fix printf compiler warnings
      nand: fix printf compiler warnings
      cdev: fix printf compiler warnings
      usb: add missing byteorder include
      add to speed
      eukrea_cpuimx35: fix printf compiler warnings
      blackfin: fix printf compiler warnings
      cs8900: fix printf compiler warnings
      dm9000: fix printf compiler warnings
      smc911x: fix printf compiler warnings
      ipe337: fix printf compiler warnings
      device tree: fix printf compiler warnings
      netx: fix printf compiler warnings
      guf-cupid: fix printf compiler warnings
      pcm043: fix printf compiler warnings
      at91: fix printf compiler warnings
      ppc: fix printf compiler warnings
      imx-esdhc: fix printf compiler warnings
      mci-core: fix printf compiler warnings
      imx-ipu-fb: fix printf compiler warnings
      mpc5xxx: fix printf compiler warnings
      i.MX nand: do not select NAND_IMX_BOOT

 arch/arm/boards/eukrea_cpuimx35/eukrea_cpuimx35.c |    2 +-
 arch/arm/boards/guf-cupid/board.c                 |    2 +-
 arch/arm/boards/pcm043/pcm043.c                   |    2 +-
 arch/arm/mach-at91/clock.c                        |    2 +-
 arch/arm/mach-imx/Kconfig                         |    1 -
 arch/arm/mach-imx/speed-imx1.c                    |   20 ++++++++++----------
 arch/arm/mach-imx/speed-imx21.c                   |   20 ++++++++++----------
 arch/arm/mach-imx/speed-imx25.c                   |   20 ++++++++++----------
 arch/arm/mach-imx/speed-imx27.c                   |   20 ++++++++++----------
 arch/arm/mach-imx/speed-imx31.c                   |    4 ++--
 arch/arm/mach-imx/speed-imx35.c                   |   18 +++++++++---------
 arch/arm/mach-netx/generic.c                      |    2 +-
 arch/blackfin/boards/ipe337/cmd_alternate.c       |    4 ++--
 arch/blackfin/lib/blackfin_linux.c                |    2 +-
 arch/blackfin/lib/traps.c                         |    2 +-
 arch/ppc/lib/ppclinux.c                           |    2 +-
 arch/ppc/mach-mpc5xxx/cpu.c                       |    2 +-
 commands/nand.c                                   |    2 +-
 common/ft_build.c                                 |    2 +-
 drivers/mci/imx-esdhc.c                           |    2 +-
 drivers/mci/mci-core.c                            |    4 ++--
 drivers/mtd/ubi/cdev.c                            |    2 +-
 drivers/net/cs8900.c                              |    6 +++---
 drivers/net/dm9000.c                              |   12 ++++++------
 drivers/net/fec_imx.c                             |    6 +++---
 drivers/net/smc911x.c                             |    2 +-
 drivers/video/imx-ipu-fb.c                        |    2 +-
 include/usb/usb.h                                 |    1 +
 net/eth.c                                         |    2 +-
 29 files changed, 84 insertions(+), 84 deletions(-)

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 01/24] i.MX: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/mach-imx/speed-imx27.c |   20 ++++++++++----------
 1 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/arch/arm/mach-imx/speed-imx27.c b/arch/arm/mach-imx/speed-imx27.c
index 0a92d29..aba5097 100644
--- a/arch/arm/mach-imx/speed-imx27.c
+++ b/arch/arm/mach-imx/speed-imx27.c
@@ -172,16 +172,16 @@ void imx_dump_clocks(void)
 	       (cid >> 28) & 0xf, (cid >> 16) & 0xfff,
 	       (cid >> 12) & 0xf, (cid >>  0) & 0xfff);
 
-	printf("mpll:    %10d Hz\n", imx_get_mpllclk());
-	printf("spll:    %10d Hz\n", imx_get_spllclk());
-	printf("arm:     %10d Hz\n", imx_get_armclk());
-	printf("perclk1: %10d Hz\n", imx_get_perclk1());
-	printf("perclk2: %10d Hz\n", imx_get_perclk2());
-	printf("perclk3: %10d Hz\n", imx_get_perclk3());
-	printf("perclk4: %10d Hz\n", imx_get_perclk4());
-	printf("clkin26: %10d Hz\n", clk_in_26m());
-	printf("ahb:     %10d Hz\n", imx_get_ahbclk());
-	printf("ipg:     %10d Hz\n", imx_get_ipgclk());
+	printf("mpll:    %10ld Hz\n", imx_get_mpllclk());
+	printf("spll:    %10ld Hz\n", imx_get_spllclk());
+	printf("arm:     %10ld Hz\n", imx_get_armclk());
+	printf("perclk1: %10ld Hz\n", imx_get_perclk1());
+	printf("perclk2: %10ld Hz\n", imx_get_perclk2());
+	printf("perclk3: %10ld Hz\n", imx_get_perclk3());
+	printf("perclk4: %10ld Hz\n", imx_get_perclk4());
+	printf("clkin26: %10ld Hz\n", clk_in_26m());
+	printf("ahb:     %10ld Hz\n", imx_get_ahbclk());
+	printf("ipg:     %10ld Hz\n", imx_get_ipgclk());
 }
 
 /*
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 02/24] net: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 net/eth.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/eth.c b/net/eth.c
index 7502e98..8333d80 100644
--- a/net/eth.c
+++ b/net/eth.c
@@ -166,7 +166,7 @@ int eth_register(struct eth_device *edev)
 
 	if (edev->get_ethaddr(edev, ethaddr) == 0) {
 		ethaddr_to_string(ethaddr, ethaddr_str);
-		printf("got MAC address from EEPROM: %s\n",&ethaddr_str);
+		printf("got MAC address from EEPROM: %s\n", ethaddr_str);
 		dev_set_param(dev, "ethaddr", ethaddr_str);
 	}
 
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 03/24] fec_imx: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/net/fec_imx.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/fec_imx.c b/drivers/net/fec_imx.c
index e63e42e..51cdb40 100644
--- a/drivers/net/fec_imx.c
+++ b/drivers/net/fec_imx.c
@@ -458,7 +458,7 @@ static int fec_send(struct eth_device *dev, void *eth_data, int data_length)
 
 	/* Check for valid length of data. */
 	if ((data_length > 1500) || (data_length <= 0)) {
-		printf("Payload (%d) to large!\n");
+		printf("Payload (%d) to large!\n", data_length);
 		return -1;
 	}
 
@@ -519,7 +519,7 @@ static int fec_recv(struct eth_device *dev)
 {
 	struct fec_priv *fec = (struct fec_priv *)dev->priv;
 	struct buffer_descriptor __iomem *rbd = &fec->rbd_base[fec->rbd_index];
-	unsigned long ievent;
+	uint32_t ievent;
 	int frame_length, len = 0;
 	struct fec_frame *frame;
 	uint16_t bd_status;
@@ -577,7 +577,7 @@ static int fec_recv(struct eth_device *dev)
 			len = frame_length;
 		} else {
 			if (bd_status & FEC_RBD_ERR) {
-				printf("error frame: 0x%08x 0x%08x\n", rbd, bd_status);
+				printf("error frame: 0x%p 0x%08x\n", rbd, bd_status);
 			}
 		}
 		/*
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 04/24] nand: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 commands/nand.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/commands/nand.c b/commands/nand.c
index 0902f21..d3921b9 100644
--- a/commands/nand.c
+++ b/commands/nand.c
@@ -63,7 +63,7 @@ static ssize_t nand_bb_read(struct cdev *cdev, void *buf, size_t count,
 			return ret;
 
 		if (ret) {
-			printf("skipping bad block at 0x%08x\n", bb->offset);
+			printf("skipping bad block at 0x%08lx\n", bb->offset);
 			bb->offset += bb->info.erasesize;
 			continue;
 		}
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 05/24] cdev: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/mtd/ubi/cdev.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/mtd/ubi/cdev.c b/drivers/mtd/ubi/cdev.c
index 30b8308..5520132 100644
--- a/drivers/mtd/ubi/cdev.c
+++ b/drivers/mtd/ubi/cdev.c
@@ -24,7 +24,7 @@ static ssize_t ubi_volume_cdev_read(struct cdev *cdev, void *buf, size_t size,
 	loff_t offp = offset;
 	int usable_leb_size = vol->usable_leb_size;
 
-	printf("%s: %d @ 0x%08x\n", __func__, size, offset);
+	printf("%s: %d @ 0x%08lx\n", __func__, size, offset);
 
 	len = size > usable_leb_size ? usable_leb_size : size;
 
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 06/24] usb: add missing byteorder include

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 include/usb/usb.h |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/include/usb/usb.h b/include/usb/usb.h
index 1b936ec..1e4d750 100644
--- a/include/usb/usb.h
+++ b/include/usb/usb.h
@@ -30,6 +30,7 @@
 
 #include <driver.h>
 #include <usb/usb_defs.h>
+#include <asm/byteorder.h>
 
 /* Everything is aribtrary */
 #define USB_ALTSETTINGALLOC		4
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 07/24] add to speed

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/mach-imx/speed-imx1.c  |   20 ++++++++++----------
 arch/arm/mach-imx/speed-imx21.c |   20 ++++++++++----------
 arch/arm/mach-imx/speed-imx25.c |   20 ++++++++++----------
 arch/arm/mach-imx/speed-imx31.c |    4 ++--
 arch/arm/mach-imx/speed-imx35.c |   18 +++++++++---------
 5 files changed, 41 insertions(+), 41 deletions(-)

diff --git a/arch/arm/mach-imx/speed-imx1.c b/arch/arm/mach-imx/speed-imx1.c
index a4b4f96..6fd549f 100644
--- a/arch/arm/mach-imx/speed-imx1.c
+++ b/arch/arm/mach-imx/speed-imx1.c
@@ -81,15 +81,15 @@ ulong imx_get_gptclk(void)
 
 void imx_dump_clocks(void)
 {
-	printf("spll:    %10d Hz\n", imx_get_spllclk());
-	printf("mpll:    %10d Hz\n", imx_get_mpllclk());
-	printf("fclk:    %10d Hz\n", imx_get_fclk());
-	printf("hclk:    %10d Hz\n", imx_get_hclk());
-	printf("bclk:    %10d Hz\n", imx_get_bclk());
-	printf("perclk1: %10d Hz\n", imx_get_perclk1());
-	printf("perclk2: %10d Hz\n", imx_get_perclk2());
-	printf("perclk3: %10d Hz\n", imx_get_perclk3());
-	printf("uart:    %10d Hz\n", imx_get_uartclk());
-	printf("gpt:     %10d Hz\n", imx_get_gptclk());
+	printf("spll:    %10ld Hz\n", imx_get_spllclk());
+	printf("mpll:    %10ld Hz\n", imx_get_mpllclk());
+	printf("fclk:    %10ld Hz\n", imx_get_fclk());
+	printf("hclk:    %10ld Hz\n", imx_get_hclk());
+	printf("bclk:    %10ld Hz\n", imx_get_bclk());
+	printf("perclk1: %10ld Hz\n", imx_get_perclk1());
+	printf("perclk2: %10ld Hz\n", imx_get_perclk2());
+	printf("perclk3: %10ld Hz\n", imx_get_perclk3());
+	printf("uart:    %10ld Hz\n", imx_get_uartclk());
+	printf("gpt:     %10ld Hz\n", imx_get_gptclk());
 }
 
diff --git a/arch/arm/mach-imx/speed-imx21.c b/arch/arm/mach-imx/speed-imx21.c
index 9ae8026..6ab1dca 100644
--- a/arch/arm/mach-imx/speed-imx21.c
+++ b/arch/arm/mach-imx/speed-imx21.c
@@ -145,16 +145,16 @@ void imx_dump_clocks(void)
 	uint32_t	cid = CID;
 
 	printf("chip id: [%08x]\n", cid);
-	printf("mpll:    %10d Hz\n", imx_get_mpllclk());
-	printf("spll:    %10d Hz\n", imx_get_spllclk());
-	printf("arm:     %10d Hz\n", imx_get_armclk());
-	printf("fclk:    %10d Hz\n", imx_get_fclk());
-	printf("nfcclk:  %10d Hz\n", imx_get_nfcclk());
-	printf("perclk1: %10d Hz\n", imx_get_perclk1());
-	printf("perclk2: %10d Hz\n", imx_get_perclk2());
-	printf("perclk3: %10d Hz\n", imx_get_perclk3());
-	printf("perclk4: %10d Hz\n", imx_get_perclk4());
-	printf("clkin26: %10d Hz\n", clk_in_26m());
+	printf("mpll:    %10ld Hz\n", imx_get_mpllclk());
+	printf("spll:    %10ld Hz\n", imx_get_spllclk());
+	printf("arm:     %10ld Hz\n", imx_get_armclk());
+	printf("fclk:    %10ld Hz\n", imx_get_fclk());
+	printf("nfcclk:  %10ld Hz\n", imx_get_nfcclk());
+	printf("perclk1: %10ld Hz\n", imx_get_perclk1());
+	printf("perclk2: %10ld Hz\n", imx_get_perclk2());
+	printf("perclk3: %10ld Hz\n", imx_get_perclk3());
+	printf("perclk4: %10ld Hz\n", imx_get_perclk4());
+	printf("clkin26: %10ld Hz\n", clk_in_26m());
 }
 
 /*
diff --git a/arch/arm/mach-imx/speed-imx25.c b/arch/arm/mach-imx/speed-imx25.c
index cb28e9f..68b50e9 100644
--- a/arch/arm/mach-imx/speed-imx25.c
+++ b/arch/arm/mach-imx/speed-imx25.c
@@ -89,16 +89,16 @@ unsigned long imx_get_mmcclk(void)
 
 void imx_dump_clocks(void)
 {
-	printf("mpll:    %10d Hz\n", imx_get_mpllclk());
-	printf("upll:    %10d Hz\n", imx_get_upllclk());
-	printf("arm:     %10d Hz\n", imx_get_armclk());
-	printf("ahb:     %10d Hz\n", imx_get_ahbclk());
-	printf("uart:    %10d Hz\n", imx_get_perclk(15));
-	printf("gpt:     %10d Hz\n", imx_get_ipgclk());
-	printf("nand:    %10d Hz\n", imx_get_perclk(8));
-	printf("lcd:     %10d Hz\n", imx_get_perclk(7));
-	printf("i2c:     %10d Hz\n", imx_get_perclk(6));
-	printf("sdhc1:   %10d Hz\n", imx_get_perclk(3));
+	printf("mpll:    %10ld Hz\n", imx_get_mpllclk());
+	printf("upll:    %10ld Hz\n", imx_get_upllclk());
+	printf("arm:     %10ld Hz\n", imx_get_armclk());
+	printf("ahb:     %10ld Hz\n", imx_get_ahbclk());
+	printf("uart:    %10ld Hz\n", imx_get_perclk(15));
+	printf("gpt:     %10ld Hz\n", imx_get_ipgclk());
+	printf("nand:    %10ld Hz\n", imx_get_perclk(8));
+	printf("lcd:     %10ld Hz\n", imx_get_perclk(7));
+	printf("i2c:     %10ld Hz\n", imx_get_perclk(6));
+	printf("sdhc1:   %10ld Hz\n", imx_get_perclk(3));
 }
 
 /*
diff --git a/arch/arm/mach-imx/speed-imx31.c b/arch/arm/mach-imx/speed-imx31.c
index 70b1330..32a517d 100644
--- a/arch/arm/mach-imx/speed-imx31.c
+++ b/arch/arm/mach-imx/speed-imx31.c
@@ -67,8 +67,8 @@ ulong imx_get_perclk1(void)
 void imx_dump_clocks(void)
 {
 	ulong cpufreq = imx_get_mcu_main_clk();
-	printf("mx31 cpu clock: %dMHz\n",cpufreq / 1000000);
-	printf("ipg clock     : %dHz\n", imx_get_perclk1());
+	printf("mx31 cpu clock: %ldMHz\n",cpufreq / 1000000);
+	printf("ipg clock     : %ldHz\n", imx_get_perclk1());
 }
 
 ulong imx_get_uartclk(void)
diff --git a/arch/arm/mach-imx/speed-imx35.c b/arch/arm/mach-imx/speed-imx35.c
index 8937ef1..b2a339c 100644
--- a/arch/arm/mach-imx/speed-imx35.c
+++ b/arch/arm/mach-imx/speed-imx35.c
@@ -187,15 +187,15 @@ ulong imx_get_i2cclk(void)
 
 void imx_dump_clocks(void)
 {
-	printf("mpll:    %10d Hz\n", imx_get_mpllclk());
-	printf("ppll:    %10d Hz\n", imx_get_ppllclk());
-	printf("arm:     %10d Hz\n", imx_get_armclk());
-	printf("gpt:     %10d Hz\n", imx_get_gptclk());
-	printf("ahb:     %10d Hz\n", imx_get_ahbclk());
-	printf("ipg:     %10d Hz\n", imx_get_ipgclk());
-	printf("ipg_per: %10d Hz\n", imx_get_ipg_perclk());
-	printf("uart:	 %10d Hz\n", imx_get_uartclk());
-	printf("sdhc1:   %10d Hz\n", imx_get_mmcclk());
+	printf("mpll:    %10ld Hz\n", imx_get_mpllclk());
+	printf("ppll:    %10ld Hz\n", imx_get_ppllclk());
+	printf("arm:     %10ld Hz\n", imx_get_armclk());
+	printf("gpt:     %10ld Hz\n", imx_get_gptclk());
+	printf("ahb:     %10ld Hz\n", imx_get_ahbclk());
+	printf("ipg:     %10ld Hz\n", imx_get_ipgclk());
+	printf("ipg_per: %10ld Hz\n", imx_get_ipg_perclk());
+	printf("uart:	 %10ld Hz\n", imx_get_uartclk());
+	printf("sdhc1:   %10ld Hz\n", imx_get_mmcclk());
 }
 
 /*
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 07/24] add to speed

[Uwe Kleine-König]

... the commit log is quite sparse.  I don't understand it when looking
at the patch.

Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 08/24] eukrea_cpuimx35: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/boards/eukrea_cpuimx35/eukrea_cpuimx35.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/boards/eukrea_cpuimx35/eukrea_cpuimx35.c b/arch/arm/boards/eukrea_cpuimx35/eukrea_cpuimx35.c
index 07f320b..f7a6169 100644
--- a/arch/arm/boards/eukrea_cpuimx35/eukrea_cpuimx35.c
+++ b/arch/arm/boards/eukrea_cpuimx35/eukrea_cpuimx35.c
@@ -386,7 +386,7 @@ static int do_cpufreq(struct command *cmdtp, int argc, char *argv[])
 		return COMMAND_ERROR_USAGE;
 	}
 
-	printf("Switched CPU frequency to %dMHz\n", freq);
+	printf("Switched CPU frequency to %ldMHz\n", freq);
 
 	return 0;
 }
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 09/24] blackfin: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/blackfin/lib/blackfin_linux.c |    2 +-
 arch/blackfin/lib/traps.c          |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/blackfin/lib/blackfin_linux.c b/arch/blackfin/lib/blackfin_linux.c
index eda3f4c..a20cf55 100644
--- a/arch/blackfin/lib/blackfin_linux.c
+++ b/arch/blackfin/lib/blackfin_linux.c
@@ -48,7 +48,7 @@ static int do_bootm_linux(struct image_data *idata)
 	image_header_t *os_header = &os_handle->header;
 
 	appl = (int (*)(char *))image_get_ep(os_header);
-	printf("Starting Kernel at 0x%08x\n", appl);
+	printf("Starting Kernel at 0x%p\n", appl);
 
 	if (relocate_image(os_handle, (void *)image_get_load(os_header)))
 		return -1;
diff --git a/arch/blackfin/lib/traps.c b/arch/blackfin/lib/traps.c
index ce479b9..14f1da9 100644
--- a/arch/blackfin/lib/traps.c
+++ b/arch/blackfin/lib/traps.c
@@ -81,7 +81,7 @@ static const char *trap_to_string(int trapnr)
 
 void trap_c (struct pt_regs *regs)
 {
-	unsigned long trapnr = (regs->seqstat) & SEQSTAT_EXCAUSE;
+	uint32_t trapnr = (regs->seqstat) & SEQSTAT_EXCAUSE;
 	const char *str;
 
 	printf("Exception occured!\n\n");
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 10/24] cs8900: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/net/cs8900.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/net/cs8900.c b/drivers/net/cs8900.c
index 8120877..b8264d7 100644
--- a/drivers/net/cs8900.c
+++ b/drivers/net/cs8900.c
@@ -364,7 +364,7 @@ static void cs8900_info(struct device_d *dev)
 	struct cs8900_priv *priv = (struct cs8900_priv *)edev->priv;
 	u16 v;
 
-	printf("%s Rev. %s (PID: 0x%04x) at 0x%08x\n", priv->chip->name,
+	printf("%s Rev. %s (PID: 0x%04x) at 0x%p\n", priv->chip->name,
 	       priv->product->rev_name, priv->product->product_id,
 	       priv->regs);
 
@@ -399,7 +399,7 @@ static int cs8900_check_id(struct cs8900_priv *priv)
 	u16 v;
 	v = cs8900_ior(priv, PP_BI_VID);
 	if (v != CRYSTAL_SEMI_EISA_ID) {
-		printf("No CS89x0 variant found at 0x%08x vid: 0x%04x\n",
+		printf("No CS89x0 variant found at 0x%p vid: 0x%04x\n",
 		       priv->regs, v);
 		return -1;
 	}
@@ -424,7 +424,7 @@ static int cs8900_check_id(struct cs8900_priv *priv)
 		printf("Invalid chip type %d\n", product->chip_type);
 		goto out;
 	}
-	printf("%s Rev. %s (PID: 0x%04x) found at 0x%08x\n", chip->name,
+	printf("%s Rev. %s (PID: 0x%04x) found at 0x%p\n", chip->name,
 	       product->rev_name, v, priv->regs);
 	priv->chip = chip;
 	priv->product = product;
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 11/24] dm9000: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/net/dm9000.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/net/dm9000.c b/drivers/net/dm9000.c
index c31cf1a..b0e8822 100644
--- a/drivers/net/dm9000.c
+++ b/drivers/net/dm9000.c
@@ -165,8 +165,8 @@
 #define IMR_PRM			(1<<0)
 
 struct dm9000_priv {
-	unsigned long iobase;
-	unsigned long iodata;
+	void __iomem *iobase;
+	void __iomem *iodata;
 	struct mii_device miidev;
 	int buswidth;
 };
@@ -246,11 +246,11 @@ static int dm9000_check_id(struct dm9000_priv *priv)
 	id_val |= DM9000_ior(priv, DM9000_PIDL) << 16;
 	id_val |= DM9000_ior(priv, DM9000_PIDH) << 24;
 	if (id_val == DM9000_ID) {
-		printf("dm9000 i/o: 0x%x, id: 0x%x \n", priv->iobase,
+		printf("dm9000 i/o: 0x%p, id: 0x%x \n", priv->iobase,
 		       id_val);
 		return 0;
 	} else {
-		printf("dm9000 not found at 0x%08x id: 0x%08x\n",
+		printf("dm9000 not found at 0x%p id: 0x%08x\n",
 		       priv->iobase, id_val);
 		return -1;
 	}
@@ -489,8 +489,8 @@ static int dm9000_probe(struct device_d *dev)
 
 	priv = edev->priv;
 	priv->buswidth = pdata->buswidth;
-	priv->iodata = pdata->iodata;
-	priv->iobase = pdata->iobase;
+	priv->iodata = (void *)pdata->iodata;
+	priv->iobase = (void *)pdata->iobase;
 
 	edev->init = dm9000_init_dev;
 	edev->open = dm9000_eth_open;
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 12/24] smc911x: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/net/smc911x.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/net/smc911x.c b/drivers/net/smc911x.c
index b559590..e5c0e7e 100644
--- a/drivers/net/smc911x.c
+++ b/drivers/net/smc911x.c
@@ -687,7 +687,7 @@ static int smc911x_probe(struct device_d *dev)
 {
 	struct eth_device *edev;
 	struct smc911x_priv *priv;
-	ulong val;
+	uint32_t val;
 	int i;
 
 	debug("smc911x_eth_init()\n");
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 13/24] ipe337: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/blackfin/boards/ipe337/cmd_alternate.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/blackfin/boards/ipe337/cmd_alternate.c b/arch/blackfin/boards/ipe337/cmd_alternate.c
index 2883c77..116f910 100644
--- a/arch/blackfin/boards/ipe337/cmd_alternate.c
+++ b/arch/blackfin/boards/ipe337/cmd_alternate.c
@@ -21,7 +21,7 @@ static int do_alternate(struct command *cmdtp, int argc, char *argv[])
 
 	ptr = buf;
 	if ((*ptr) != MAGIC) {
-		printf("Wrong magic! Expected 0x%08x, got 0x%08x.\n", MAGIC, *ptr);
+		printf("Wrong magic! Expected 0x%08x, got 0x%08lx.\n", MAGIC, *ptr);
 		return 1;
 	}
 
@@ -38,7 +38,7 @@ static int do_alternate(struct command *cmdtp, int argc, char *argv[])
 		} while (val >>= 1);
 	}
 
-	printf("Bitcount : %d\n", bitcount);
+	printf("Bitcount : %ld\n", bitcount);
 
 	free(buf);
 	return (bitcount & 1) ? 3 : 2;
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 14/24] device tree: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 common/ft_build.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/common/ft_build.c b/common/ft_build.c
index 4a7ff31..4c27cfa 100644
--- a/common/ft_build.c
+++ b/common/ft_build.c
@@ -311,7 +311,7 @@ void ft_dump_blob(const void *bphp)
 		}
 
 		if (tag != OF_DT_PROP) {
-			fprintf(stderr, "%*s ** Unknown tag 0x%08x at 0x%x\n",
+			fprintf(stderr, "%*s ** Unknown tag 0x%08x at 0x%p\n",
 				depth * shift, "", tag, --p);
 			break;
 		}
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 15/24] netx: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/mach-netx/generic.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-netx/generic.c b/arch/arm/mach-netx/generic.c
index de1e1b0..6966def 100644
--- a/arch/arm/mach-netx/generic.c
+++ b/arch/arm/mach-netx/generic.c
@@ -41,7 +41,7 @@ static int xc_check_ptr(int xcno, unsigned long adr, unsigned int size)
 	    adr + size < NETX_PA_XPEC(xcno) + XPEC_MEM_SIZE)
 		return 0;
 
-	printf("%s: illegal pointer %p\n",__FUNCTION__,adr);
+	printf("%s: illegal pointer 0x%08lx\n", __func__ ,adr);
 	return -1;
 }
 
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 16/24] guf-cupid: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/boards/guf-cupid/board.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/boards/guf-cupid/board.c b/arch/arm/boards/guf-cupid/board.c
index 6d7a99b..e30f8e5 100644
--- a/arch/arm/boards/guf-cupid/board.c
+++ b/arch/arm/boards/guf-cupid/board.c
@@ -408,7 +408,7 @@ static int do_cpufreq(struct command *cmdtp, int argc, char *argv[])
 		return COMMAND_ERROR_USAGE;
 	}
 
-	printf("Switched CPU frequency to %dMHz\n", freq);
+	printf("Switched CPU frequency to %ldMHz\n", freq);
 
 	return 0;
 }
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 17/24] pcm043: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/boards/pcm043/pcm043.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/boards/pcm043/pcm043.c b/arch/arm/boards/pcm043/pcm043.c
index 38f1b8f..36bde45 100644
--- a/arch/arm/boards/pcm043/pcm043.c
+++ b/arch/arm/boards/pcm043/pcm043.c
@@ -354,7 +354,7 @@ static int do_cpufreq(struct command *cmdtp, int argc, char *argv[])
 		return COMMAND_ERROR_USAGE;
 	}
 
-	printf("Switched CPU frequency to %dMHz\n", freq);
+	printf("Switched CPU frequency to %ldMHz\n", freq);
 
 	return 0;
 }
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 18/24] at91: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/mach-at91/clock.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-at91/clock.c b/arch/arm/mach-at91/clock.c
index 6fd09d5..4d77a37 100644
--- a/arch/arm/mach-at91/clock.c
+++ b/arch/arm/mach-at91/clock.c
@@ -679,7 +679,7 @@ static int at91_clock_display(void)
 	if (pll_overclock)
 		pr_info("Clocks: PLLA overclocked, %ld MHz\n", plla.rate_hz / 1000000);
 
-	printf("Clocks: CPU %u MHz, master %u MHz, main %u.%03u MHz\n",
+	printf("Clocks: CPU %lu MHz, master %u MHz, main %u.%03u MHz\n",
 		mck.parent->rate_hz / 1000000, (unsigned) mck.rate_hz / 1000000,
 		(unsigned) main_clk.rate_hz / 1000000,
 		((unsigned) main_clk.rate_hz % 1000000) / 1000);
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 19/24] ppc: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/ppc/lib/ppclinux.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/ppc/lib/ppclinux.c b/arch/ppc/lib/ppclinux.c
index fc22a87..f27955d 100644
--- a/arch/ppc/lib/ppclinux.c
+++ b/arch/ppc/lib/ppclinux.c
@@ -219,7 +219,7 @@ static int do_bootm_linux(struct image_data *idata)
 		else
 			of_start  = (ulong)kbd - of_len - 8192;
 		of_start &= ~(4096 - 1);	/* align on page */
-		debug ("## device tree at 0x%08lX ... 0x%08lX (len=%ld=0x%lX)\n",
+		debug ("## device tree at 0x%p ... 0x%p (len=%ld=0x%lX)\n",
 			of_data, of_data + of_len - 1, of_len, of_len);
 
 		of_flat_tree = (char *)of_start;
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 20/24] imx-esdhc: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/mci/imx-esdhc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/mci/imx-esdhc.c b/drivers/mci/imx-esdhc.c
index 63cd059..2937156 100644
--- a/drivers/mci/imx-esdhc.c
+++ b/drivers/mci/imx-esdhc.c
@@ -355,7 +355,7 @@ void set_sysctl(struct mci_host *mci, u32 clock)
 	div -= 1;
 	pre_div >>= 1;
 
-	dev_dbg(host->dev, "set clock: wanted: %d got: %d\n", clock, host->cur_clock);
+	dev_dbg(host->dev, "set clock: wanted: %d got: %ld\n", clock, host->cur_clock);
 	dev_dbg(host->dev, "pre_div: %d div: %d\n", pre_div, div);
 
 	clk = (pre_div << 8) | (div << 4);
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 21/24] mci-core: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/mci/mci-core.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mci/mci-core.c b/drivers/mci/mci-core.c
index 04a1e4c..c562700 100644
--- a/drivers/mci/mci-core.c
+++ b/drivers/mci/mci-core.c
@@ -1001,13 +1001,13 @@ static int mci_sd_read(struct device_d *disk_dev, uint64_t sector_start,
 	while (sector_count) {
 		int now = min(sector_count, 32U);
 		if (sector_start > MAX_BUFFER_NUMBER) {
-			pr_err("Cannot handle block number %lu. Too large!\n",
+			pr_err("Cannot handle block number %u. Too large!\n",
 				(unsigned)sector_start);
 			return -EINVAL;
 		}
 		rc = mci_read_block(mci_dev, buffer, (unsigned)sector_start, now);
 		if (rc != 0) {
-			pr_err("Reading block %lu failed with %d\n", (unsigned)sector_start, rc);
+			pr_err("Reading block %u failed with %d\n", (unsigned)sector_start, rc);
 			return rc;
 		}
 		sector_count -= now;
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 22/24] imx-ipu-fb: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/video/imx-ipu-fb.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/video/imx-ipu-fb.c b/drivers/video/imx-ipu-fb.c
index c38082d..d9e1e21 100644
--- a/drivers/video/imx-ipu-fb.c
+++ b/drivers/video/imx-ipu-fb.c
@@ -488,7 +488,7 @@ static int sdc_init_panel(struct fb_info *info, enum pixel_fmt pixel_fmt)
 		div = 0x40;
 	}
 
-	dev_dbg(&info->dev, "pixel clk = %u, divider %u.%u\n",
+	dev_dbg(&info->dev, "pixel clk = %lu, divider %u.%u\n",
 		pixel_clk, div >> 4, (div & 7) * 125);
 
 	/*
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 23/24] mpc5xxx: fix printf compiler warnings

[Sascha Hauer]

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/ppc/mach-mpc5xxx/cpu.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/ppc/mach-mpc5xxx/cpu.c b/arch/ppc/mach-mpc5xxx/cpu.c
index 4d08c55..be7c19d 100644
--- a/arch/ppc/mach-mpc5xxx/cpu.c
+++ b/arch/ppc/mach-mpc5xxx/cpu.c
@@ -65,7 +65,7 @@ int checkcpu (void)
 	printf (" v%d.%d, Core v%d.%d", SVR_MJREV (svr), SVR_MNREV (svr),
 		PVR_MAJ(pvr), PVR_MIN(pvr));
 #endif
-	printf (" at %s Hz\n", clock);
+	printf (" at %ld Hz\n", clock);
 	return 0;
 }
 
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

[PATCH 24/24] i.MX nand: do not select NAND_IMX_BOOT

[Sascha Hauer]

NAND_IMX_BOOT is related to the external boot mode, so do not
select it when using internal boot mode.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 arch/arm/mach-imx/Kconfig |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
index f3506af..c9f55b5 100644
--- a/arch/arm/mach-imx/Kconfig
+++ b/arch/arm/mach-imx/Kconfig
@@ -56,7 +56,6 @@ choice
 
 config ARCH_IMX_INTERNAL_BOOT_NAND
 	bool "NAND, SD/MMC, Serial ROM"
-	select NAND_IMX_BOOT
 
 config ARCH_IMX_INTERNAL_BOOT_NOR
 	bool "NOR"
-- 
1.7.2.3


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 24/24] i.MX nand: do not select NAND_IMX_BOOT

[Juergen Beisert]

Sascha Hauer wrote:
> NAND_IMX_BOOT is related to the external boot mode, so do not
> select it when using internal boot mode.
>
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> ---
>  arch/arm/mach-imx/Kconfig |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> index f3506af..c9f55b5 100644
> --- a/arch/arm/mach-imx/Kconfig
> +++ b/arch/arm/mach-imx/Kconfig
> @@ -56,7 +56,6 @@ choice
>
>  config ARCH_IMX_INTERNAL_BOOT_NAND
>  	bool "NAND, SD/MMC, Serial ROM"
> -	select NAND_IMX_BOOT
>
>  config ARCH_IMX_INTERNAL_BOOT_NOR
>  	bool "NOR"

Seems not related to the 'printf' topic?

jbe

-- 
Pengutronix e.K.                              | Juergen Beisert             |
Linux Solutions for Science and Industry      | Phone: +49-8766-939 228     |
Vertretung Sued/Muenchen, Germany             | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686              | http://www.pengutronix.de/  |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH 24/24] i.MX nand: do not select NAND_IMX_BOOT

[Sascha Hauer]

On Fri, Jan 07, 2011 at 12:05:13PM +0100, Juergen Beisert wrote:
> Sascha Hauer wrote:
> > NAND_IMX_BOOT is related to the external boot mode, so do not
> > select it when using internal boot mode.
> >
> > Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> > ---
> >  arch/arm/mach-imx/Kconfig |    1 -
> >  1 files changed, 0 insertions(+), 1 deletions(-)
> >
> > diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig
> > index f3506af..c9f55b5 100644
> > --- a/arch/arm/mach-imx/Kconfig
> > +++ b/arch/arm/mach-imx/Kconfig
> > @@ -56,7 +56,6 @@ choice
> >
> >  config ARCH_IMX_INTERNAL_BOOT_NAND
> >  	bool "NAND, SD/MMC, Serial ROM"
> > -	select NAND_IMX_BOOT
> >
> >  config ARCH_IMX_INTERNAL_BOOT_NOR
> >  	bool "NOR"
> 
> Seems not related to the 'printf' topic?

Pssst :)

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox