From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932545Ab2DTDch (ORCPT ); Thu, 19 Apr 2012 23:32:37 -0400 Received: from mail-pb0-f46.google.com ([209.85.160.46]:51662 "EHLO mail-pb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932268Ab2DTDcP (ORCPT ); Thu, 19 Apr 2012 23:32:15 -0400 Date: Thu, 19 Apr 2012 20:32:09 -0700 From: Greg KH To: Kyungmin Park Cc: MyungJoo Ham , Arnd Bergmann , LKML , NeilBrown , Randy Dunlap , Mike Lockwood , Arve =?iso-8859-1?Q?Hj=F8nnevag?= , Linus Walleij , Dmitry Torokhov , Morten CHRISTIANSEN , Mark Brown , John Stultz , Joerg Roedel , myungjoo.ham@gmail.com, cw00.choi@samsung.com Subject: Re: [PATCH v8 1/6] Extcon (external connector): import Android's switch class and modify. Message-ID: <20120420033209.GA17930@kroah.com> References: <20120418214703.GA31606@kroah.com> <1334803298-24260-1-git-send-email-myungjoo.ham@samsung.com> <1334803298-24260-2-git-send-email-myungjoo.ham@samsung.com> <20120420023704.GA1399@kroah.com> <20120420024920.GA17151@kroah.com> <20120420030046.GA17274@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Apr 20, 2012 at 12:08:32PM +0900, Kyungmin Park wrote: > On 4/20/12, Greg KH wrote: > > On Fri, Apr 20, 2012 at 11:57:26AM +0900, Kyungmin Park wrote: > >> On 4/20/12, Greg KH wrote: > >> > On Fri, Apr 20, 2012 at 11:46:25AM +0900, Kyungmin Park wrote: > >> >> On 4/20/12, Greg KH wrote: > >> >> > On Thu, Apr 19, 2012 at 11:41:33AM +0900, MyungJoo Ham wrote: > >> >> >> External connector class (extcon) is based on and an extension of > >> >> >> Android kernel's switch class located at linux/drivers/switch/. > >> >> > > >> >> > Oh come on, this still breaks the build, it is as if you didn't even > >> >> > test this at all, or modify anything since I reported it as a problem > >> >> > to > >> >> > you yesterday: > >> >> > > >> >> > drivers/extcon/extcon_class.c:79:6: error: redefinition of > >> >> > ‘extcon_set_state’ > >> >> > include/linux/extcon.h:80:20: note: previous definition of > >> >> > ‘extcon_set_state’ was here > >> >> > drivers/extcon/extcon_class.c:170:5: error: redefinition of > >> >> > ‘extcon_dev_register’ > >> >> > include/linux/extcon.h:67:19: note: previous definition of > >> >> > ‘extcon_dev_register’ was here > >> >> > drivers/extcon/extcon_class.c:214:6: error: redefinition of > >> >> > ‘extcon_dev_unregister’ > >> >> > include/linux/extcon.h:73:20: note: previous definition of > >> >> > ‘extcon_dev_unregister’ was here > >> >> > > >> >> > You can do better than this, I know you can. > >> >> > >> >> Umm. can you show your EXTCON config? I think you build it as module. > >> >> it's tested it extcon on or not. anyway I'll test it as module and > >> >> check it. > >> > > >> > Yes, I selected to build it as a module, which, according to your patch, > >> > is a valid selection. > >> > >> Found, need to add CONFIG_EXTCON_MODULE at below. > > > > You do know we have a new macro that handles all of this for you, > > IS_ENABLED(). It handles the _MODULE stuff and the other stuff as well, > > and you can put it in a .c expression also. > > > > Try using that instead. > > No problem, send patches with it. > > BTW, can we post it after one hour? You can send it whenever you want, I do not impose any deadlines. But note, I'll not be reading email for at least 10 more hours, it's getting late in my timezone, so take your time and get it right, no rush. thanks, greg k-h