From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755031AbaHGSbC (ORCPT ); Thu, 7 Aug 2014 14:31:02 -0400 Received: from senator.holtmann.net ([87.106.208.187]:36386 "EHLO mail.holtmann.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750754AbaHGSbA convert rfc822-to-8bit (ORCPT ); Thu, 7 Aug 2014 14:31:00 -0400 Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 7.3 \(1878.6\)) Subject: Re: [PATCH] 6lowpan: Allow 6LoWPAN to be modular From: Marcel Holtmann In-Reply-To: Date: Thu, 7 Aug 2014 11:31:11 -0700 Cc: Alexander Aring , "David S. Miller" , linux-zigbee-devel@lists.sourceforge.net, BlueZ development , linux-wpan@vger.kernel.org, "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" Content-Transfer-Encoding: 8BIT Message-Id: <144B79BA-C835-4EF4-8FF5-533D535A5143@holtmann.org> References: <1407424003-15073-1-git-send-email-geert@linux-m68k.org> To: Geert Uytterhoeven X-Mailer: Apple Mail (2.1878.6) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Geert, >>> Change config symbol 6LOWPAN from type bool to type tristate, so >>> 6LoWPAN can be built modular, just like IPV6 >>> >>> Signed-off-by: Geert Uytterhoeven >>> --- >>> net/6lowpan/Kconfig | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/net/6lowpan/Kconfig b/net/6lowpan/Kconfig >>> index 028a5c6d1f61..e4a02ef55102 100644 >>> --- a/net/6lowpan/Kconfig >>> +++ b/net/6lowpan/Kconfig >>> @@ -1,5 +1,5 @@ >>> config 6LOWPAN >>> - bool "6LoWPAN Support" >>> + tristate "6LoWPAN Support" >> >> is this a regression from the previous CONFIG_6LOWPAN separation that we introduced? If it is, then it might be easiest if Dave just takes it into his net tree. Otherwise I take it through bluetooth-next. > > I hadn't thought about that... Checking... > > Yes, I believe this is a regression. > > In v3.16, my allmodconfig build had: > > CONFIG_IEEE802154_6LOWPAN=m > CONFIG_6LOWPAN_IPHC=m > CONFIG_BT_6LOWPAN=y > > Without my patch it has: > > CONFIG_6LOWPAN=y > CONFIG_IEEE802154_6LOWPAN=m > CONFIG_BT_6LOWPAN=m > > and iphc is what's in net/6lowpan/built-in.o now. Acked-by: Marcel Holtmann Dave, I am fine if you just queue this up for 3.17-rc1. Regards Marcel