[B.A.T.M.A.N.] [PATCH] batman-adv: Document multicast compile time option

[B.A.T.M.A.N.] [PATCH] batman-adv: Document multicast compile time option

[Sven Eckelmann]

The README.external lists all options which can be enabled/disabled during
compile time. The CONFIG_BATMAN_ADV_MCAST option wasn't document in this
list.

Signed-off-by: Sven Eckelmann <sven@narfation.org>
---
 Makefile        | 1 +
 README.external | 1 +
 2 files changed, 2 insertions(+)

diff --git a/Makefile b/Makefile
index 8a04d12..d8bd4ef 100644
--- a/Makefile
+++ b/Makefile
@@ -59,6 +59,7 @@ BUILD_FLAGS := \
 	CONFIG_BATMAN_ADV_NC=$(CONFIG_BATMAN_ADV_NC) \
 	CONFIG_BATMAN_ADV_MCAST=$(CONFIG_BATMAN_ADV_MCAST) \
 	INSTALL_MOD_DIR=updates/net/batman-adv/
+export batman-adv-y += ../../compat.o
 
 all: config
 	$(MAKE) -C $(KERNELPATH) $(BUILD_FLAGS)	modules
diff --git a/README.external b/README.external
index e3b533d..f068273 100644
--- a/README.external
+++ b/README.external
@@ -38,6 +38,7 @@ module).  Available  options  and  their    possible   values are
  * CONFIG_BATMAN_ADV_DEBUG=[y|n*] (B.A.T.M.A.N. debugging)
  * CONFIG_BATMAN_ADV_BLA=[y*|n] (B.A.T.M.A.N. bridge loop avoidance)
  * CONFIG_BATMAN_ADV_DAT=[y*|n] (B.A.T.M.A.N. Distributed ARP Table)
+ * CONFIG_BATMAN_ADV_MCAST=[y*|n] (B.A.T.M.A.N. multicast optimizations)
  * CONFIG_BATMAN_ADV_NC=[y|n*] (B.A.T.M.A.N. Network Coding)
 
 e.g., debugging can be enabled by
-- 
2.1.4

Re: [B.A.T.M.A.N.] [PATCH] batman-adv: Document multicast compile time option

[Marek Lindner]

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

On Tuesday, April 21, 2015 12:42:09 Sven Eckelmann wrote:
> --- a/Makefile
> +++ b/Makefile
> @@ -59,6 +59,7 @@ BUILD_FLAGS := \
>         CONFIG_BATMAN_ADV_NC=$(CONFIG_BATMAN_ADV_NC) \
>         CONFIG_BATMAN_ADV_MCAST=$(CONFIG_BATMAN_ADV_MCAST) \
>         INSTALL_MOD_DIR=updates/net/batman-adv/
> +export batman-adv-y += ../../compat.o
>  
>  all: config
>         $(MAKE) -C $(KERNELPATH) $(BUILD_FLAGS) modules

This hunk does not look like it should be part of this patch according to the 
patch description ?!

Cheers,
Marek

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

Re: [B.A.T.M.A.N.] [PATCH] batman-adv: Document multicast compile time option

[Marek Lindner]

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

On Wednesday, April 22, 2015 10:08:49 Marek Lindner wrote:
> On Tuesday, April 21, 2015 12:42:09 Sven Eckelmann wrote:
> > --- a/Makefile
> > +++ b/Makefile
> > @@ -59,6 +59,7 @@ BUILD_FLAGS := \
> >
> >         CONFIG_BATMAN_ADV_NC=$(CONFIG_BATMAN_ADV_NC) \
> >         CONFIG_BATMAN_ADV_MCAST=$(CONFIG_BATMAN_ADV_MCAST) \
> >         INSTALL_MOD_DIR=updates/net/batman-adv/
> >
> > +export batman-adv-y += ../../compat.o
> >
> >  
> >  all: config
> >         $(MAKE) -C $(KERNELPATH) $(BUILD_FLAGS) modules
> 
> This hunk does not look like it should be part of this patch according to
> the  patch description ?!

Scratch that. You had already sent v2 without the hunk.

Cheers,
Marek

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