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 <hans@xxxxxxxxxx> > --- > 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: 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. I have no idea why gcc did not complain as %08ul is also wrong. Larry -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
