UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

[PATCH] hostap update 

Warning fix for 64-bit platforms

Hello!

The patch fixes following warning seen on 64-bit platforms (in my case -
x86_64, gcc-4.0):

In file included from /usr/local/src/hostap/driver/modules/hostap_cs.c:203:
/usr/local/src/hostap/driver/modules/hostap_hw.c: In function ?prism2_transmit_cb?:
/usr/local/src/hostap/driver/modules/hostap_hw.c:1674: warning: cast from pointer to integer of different size
/usr/local/src/hostap/driver/modules/hostap_hw.c: In function ?prism2_transmit?:
/usr/local/src/hostap/driver/modules/hostap_hw.c:1758: warning: cast to pointer from integer of different size

prism2_transmit_cb uses a (void *) argument to get an integer.   A
simple fix would be to use double cast from pointer to long and then to
int (and vice versa when int is passed as a pointer).  But I prefer a
slightly longer patch.

I believe that whenever an argument can hold both a pointer and an
integer, it should be declared long.  long can hold both pointers and
integers (except win64, but we are not coding for Windows), it can be
cast to both of them and it's never assumed to be a valid pointer, which
could be useful for some automatic code checkers.

Signed-off-by: Pavel Roskin <proski@gnu.org>
Signed-off-by: Jouni Malinen <jkmaline@cc.hut.fi>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
This commit is contained in:
Pavel Roskin 2005-07-30 12:50:05 -07:00 committed by Jeff Garzik
parent 3e1d393240
commit b15eff2632
2 changed files with 11 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
drivers/net/wireless/hostap/hostap_hw.c
View File

@ -507,7 +507,7 @@ static int hfa384x_cmd(struct net_device *dev, u16 cmd, u16 param0,
* @cmd: Prism2 command code (HFA384X_CMD_CODE_*) * @cmd: Prism2 command code (HFA384X_CMD_CODE_*)
* @param0: value for Param0 register * @param0: value for Param0 register
* @callback: command completion callback function (%NULL = no callback) * @callback: command completion callback function (%NULL = no callback)
* @context: data pointer to be given to callback function * @context: context data to be given to the callback function
* *
* Issue given command (possibly after waiting in command queue) and use * Issue given command (possibly after waiting in command queue) and use
* callback function to indicate command completion. This can be called both * callback function to indicate command completion. This can be called both
@ -517,9 +517,9 @@ static int hfa384x_cmd(struct net_device *dev, u16 cmd, u16 param0,
*/ */
static int hfa384x_cmd_callback(struct net_device *dev, u16 cmd, u16 param0, static int hfa384x_cmd_callback(struct net_device *dev, u16 cmd, u16 param0,
void (*callback)(struct net_device *dev, void (*callback)(struct net_device *dev,
void *context, u16 resp0, long context, u16 resp0,
u16 status), u16 status),
void *context) long context)
{ {
struct hostap_interface *iface; struct hostap_interface *iface;
local_info_t *local; local_info_t *local;
@ -1710,7 +1710,7 @@ static int prism2_get_txfid_idx(local_info_t *local)
/* Called only from hardware IRQ */ /* Called only from hardware IRQ */
static void prism2_transmit_cb(struct net_device *dev, void *context, static void prism2_transmit_cb(struct net_device *dev, long context,
u16 resp0, u16 res) u16 resp0, u16 res)
{ {
struct hostap_interface *iface; struct hostap_interface *iface;
@ -1805,7 +1805,7 @@ static int prism2_transmit(struct net_device *dev, int idx)
dev, dev,
HFA384X_CMDCODE_TRANSMIT | HFA384X_CMD_TX_RECLAIM, HFA384X_CMDCODE_TRANSMIT | HFA384X_CMD_TX_RECLAIM,
local->txfid[idx], local->txfid[idx],
prism2_transmit_cb, (void *) idx); prism2_transmit_cb, (long) idx);
if (res) { if (res) {
struct net_device_stats *stats; struct net_device_stats *stats;
@ -1951,7 +1951,7 @@ static int prism2_tx_80211(struct sk_buff *skb, struct net_device *dev)
/* Any RX packet seems to break something with TX bus /* Any RX packet seems to break something with TX bus
* mastering; enable command is enough to fix this.. */ * mastering; enable command is enough to fix this.. */
if (hfa384x_cmd_callback(dev, HFA384X_CMDCODE_ENABLE, 0, if (hfa384x_cmd_callback(dev, HFA384X_CMDCODE_ENABLE, 0,
prism2_tx_cb, (void *) buf_len)) { prism2_tx_cb, (long) buf_len)) {
printk(KERN_DEBUG "%s: TX: enable port0 failed\n", printk(KERN_DEBUG "%s: TX: enable port0 failed\n",
dev->name); dev->name);
} }
@ -2697,7 +2697,7 @@ static void prism2_infdrop(struct net_device *dev)
* get out of this state by inquiring CommTallies. */ * get out of this state by inquiring CommTallies. */
if (!last_inquire || time_after(jiffies, last_inquire + HZ)) { if (!last_inquire || time_after(jiffies, last_inquire + HZ)) {
hfa384x_cmd_callback(dev, HFA384X_CMDCODE_INQUIRE, hfa384x_cmd_callback(dev, HFA384X_CMDCODE_INQUIRE,
HFA384X_INFO_COMMTALLIES, NULL, NULL); HFA384X_INFO_COMMTALLIES, NULL, 0);
last_inquire = jiffies; last_inquire = jiffies;
} }
} }
@ -3055,7 +3055,7 @@ static void hostap_passive_scan(unsigned long data)
if (hfa384x_cmd_callback(dev, HFA384X_CMDCODE_TEST | if (hfa384x_cmd_callback(dev, HFA384X_CMDCODE_TEST |
(HFA384X_TEST_CHANGE_CHANNEL << 8), (HFA384X_TEST_CHANGE_CHANNEL << 8),
channel, NULL, NULL)) channel, NULL, 0))
printk(KERN_ERR "%s: passive scan channel set %d " printk(KERN_ERR "%s: passive scan channel set %d "
"failed\n", dev->name, channel); "failed\n", dev->name, channel);
@ -3088,7 +3088,7 @@ static void hostap_tick_timer(unsigned long data)
!local->hw_downloading && local->hw_ready && !local->hw_downloading && local->hw_ready &&
!local->hw_resetting && local->dev_enabled) { !local->hw_resetting && local->dev_enabled) {
hfa384x_cmd_callback(local->dev, HFA384X_CMDCODE_INQUIRE, hfa384x_cmd_callback(local->dev, HFA384X_CMDCODE_INQUIRE,
HFA384X_INFO_COMMTALLIES, NULL, NULL); HFA384X_INFO_COMMTALLIES, NULL, 0);
last_inquire = jiffies; last_inquire = jiffies;
} }

4
drivers/net/wireless/hostap/hostap_wlan.h
View File

@ -545,9 +545,9 @@ struct hostap_cmd_queue {
struct list_head list; struct list_head list;
wait_queue_head_t compl; wait_queue_head_t compl;
volatile enum { CMD_SLEEP, CMD_CALLBACK, CMD_COMPLETED } type; volatile enum { CMD_SLEEP, CMD_CALLBACK, CMD_COMPLETED } type;
void (*callback)(struct net_device *dev, void *context, u16 resp0, void (*callback)(struct net_device *dev, long context, u16 resp0,
u16 res); u16 res);
void *context; long context;
u16 cmd, param0, param1; u16 cmd, param0, param1;
u16 resp0, res; u16 resp0, res;
volatile int issued, issuing; volatile int issued, issuing;