[PATCH 1/1] iwl4965: fix %d confusingly prefixed with 0x in format string

2014-09-05 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/net/wireless/iwlegacy/4965-mac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/iwlegacy/4965-mac.c 
b/drivers/net/wireless/iwlegacy/4965-mac.c
index 3dcbe2c..cf7996f 100644
--- a/drivers/net/wireless/iwlegacy/4965-mac.c
+++ b/drivers/net/wireless/iwlegacy/4965-mac.c
@@ -4633,7 +4633,7 @@ il4965_store_tx_power(struct device *d, struct 
device_attribute *attr,
else {
ret = il_set_tx_power(il, val, false);
if (ret)
-   IL_ERR("failed setting tx power (0x%d).\n", ret);
+   IL_ERR("failed setting tx power (0x%08x).\n", ret);
else
ret = count;
}
-- 
2.1.0.rc2.206.gedb03e5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 1/1] rtlwifi: fix %d confusingly prefixed with 0x in format strings

2014-09-05 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/net/wireless/rtlwifi/pci.c   | 6 +++---
 drivers/net/wireless/rtlwifi/rtl8192de/phy.c | 8 
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/net/wireless/rtlwifi/pci.c 
b/drivers/net/wireless/rtlwifi/pci.c
index 67d1ee6..74a8ba4 100644
--- a/drivers/net/wireless/rtlwifi/pci.c
+++ b/drivers/net/wireless/rtlwifi/pci.c
@@ -646,7 +646,7 @@ static void _rtl_pci_tx_isr(struct ieee80211_hw *hw, int 
prio)
== 2) {
 
RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD,
-"more desc left, wake skb_queue@%d, ring->idx 
= %d, skb_queue_len = 0x%d\n",
+"more desc left, wake skb_queue@%d, ring->idx 
= %d, skb_queue_len = 0x%x\n",
 prio, ring->idx,
 skb_queue_len(&ring->queue));
 
@@ -1469,7 +1469,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
 
if ((own == 1) && (hw_queue != BEACON_QUEUE)) {
RT_TRACE(rtlpriv, COMP_ERR, DBG_WARNING,
-"No more TX desc@%d, ring->idx = %d, idx = %d, 
skb_queue_len = 0x%d\n",
+"No more TX desc@%d, ring->idx = %d, idx = %d, 
skb_queue_len = 0x%x\n",
 hw_queue, ring->idx, idx,
 skb_queue_len(&ring->queue));
 
@@ -1511,7 +1511,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
if ((ring->entries - skb_queue_len(&ring->queue)) < 2 &&
hw_queue != BEACON_QUEUE) {
RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD,
-"less desc left, stop skb_queue@%d, ring->idx = %d, 
idx = %d, skb_queue_len = 0x%d\n",
+"less desc left, stop skb_queue@%d, ring->idx = %d, 
idx = %d, skb_queue_len = 0x%x\n",
 hw_queue, ring->idx, idx,
 skb_queue_len(&ring->queue));
 
diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/phy.c 
b/drivers/net/wireless/rtlwifi/rtl8192de/phy.c
index 592125a..1961b8e 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192de/phy.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192de/phy.c
@@ -677,7 +677,7 @@ static void _rtl92d_store_pwrindex_diffrate_offset(struct 
ieee80211_hw *hw,
 
rtlphy->mcs_offset[rtlphy->pwrgroup_cnt][index] = data;
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
-"MCSTxPowerLevelOriginalOffset[%d][%d] = 0x%ulx\n",
+"MCSTxPowerLevelOriginalOffset[%d][%d] = 0x%x\n",
 rtlphy->pwrgroup_cnt, index,
 rtlphy->mcs_offset[rtlphy->pwrgroup_cnt][index]);
if (index == 13)
@@ -2531,7 +2531,7 @@ static void _rtl92d_phy_reload_lck_setting(struct 
ieee80211_hw *hw,
if (rtlpriv->rtlhal.current_bandtype == BAND_ON_5G) {/* Path-A for 5G */
u4tmp = curveindex_5g[channel-1];
RTPRINT(rtlpriv, FINIT, INIT_IQK,
-   "ver 1 set RF-A, 5G,0x28 = 0x%ulx !!\n", u4tmp);
+   "ver 1 set RF-A, 5G,0x28 = 0x%x !!\n", u4tmp);
if (rtlpriv->rtlhal.macphymode == DUALMAC_DUALPHY &&
rtlpriv->rtlhal.interfaceindex == 1) {
bneed_powerdown_radio =
@@ -2550,7 +2550,7 @@ static void _rtl92d_phy_reload_lck_setting(struct 
ieee80211_hw *hw,
} else if (rtlpriv->rtlhal.current_bandtype == BAND_ON_2_4G) {
u4tmp = curveindex_2g[channel-1];
RTPRINT(rtlpriv, FINIT, INIT_IQK,
-   "ver 3 set RF-B, 2G, 0x28 = 0x%ulx !!\n", u4tmp);
+   "ver 3 set RF-B, 2G, 0x28 = 0x%x !!\n", u4tmp);
if (rtlpriv->rtlhal.macphymode == DUALMAC_DUALPHY &&
rtlpriv->rtlhal.interfaceindex == 0) {
bneed_powerdown_radio =
@@ -2562,7 +2562,7 @@ static void _rtl92d_phy_reload_lck_setting(struct 
ieee80211_hw *hw,
}
rtl_set_rfreg(hw, erfpath, RF_SYN_G4, 0x3f800, u4tmp);
RTPRINT(rtlpriv, FINIT, INIT_IQK,
-   "ver 3 set RF-B, 2G, 0x28 = 0x%ulx !!\n",
+   "ver 3 set RF-B, 2G, 0x28 = 0x%x !!\n",
rtl_get_rfreg(hw,  erfpath, RF_SYN_G4, 0x3f800));
if (bneed_powerdown_radio)
_rtl92d_phy_restore_rf_env(hw, erfpath, &u4regvalue);
-- 
2.1.0.rc2.206.gedb03e5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 1/1] ath6kl: fix %d confusingly prefixed with 0x in format strings

2014-08-17 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/net/wireless/ath/ath6kl/init.c | 2 +-
 drivers/net/wireless/ath/ath6kl/main.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ath/ath6kl/init.c 
b/drivers/net/wireless/ath/ath6kl/init.c
index fffd523..6e473fa 100644
--- a/drivers/net/wireless/ath/ath6kl/init.c
+++ b/drivers/net/wireless/ath/ath6kl/init.c
@@ -1049,7 +1049,7 @@ static int ath6kl_fetch_fw_apin(struct ath6kl *ar, const 
char *name)
ar->hw.reserved_ram_size = le32_to_cpup(val);
 
ath6kl_dbg(ATH6KL_DBG_BOOT,
-  "found reserved ram size ie 0x%d\n",
+  "found reserved ram size ie %d\n",
   ar->hw.reserved_ram_size);
break;
case ATH6KL_FW_IE_CAPABILITIES:
diff --git a/drivers/net/wireless/ath/ath6kl/main.c 
b/drivers/net/wireless/ath/ath6kl/main.c
index 21516bc..933aef0 100644
--- a/drivers/net/wireless/ath/ath6kl/main.c
+++ b/drivers/net/wireless/ath/ath6kl/main.c
@@ -225,7 +225,7 @@ int ath6kl_diag_write32(struct ath6kl *ar, u32 address, 
__le32 value)
ret = ath6kl_hif_diag_write32(ar, address, value);
 
if (ret) {
-   ath6kl_err("failed to write 0x%x during diagnose window to 
0x%d\n",
+   ath6kl_err("failed to write 0x%x during diagnose window to 
0x%x\n",
   address, value);
return ret;
}
-- 
2.1.0.rc2.206.gedb03e5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: staging: rtl8192ee: fix %d confusingly prefixed with 0x in format strings

2014-08-10 Thread Hans Wennborg 
Re-sending patch based off of
https://kernel.googlesource.com/pub/scm/linux/kernel/git/gregkh/staging

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH] staging: rtl8192ee: fix %d confusingly prefixed with 0x in format strings

2014-08-10 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/staging/rtl8192ee/pci.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8192ee/pci.c b/drivers/staging/rtl8192ee/pci.c
index 0215aef..349e636 100644
--- a/drivers/staging/rtl8192ee/pci.c
+++ b/drivers/staging/rtl8192ee/pci.c
@@ -649,7 +649,7 @@ static void _rtl_pci_tx_isr(struct ieee80211_hw *hw, int 
prio)
 
if ((ring->entries - skb_queue_len(&ring->queue)) == 2) {
RT_TRACE(COMP_ERR, DBG_LOUD,
-("more desc left, wake skb_queue@%d,ring->idx 
= %d, skb_queue_len = 0x%d\n",
+("more desc left, wake skb_queue@%d,ring->idx 
= %d, skb_queue_len = 0x%x\n",
 prio, ring->idx,
 skb_queue_len(&ring->queue)));
 
@@ -1628,7 +1628,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
 
if ((own == 1) && (hw_queue != BEACON_QUEUE)) {
RT_TRACE(COMP_ERR, DBG_WARNING,
-("No more TX desc@%d, ring->idx = %d, idx = 
%d, skb_queue_len = 0x%d\n",
+("No more TX desc@%d, ring->idx = %d, idx = 
%d, skb_queue_len = 0x%x\n",
  hw_queue, ring->idx, idx,
  skb_queue_len(&ring->queue)));
 
@@ -1670,7 +1670,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
if ((ring->entries - skb_queue_len(&ring->queue)) < 2 &&
hw_queue != BEACON_QUEUE) {
RT_TRACE(COMP_ERR, DBG_LOUD,
-("less desc left, stop skb_queue@%d, ring->idx = %d, 
idx = %d, skb_queue_len = 0x%d\n",
+("less desc left, stop skb_queue@%d, ring->idx = %d, 
idx = %d, skb_queue_len = 0x%x\n",
  hw_queue, ring->idx, idx,
  skb_queue_len(&ring->queue)));
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 16/19] staging: rtl8192ee: fix %d confusingly prefixed with 0x in format strings

2014-08-10 Thread Hans Wennborg 

On 08/09/2014 09:23 PM, Greg KH wrote:

On Sun, Aug 03, 2014 at 05:20:58PM -0700, Hans Wennborg wrote:

Signed-off-by: Hans Wennborg 
---
  drivers/staging/rtl8192ee/pci.c | 6 +++---
  1 file changed, 3 insertions(+), 3 deletions(-)


This patch fails to apply to my tree :(



Sorry about that. I'll try to rebase and re-post the patch.

 - Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 02/15] cpufreq: fix decimal printf format specifiers prefixed with 0x

2014-08-06 Thread Hans Wennborg 

On 08/06/2014 01:35 PM, Rafael J. Wysocki wrote:

On Tuesday, August 05, 2014 09:41:48 PM Hans Wennborg wrote:

The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 


This triggers a build warning related to the formats.  Care to fix up?


Sure. Sending a new patch, hopefully attached to the same thread (still 
working on my git send-mail skills).


 - Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 1/1] cpufreq: fix decimal printf specifiers prefixed with 0x

2014-08-06 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that. Also, these are 32-bit values,
so drop the l characters.

Signed-off-by: Hans Wennborg 
---
 drivers/cpufreq/speedstep-smi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/cpufreq/speedstep-smi.c b/drivers/cpufreq/speedstep-smi.c
index 8635eec..5fc96d5 100644
--- a/drivers/cpufreq/speedstep-smi.c
+++ b/drivers/cpufreq/speedstep-smi.c
@@ -324,8 +324,8 @@ static int __init speedstep_init(void)
return -ENODEV;
}
 
-   pr_debug("signature:0x%.8ulx, command:0x%.8ulx, "
-   "event:0x%.8ulx, perf_level:0x%.8ulx.\n",
+   pr_debug("signature:0x%.8x, command:0x%.8x, "
+   "event:0x%.8x, perf_level:0x%.8x.\n",
ist_info.signature, ist_info.command,
ist_info.event, ist_info.perf_level);
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 2/2] [media] dvb: return the error from i2c_transfer if negative

2014-08-06 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/media/dvb-frontends/mb86a16.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/media/dvb-frontends/mb86a16.c 
b/drivers/media/dvb-frontends/mb86a16.c
index 1f7fce7..0e65d35 100644
--- a/drivers/media/dvb-frontends/mb86a16.c
+++ b/drivers/media/dvb-frontends/mb86a16.c
@@ -118,6 +118,8 @@ static int mb86a16_read(struct mb86a16_state *state, u8 
reg, u8 *val)
dprintk(verbose, MB86A16_ERROR, 1, "read error(reg=0x%02x, 
ret=%i)",
reg, ret);
 
+   if (ret < 0)
+   return ret;
return -EREMOTEIO;
}
*val = b1[0];
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 1/2] [media] dvb: remove 0x prefix from decimal value in printf

2014-08-06 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/media/dvb-frontends/mb86a16.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/dvb-frontends/mb86a16.c 
b/drivers/media/dvb-frontends/mb86a16.c
index 9ae40ab..1f7fce7 100644
--- a/drivers/media/dvb-frontends/mb86a16.c
+++ b/drivers/media/dvb-frontends/mb86a16.c
@@ -115,7 +115,7 @@ static int mb86a16_read(struct mb86a16_state *state, u8 
reg, u8 *val)
};
ret = i2c_transfer(state->i2c_adap, msg, 2);
if (ret != 2) {
-   dprintk(verbose, MB86A16_ERROR, 1, "read error(reg=0x%02x, 
ret=0x%i)",
+   dprintk(verbose, MB86A16_ERROR, 1, "read error(reg=0x%02x, 
ret=%i)",
reg, ret);
 
return -EREMOTEIO;
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 08/15] rtlwifi: rtl8192de: fix decimal printf format specifiers prefixed with 0x

2014-08-06 Thread Hans Wennborg 

On 08/06/2014 07:38 AM, Larry Finger wrote:

On 08/05/2014 11:43 PM, Hans Wennborg wrote:

The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
  drivers/net/wireless/rtlwifi/rtl8192de/fw.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/fw.c 
b/drivers/net/wireless/rtlwifi/rtl8192de/fw.c
index 2317707..66e1760 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192de/fw.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192de/fw.c
@@ -234,7 +234,7 @@ static int _rtl92d_fw_init(struct ieee80211_hw *hw)
 rtl_read_byte(rtlpriv, FW_MAC1_READY));
}
RT_TRACE(rtlpriv, COMP_FW, DBG_DMESG,
-"Polling FW ready fail!! REG_MCUFWDL:0x%08ul\n",
+"Polling FW ready fail!! REG_MCUFWDL:0x%08lx\n",
 rtl_read_dword(rtlpriv, REG_MCUFWDL));
