[lkml]   [2012]   [Jul]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] apple-gmux: Fix kconfig dependencies
    On Mon, Jul 16, 2012 at 04:50:34PM -0700, Seth Forshee wrote:
    > On Mon, Jul 02, 2012 at 09:55:29AM -0500, Seth Forshee wrote:
    > > Fix the dependencies of apple-gmux to prevent it from being built-in
    > > when one or more of its dependencies is built as a module. Otherwise it
    > > can fail to build due to missing symbols.
    > Matthew: Are you okay with this, or is there some better way to do it?

    I think this is probably about as good as it gets.

    > I'm looking at Andreas's code for switcharoo support, and something I
    > want to do along with it is make apple-gmux not dependent on backlight
    > support (i.e. you can still build switcharoo support if
    > BACKLIGHT_CLASS_DEVICE is disabled). I'm assuming this patch as a
    > prerequisite. But things do get really monstrously ugly, so if there's a
    > cleaner way to enforce this that I haven't found I'd love to hear about
    > it.

    Why do you want to support that case? One of the problems we have with
    the kernel at present is that it's trivial to come up with non-sensical
    configurations. I can't think of a single case where you'd want the gmux
    support without also having the backlight support code.

    Matthew Garrett |

     \ /
      Last update: 2012-07-23 16:21    [W:2.562 / U:0.072 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site