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