Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753384AbYKNGyr (ORCPT ); Fri, 14 Nov 2008 01:54:47 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1750813AbYKNGyi (ORCPT ); Fri, 14 Nov 2008 01:54:38 -0500 Received: from mx1.suse.de ([195.135.220.2]:37968 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750727AbYKNGyh (ORCPT ); Fri, 14 Nov 2008 01:54:37 -0500 Date: Fri, 14 Nov 2008 07:54:35 +0100 Message-ID: From: Takashi Iwai To: Andres Salomon Cc: Ingo Molnar , Pavel Machek , Jeremy Katz , Thomas Gleixner , Jordan Crouse , linux-kernel@vger.kernel.org, dsaxena@laptop.org Subject: Re: [PATCH] ALSA: cs5535audio: only build OLPC support if MGEODE_LX is defined In-Reply-To: <20081113213852.16561ed2@ephemeral> References: <20081109201710.4c490f04@debxo> <1226498050.12791.45.camel@aglarond.local> <1226547898.13077.83.camel@aglarond.local> <20081113153714.GA1520@ucw.cz> <20081113111428.59aa36cb@ephemeral> <20081113163802.GA1734@ucw.cz> <20081113120147.2ecde3c8@ephemeral> <20081113190339.GE17851@elte.hu> <20081113213852.16561ed2@ephemeral> User-Agent: Wanderlust/2.12.0 (Your Wildest Dreams) SEMI/1.14.6 (Maruoka) FLIM/1.14.7 (=?ISO-8859-4?Q?Sanj=F2?=) APEL/10.6 Emacs/22.3 (x86_64-suse-linux-gnu) MULE/5.0 (SAKAKI) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2803 Lines: 81 At Thu, 13 Nov 2008 21:38:52 -0500, Andres Salomon wrote: > > On Fri, 14 Nov 2008 00:30:48 +0100 > Takashi Iwai wrote: > > > At Thu, 13 Nov 2008 20:03:39 +0100, > > Ingo Molnar wrote: > > > > > > > > > * Andres Salomon wrote: > > > > > > > On Thu, 13 Nov 2008 17:38:02 +0100 > > > > Pavel Machek wrote: > > > > > > > > > On Thu 2008-11-13 11:14:28, Andres Salomon wrote: > > > > > > On Thu, 13 Nov 2008 16:54:19 +0100 > > > > > > Takashi Iwai 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 -- 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/