On Sun, Jul 28, 2013 at 06:19:29PM +0200, walter harms wrote:
>
>
> Am 28.07.2013 00:38, schrieb Emil Goode:
> > This patch fixes a lot of non-ANSI sparse warnings by adding void to
> > parameterless functions.
> >
> > Signed-off-by: Emil Goode <emilgoode@xxxxxxxxx>
> > ---
> > v2: A bit more specific commit message.
> >
> > .../lustre/lnet/klnds/socklnd/socklnd_lib-linux.c | 6 +++---
> > drivers/staging/lustre/lnet/lnet/api-ni.c | 2 +-
> > drivers/staging/lustre/lnet/selftest/console.c | 2 +-
> > .../staging/lustre/lustre/libcfs/linux/linux-tracefile.c | 14 +++++++-------
> > drivers/staging/lustre/lustre/ptlrpc/pack_generic.c | 2 +-
> > drivers/staging/lustre/lustre/ptlrpc/pinger.c | 4 ++--
> > 6 files changed, 15 insertions(+), 15 deletions(-)
> >
> > diff --git a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_lib-linux.c b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_lib-linux.c
> > index 3e08fe2..76e442b 100644
> > --- a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_lib-linux.c
> > +++ b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_lib-linux.c
> > @@ -325,20 +325,20 @@ ksocknal_lib_tunables_init ()
> > }
> >
> > void
> > -ksocknal_lib_tunables_fini ()
> > +ksocknal_lib_tunables_fini(void)
> > {
> > if (ksocknal_tunables.ksnd_sysctl != NULL)
> > unregister_sysctl_table(ksocknal_tunables.ksnd_sysctl);
> > }
>
> this is not needed as unregister_sysctl_table() can handle NULL
There's lots of cleanup needed in this code, but as that's a separate
thing from what the original poster was doing here, it isn't relevant
for this patch.
Follow-on patches are always welcome...
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