return -1;
  }


It would be best to actually build with your patches installed. This one yields
the following warning:


Terribly sorry about that, and thanks for catching it. I naively assumed 
that the code was correct besides the decimal vs. hex issue.



   CC [M]  drivers/net/wireless/rtlwifi/rtl8192de/fw.o
drivers/net/wireless/rtlwifi/rtl8192de/fw.c: In function ‘_rtl92d_fw_init’:
drivers/net/wireless/rtlwifi/rtl8192de/fw.c:236:2: warning: format ‘%lx’ expects
argument of type ‘long unsigned int’, but argument 5 has type ‘u32’ [-Wformat=]
   RT_TRACE(rtlpriv, COMP_FW, DBG_DMESG,
   ^

The format specifier should be %04x, not %08lx.


Shouldn't it be %08x since it's a 32-bit value?

> I have no idea why gcc did not complain as %08ul is also wrong.

I assume the compiler would have warned about %08lu, but %08ul takes an 
unsigned and just prints an l afterwards.


 - Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 04/15] [media] dvb: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/media/dvb-frontends/mb86a16.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/dvb-frontends/mb86a16.c 
b/drivers/media/dvb-frontends/mb86a16.c
index 9ae40ab..5939133 100644
--- a/drivers/media/dvb-frontends/mb86a16.c
+++ b/drivers/media/dvb-frontends/mb86a16.c
@@ -115,7 +115,7 @@ static int mb86a16_read(struct mb86a16_state *state, u8 
reg, u8 *val)
};
ret = i2c_transfer(state->i2c_adap, msg, 2);
if (ret != 2) {
-   dprintk(verbose, MB86A16_ERROR, 1, "read error(reg=0x%02x, 
ret=0x%i)",
+   dprintk(verbose, MB86A16_ERROR, 1, "read error(reg=0x%02x, 
ret=0x%x)",
reg, ret);
 
return -EREMOTEIO;
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 07/15] mwifiex: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/net/wireless/mwifiex/cfg80211.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/mwifiex/cfg80211.c 
b/drivers/net/wireless/mwifiex/cfg80211.c
index e2e6bf1..2856f0e 100644
--- a/drivers/net/wireless/mwifiex/cfg80211.c
+++ b/drivers/net/wireless/mwifiex/cfg80211.c
@@ -246,7 +246,7 @@ mwifiex_cfg80211_remain_on_channel(struct wiphy *wiphy,
}
 
if (priv->roc_cfg.cookie) {
-   wiphy_dbg(wiphy, "info: ongoing ROC, cookie = 0x%llu\n",
+   wiphy_dbg(wiphy, "info: ongoing ROC, cookie = 0x%llx\n",
  priv->roc_cfg.cookie);
return -EBUSY;
}
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 08/15] rtlwifi: rtl8192de: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/net/wireless/rtlwifi/rtl8192de/fw.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/fw.c 
b/drivers/net/wireless/rtlwifi/rtl8192de/fw.c
index 2317707..66e1760 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192de/fw.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192de/fw.c
@@ -234,7 +234,7 @@ static int _rtl92d_fw_init(struct ieee80211_hw *hw)
 rtl_read_byte(rtlpriv, FW_MAC1_READY));
}
RT_TRACE(rtlpriv, COMP_FW, DBG_DMESG,
-"Polling FW ready fail!! REG_MCUFWDL:0x%08ul\n",
+"Polling FW ready fail!! REG_MCUFWDL:0x%08lx\n",
 rtl_read_dword(rtlpriv, REG_MCUFWDL));
