From: Tilman Schmidt <tilman@imap.cc>
To: Joe Perches <joe@perches.com>
Cc: David Miller <davem@davemloft.net>,
netdev@vger.kernel.org, Hansjoerg Lipp <hjlipp@web.de>,
Karsten Keil <isdn@linux-pingi.de>,
gigaset307x-common@lists.sourceforge.net
Subject: Re: [PATCH net-next] drivers/isdn/gigaset: tasklet_init - Remove unnecessary leading & from second arg
Date: Sat, 21 Nov 2009 19:08:14 +0100 [thread overview]
Message-ID: <4B082C8E.2030801@imap.cc> (raw)
In-Reply-To: <1258660553.3863.18.camel@Joe-Laptop.home>
[-- Attachment #1: Type: text/plain, Size: 3970 bytes --]
Am 19.11.2009 20:55 schrieb Joe Perches:
> Changed function pointer use from non-majority address-of style
> to majority short form without & via:
>
> grep -rPl "\btasklet_init\s*\([^,\)]+,\s*\&" drivers/isdn | while read file ; do \
> perl -i -e 'local $/; while (<>) { s@(\btasklet_init\s*\([^,\)]+,\s*)\&@\1@g ; print ; }' $file ;\
> done
>
> Compile tested allyesconfig x86
>
> Signed-off-by: Joe Perches <joe@perches.com>
Acked-by: Tilman Schmidt <tilman@imap.cc>
Thanks,
Tilman
>
> drivers/isdn/gigaset/bas-gigaset.c | 4 ++--
> drivers/isdn/gigaset/common.c | 2 +-
> drivers/isdn/gigaset/interface.c | 2 +-
> drivers/isdn/gigaset/ser-gigaset.c | 2 +-
> drivers/isdn/gigaset/usb-gigaset.c | 2 +-
> 5 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/isdn/gigaset/bas-gigaset.c b/drivers/isdn/gigaset/bas-gigaset.c
> index 9fd19db..95ebc51 100644
> --- a/drivers/isdn/gigaset/bas-gigaset.c
> +++ b/drivers/isdn/gigaset/bas-gigaset.c
> @@ -2117,7 +2117,7 @@ static int gigaset_initbcshw(struct bc_state *bcs)
> return 0;
> }
> tasklet_init(&ubc->sent_tasklet,
> - &write_iso_tasklet, (unsigned long) bcs);
> + write_iso_tasklet, (unsigned long) bcs);
>
> spin_lock_init(&ubc->isoinlock);
> for (i = 0; i < BAS_INURBS; ++i)
> @@ -2138,7 +2138,7 @@ static int gigaset_initbcshw(struct bc_state *bcs)
> ubc->shared0s = 0;
> ubc->stolen0s = 0;
> tasklet_init(&ubc->rcvd_tasklet,
> - &read_iso_tasklet, (unsigned long) bcs);
> + read_iso_tasklet, (unsigned long) bcs);
> return 1;
> }
>
> diff --git a/drivers/isdn/gigaset/common.c b/drivers/isdn/gigaset/common.c
> index c438cfc..82ed1cd 100644
> --- a/drivers/isdn/gigaset/common.c
> +++ b/drivers/isdn/gigaset/common.c
> @@ -727,7 +727,7 @@ struct cardstate *gigaset_initcs(struct gigaset_driver *drv, int channels,
> cs->ev_tail = 0;
> cs->ev_head = 0;
>
> - tasklet_init(&cs->event_tasklet, &gigaset_handle_event,
> + tasklet_init(&cs->event_tasklet, gigaset_handle_event,
> (unsigned long) cs);
> cs->commands_pending = 0;
> cs->cur_at_seq = 0;
> diff --git a/drivers/isdn/gigaset/interface.c b/drivers/isdn/gigaset/interface.c
> index 577809c..d2260b0 100644
> --- a/drivers/isdn/gigaset/interface.c
> +++ b/drivers/isdn/gigaset/interface.c
> @@ -584,7 +584,7 @@ void gigaset_if_init(struct cardstate *cs)
> if (!drv->have_tty)
> return;
>
> - tasklet_init(&cs->if_wake_tasklet, &if_wake, (unsigned long) cs);
> + tasklet_init(&cs->if_wake_tasklet, if_wake, (unsigned long) cs);
>
> mutex_lock(&cs->mutex);
> cs->tty_dev = tty_register_device(drv->tty, cs->minor_index, NULL);
> diff --git a/drivers/isdn/gigaset/ser-gigaset.c b/drivers/isdn/gigaset/ser-gigaset.c
> index ac3409e..168d585 100644
> --- a/drivers/isdn/gigaset/ser-gigaset.c
> +++ b/drivers/isdn/gigaset/ser-gigaset.c
> @@ -434,7 +434,7 @@ static int gigaset_initcshw(struct cardstate *cs)
> dev_set_drvdata(&cs->hw.ser->dev.dev, cs);
>
> tasklet_init(&cs->write_tasklet,
> - &gigaset_modem_fill, (unsigned long) cs);
> + gigaset_modem_fill, (unsigned long) cs);
> return 1;
> }
>
> diff --git a/drivers/isdn/gigaset/usb-gigaset.c b/drivers/isdn/gigaset/usb-gigaset.c
> index f56b2a8..3ab1dae 100644
> --- a/drivers/isdn/gigaset/usb-gigaset.c
> +++ b/drivers/isdn/gigaset/usb-gigaset.c
> @@ -614,7 +614,7 @@ static int gigaset_initcshw(struct cardstate *cs)
> ucs->bulk_out_urb = NULL;
> ucs->read_urb = NULL;
> tasklet_init(&cs->write_tasklet,
> - &gigaset_modem_fill, (unsigned long) cs);
> + gigaset_modem_fill, (unsigned long) cs);
>
> return 1;
> }
>
>
--
Tilman Schmidt E-Mail: tilman@imap.cc
Bonn, Germany
Diese Nachricht besteht zu 100% aus wiederverwerteten Bits.
Ungeöffnet mindestens haltbar bis: (siehe Rückseite)
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 259 bytes --]
prev parent reply other threads:[~2009-11-21 18:08 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-11-19 19:55 [PATCH net-next] drivers/isdn/gigaset: tasklet_init - Remove unnecessary leading & from second arg Joe Perches
2009-11-19 20:29 ` David Miller
2009-11-21 18:08 ` Tilman Schmidt [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4B082C8E.2030801@imap.cc \
--to=tilman@imap.cc \
--cc=davem@davemloft.net \
--cc=gigaset307x-common@lists.sourceforge.net \
--cc=hjlipp@web.de \
--cc=isdn@linux-pingi.de \
--cc=joe@perches.com \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.