Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757669AbYGFOaA (ORCPT ); Sun, 6 Jul 2008 10:30:00 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756736AbYGFO3w (ORCPT ); Sun, 6 Jul 2008 10:29:52 -0400 Received: from nf-out-0910.google.com ([64.233.182.187]:34225 "EHLO nf-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756740AbYGFO3v (ORCPT ); Sun, 6 Jul 2008 10:29:51 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=hXEBWniBs+9OVzxBIOhC8P19OOkodtqOJU/IDUXl+q9MI0h9MyhCtXyl8wWnsFBV6D 3Tq3EEjOVeDxUruKYsFszQIxVizryPLdZeGVNOluvRuj9dCBLsHx8CdIZ2/6ivBg9VGY y/r0xJ1eqLABvx10cllTk0vXRkC52XBWWVIac= Message-ID: <412bdbff0807060729o639f98b1s383d146856feb6da@mail.gmail.com> Date: Sun, 6 Jul 2008 10:29:48 -0400 From: "Devin Heitmueller" To: "David Woodhouse" Subject: Re: [linux-dvb] [PATCH] Remove fdump tool for av7110 firmware Cc: "Klaus Schmidinger" , kernelnewbies , kernel-janitors , LKML , "Jaswinder Singh" , linux-dvb@linuxtv.org, "Alan Cox" In-Reply-To: <1215343022.10393.945.camel@pmac.infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <1214139259.2994.8.camel@jaswinder.satnam> <200807060315.51736@orion.escape-edv.de> <48708BBF.9050400@cadsoft.de> <1215343022.10393.945.camel@pmac.infradead.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2445 Lines: 61 On Sun, Jul 6, 2008 at 7:17 AM, David Woodhouse wrote: > On Sun, 2008-07-06 at 11:09 +0200, Klaus Schmidinger wrote: >> On 07/06/08 03:15, Oliver Endriss wrote: >> > Jaswinder Singh wrote: >> >> There's no point in this, since the user can use the BUILTIN_FIRMWARE >> >> option to include arbitrary firmware files directly in the kernel image. >> > >> > NAK! This option allows to compile the firmware into the _driver_, >> > which is very useful if you want to test various driver/firmware >> > combinations. Having the firmware in the _kernel_ does not help! >> >> I strongly support Oliver's request! >> Working with various driver versions is much easier with the >> firmware compiled into the driver! > > That's strange; I've found exactly the opposite to be the case. > > If I want to test permutations of driver and firmware, as I've done for > the libertas driver a number of times, I find it _much_ better to > preserve the modularity. I can build each version of the driver and can > test that against various firmware versions without having to rebuild > it, and with much less chance of something going wrong so that I'm not > actually testing what I think I'm testing. > > Perhaps I'm missing something that would help me work better? Please > could you help me understand how you currently work, and I'll attempt to > make it easier for you. Can you talk me through an example of a session > where you had to do this testing of 'various driver/firmware > combinations'? > > > -- > dwmw2 > > > _______________________________________________ > linux-dvb mailing list > linux-dvb@linuxtv.org > http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb > Correct me if I'm wrong, but doesn't this also affect those distributions that consider kernels with binary firmware blobs to not be free software? Those distributions take the stance that the firmware must be loadable by userland, in which case the proposed patch removes this capability. Is there some downside to leaving this functionality in there? Are there known bugs or maintainability issues with the code as-is? Devin -- Devin J. Heitmueller http://www.devinheitmueller.com AIM: devinheitmueller -- 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/