return -1;
 }
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 09/15] pinctrl: pinctrl-at91.c: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/pinctrl/pinctrl-at91.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/pinctrl/pinctrl-at91.c b/drivers/pinctrl/pinctrl-at91.c
index af1ba4f..60464a2 100644
--- a/drivers/pinctrl/pinctrl-at91.c
+++ b/drivers/pinctrl/pinctrl-at91.c
@@ -497,10 +497,10 @@ static struct at91_pinctrl_mux_ops at91sam9x5_ops = {
 static void at91_pin_dbg(const struct device *dev, const struct at91_pmx_pin 
*pin)
 {
if (pin->mux) {
-   dev_dbg(dev, "pio%c%d configured as periph%c with conf = 
0x%lu\n",
+   dev_dbg(dev, "pio%c%d configured as periph%c with conf = 
0x%lx\n",
pin->bank + 'A', pin->pin, pin->mux - 1 + 'A', 
pin->conf);
} else {
-   dev_dbg(dev, "pio%c%d configured as gpio with conf = 0x%lu\n",
+   dev_dbg(dev, "pio%c%d configured as gpio with conf = 0x%lx\n",
pin->bank + 'A', pin->pin, pin->conf);
}
 }
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 11/15] TTY: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/tty/moxa.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/tty/moxa.c b/drivers/tty/moxa.c
index 1deaca4..14c54e0 100644
--- a/drivers/tty/moxa.c
+++ b/drivers/tty/moxa.c
@@ -1096,7 +1096,7 @@ static int __init moxa_init(void)
continue;
}
 
-   printk(KERN_INFO "MOXA isa board found at 0x%.8lu and "
+   printk(KERN_INFO "MOXA isa board found at 0x%.8lx and "
"ready (%u ports, firmware loaded)\n",
baseaddr[i], brd->numPorts);
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 15/15] ASoC: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 sound/soc/atmel/atmel_ssc_dai.c | 2 +-
 sound/soc/sh/dma-sh7760.c   | 2 +-
 sound/soc/sh/rcar/gen.c | 6 +++---
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/sound/soc/atmel/atmel_ssc_dai.c b/sound/soc/atmel/atmel_ssc_dai.c
index f403f39..eb2ed21 100644
--- a/sound/soc/atmel/atmel_ssc_dai.c
+++ b/sound/soc/atmel/atmel_ssc_dai.c
@@ -201,7 +201,7 @@ static int atmel_ssc_startup(struct snd_pcm_substream 
*substream,
struct atmel_pcm_dma_params *dma_params;
int dir, dir_mask;
 
-   pr_debug("atmel_ssc_startup: SSC_SR=0x%u\n",
+   pr_debug("atmel_ssc_startup: SSC_SR=0x%x\n",
ssc_readl(ssc_p->ssc->regs, SR));
 
if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) {
diff --git a/sound/soc/sh/dma-sh7760.c b/sound/soc/sh/dma-sh7760.c
index c85f8eb..5a71f74 100644
--- a/sound/soc/sh/dma-sh7760.c
+++ b/sound/soc/sh/dma-sh7760.c
@@ -204,7 +204,7 @@ static int camelot_prepare(struct snd_pcm_substream 
*substream)
struct snd_soc_pcm_runtime *rtd = substream->private_data;
struct camelot_pcm *cam = &cam_pcm_data[rtd->cpu_dai->id];
 
-   pr_debug("PCM data: addr 0x%08ulx len %d\n",
+   pr_debug("PCM data: addr 0x%08lx len %d\n",
 (u32)runtime->dma_addr, runtime->dma_bytes);
  
if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) {
diff --git a/sound/soc/sh/rcar/gen.c b/sound/soc/sh/rcar/gen.c
index 3fdf3be..0f25ee6 100644
--- a/sound/soc/sh/rcar/gen.c
+++ b/sound/soc/sh/rcar/gen.c
@@ -69,7 +69,7 @@ u32 rsnd_read(struct rsnd_priv *priv,
 
regmap_fields_read(gen->regs[reg], rsnd_mod_id(mod), &val);
 
-   dev_dbg(dev, "r %s - 0x%04d : %08x\n", rsnd_mod_name(mod), reg, val);
+   dev_dbg(dev, "r %s - 0x%04x : %08x\n", rsnd_mod_name(mod), reg, val);
 
return val;
 }
@@ -86,7 +86,7 @@ void rsnd_write(struct rsnd_priv *priv,
 
regmap_fields_write(gen->regs[reg], rsnd_mod_id(mod), data);
 
-   dev_dbg(dev, "w %s - 0x%04d : %08x\n", rsnd_mod_name(mod), reg, data);
+   dev_dbg(dev, "w %s - 0x%04x : %08x\n", rsnd_mod_name(mod), reg, data);
 }
 
 void rsnd_bset(struct rsnd_priv *priv, struct rsnd_mod *mod,
@@ -101,7 +101,7 @@ void rsnd_bset(struct rsnd_priv *priv, struct rsnd_mod *mod,
regmap_fields_update_bits(gen->regs[reg], rsnd_mod_id(mod),
  mask, data);
 
-   dev_dbg(dev, "b %s - 0x%04d : %08x/%08x\n",
+   dev_dbg(dev, "b %s - 0x%04x : %08x/%08x\n",
rsnd_mod_name(mod), reg, data, mask);
 }
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 14/15] xfs: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 fs/xfs/xfs_discard.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/xfs/xfs_discard.c b/fs/xfs/xfs_discard.c
index 13d08a1..b4a2415 100644
--- a/fs/xfs/xfs_discard.c
+++ b/fs/xfs/xfs_discard.c
@@ -228,7 +228,7 @@ xfs_discard_extents(
GFP_NOFS, 0);
if (error && error != -EOPNOTSUPP) {
xfs_info(mp,
-"discard failed for extent [0x%llu,%u], error %d",
+"discard failed for extent [0x%llx,%u], error %d",
 (unsigned long long)busyp->bno,
 busyp->length,
 error);
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 10/15] scsi: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/scsi/cxgbi/libcxgbi.c | 2 +-
 drivers/scsi/nsp32.c  | 2 +-
 drivers/scsi/qla2xxx/qla_target.c | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/scsi/cxgbi/libcxgbi.c b/drivers/scsi/cxgbi/libcxgbi.c
index d65df6d..4cc9787 100644
--- a/drivers/scsi/cxgbi/libcxgbi.c
+++ b/drivers/scsi/cxgbi/libcxgbi.c
@@ -1807,7 +1807,7 @@ static void csk_return_rx_credits(struct cxgbi_sock *csk, 
int copied)
u32 credits;
 
log_debug(1 << CXGBI_DBG_PDU_RX,
-   "csk 0x%p,%u,0x%lu,%u, seq %u, wup %u, thre %u, %u.\n",
+   "csk 0x%p,%u,0x%lx,%u, seq %u, wup %u, thre %u, %u.\n",
csk, csk->state, csk->flags, csk->tid, csk->copied_seq,
csk->rcv_wup, cdev->rx_credit_thres,
cdev->rcv_win);
diff --git a/drivers/scsi/nsp32.c b/drivers/scsi/nsp32.c
index 50b086a..198f754 100644
--- a/drivers/scsi/nsp32.c
+++ b/drivers/scsi/nsp32.c
@@ -915,7 +915,7 @@ static int nsp32_queuecommand_lck(struct scsi_cmnd *SCpnt, 
void (*done)(struct s
int ret;
 
nsp32_dbg(NSP32_DEBUG_QUEUECOMMAND,
- "enter. target: 0x%x LUN: 0x%llu cmnd: 0x%x cmndlen: 0x%x "
+ "enter. target: 0x%x LUN: 0x%llx cmnd: 0x%x cmndlen: 0x%x "
  "use_sg: 0x%x reqbuf: 0x%lx reqlen: 0x%x",
  SCpnt->device->id, SCpnt->device->lun, SCpnt->cmnd[0], 
SCpnt->cmd_len,
  scsi_sg_count(SCpnt), scsi_sglist(SCpnt), 
scsi_bufflen(SCpnt));
diff --git a/drivers/scsi/qla2xxx/qla_target.c 
b/drivers/scsi/qla2xxx/qla_target.c
index e632e14..aebe62c 100644
--- a/drivers/scsi/qla2xxx/qla_target.c
+++ b/drivers/scsi/qla2xxx/qla_target.c
@@ -2503,7 +2503,7 @@ qlt_handle_dif_error(struct scsi_qla_host *vha, struct 
qla_tgt_cmd *cmd,
"iocb(s) %p Returned STATUS.\n", sts);
 
ql_dbg(ql_dbg_tgt, vha, 0xf075,
-   "dif check TGT cdb 0x%x lba 0x%llu: [Actual|Expected] Ref 
Tag[0x%x|0x%x], App Tag [0x%x|0x%x], Guard [0x%x|0x%x]\n",
+   "dif check TGT cdb 0x%x lba 0x%llx: [Actual|Expected] Ref 
Tag[0x%x|0x%x], App Tag [0x%x|0x%x], Guard [0x%x|0x%x]\n",
cmd->atio.u.isp24.fcp_cmnd.cdb[0], lba,
a_ref_tag, e_ref_tag, a_app_tag, e_app_tag, a_guard, e_guard);
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 13/15] exofs: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 fs/exofs/dir.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/exofs/dir.c b/fs/exofs/dir.c
index 49f51ab..54ab575 100644
--- a/fs/exofs/dir.c
+++ b/fs/exofs/dir.c
@@ -143,7 +143,7 @@ Espan:
 bad_entry:
EXOFS_ERR(
"ERROR [exofs_check_page]: bad entry in directory(0x%lx): %s - "
-   "offset=%lu, inode=0x%llu, rec_len=%d, name_len=%d\n",
+   "offset=%lu, inode=0x%llx, rec_len=%d, name_len=%d\n",
dir->i_ino, error, (page->index<inode_no)),
rec_len, p->name_len);
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 12/15] usb: musb: ux500: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/usb/musb/ux500_dma.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/musb/ux500_dma.c b/drivers/usb/musb/ux500_dma.c
index 9aad00f..221faed 100644
--- a/drivers/usb/musb/ux500_dma.c
+++ b/drivers/usb/musb/ux500_dma.c
@@ -96,7 +96,7 @@ static bool ux500_configure_channel(struct dma_channel 
*channel,
struct musb *musb = ux500_channel->controller->private_data;
 
dev_dbg(musb->controller,
-   "packet_sz=%d, mode=%d, dma_addr=0x%llu, len=%d is_tx=%d\n",
+   "packet_sz=%d, mode=%d, dma_addr=0x%llx, len=%d is_tx=%d\n",
packet_sz, mode, (unsigned long long) dma_addr,
len, ux500_channel->is_tx);
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 06/15] vmxnet3: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/net/vmxnet3/vmxnet3_drv.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/vmxnet3/vmxnet3_drv.c 
b/drivers/net/vmxnet3/vmxnet3_drv.c
index b76f7dc..d0db371 100644
--- a/drivers/net/vmxnet3/vmxnet3_drv.c
+++ b/drivers/net/vmxnet3/vmxnet3_drv.c
@@ -766,7 +766,7 @@ vmxnet3_map_pkt(struct sk_buff *skb, struct vmxnet3_tx_ctx 
*ctx,
gdesc->dword[3] = 0;
 
netdev_dbg(adapter->netdev,
-   "txd[%u]: 0x%llu %u %u\n",
+   "txd[%u]: 0x%llx %u %u\n",
tq->tx_ring.next2fill, 
le64_to_cpu(gdesc->txd.addr),
le32_to_cpu(gdesc->dword[2]), gdesc->dword[3]);
vmxnet3_cmd_ring_adv_next2fill(&tq->tx_ring);
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 05/15] mfd: rtsx_usb: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/mfd/rtsx_usb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/rtsx_usb.c b/drivers/mfd/rtsx_usb.c
index 6352bec..2d488e9 100644
--- a/drivers/mfd/rtsx_usb.c
+++ b/drivers/mfd/rtsx_usb.c
@@ -684,7 +684,7 @@ static int rtsx_usb_suspend(struct usb_interface *intf, 
pm_message_t message)
struct rtsx_ucr *ucr =
(struct rtsx_ucr *)usb_get_intfdata(intf);
 
-   dev_dbg(&intf->dev, "%s called with pm message 0x%04u\n",
+   dev_dbg(&intf->dev, "%s called with pm message 0x%04x\n",
__func__, message.event);
 
/*
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 03/15] iommu/arm-smmu: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/iommu/arm-smmu.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/iommu/arm-smmu.c b/drivers/iommu/arm-smmu.c
index ca18d6d..d84f983 100644
--- a/drivers/iommu/arm-smmu.c
+++ b/drivers/iommu/arm-smmu.c
@@ -623,7 +623,7 @@ static irqreturn_t arm_smmu_context_fault(int irq, void 
*dev)
 
if (fsr & FSR_IGN)
dev_err_ratelimited(smmu->dev,
-   "Unexpected context fault (fsr 0x%u)\n",
+   "Unexpected context fault (fsr 0x%x)\n",
fsr);
 
fsynr = readl_relaxed(cb_base + ARM_SMMU_CB_FSYNR0);
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 02/15] cpufreq: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 drivers/cpufreq/speedstep-smi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/cpufreq/speedstep-smi.c b/drivers/cpufreq/speedstep-smi.c
index 8635eec..fda9beb 100644
--- a/drivers/cpufreq/speedstep-smi.c
+++ b/drivers/cpufreq/speedstep-smi.c
@@ -324,8 +324,8 @@ static int __init speedstep_init(void)
return -ENODEV;
}
 
-   pr_debug("signature:0x%.8ulx, command:0x%.8ulx, "
-   "event:0x%.8ulx, perf_level:0x%.8ulx.\n",
+   pr_debug("signature:0x%.8lx, command:0x%.8lx, "
+   "event:0x%.8lx, perf_level:0x%.8lx.\n",
ist_info.signature, ist_info.command,
ist_info.event, ist_info.perf_level);
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 01/15] sparc: fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
The prefix suggests the number should be printed in hex, so use
the %x specifier to do that.

Found by using regex suggested by Joe Perches.

Signed-off-by: Hans Wennborg 
---
 arch/sparc/kernel/smp_32.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/sparc/kernel/smp_32.c b/arch/sparc/kernel/smp_32.c
index 7958242..b3a5d81 100644
--- a/arch/sparc/kernel/smp_32.c
+++ b/arch/sparc/kernel/smp_32.c
@@ -68,7 +68,7 @@ void smp_store_cpu_info(int id)
mid = cpu_get_hwmid(cpu_node);
 
if (mid < 0) {
-   printk(KERN_NOTICE "No MID found for CPU%d at node 0x%08d", id, 
cpu_node);
+   printk(KERN_NOTICE "No MID found for CPU%d at node 0x%08x", id, 
cpu_node);
mid = 0;
}
cpu_data(id).mid = mid;
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 00/15] fix decimal printf format specifiers prefixed with 0x

2014-08-05 Thread Hans Wennborg 
This is the second round of fixes, covering all the instances found
with Joe Perches's regex from https://lkml.org/lkml/2014/8/3/128 in
linux-next.

I have tried to split it across the different subsystems and address
the appropriate folks on each patch.

Hans Wennborg (15):
  sparc: fix decimal printf format specifiers prefixed with 0x
  cpufreq: fix decimal printf format specifiers prefixed with 0x
  iommu/arm-smmu: fix decimal printf format specifiers prefixed with 0x
  [media] dvb: fix decimal printf format specifiers prefixed with 0x
  mfd: rtsx_usb: fix decimal printf format specifiers prefixed with 0x
  vmxnet3: fix decimal printf format specifiers prefixed with 0x
  mwifiex: fix decimal printf format specifiers prefixed with 0x
  rtlwifi: rtl8192de: fix decimal printf format specifiers prefixed with 0x
  pinctrl: pinctrl-at91.c: fix decimal printf format specifiers prefixed with 0x
  scsi: fix decimal printf format specifiers prefixed with 0x
  TTY: fix decimal printf format specifiers prefixed with 0x
  usb: musb: ux500: fix decimal printf format specifiers prefixed with 0x
  exofs: fix decimal printf format specifiers prefixed with 0x
  xfs: fix decimal printf format specifiers prefixed with 0x
  ASoC: fix decimal printf format specifiers prefixed with 0x

 arch/sparc/kernel/smp_32.c  | 2 +-
 drivers/cpufreq/speedstep-smi.c | 4 ++--
 drivers/iommu/arm-smmu.c| 2 +-
 drivers/media/dvb-frontends/mb86a16.c   | 2 +-
 drivers/mfd/rtsx_usb.c  | 2 +-
 drivers/net/vmxnet3/vmxnet3_drv.c   | 2 +-
 drivers/net/wireless/mwifiex/cfg80211.c | 2 +-
 drivers/net/wireless/rtlwifi/rtl8192de/fw.c | 2 +-
 drivers/pinctrl/pinctrl-at91.c  | 4 ++--
 drivers/scsi/cxgbi/libcxgbi.c   | 2 +-
 drivers/scsi/nsp32.c| 2 +-
 drivers/scsi/qla2xxx/qla_target.c   | 2 +-
 drivers/tty/moxa.c  | 2 +-
 drivers/usb/musb/ux500_dma.c| 2 +-
 fs/exofs/dir.c  | 2 +-
 fs/xfs/xfs_discard.c| 2 +-
 sound/soc/atmel/atmel_ssc_dai.c | 2 +-
 sound/soc/sh/dma-sh7760.c   | 2 +-
 sound/soc/sh/rcar/gen.c | 6 +++---
 19 files changed, 23 insertions(+), 23 deletions(-)

-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] checkpatch: Add test for printf formats with 0x that emit decimal

2014-08-04 Thread Hans Wennborg 

On 08/03/2014 08:46 PM, Joe Perches wrote:

On Sun, 2014-08-03 at 20:03 -0700, Hans Wennborg wrote:

On 08/03/2014 07:50 PM, Joe Perches wrote:

0x% should be used to emit hexadecimal values.

Uses of 0x%[udi] emit decimal values but these should
probably instead use 0x%x variants.

Warn on these uses.


Good idea!

[]

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl

[]

@@ -4985,6 +4985,10 @@ sub process {
while ($line =~ /(?:^|")([X\t]*)(?:"|$)/g) {
$string = substr($rawline, $-[1], $+[1] - $-[1]);
$string =~ s/%%/__/g;
+   if ($string =~ 
/(0x(?

Maybe the regex should have a \b to check for a word boundary before the
0 to avoid matching things like "800x%d"? (I don't know if that occurs
in the kernel, but I've seen it elsewhere.)


Maybe.

Code it to do the appropriate thing and test it too.
See if there any other cases that should be emitted.


I grepped with your regex over the code and didn't get any false 
positives, so it seems fine. Feel free to ignore my previous comment.


Thanks,
Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 1/1] gru: fix %d confusingly prefixed with 0x in format string

2014-08-04 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/misc/sgi-gru/grumain.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/misc/sgi-gru/grumain.c b/drivers/misc/sgi-gru/grumain.c
index ae16c8c..161a940 100644
--- a/drivers/misc/sgi-gru/grumain.c
+++ b/drivers/misc/sgi-gru/grumain.c
@@ -283,7 +283,7 @@ static void gru_unload_mm_tracker(struct gru_state *gru,
spin_lock(&gru->gs_asid_lock);
BUG_ON((asids->mt_ctxbitmap & ctxbitmap) != ctxbitmap);
asids->mt_ctxbitmap ^= ctxbitmap;
-   gru_dbg(grudev, "gid %d, gts %p, gms %p, ctxnum 0x%d, asidmap 0x%lx\n",
+   gru_dbg(grudev, "gid %d, gts %p, gms %p, ctxnum %d, asidmap 0x%lx\n",
gru->gs_gid, gts, gms, gts->ts_ctxnum, gms->ms_asidmap[0]);
spin_unlock(&gru->gs_asid_lock);
spin_unlock(&gms->ms_asid_lock);
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH] checkpatch: Add test for printf formats with 0x that emit decimal

2014-08-03 Thread Hans Wennborg 

On 08/03/2014 07:50 PM, Joe Perches wrote:

0x% should be used to emit hexadecimal values.

Uses of 0x%[udi] emit decimal values but these should
probably instead use 0x%x variants.

Warn on these uses.


Good idea!



Signed-off-by: Joe Perches 
Noticed-by: Hans Wennborg 
---
  scripts/checkpatch.pl | 4 
  1 file changed, 4 insertions(+)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index da74e65..0178fe0 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -4985,6 +4985,10 @@ sub process {
while ($line =~ /(?:^|")([X\t]*)(?:"|$)/g) {
$string = substr($rawline, $-[1], $+[1] - $-[1]);
$string =~ s/%%/__/g;
+   if ($string =~ 
/(0x(?

Maybe the regex should have a \b to check for a word boundary before the 
0 to avoid matching things like "800x%d"? (I don't know if that occurs 
in the kernel, but I've seen it elsewhere.)


 - Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 1/1] cris: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 

On 08/02/2014 11:10 PM, Joe Perches wrote:

On Sat, 2014-08-02 at 18:19 -0700, Hans Wennborg wrote:

Yes, I have a patch for a bunch of these, but I figured it would be
easier to get it merged if I split it up. (Complete kernel newbie here.)


You as a kernel newbie did good, thanks.

A small tip would be to do all of them in a single series
cc'ing each individual patch to the appropriate maintainers
and mailing lists.


Thanks for the advice! I have split up and sent out the big patch as a 
series, with hopefully the right folks CC'd.


 - Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 19/19] alsa: riptide: fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 sound/firewire/fireworks/fireworks_proc.c | 4 ++--
 sound/pci/riptide/riptide.c   | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/sound/firewire/fireworks/fireworks_proc.c 
b/sound/firewire/fireworks/fireworks_proc.c
index 02bf394..23bb274 100644
--- a/sound/firewire/fireworks/fireworks_proc.c
+++ b/sound/firewire/fireworks/fireworks_proc.c
@@ -64,7 +64,7 @@ proc_read_hwinfo(struct snd_info_entry *entry, struct 
snd_info_buffer *buffer)
hwinfo->phys_in_grp_count);
for (i = 0; i < hwinfo->phys_in_grp_count; i++) {
snd_iprintf(buffer,
-   "phys in grp[0x%d]: type 0x%d, count 0x%x\n",
+   "phys in grp[0x%x]: type 0x%x, count 0x%x\n",
i, hwinfo->phys_out_grps[i].type,
hwinfo->phys_out_grps[i].count);
}
@@ -73,7 +73,7 @@ proc_read_hwinfo(struct snd_info_entry *entry, struct 
snd_info_buffer *buffer)
hwinfo->phys_out_grp_count);
for (i = 0; i < hwinfo->phys_out_grp_count; i++) {
snd_iprintf(buffer,
-   "phys out grps[0x%d]: type 0x%d, count 0x%x\n",
+   "phys out grps[0x%x]: type 0x%x, count 0x%x\n",
i, hwinfo->phys_out_grps[i].type,
hwinfo->phys_out_grps[i].count);
}
diff --git a/sound/pci/riptide/riptide.c b/sound/pci/riptide/riptide.c
index b4a8278..f0315c3 100644
--- a/sound/pci/riptide/riptide.c
+++ b/sound/pci/riptide/riptide.c
@@ -941,7 +941,7 @@ setmixer(struct cmdif *cif, short num, unsigned short rval, 
unsigned short lval)
union cmdret rptr = CMDRET_ZERO;
int i = 0;
 
-   snd_printdd("sent mixer %d: 0x%d 0x%d\n", num, rval, lval);
+   snd_printdd("sent mixer %d: 0x%x 0x%x\n", num, rval, lval);
do {
SEND_SDGV(cif, num, num, rval, lval);
SEND_RDGV(cif, num, num, &rptr);
@@ -1080,7 +1080,7 @@ getmixer(struct cmdif *cif, short num, unsigned short 
*rval,
return -EIO;
*rval = rptr.retwords[0];
*lval = rptr.retwords[1];
-   snd_printdd("got mixer %d: 0x%d 0x%d\n", num, *rval, *lval);
+   snd_printdd("got mixer %d: 0x%x 0x%x\n", num, *rval, *lval);
return 0;
 }
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 17/19] staging: rtl8821ae: fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/staging/rtl8821ae/pci.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8821ae/pci.c b/drivers/staging/rtl8821ae/pci.c
index 26d7b2f..b818788 100644
--- a/drivers/staging/rtl8821ae/pci.c
+++ b/drivers/staging/rtl8821ae/pci.c
@@ -662,7 +662,7 @@ static void _rtl_pci_tx_isr(struct ieee80211_hw *hw, int 
prio)
RT_TRACE(COMP_ERR, DBG_LOUD,
("more desc left, wake"
 "skb_queue@%d,ring->idx = %d,"
-"skb_queue_len = 0x%d\n",
+"skb_queue_len = 0x%x\n",
 prio, ring->idx,
 skb_queue_len(&ring->queue)));
 
@@ -1650,7 +1650,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
if ((own == 1) && (hw_queue != BEACON_QUEUE)) {
RT_TRACE(COMP_ERR, DBG_WARNING,
 ("No more TX desc@%d, ring->idx = %d,"
- "idx = %d, skb_queue_len = 0x%d\n",
+ "idx = %d, skb_queue_len = 0x%x\n",
  hw_queue, ring->idx, idx,
  skb_queue_len(&ring->queue)));
 
@@ -1695,7 +1695,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
RT_TRACE(COMP_ERR, DBG_LOUD,
 ("less desc left, stop skb_queue@%d, "
  "ring->idx = %d,"
- "idx = %d, skb_queue_len = 0x%d\n",
+ "idx = %d, skb_queue_len = 0x%x\n",
  hw_queue, ring->idx, idx,
  skb_queue_len(&ring->queue)));
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 18/19] sound: fireworks: fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 sound/firewire/fireworks/fireworks_proc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sound/firewire/fireworks/fireworks_proc.c 
b/sound/firewire/fireworks/fireworks_proc.c
index f29d4aa..02bf394 100644
--- a/sound/firewire/fireworks/fireworks_proc.c
+++ b/sound/firewire/fireworks/fireworks_proc.c
@@ -64,7 +64,7 @@ proc_read_hwinfo(struct snd_info_entry *entry, struct 
snd_info_buffer *buffer)
hwinfo->phys_in_grp_count);
for (i = 0; i < hwinfo->phys_in_grp_count; i++) {
snd_iprintf(buffer,
-   "phys in grp[0x%d]: type 0x%d, count 0x%d\n",
+   "phys in grp[0x%d]: type 0x%d, count 0x%x\n",
i, hwinfo->phys_out_grps[i].type,
hwinfo->phys_out_grps[i].count);
}
@@ -73,7 +73,7 @@ proc_read_hwinfo(struct snd_info_entry *entry, struct 
snd_info_buffer *buffer)
hwinfo->phys_out_grp_count);
for (i = 0; i < hwinfo->phys_out_grp_count; i++) {
snd_iprintf(buffer,
-   "phys out grps[0x%d]: type 0x%d, count 0x%d\n",
+   "phys out grps[0x%d]: type 0x%d, count 0x%x\n",
i, hwinfo->phys_out_grps[i].type,
hwinfo->phys_out_grps[i].count);
}
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 15/19] staging: nokia_h4p: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/staging/nokia_h4p/nokia_core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/nokia_h4p/nokia_core.c 
b/drivers/staging/nokia_h4p/nokia_core.c
index 775e1d0..240da0c 100644
--- a/drivers/staging/nokia_h4p/nokia_core.c
+++ b/drivers/staging/nokia_h4p/nokia_core.c
@@ -1113,7 +1113,7 @@ static int hci_h4p_probe(struct platform_device *pdev)
GPIOF_OUT_INIT_LOW, "bt_wakeup");
 
if (err < 0) {
-   dev_err(info->dev, "Cannot get GPIO line 0x%d",
+   dev_err(info->dev, "Cannot get GPIO line 0x%x",
info->bt_wakeup_gpio);
return err;
}
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 16/19] staging: rtl8192ee: fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/staging/rtl8192ee/pci.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/rtl8192ee/pci.c b/drivers/staging/rtl8192ee/pci.c
index 0215aef..349e636 100644
--- a/drivers/staging/rtl8192ee/pci.c
+++ b/drivers/staging/rtl8192ee/pci.c
@@ -649,7 +649,7 @@ static void _rtl_pci_tx_isr(struct ieee80211_hw *hw, int 
prio)
 
if ((ring->entries - skb_queue_len(&ring->queue)) == 2) {
RT_TRACE(COMP_ERR, DBG_LOUD,
-("more desc left, wake skb_queue@%d,ring->idx 
= %d, skb_queue_len = 0x%d\n",
+("more desc left, wake skb_queue@%d,ring->idx 
= %d, skb_queue_len = 0x%x\n",
 prio, ring->idx,
 skb_queue_len(&ring->queue)));
 
@@ -1628,7 +1628,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
 
if ((own == 1) && (hw_queue != BEACON_QUEUE)) {
RT_TRACE(COMP_ERR, DBG_WARNING,
-("No more TX desc@%d, ring->idx = %d, idx = 
%d, skb_queue_len = 0x%d\n",
+("No more TX desc@%d, ring->idx = %d, idx = 
%d, skb_queue_len = 0x%x\n",
  hw_queue, ring->idx, idx,
  skb_queue_len(&ring->queue)));
 
@@ -1670,7 +1670,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
if ((ring->entries - skb_queue_len(&ring->queue)) < 2 &&
hw_queue != BEACON_QUEUE) {
RT_TRACE(COMP_ERR, DBG_LOUD,
-("less desc left, stop skb_queue@%d, ring->idx = %d, 
idx = %d, skb_queue_len = 0x%d\n",
+("less desc left, stop skb_queue@%d, ring->idx = %d, 
idx = %d, skb_queue_len = 0x%x\n",
  hw_queue, ring->idx, idx,
  skb_queue_len(&ring->queue)));
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 11/19] iwl4965: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/net/wireless/iwlegacy/4965-mac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/iwlegacy/4965-mac.c 
b/drivers/net/wireless/iwlegacy/4965-mac.c
index c159c05..7d6fd59 100644
--- a/drivers/net/wireless/iwlegacy/4965-mac.c
+++ b/drivers/net/wireless/iwlegacy/4965-mac.c
@@ -4633,7 +4633,7 @@ il4965_store_tx_power(struct device *d, struct 
device_attribute *attr,
else {
ret = il_set_tx_power(il, val, false);
if (ret)
-   IL_ERR("failed setting tx power (0x%d).\n", ret);
+   IL_ERR("failed setting tx power (0x%x).\n", ret);
else
ret = count;
}
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 13/19] drivers: parisc: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/parisc/dino.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/parisc/dino.c b/drivers/parisc/dino.c
index 9eae983..a0580af 100644
--- a/drivers/parisc/dino.c
+++ b/drivers/parisc/dino.c
@@ -913,7 +913,7 @@ static int __init dino_probe(struct parisc_device *dev)
printk("%s version %s found at 0x%lx\n", name, version, hpa);
 
if (!request_mem_region(hpa, PAGE_SIZE, name)) {
-   printk(KERN_ERR "DINO: Hey! Someone took my MMIO space 
(0x%ld)!\n",
+   printk(KERN_ERR "DINO: Hey! Someone took my MMIO space 
(0x%lx)!\n",
hpa);
return 1;
}
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 12/19] rtlwifi: fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/net/wireless/rtlwifi/pci.c   | 6 +++---
 drivers/net/wireless/rtlwifi/rtl8192de/phy.c | 8 
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/net/wireless/rtlwifi/pci.c 
b/drivers/net/wireless/rtlwifi/pci.c
index 67d1ee6..74a8ba4 100644
--- a/drivers/net/wireless/rtlwifi/pci.c
+++ b/drivers/net/wireless/rtlwifi/pci.c
@@ -646,7 +646,7 @@ static void _rtl_pci_tx_isr(struct ieee80211_hw *hw, int 
prio)
== 2) {
 
RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD,
-"more desc left, wake skb_queue@%d, ring->idx 
= %d, skb_queue_len = 0x%d\n",
+"more desc left, wake skb_queue@%d, ring->idx 
= %d, skb_queue_len = 0x%x\n",
 prio, ring->idx,
 skb_queue_len(&ring->queue));
 
@@ -1469,7 +1469,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
 
if ((own == 1) && (hw_queue != BEACON_QUEUE)) {
RT_TRACE(rtlpriv, COMP_ERR, DBG_WARNING,
-"No more TX desc@%d, ring->idx = %d, idx = %d, 
skb_queue_len = 0x%d\n",
+"No more TX desc@%d, ring->idx = %d, idx = %d, 
skb_queue_len = 0x%x\n",
 hw_queue, ring->idx, idx,
 skb_queue_len(&ring->queue));
 
@@ -1511,7 +1511,7 @@ static int rtl_pci_tx(struct ieee80211_hw *hw,
if ((ring->entries - skb_queue_len(&ring->queue)) < 2 &&
hw_queue != BEACON_QUEUE) {
RT_TRACE(rtlpriv, COMP_ERR, DBG_LOUD,
-"less desc left, stop skb_queue@%d, ring->idx = %d, 
idx = %d, skb_queue_len = 0x%d\n",
+"less desc left, stop skb_queue@%d, ring->idx = %d, 
idx = %d, skb_queue_len = 0x%x\n",
 hw_queue, ring->idx, idx,
 skb_queue_len(&ring->queue));
 
diff --git a/drivers/net/wireless/rtlwifi/rtl8192de/phy.c 
b/drivers/net/wireless/rtlwifi/rtl8192de/phy.c
index 592125a..7cfdfc9 100644
--- a/drivers/net/wireless/rtlwifi/rtl8192de/phy.c
+++ b/drivers/net/wireless/rtlwifi/rtl8192de/phy.c
@@ -677,7 +677,7 @@ static void _rtl92d_store_pwrindex_diffrate_offset(struct 
ieee80211_hw *hw,
 
rtlphy->mcs_offset[rtlphy->pwrgroup_cnt][index] = data;
RT_TRACE(rtlpriv, COMP_INIT, DBG_TRACE,
-"MCSTxPowerLevelOriginalOffset[%d][%d] = 0x%ulx\n",
+"MCSTxPowerLevelOriginalOffset[%d][%d] = 0x%lx\n",
 rtlphy->pwrgroup_cnt, index,
 rtlphy->mcs_offset[rtlphy->pwrgroup_cnt][index]);
if (index == 13)
@@ -2531,7 +2531,7 @@ static void _rtl92d_phy_reload_lck_setting(struct 
ieee80211_hw *hw,
if (rtlpriv->rtlhal.current_bandtype == BAND_ON_5G) {/* Path-A for 5G */
u4tmp = curveindex_5g[channel-1];
RTPRINT(rtlpriv, FINIT, INIT_IQK,
-   "ver 1 set RF-A, 5G,0x28 = 0x%ulx !!\n", u4tmp);
+   "ver 1 set RF-A, 5G,0x28 = 0x%lx !!\n", u4tmp);
if (rtlpriv->rtlhal.macphymode == DUALMAC_DUALPHY &&
rtlpriv->rtlhal.interfaceindex == 1) {
bneed_powerdown_radio =
@@ -2550,7 +2550,7 @@ static void _rtl92d_phy_reload_lck_setting(struct 
ieee80211_hw *hw,
} else if (rtlpriv->rtlhal.current_bandtype == BAND_ON_2_4G) {
u4tmp = curveindex_2g[channel-1];
RTPRINT(rtlpriv, FINIT, INIT_IQK,
-   "ver 3 set RF-B, 2G, 0x28 = 0x%ulx !!\n", u4tmp);
+   "ver 3 set RF-B, 2G, 0x28 = 0x%lx !!\n", u4tmp);
if (rtlpriv->rtlhal.macphymode == DUALMAC_DUALPHY &&
rtlpriv->rtlhal.interfaceindex == 0) {
bneed_powerdown_radio =
@@ -2562,7 +2562,7 @@ static void _rtl92d_phy_reload_lck_setting(struct 
ieee80211_hw *hw,
}
rtl_set_rfreg(hw, erfpath, RF_SYN_G4, 0x3f800, u4tmp);
RTPRINT(rtlpriv, FINIT, INIT_IQK,
-   "ver 3 set RF-B, 2G, 0x28 = 0x%ulx !!\n",
+   "ver 3 set RF-B, 2G, 0x28 = 0x%lx !!\n",
rtl_get_rfreg(hw,  erfpath, RF_SYN_G4, 0x3f800));
if (bneed_powerdown_radio)
_rtl92d_phy_restore_rf_env(hw, erfpath, &u4regvalue);
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 09/19] net: smc911x: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/net/ethernet/smsc/smc911x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/ethernet/smsc/smc911x.c 
b/drivers/net/ethernet/smsc/smc911x.c
index 1c44e67..9778cba 100644
--- a/drivers/net/ethernet/smsc/smc911x.c
+++ b/drivers/net/ethernet/smsc/smc911x.c
@@ -730,7 +730,7 @@ static void smc911x_phy_detect(struct net_device *dev)
lp->phy_type = id1 << 16 | id2;
}
 
-   DBG(SMC_DEBUG_MISC, dev, "phy_id1=0x%x, phy_id2=0x%x phyaddr=0x%d\n",
+   DBG(SMC_DEBUG_MISC, dev, "phy_id1=0x%x, phy_id2=0x%x phyaddr=0x%x\n",
id1, id2, lp->mii.phy_id);
 }
 
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 14/19] hpsa: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/scsi/hpsa.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c
index 6b35d0d..47358b5 100644
--- a/drivers/scsi/hpsa.c
+++ b/drivers/scsi/hpsa.c
@@ -6088,7 +6088,7 @@ static void print_cfg_table(struct device *dev, struct 
CfgTable *tb)
   readl(&(tb->HostWrite.CoalIntDelay)));
dev_info(dev, "   Coalesce Interrupt Count = 0x%x\n",
   readl(&(tb->HostWrite.CoalIntCount)));
-   dev_info(dev, "   Max outstanding commands = 0x%d\n",
+   dev_info(dev, "   Max outstanding commands = 0x%x\n",
   readl(&(tb->CmdsOutMax)));
dev_info(dev, "   Bus Types = 0x%x\n", readl(&(tb->BusTypes)));
for (i = 0; i < 16; i++)
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 10/19] ath6kl: fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/net/wireless/ath/ath6kl/init.c | 2 +-
 drivers/net/wireless/ath/ath6kl/main.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/net/wireless/ath/ath6kl/init.c 
b/drivers/net/wireless/ath/ath6kl/init.c
index fffd523..91af845 100644
--- a/drivers/net/wireless/ath/ath6kl/init.c
+++ b/drivers/net/wireless/ath/ath6kl/init.c
@@ -1049,7 +1049,7 @@ static int ath6kl_fetch_fw_apin(struct ath6kl *ar, const 
char *name)
ar->hw.reserved_ram_size = le32_to_cpup(val);
 
ath6kl_dbg(ATH6KL_DBG_BOOT,
-  "found reserved ram size ie 0x%d\n",
+  "found reserved ram size ie 0x%x\n",
   ar->hw.reserved_ram_size);
break;
case ATH6KL_FW_IE_CAPABILITIES:
diff --git a/drivers/net/wireless/ath/ath6kl/main.c 
b/drivers/net/wireless/ath/ath6kl/main.c
index 21516bc..933aef0 100644
--- a/drivers/net/wireless/ath/ath6kl/main.c
+++ b/drivers/net/wireless/ath/ath6kl/main.c
@@ -225,7 +225,7 @@ int ath6kl_diag_write32(struct ath6kl *ar, u32 address, 
__le32 value)
ret = ath6kl_hif_diag_write32(ar, address, value);
 
if (ret) {
-   ath6kl_err("failed to write 0x%x during diagnose window to 
0x%d\n",
+   ath6kl_err("failed to write 0x%x during diagnose window to 
0x%x\n",
   address, value);
return ret;
}
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 07/19] mfd: omap-usb-host: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/mfd/omap-usb-host.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/omap-usb-host.c b/drivers/mfd/omap-usb-host.c
index 33a9234..83dab2f 100644
--- a/drivers/mfd/omap-usb-host.c
+++ b/drivers/mfd/omap-usb-host.c
@@ -647,7 +647,7 @@ static int usbhs_omap_probe(struct platform_device *pdev)
default:
omap->nports = OMAP3_HS_USB_PORTS;
dev_dbg(dev,
-"USB HOST Rev:0x%d not recognized, assuming %d 
ports\n",
+"USB HOST Rev:0x%x not recognized, assuming %d 
ports\n",
 omap->usbhs_rev, omap->nports);
break;
}
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 08/19] gru: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/misc/sgi-gru/grumain.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/misc/sgi-gru/grumain.c b/drivers/misc/sgi-gru/grumain.c
index ae16c8c..a1ce324 100644
--- a/drivers/misc/sgi-gru/grumain.c
+++ b/drivers/misc/sgi-gru/grumain.c
@@ -283,7 +283,7 @@ static void gru_unload_mm_tracker(struct gru_state *gru,
spin_lock(&gru->gs_asid_lock);
BUG_ON((asids->mt_ctxbitmap & ctxbitmap) != ctxbitmap);
asids->mt_ctxbitmap ^= ctxbitmap;
-   gru_dbg(grudev, "gid %d, gts %p, gms %p, ctxnum 0x%d, asidmap 0x%lx\n",
+   gru_dbg(grudev, "gid %d, gts %p, gms %p, ctxnum 0x%x, asidmap 0x%lx\n",
gru->gs_gid, gts, gms, gts->ts_ctxnum, gms->ms_asidmap[0]);
spin_unlock(&gru->gs_asid_lock);
spin_unlock(&gms->ms_asid_lock);
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 05/19] media: s5p-mfs: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c 
b/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c
index c1c12f8..e9175ab 100644
--- a/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c
+++ b/drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c
@@ -525,7 +525,7 @@ static int s5p_mfc_set_enc_stream_buffer_v6(struct 
s5p_mfc_ctx *ctx,
WRITEL(addr, mfc_regs->e_stream_buffer_addr); /* 16B align */
WRITEL(size, mfc_regs->e_stream_buffer_size);
 
-   mfc_debug(2, "stream buf addr: 0x%08lx, size: 0x%d\n",
+   mfc_debug(2, "stream buf addr: 0x%08lx, size: 0x%x\n",
  addr, size);
 
return 0;
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 06/19] mfd: htc-i2cpld: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/mfd/htc-i2cpld.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mfd/htc-i2cpld.c b/drivers/mfd/htc-i2cpld.c
index b44f020..6bdb78c 100644
--- a/drivers/mfd/htc-i2cpld.c
+++ b/drivers/mfd/htc-i2cpld.c
@@ -404,7 +404,7 @@ static int htcpld_register_chip_i2c(
}
 
i2c_set_clientdata(client, chip);
-   snprintf(client->name, I2C_NAME_SIZE, "Chip_0x%d", client->addr);
+   snprintf(client->name, I2C_NAME_SIZE, "Chip_0x%x", client->addr);
chip->client = client;
 
/* Reset the chip */
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 04/19] drivers: cciss: fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/block/cciss.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c
index ff20f19..99f778d 100644
--- a/drivers/block/cciss.c
+++ b/drivers/block/cciss.c
@@ -3838,7 +3838,7 @@ static void print_cfg_table(ctlr_info_t *h)
   readl(&(tb->HostWrite.CoalIntDelay)));
