public inbox for b.a.t.m.a.n@lists.open-mesh.org
 help / color / mirror / Atom feed
* [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: Remove unused work_struct forward declaration
@ 2016-05-05 10:35 Sven Eckelmann
  2016-05-05 10:35 ` [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: Add missing include for container_of Sven Eckelmann
  2016-05-05 14:57 ` [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: Remove unused work_struct forward declaration Marek Lindner
  0 siblings, 2 replies; 4+ messages in thread
From: Sven Eckelmann @ 2016-05-05 10:35 UTC (permalink / raw)
  To: b.a.t.m.a.n; +Cc: Sven Eckelmann

The only function declaration using the "struct work_struct" was removed
and thus this forward declaration should also be removed.

Fixes: 0d8468553c3c ("batman-adv: remove ogm_emit and ogm_schedule API calls")
Signed-off-by: Sven Eckelmann <sven.eckelmann@open-mesh.com>
---
Problem was found by https://lists.open-mesh.org/pipermail/linux-merge/2016-May/003146.html

 net/batman-adv/send.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/net/batman-adv/send.h b/net/batman-adv/send.h
index f6fda19..7cecb75 100644
--- a/net/batman-adv/send.h
+++ b/net/batman-adv/send.h
@@ -26,7 +26,6 @@
 #include "packet.h"
 
 struct sk_buff;
-struct work_struct;
 
 void batadv_forw_packet_free(struct batadv_forw_packet *forw_packet);
 int batadv_send_skb_to_orig(struct sk_buff *skb,
-- 
2.8.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: Add missing include for container_of
  2016-05-05 10:35 [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: Remove unused work_struct forward declaration Sven Eckelmann
@ 2016-05-05 10:35 ` Sven Eckelmann
  2016-05-05 14:58   ` Marek Lindner
  2016-05-05 14:57 ` [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: Remove unused work_struct forward declaration Marek Lindner
  1 sibling, 1 reply; 4+ messages in thread
From: Sven Eckelmann @ 2016-05-05 10:35 UTC (permalink / raw)
  To: b.a.t.m.a.n; +Cc: Sven Eckelmann

Fixes: 0d8468553c3c ("batman-adv: remove ogm_emit and ogm_schedule API calls")
Signed-off-by: Sven Eckelmann <sven.eckelmann@open-mesh.com>
---
Problem was found by https://lists.open-mesh.org/pipermail/linux-merge/2016-May/003146.html

 net/batman-adv/bat_iv_ogm.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/batman-adv/bat_iv_ogm.c b/net/batman-adv/bat_iv_ogm.c
index 67bdba1..4815db9 100644
--- a/net/batman-adv/bat_iv_ogm.c
+++ b/net/batman-adv/bat_iv_ogm.c
@@ -30,6 +30,7 @@
 #include <linux/if_ether.h>
 #include <linux/init.h>
 #include <linux/jiffies.h>
+#include <linux/kernel.h>
 #include <linux/list.h>
 #include <linux/kref.h>
 #include <linux/lockdep.h>
-- 
2.8.1


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: Remove unused work_struct forward declaration
  2016-05-05 10:35 [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: Remove unused work_struct forward declaration Sven Eckelmann
  2016-05-05 10:35 ` [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: Add missing include for container_of Sven Eckelmann
@ 2016-05-05 14:57 ` Marek Lindner
  1 sibling, 0 replies; 4+ messages in thread
From: Marek Lindner @ 2016-05-05 14:57 UTC (permalink / raw)
  To: b.a.t.m.a.n; +Cc: Sven Eckelmann

[-- Attachment #1: Type: text/plain, Size: 553 bytes --]

On Thursday, May 05, 2016 12:35:18 Sven Eckelmann wrote:
> The only function declaration using the "struct work_struct" was removed
> and thus this forward declaration should also be removed.
> 
> Fixes: 0d8468553c3c ("batman-adv: remove ogm_emit and ogm_schedule API
> calls") Signed-off-by: Sven Eckelmann <sven.eckelmann@open-mesh.com>
> ---
> Problem was found by
> https://lists.open-mesh.org/pipermail/linux-merge/2016-May/003146.html
> 
>  net/batman-adv/send.h | 1 -
>  1 file changed, 1 deletion(-)

Applied in revision 2ceab95.

Thanks,
Marek

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: Add missing include for container_of
  2016-05-05 10:35 ` [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: Add missing include for container_of Sven Eckelmann
@ 2016-05-05 14:58   ` Marek Lindner
  0 siblings, 0 replies; 4+ messages in thread
From: Marek Lindner @ 2016-05-05 14:58 UTC (permalink / raw)
  To: b.a.t.m.a.n; +Cc: Sven Eckelmann

[-- Attachment #1: Type: text/plain, Size: 422 bytes --]

On Thursday, May 05, 2016 12:35:19 Sven Eckelmann wrote:
> Fixes: 0d8468553c3c ("batman-adv: remove ogm_emit and ogm_schedule API
> calls") Signed-off-by: Sven Eckelmann <sven.eckelmann@open-mesh.com>
> ---
> Problem was found by
> https://lists.open-mesh.org/pipermail/linux-merge/2016-May/003146.html
> 
>  net/batman-adv/bat_iv_ogm.c | 1 +
>  1 file changed, 1 insertion(+)

Applied in revision ba17db2.

Thanks,
Marek

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-05-05 14:58 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-05 10:35 [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: Remove unused work_struct forward declaration Sven Eckelmann
2016-05-05 10:35 ` [B.A.T.M.A.N.] [PATCH 2/2] batman-adv: Add missing include for container_of Sven Eckelmann
2016-05-05 14:58   ` Marek Lindner
2016-05-05 14:57 ` [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: Remove unused work_struct forward declaration Marek Lindner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox