* [patch] wl1271_cmd.c: cleanup char => u8
@ 2009-12-23 13:23 Dan Carpenter
2009-12-27 8:33 ` Kalle Valo
0 siblings, 1 reply; 2+ messages in thread
From: Dan Carpenter @ 2009-12-23 13:23 UTC (permalink / raw)
To: Luciano Coelho; +Cc: linux-wireless, John W. Linville
This is just a clean up and doesn't make a functional difference. It keeps the
lint checkers happy.
Signed-off-by: Dan Carpenter <error27@gmail.com>
--- orig/drivers/net/wireless/wl12xx/wl1271_cmd.c 2009-12-22 22:59:58.000000000 +0200
+++ devel/drivers/net/wireless/wl12xx/wl1271_cmd.c 2009-12-22 23:00:29.000000000 +0200
@@ -777,7 +777,7 @@ out:
return ret;
}
-static int wl1271_build_basic_rates(char *rates, u8 band)
+static int wl1271_build_basic_rates(u8 *rates, u8 band)
{
u8 index = 0;
@@ -804,7 +804,7 @@ static int wl1271_build_basic_rates(char
return index;
}
-static int wl1271_build_extended_rates(char *rates, u8 band)
+static int wl1271_build_extended_rates(u8 *rates, u8 band)
{
u8 index = 0;
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [patch] wl1271_cmd.c: cleanup char => u8
2009-12-23 13:23 [patch] wl1271_cmd.c: cleanup char => u8 Dan Carpenter
@ 2009-12-27 8:33 ` Kalle Valo
0 siblings, 0 replies; 2+ messages in thread
From: Kalle Valo @ 2009-12-27 8:33 UTC (permalink / raw)
To: Dan Carpenter; +Cc: Luciano Coelho, linux-wireless, John W. Linville
Dan Carpenter <error27@gmail.com> writes:
> This is just a clean up and doesn't make a functional difference. It
> keeps the lint checkers happy.
Thanks, looks good.
The "wl1271_cmd.c:" prefix is a bit ugly, though. John, if possible,
can you change that to "wl1271:" before you apply the patch, please?
> Signed-off-by: Dan Carpenter <error27@gmail.com>
Luciano is on vacation right now (the lucky ba^H^Hguy), so I'll act as
a backup:
Reviewed-by: Kalle Valo <kalle.valo@nokia.com>
--
Kalle Valo
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-12-27 8:33 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-12-23 13:23 [patch] wl1271_cmd.c: cleanup char => u8 Dan Carpenter
2009-12-27 8:33 ` Kalle Valo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).