* [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
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 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.