lkml.org 
[lkml]   [2008]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] ALSA: cs5535audio: only build OLPC support if MGEODE_LX is defined
At Thu, 13 Nov 2008 21:38:52 -0500,
Andres Salomon wrote:
>
> On Fri, 14 Nov 2008 00:30:48 +0100
> Takashi Iwai <tiwai@suse.de> wrote:
>
> > At Thu, 13 Nov 2008 20:03:39 +0100,
> > Ingo Molnar wrote:
> > >
> > >
> > > * Andres Salomon <dilinger@queued.net> wrote:
> > >
> > > > On Thu, 13 Nov 2008 17:38:02 +0100
> > > > Pavel Machek <pavel@suse.cz> wrote:
> > > >
> > > > > On Thu 2008-11-13 11:14:28, Andres Salomon wrote:
> > > > > > On Thu, 13 Nov 2008 16:54:19 +0100
> > > > > > Takashi Iwai <tiwai@suse.de> wrote:
> > > > > >
> > > > > > > At Thu, 13 Nov 2008 16:37:14 +0100,
> > > > > > > Pavel Machek wrote:
> > > > > > [...]
> > > > > > > >
> > > > > > > > > IMO, the basic problem is rather that the platform and
> > > > > > > > > the CPU selection is mixed up. The geode GPIO stuff
> > > > > > > > > shouldn't belong to CPU selection.
> > > > > > > >
> > > > > > > > ACK... and suggested patch makes it worse.
> > > > > > > >
> > > > > > > > There should be option 'support OLPC' somewhere, and that
> > > > > > > > should enable the gpio bits. Alsa driver should depend on
> > > > > > > > that one.
> > > > > > >
> > > > > > > Any better fix patch is welcome, of course :)
> > > > > > > This would make save a lot time than writing mails...
> > > > >
> > > > > >
> > > > > > Okay, how about something like the following?
> > > > >
> > > > > It is wrong. It is valid & useful to compile kernel for i386 and
> > > > > expect all the features on OLPC (with small performance hit).
> > > >
> > > > No one's debating that. That requires additional work that will
> > > > take longer, however.
> > >
> > > i've zapped this patch meanwhile:
> > >
> > > 1355c96: x86/olpc: make CONFIG_OLPC dependent on CONFIG_MGEODE_LX
> > >
> > > because it cripples the ability to run distribution kernels on the
> > > OLPC.
> >
> > OK, I reverted also all relevant changes for cs5535audio driver now.
> > The patches are saved in topic/cs5535audio branch, though.
> >
> > Let's fix OLPC-geode coupling first.
> >
>
>
> Hm, I'd really rather prefer this to be upstream. The patch I sent adds
> no regressions,

It doesn't build as is, a clear regression :)

> allows the driver to happily coexist with existing
> stuff, and *does* add support if you configure OLPC with MGEODE_LX
> (generic kernels don't get the additional benefits, but those configured
> specifically for OLPC do).

If this dependency issue is solved, of course I'm willing to take it
back. That's why the patches are still kept in the topic branch.


thanks,

Takashi


\
 
 \ /
  Last update: 2008-11-14 07:57    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site