Subject: Re: [linux-dvb-maintainer] Re: [PATCH 26/37] dvb: add support for
	plls used by nxt200x
From: Andreas Oberritter <obi@linuxtv.org>
To: Manu Abraham <manu@linuxtv.org>
Cc: linux-kernel@vger.kernel.org, Mike Krufky <mkrufky@linuxtv.org>,
       linux-dvb-maintainer@linuxtv.org, Andrew Morton <akpm@osdl.org>
In-Reply-To: <436AA148.8090705@linuxtv.org>
References: <4367241A.1060300@m1k.net>
	 <20051103135910.3bf893d9.akpm@osdl.org> <436A96A8.4080906@linuxtv.org>
	 <436AA148.8090705@linuxtv.org>
Content-Type: text/plain
Date: Fri, 04 Nov 2005 01:51:08 +0100
Message-Id: <1131065468.9376.23.camel@ip6-localhost>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1387
Lines: 31

On Fri, 2005-11-04 at 03:46 +0400, Manu Abraham wrote:
> We have in the DVB subsystem most of the exported symbols as 
> EXPORT_SYMBOL itself, rather than EXPORT_SYMBOL_GPL. I think if this 
> needs to be changed, we would require a global change of all symbols to 
> the same to maintain consistency. If you require that change we can have 
> a change but i would think that the discussions be done with the 
> relevant copyright holders too, eventhough probably most of the authors 
> won't have any objection.

I don't know if I ever contributed code to the DVB subsystem which is
actually exported, but in case I did, then I am against changing the
affected EXPORT_SYMBOLs.

This would make it impossible to the use source code of most hardware
vendors for embedded products because they usually have different
licenses for their "run-on-every-embedded-platform-and-even-on-windows"
drivers.

Also I remember people telling on lkml that EXPORT_SYMBOL_GPL was used
for new kernel internal code only and I can't see how this applies to
dvb-pll or any other part of the dvb subsystem which grew up outside the
kernel tree.

Regards,
Andreas

-
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/