linux-kernel - Re: [PATCH v8 1/6] Extcon (external connector): import Android's switch class and modify.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20120420033209.GA17930@kroah.com>
Date:	Thu, 19 Apr 2012 20:32:09 -0700
From:	Greg KH <gregkh@...uxfoundation.org>
To:	Kyungmin Park <kyungmin.park@...sung.com>
Cc:	MyungJoo Ham <myungjoo.ham@...sung.com>,
	Arnd Bergmann <arnd@...db.de>,
	LKML <linux-kernel@...r.kernel.org>, NeilBrown <neilb@...e.de>,
	Randy Dunlap <rdunlap@...otime.net>,
	Mike Lockwood <lockwood@...roid.com>,
	Arve Hjønnevag <arve@...roid.com>,
	Linus Walleij <linus.walleij@...aro.org>,
	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	Morten CHRISTIANSEN <morten.christiansen@...ricsson.com>,
	Mark Brown <broonie@...nsource.wolfsonmicro.com>,
	John Stultz <john.stultz@...aro.org>,
	Joerg Roedel <joerg.roedel@....com>, myungjoo.ham@...il.com,
	cw00.choi@...sung.com
Subject: Re: [PATCH v8 1/6] Extcon (external connector): import Android's
 switch class and modify.

On Fri, Apr 20, 2012 at 12:08:32PM +0900, Kyungmin Park wrote:
> On 4/20/12, Greg KH <gregkh@...uxfoundation.org> wrote:
> > On Fri, Apr 20, 2012 at 11:57:26AM +0900, Kyungmin Park wrote:
> >> On 4/20/12, Greg KH <gregkh@...uxfoundation.org> wrote:
> >> > On Fri, Apr 20, 2012 at 11:46:25AM +0900, Kyungmin Park wrote:
> >> >> On 4/20/12, Greg KH <gregkh@...uxfoundation.org> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/