2019-02-23 11:20:01

by Jules Irenge

[permalink] [raw]
Subject: [PATCH] net: ax25: Fix multiple blank lines warning

Signed-off-by: Jules Irenge <[email protected]>
---
net/ax25/af_ax25.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/net/ax25/af_ax25.c b/net/ax25/af_ax25.c
index 5048790a75f6..2da349b7854a 100644
--- a/net/ax25/af_ax25.c
+++ b/net/ax25/af_ax25.c
@@ -49,8 +49,6 @@
#include <net/ip.h>
#include <net/arp.h>

-
-
HLIST_HEAD(ax25_list);
DEFINE_SPINLOCK(ax25_list_lock);

--
2.20.1



2019-02-23 13:00:42

by Greg KH

[permalink] [raw]
Subject: Re: [Outreachy kernel] [PATCH] net: ax25: Fix multiple blank lines warning

On Sat, Feb 23, 2019 at 11:17:51AM +0000, Jules Irenge wrote:
> Signed-off-by: Jules Irenge <[email protected]>
> ---
> net/ax25/af_ax25.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/net/ax25/af_ax25.c b/net/ax25/af_ax25.c
> index 5048790a75f6..2da349b7854a 100644
> --- a/net/ax25/af_ax25.c
> +++ b/net/ax25/af_ax25.c
> @@ -49,8 +49,6 @@
> #include <net/ip.h>
> #include <net/arp.h>
>
> -
> -
> HLIST_HEAD(ax25_list);
> DEFINE_SPINLOCK(ax25_list_lock);

Why are you doing outreachy application patches in net/? We only accept
them in drivers/staging/ Always start there first.

thanks,

greg k-h

2019-02-23 13:25:05

by Ivan Ivanov

[permalink] [raw]
Subject: Re: [Outreachy kernel] [PATCH] net: ax25: Fix multiple blank lines warning

as usual, Outreachy is doing stiuupid almost-useless commits like
remove two blank lines or fix a tiny typo. They are trying hard to
pretend that they are useful by doing many lame patches like this,
while promoting their SJW madness, anti-male sieexism and anti-white
riaacism. But with the lame stuff like this patch, they are failing
miserably. Maybe hire a ghostwriter (from NSA) to write some drivers
for you?

P.S. and no I am not going to write them for you because I disagree
with your Ci0ck of Conduct and now have to write from anonymous
account

On Sat, Feb 23, 2019, Greg KH <[email protected]> wrote:
>
> On Sat, Feb 23, 2019 at 11:17:51AM +0000, Jules Irenge wrote:
> > Signed-off-by: Jules Irenge <[email protected]>
> > ---
> > net/ax25/af_ax25.c | 2 --
> > 1 file changed, 2 deletions(-)
> >
> > diff --git a/net/ax25/af_ax25.c b/net/ax25/af_ax25.c
> > index 5048790a75f6..2da349b7854a 100644
> > --- a/net/ax25/af_ax25.c
> > +++ b/net/ax25/af_ax25.c
> > @@ -49,8 +49,6 @@
> > #include <net/ip.h>
> > #include <net/arp.h>
> >
> > -
> > -
> > HLIST_HEAD(ax25_list);
> > DEFINE_SPINLOCK(ax25_list_lock);
>
> Why are you doing outreachy application patches in net/? We only accept
> them in drivers/staging/ Always start there first.
>
> thanks,
>
> greg k-h

2019-02-23 13:28:13

by Julia Lawall

[permalink] [raw]
Subject: Re: [Outreachy kernel] [PATCH] net: ax25: Fix multiple blank lines warning

On Sat, 23 Feb 2019, Jules Irenge wrote:

You need a commit message here.

> Signed-off-by: Jules Irenge <[email protected]>
> ---
> net/ax25/af_ax25.c | 2 --

This is the same file as in your previous patch, so they should be in a
series.

But please work on files in drivers/staging instead.

julia

> 1 file changed, 2 deletions(-)
>
> diff --git a/net/ax25/af_ax25.c b/net/ax25/af_ax25.c
> index 5048790a75f6..2da349b7854a 100644
> --- a/net/ax25/af_ax25.c
> +++ b/net/ax25/af_ax25.c
> @@ -49,8 +49,6 @@
> #include <net/ip.h>
> #include <net/arp.h>
>
> -
> -
> HLIST_HEAD(ax25_list);
> DEFINE_SPINLOCK(ax25_list_lock);
>
> --
> 2.20.1
>
> --
> You received this message because you are subscribed to the Google Groups "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
> To post to this group, send email to [email protected].
> To view this discussion on the web visit https://groups.google.com/d/msgid/outreachy-kernel/20190223111751.14620-1-jbi.octave%40gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>