Commit 3796e4ee authored by Martin Dalecki's avatar Martin Dalecki Committed by Linus Torvalds

[PATCH] kill warnings 8/19

- Fix improper __FUNCTION__ usage in irda_device.
- Whitepsace cleanup there too.
parent 8192cb91
/********************************************************************* /*********************************************************************
* *
* Filename: irda_device.c * Filename: irda_device.c
* Version: 0.9 * Version: 0.9
* Description: Utility functions used by the device drivers * Description: Utility functions used by the device drivers
...@@ -8,25 +8,25 @@ ...@@ -8,25 +8,25 @@
* Created at: Sat Oct 9 09:22:27 1999 * Created at: Sat Oct 9 09:22:27 1999
* Modified at: Sun Jan 23 17:41:24 2000 * Modified at: Sun Jan 23 17:41:24 2000
* Modified by: Dag Brattli <dagb@cs.uit.no> * Modified by: Dag Brattli <dagb@cs.uit.no>
* *
* Copyright (c) 1999-2000 Dag Brattli, All Rights Reserved. * Copyright (c) 1999-2000 Dag Brattli, All Rights Reserved.
* Copyright (c) 2000-2001 Jean Tourrilhes <jt@hpl.hp.com> * Copyright (c) 2000-2001 Jean Tourrilhes <jt@hpl.hp.com>
* *
* This program is free software; you can redistribute it and/or * This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License as * modify it under the terms of the GNU General Public License as
* published by the Free Software Foundation; either version 2 of * published by the Free Software Foundation; either version 2 of
* the License, or (at your option) any later version. * the License, or (at your option) any later version.
* *
* This program is distributed in the hope that it will be useful, * This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of * but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details. * GNU General Public License for more details.
* *
* You should have received a copy of the GNU General Public License * You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software * along with this program; if not, write to the Free Software
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, * Foundation, Inc., 59 Temple Place, Suite 330, Boston,
* MA 02111-1307 USA * MA 02111-1307 USA
* *
********************************************************************/ ********************************************************************/
#include <linux/config.h> #include <linux/config.h>
...@@ -83,7 +83,7 @@ const char *infrared_mode[] = { ...@@ -83,7 +83,7 @@ const char *infrared_mode[] = {
#ifdef CONFIG_IRDA_DEBUG #ifdef CONFIG_IRDA_DEBUG
static const char *task_state[] = { static const char *task_state[] = {
"IRDA_TASK_INIT", "IRDA_TASK_INIT",
"IRDA_TASK_DONE", "IRDA_TASK_DONE",
"IRDA_TASK_WAIT", "IRDA_TASK_WAIT",
"IRDA_TASK_WAIT1", "IRDA_TASK_WAIT1",
"IRDA_TASK_WAIT2", "IRDA_TASK_WAIT2",
...@@ -97,7 +97,7 @@ static const char *task_state[] = { ...@@ -97,7 +97,7 @@ static const char *task_state[] = {
static void irda_task_timer_expired(void *data); static void irda_task_timer_expired(void *data);
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
int irda_device_proc_read(char *buf, char **start, off_t offset, int len, int irda_device_proc_read(char *buf, char **start, off_t offset, int len,
int unused); int unused);
#endif /* CONFIG_PROC_FS */ #endif /* CONFIG_PROC_FS */
...@@ -106,21 +106,19 @@ int __init irda_device_init( void) ...@@ -106,21 +106,19 @@ int __init irda_device_init( void)
{ {
dongles = hashbin_new(HB_GLOBAL); dongles = hashbin_new(HB_GLOBAL);
if (dongles == NULL) { if (dongles == NULL) {
printk(KERN_WARNING printk(KERN_WARNING "IrDA: Can't allocate dongles hashbin!\n");
"IrDA: Can't allocate dongles hashbin!\n");
return -ENOMEM; return -ENOMEM;
} }
tasks = hashbin_new(HB_GLOBAL); tasks = hashbin_new(HB_GLOBAL);
if (tasks == NULL) { if (tasks == NULL) {
printk(KERN_WARNING printk(KERN_WARNING "IrDA: Can't allocate tasks hashbin!\n");
"IrDA: Can't allocate tasks hashbin!\n");
return -ENOMEM; return -ENOMEM;
} }
/* /*
* Call the init function of the device drivers that has not been * Call the init function of the device drivers that has not been
* compiled as a module * compiled as a module
*/ */
#ifdef CONFIG_IRTTY_SIR #ifdef CONFIG_IRTTY_SIR
irtty_init(); irtty_init();
...@@ -156,10 +154,10 @@ int __init irda_device_init( void) ...@@ -156,10 +154,10 @@ int __init irda_device_init( void)
litelink_init(); litelink_init();
#endif #endif
#ifdef CONFIG_OLD_BELKIN #ifdef CONFIG_OLD_BELKIN
old_belkin_init(); old_belkin_init();
#endif #endif
#ifdef CONFIG_EP7211_IR #ifdef CONFIG_EP7211_IR
ep7211_ir_init(); ep7211_ir_init();
#endif #endif
return 0; return 0;
} }
...@@ -178,7 +176,7 @@ void irda_device_cleanup(void) ...@@ -178,7 +176,7 @@ void irda_device_cleanup(void)
* Called when we have detected that another station is transmiting * Called when we have detected that another station is transmiting
* in contention mode. * in contention mode.
*/ */
void irda_device_set_media_busy(struct net_device *dev, int status) void irda_device_set_media_busy(struct net_device *dev, int status)
{ {
struct irlap_cb *self; struct irlap_cb *self;
...@@ -203,15 +201,15 @@ void irda_device_set_media_busy(struct net_device *dev, int status) ...@@ -203,15 +201,15 @@ void irda_device_set_media_busy(struct net_device *dev, int status)
} }
int irda_device_set_dtr_rts(struct net_device *dev, int dtr, int rts) int irda_device_set_dtr_rts(struct net_device *dev, int dtr, int rts)
{ {
struct if_irda_req req; struct if_irda_req req;
int ret; int ret;
IRDA_DEBUG(2, __FUNCTION__ "()\n"); IRDA_DEBUG(2, __FUNCTION__ "()\n");
if (!dev->do_ioctl) { if (!dev->do_ioctl) {
ERROR(__FUNCTION__ "(), do_ioctl not impl. by " ERROR("%s: do_ioctl not impl. by device driver\n",
"device driver\n"); __FUNCTION__);
return -1; return -1;
} }
...@@ -224,15 +222,15 @@ int irda_device_set_dtr_rts(struct net_device *dev, int dtr, int rts) ...@@ -224,15 +222,15 @@ int irda_device_set_dtr_rts(struct net_device *dev, int dtr, int rts)
} }
int irda_device_change_speed(struct net_device *dev, __u32 speed) int irda_device_change_speed(struct net_device *dev, __u32 speed)
{ {
struct if_irda_req req; struct if_irda_req req;
int ret; int ret;
IRDA_DEBUG(2, __FUNCTION__ "()\n"); IRDA_DEBUG(2, __FUNCTION__ "()\n");
if (!dev->do_ioctl) { if (!dev->do_ioctl) {
ERROR(__FUNCTION__ "(), do_ioctl not impl. by " ERROR("%s: do_ioctl not impl. by device driver\n",
"device driver\n"); __FUNCTION__);
return -1; return -1;
} }
...@@ -257,8 +255,8 @@ int irda_device_is_receiving(struct net_device *dev) ...@@ -257,8 +255,8 @@ int irda_device_is_receiving(struct net_device *dev)
IRDA_DEBUG(2, __FUNCTION__ "()\n"); IRDA_DEBUG(2, __FUNCTION__ "()\n");
if (!dev->do_ioctl) { if (!dev->do_ioctl) {
ERROR(__FUNCTION__ "(), do_ioctl not impl. by " ERROR("%s: do_ioctl not impl. by device driver\n",
"device driver\n"); __FUNCTION__);
return -1; return -1;
} }
...@@ -279,7 +277,7 @@ void irda_task_next_state(struct irda_task *task, IRDA_TASK_STATE state) ...@@ -279,7 +277,7 @@ void irda_task_next_state(struct irda_task *task, IRDA_TASK_STATE state)
static void __irda_task_delete(struct irda_task *task) static void __irda_task_delete(struct irda_task *task)
{ {
del_timer(&task->timer); del_timer(&task->timer);
kfree(task); kfree(task);
} }
...@@ -314,14 +312,14 @@ int irda_task_kick(struct irda_task *task) ...@@ -314,14 +312,14 @@ int irda_task_kick(struct irda_task *task)
do { do {
timeout = task->function(task); timeout = task->function(task);
if (count++ > 100) { if (count++ > 100) {
ERROR(__FUNCTION__ "(), error in task handler!\n"); ERROR("%s: error in task handler!\n", __FUNCTION__);
irda_task_delete(task); irda_task_delete(task);
return TRUE; return TRUE;
} }
} while ((timeout == 0) && (task->state != IRDA_TASK_DONE)); } while ((timeout == 0) && (task->state != IRDA_TASK_DONE));
if (timeout < 0) { if (timeout < 0) {
ERROR(__FUNCTION__ "(), Error executing task!\n"); ERROR("%s: Error executing task!\n", __FUNCTION__);
irda_task_delete(task); irda_task_delete(task);
return TRUE; return TRUE;
} }
...@@ -346,14 +344,14 @@ int irda_task_kick(struct irda_task *task) ...@@ -346,14 +344,14 @@ int irda_task_kick(struct irda_task *task)
/* Kick parent task */ /* Kick parent task */
irda_task_kick(task->parent); irda_task_kick(task->parent);
} }
} }
irda_task_delete(task); irda_task_delete(task);
} else if (timeout > 0) { } else if (timeout > 0) {
irda_start_timer(&task->timer, timeout, (void *) task, irda_start_timer(&task->timer, timeout, (void *) task,
irda_task_timer_expired); irda_task_timer_expired);
finished = FALSE; finished = FALSE;
} else { } else {
IRDA_DEBUG(0, __FUNCTION__ IRDA_DEBUG(0, __FUNCTION__
"(), not finished, and no timeout!\n"); "(), not finished, and no timeout!\n");
finished = FALSE; finished = FALSE;
} }
...@@ -367,7 +365,7 @@ int irda_task_kick(struct irda_task *task) ...@@ -367,7 +365,7 @@ int irda_task_kick(struct irda_task *task)
* This function registers and tries to execute tasks that may take some * This function registers and tries to execute tasks that may take some
* time to complete. We do it this hairy way since we may have been * time to complete. We do it this hairy way since we may have been
* called from interrupt context, so it's not possible to use * called from interrupt context, so it's not possible to use
* schedule_timeout() * schedule_timeout()
* Two important notes : * Two important notes :
* o Make sure you irda_task_delete(task); in case you delete the * o Make sure you irda_task_delete(task); in case you delete the
* calling instance. * calling instance.
...@@ -375,9 +373,9 @@ int irda_task_kick(struct irda_task *task) ...@@ -375,9 +373,9 @@ int irda_task_kick(struct irda_task *task)
* want to lock within the task handler. * want to lock within the task handler.
* Jean II * Jean II
*/ */
struct irda_task *irda_task_execute(void *instance, struct irda_task *irda_task_execute(void *instance,
IRDA_TASK_CALLBACK function, IRDA_TASK_CALLBACK function,
IRDA_TASK_CALLBACK finished, IRDA_TASK_CALLBACK finished,
struct irda_task *parent, void *param) struct irda_task *parent, void *param)
{ {
struct irda_task *task; struct irda_task *task;
...@@ -394,7 +392,7 @@ struct irda_task *irda_task_execute(void *instance, ...@@ -394,7 +392,7 @@ struct irda_task *irda_task_execute(void *instance,
task->function = function; task->function = function;
task->finished = finished; task->finished = finished;
task->parent = parent; task->parent = parent;
task->param = param; task->param = param;
task->magic = IRDA_TASK_MAGIC; task->magic = IRDA_TASK_MAGIC;
init_timer(&task->timer); init_timer(&task->timer);
...@@ -433,7 +431,7 @@ static void irda_task_timer_expired(void *data) ...@@ -433,7 +431,7 @@ static void irda_task_timer_expired(void *data)
* This function should be used by low level device drivers in a similar way * This function should be used by low level device drivers in a similar way
* as ether_setup() is used by normal network device drivers * as ether_setup() is used by normal network device drivers
*/ */
int irda_device_setup(struct net_device *dev) int irda_device_setup(struct net_device *dev)
{ {
ASSERT(dev != NULL, return -1;); ASSERT(dev != NULL, return -1;);
...@@ -445,7 +443,7 @@ int irda_device_setup(struct net_device *dev) ...@@ -445,7 +443,7 @@ int irda_device_setup(struct net_device *dev)
dev->type = ARPHRD_IRDA; dev->type = ARPHRD_IRDA;
dev->tx_queue_len = 8; /* Window size + 1 s-frame */ dev->tx_queue_len = 8; /* Window size + 1 s-frame */
memset(dev->broadcast, 0xff, 4); memset(dev->broadcast, 0xff, 4);
dev->mtu = 2048; dev->mtu = 2048;
...@@ -491,7 +489,7 @@ dongle_t *irda_device_dongle_init(struct net_device *dev, int type) ...@@ -491,7 +489,7 @@ dongle_t *irda_device_dongle_init(struct net_device *dev, int type)
sprintf(modname, "irda-dongle-%d", type); sprintf(modname, "irda-dongle-%d", type);
request_module(modname); request_module(modname);
} }
#endif /* CONFIG_KMOD */ #endif
if (!(reg = hashbin_find(dongles, type, NULL))) { if (!(reg = hashbin_find(dongles, type, NULL))) {
ERROR("IrDA: Unable to find requested dongle\n"); ERROR("IrDA: Unable to find requested dongle\n");
...@@ -514,9 +512,6 @@ dongle_t *irda_device_dongle_init(struct net_device *dev, int type) ...@@ -514,9 +512,6 @@ dongle_t *irda_device_dongle_init(struct net_device *dev, int type)
/* /*
* Function irda_device_dongle_cleanup (dongle) * Function irda_device_dongle_cleanup (dongle)
*
*
*
*/ */
int irda_device_dongle_cleanup(dongle_t *dongle) int irda_device_dongle_cleanup(dongle_t *dongle)
{ {
...@@ -531,21 +526,18 @@ int irda_device_dongle_cleanup(dongle_t *dongle) ...@@ -531,21 +526,18 @@ int irda_device_dongle_cleanup(dongle_t *dongle)
/* /*
* Function irda_device_register_dongle (dongle) * Function irda_device_register_dongle (dongle)
*
*
*
*/ */
int irda_device_register_dongle(struct dongle_reg *new) int irda_device_register_dongle(struct dongle_reg *new)
{ {
/* Check if this dongle has been registred before */ /* Check if this dongle has been registred before */
if (hashbin_find(dongles, new->type, NULL)) { if (hashbin_find(dongles, new->type, NULL)) {
MESSAGE(__FUNCTION__ "(), Dongle already registered\n"); MESSAGE("%s: Dongle already registered\n", __FUNCTION__);
return 0; return 0;
} }
/* Insert IrDA dongle into hashbin */ /* Insert IrDA dongle into hashbin */
hashbin_insert(dongles, (irda_queue_t *) new, new->type, NULL); hashbin_insert(dongles, (irda_queue_t *) new, new->type, NULL);
return 0; return 0;
} }
...@@ -561,7 +553,7 @@ void irda_device_unregister_dongle(struct dongle_reg *dongle) ...@@ -561,7 +553,7 @@ void irda_device_unregister_dongle(struct dongle_reg *dongle)
node = hashbin_remove(dongles, dongle->type, NULL); node = hashbin_remove(dongles, dongle->type, NULL);
if (!node) { if (!node) {
ERROR(__FUNCTION__ "(), dongle not found!\n"); ERROR("%s: dongle not found!\n", __FUNCTION__);
return; return;
} }
} }
...@@ -574,22 +566,22 @@ void irda_device_unregister_dongle(struct dongle_reg *dongle) ...@@ -574,22 +566,22 @@ void irda_device_unregister_dongle(struct dongle_reg *dongle)
* driver to find out which modes it support. * driver to find out which modes it support.
*/ */
int irda_device_set_mode(struct net_device* dev, int mode) int irda_device_set_mode(struct net_device* dev, int mode)
{ {
struct if_irda_req req; struct if_irda_req req;
int ret; int ret;
IRDA_DEBUG(0, __FUNCTION__ "()\n"); IRDA_DEBUG(0, __FUNCTION__ "()\n");
if (!dev->do_ioctl) { if (!dev->do_ioctl) {
ERROR(__FUNCTION__ "(), set_raw_mode not impl. by " ERROR("%s: set_raw_mode not impl. by device driver\n",
"device driver\n"); __FUNCTION__);
return -1; return -1;
} }
req.ifr_mode = mode; req.ifr_mode = mode;
ret = dev->do_ioctl(dev, (struct ifreq *) &req, SIOCSMODE); ret = dev->do_ioctl(dev, (struct ifreq *) &req, SIOCSMODE);
return ret; return ret;
} }
...@@ -602,9 +594,9 @@ int irda_device_set_mode(struct net_device* dev, int mode) ...@@ -602,9 +594,9 @@ int irda_device_set_mode(struct net_device* dev, int mode)
void setup_dma(int channel, char *buffer, int count, int mode) void setup_dma(int channel, char *buffer, int count, int mode)
{ {
unsigned long flags; unsigned long flags;
flags = claim_dma_lock(); flags = claim_dma_lock();
disable_dma(channel); disable_dma(channel);
clear_dma_ff(channel); clear_dma_ff(channel);
set_dma_mode(channel, mode); set_dma_mode(channel, mode);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment