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

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

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

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

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

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

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

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 de

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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