lkml.org 
[lkml]   [2012]   [Apr]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [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@linuxfoundation.org> wrote:
    > > On Fri, Apr 20, 2012 at 11:57:26AM +0900, Kyungmin Park wrote:
    > >> On 4/20/12, Greg KH <gregkh@linuxfoundation.org> wrote:
    > >> > On Fri, Apr 20, 2012 at 11:46:25AM +0900, Kyungmin Park wrote:
    > >> >> On 4/20/12, Greg KH <gregkh@linuxfoundation.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@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2012-04-20 05:35    [W:2.888 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site