dev_dbg(&h->pdev->dev, "   Coalesce Interrupt Count = 0x%x\n",
   readl(&(tb->HostWrite.CoalIntCount)));
-   dev_dbg(&h->pdev->dev, "   Max outstanding commands = 0x%d\n",
+   dev_dbg(&h->pdev->dev, "   Max outstanding commands = 0x%x\n",
   readl(&(tb->CmdsOutMax)));
dev_dbg(&h->pdev->dev, "   Bus Types = 0x%x\n",
readl(&(tb->BusTypes)));
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 03/19] drivers: DAC960 fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/block/DAC960.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/block/DAC960.c b/drivers/block/DAC960.c
index 811e11c..d9b32f2 100644
--- a/drivers/block/DAC960.c
+++ b/drivers/block/DAC960.c
@@ -2954,7 +2954,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
case DAC960_PD_Controller:
  if (!request_region(Controller->IO_Address, 0x80,
  Controller->FullModelName)) {
-   DAC960_Error("IO port 0x%d busy for Controller at\n",
+   DAC960_Error("IO port 0x%x busy for Controller at\n",
 Controller, Controller->IO_Address);
goto Failure;
  }
@@ -2990,7 +2990,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
case DAC960_P_Controller:
  if (!request_region(Controller->IO_Address, 0x80,
  Controller->FullModelName)){
-   DAC960_Error("IO port 0x%d busy for Controller at\n",
+   DAC960_Error("IO port 0x%x busy for Controller at\n",
 Controller, Controller->IO_Address);
goto Failure;
  }
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 02/19] drivers: atm: fix %d confusingly prefixed with 0x in format strings

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 drivers/atm/eni.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/atm/eni.c b/drivers/atm/eni.c
index b1955ba..d65975a 100644
--- a/drivers/atm/eni.c
+++ b/drivers/atm/eni.c
@@ -2155,7 +2155,7 @@ static int eni_proc_read(struct atm_dev *dev,loff_t 
*pos,char *page)
 
if (!tx->send) continue;
if (!--left) {
-   return sprintf(page,"tx[%d]:0x%ld-0x%ld "
+   return sprintf(page, "tx[%d]:0x%lx-0x%lx "
"(%6ld bytes), rsv %d cps, shp %d cps%s\n",i,
(unsigned long) (tx->send - eni_dev->ram),
tx->send-eni_dev->ram+tx->words*4-1,tx->words*4,
@@ -2181,7 +2181,7 @@ static int eni_proc_read(struct atm_dev *dev,loff_t 
*pos,char *page)
if (--left) continue;
length = sprintf(page,"vcc %4d: ",vcc->vci);
if (eni_vcc->rx) {
-   length += sprintf(page+length,"0x%ld-0x%ld "
+   length += sprintf(page+length, "0x%lx-0x%lx "
"(%6ld bytes)",
(unsigned long) (eni_vcc->recv - 
eni_dev->ram),

eni_vcc->recv-eni_dev->ram+eni_vcc->words*4-1,
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 01/19] ARM: OMAP: fix %d confusingly prefixed with 0x in format string

2014-08-03 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 arch/arm/mach-omap2/id.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/mach-omap2/id.c b/arch/arm/mach-omap2/id.c
index d42022f..53841de 100644
--- a/arch/arm/mach-omap2/id.c
+++ b/arch/arm/mach-omap2/id.c
@@ -663,7 +663,7 @@ void __init dra7xxx_check_revision(void)
 
default:
/* Unknown default to latest silicon rev as default*/
-   pr_warn("%s: unknown idcode=0x%08x (hawkeye=0x%08x,rev=0x%d)\n",
+   pr_warn("%s: unknown idcode=0x%08x (hawkeye=0x%08x,rev=0x%x)\n",
__func__, idcode, hawkeye, rev);
omap_revision = DRA752_REV_ES1_1;
}
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Re: [PATCH 1/1] cris: fix %d confusingly prefixed with 0x in format string

2014-08-02 Thread Hans Wennborg 



On 08/02/2014 05:46 PM, Joe Perches wrote:

On Sat, 2014-08-02 at 15:20 -0700, Hans Wennborg wrote:

Signed-off-by: Hans Wennborg 
---
  arch/cris/arch-v32/drivers/cryptocop.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/cris/arch-v32/drivers/cryptocop.c 
b/arch/cris/arch-v32/drivers/cryptocop.c

[]

@@ -3155,7 +3155,7 @@ static void print_dma_descriptors(struct 
cryptocop_int_operation *iop)
printk("print_dma_descriptors start\n");

printk("iop:\n");
-   printk("\tsid: 0x%lld\n", iop->sid);
+   printk("\tsid: 0x%llx\n", iop->sid);

printk("\tcdesc_out: 0x%p\n", iop->cdesc_out);
printk("\tcdesc_in: 0x%p\n", iop->cdesc_in);


Thanks.

This is a surprisingly common error:

$ /usr/bin/git grep -P 
'0x\%(?!ull\b|ul\b|u\b|lu\b|llu\b|llux\b|lux\b|lx\b|lluX\b|luX\b|lX\b)[ul]*[ud]'

[...]

Yes, I have a patch for a bunch of these, but I figured it would be 
easier to get it merged if I split it up. (Complete kernel newbie here.)


 - Hans
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 1/1] cris: fix %d confusingly prefixed with 0x in format string

2014-08-02 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 arch/cris/arch-v32/drivers/cryptocop.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/cris/arch-v32/drivers/cryptocop.c 
b/arch/cris/arch-v32/drivers/cryptocop.c
index 877da19..0ae4f65 100644
--- a/arch/cris/arch-v32/drivers/cryptocop.c
+++ b/arch/cris/arch-v32/drivers/cryptocop.c
@@ -3155,7 +3155,7 @@ static void print_dma_descriptors(struct 
cryptocop_int_operation *iop)
printk("print_dma_descriptors start\n");
 
printk("iop:\n");
-   printk("\tsid: 0x%lld\n", iop->sid);
+   printk("\tsid: 0x%llx\n", iop->sid);
 
printk("\tcdesc_out: 0x%p\n", iop->cdesc_out);
printk("\tcdesc_in: 0x%p\n", iop->cdesc_in);
-- 
2.0.0.526.g5318336

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[PATCH 1/1] Fix confusing uses of %d prefixed by 0x in format strings

2014-05-26 Thread Hans Wennborg 
Signed-off-by: Hans Wennborg 
---
 arch/cris/arch-v32/drivers/cryptocop.c  | 2 +-
 drivers/atm/eni.c   | 4 ++--
 drivers/block/DAC960.c  | 4 ++--
 drivers/block/cciss.c   | 2 +-
 drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c | 2 +-
 drivers/mfd/htc-i2cpld.c| 2 +-
 drivers/mfd/omap-usb-host.c | 2 +-
 drivers/misc/sgi-gru/grumain.c  | 2 +-
 drivers/net/ethernet/smsc/smc911x.c | 2 +-
 drivers/net/wireless/ath/ath6kl/init.c  | 2 +-
 drivers/net/wireless/ath/ath6kl/main.c  | 2 +-
 drivers/net/wireless/iwlegacy/4965-mac.c| 2 +-
 drivers/net/wireless/rtlwifi/pci.c  | 6 +++---
 drivers/parisc/dino.c   | 2 +-
 drivers/scsi/hpsa.c | 2 +-
 drivers/staging/nokia_h4p/nokia_core.c  | 2 +-
 drivers/staging/rtl8821ae/pci.c | 6 +++---
 fs/cifs/cifs_debug.c| 2 +-
 fs/cifs/misc.c  | 4 ++--
 fs/cifs/smb2misc.c  | 6 +++---
 sound/pci/riptide/riptide.c | 4 ++--
 21 files changed, 31 insertions(+), 31 deletions(-)

diff --git a/arch/cris/arch-v32/drivers/cryptocop.c 
b/arch/cris/arch-v32/drivers/cryptocop.c
index 877da19..0ae4f65 100644
--- a/arch/cris/arch-v32/drivers/cryptocop.c
+++ b/arch/cris/arch-v32/drivers/cryptocop.c
@@ -3155,7 +3155,7 @@ static void print_dma_descriptors(struct 
cryptocop_int_operation *iop)
printk("print_dma_descriptors start\n");
 
printk("iop:\n");
-   printk("\tsid: 0x%lld\n", iop->sid);
+   printk("\tsid: 0x%llx\n", iop->sid);
 
printk("\tcdesc_out: 0x%p\n", iop->cdesc_out);
printk("\tcdesc_in: 0x%p\n", iop->cdesc_in);
diff --git a/drivers/atm/eni.c b/drivers/atm/eni.c
index b1955ba..d65975a 100644
--- a/drivers/atm/eni.c
+++ b/drivers/atm/eni.c
@@ -2155,7 +2155,7 @@ static int eni_proc_read(struct atm_dev *dev,loff_t 
*pos,char *page)
 
if (!tx->send) continue;
if (!--left) {
-   return sprintf(page,"tx[%d]:0x%ld-0x%ld "
+   return sprintf(page, "tx[%d]:0x%lx-0x%lx "
"(%6ld bytes), rsv %d cps, shp %d cps%s\n",i,
(unsigned long) (tx->send - eni_dev->ram),
tx->send-eni_dev->ram+tx->words*4-1,tx->words*4,
@@ -2181,7 +2181,7 @@ static int eni_proc_read(struct atm_dev *dev,loff_t 
*pos,char *page)
if (--left) continue;
length = sprintf(page,"vcc %4d: ",vcc->vci);
if (eni_vcc->rx) {
-   length += sprintf(page+length,"0x%ld-0x%ld "
+   length += sprintf(page+length, "0x%lx-0x%lx "
"(%6ld bytes)",
(unsigned long) (eni_vcc->recv - 
eni_dev->ram),

eni_vcc->recv-eni_dev->ram+eni_vcc->words*4-1,
diff --git a/drivers/block/DAC960.c b/drivers/block/DAC960.c
index 125d845..3734545 100644
--- a/drivers/block/DAC960.c
+++ b/drivers/block/DAC960.c
@@ -2954,7 +2954,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
case DAC960_PD_Controller:
  if (!request_region(Controller->IO_Address, 0x80,
  Controller->FullModelName)) {
-   DAC960_Error("IO port 0x%d busy for Controller at\n",
+   DAC960_Error("IO port 0x%x busy for Controller at\n",
 Controller, Controller->IO_Address);
goto Failure;
  }
@@ -2990,7 +2990,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
case DAC960_P_Controller:
  if (!request_region(Controller->IO_Address, 0x80,
  Controller->FullModelName)){
-   DAC960_Error("IO port 0x%d busy for Controller at\n",
+   DAC960_Error("IO port 0x%x busy for Controller at\n",
 Controller, Controller->IO_Address);
goto Failure;
  }
diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c
index 4595c22..ca88ebe 100644
--- a/drivers/block/cciss.c
+++ b/drivers/block/cciss.c
@@ -3841,7 +3841,7 @@ static void print_cfg_table(ctlr_info_t *h)
   readl(&(tb->HostWrite.CoalIntDelay)));
dev_dbg(&h->pdev->dev, "   Coalesce Interrupt Count = 0x%x\n",
   readl(&(tb->HostWrite.CoalIntCount)));
-   dev_dbg(&h->pdev->dev, "   Max outstanding commands =