All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Norbert Preining <preining@logic.at>
Cc: "Nemosoft Unv." <nemosoft@smcc.demon.nl>,
	Linux USB Mailing List  <linux-usb-devel@lists.sourceforge.net>,
	linux-kernel@vger.kernel.org
Subject: Re: [linux-usb-devel] Re: kernel 2.6.8 pwc patches and counterpatches
Date: Tue, 24 Aug 2004 22:52:08 -0700	[thread overview]
Message-ID: <20040825055208.GB16147@kroah.com> (raw)
In-Reply-To: <20040825053516.GA19771@gamma.logic.tuwien.ac.at>

On Wed, Aug 25, 2004 at 07:35:16AM +0200, Norbert Preining wrote:
> 
> Bummer. This is rubbish. And I am sure that this is not the intention of
> Linus comments. IF the module would be NON functional without the closed
> plugin, then yes, rip it out. But it is useable, and this hook *can* be
> used for closed modules, but also for other modules. 

But it isn't, that's the point.

> It really looks like personal stuff going on here, not really objective
> discussion on this point.

Nothing personal here.  The GPL code has a exported symbol explicitly to
be able to load a closed source decoder module.  Because of that, I've
deleted that symbol (and the surrounding logic, as it's no longer
needed.)  One could also argue that no in-kernel code needs that symbol
exported, so it should be removed for that reason alone.

That's all I've done.  The in-kernel module still works the same as it
always did if you never used the closed source decoder.

thanks,

greg k-h

  reply	other threads:[~2004-08-25  6:10 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1092793392.17286.75.camel@localhost>
     [not found] ` <1092845135.8044.22.camel@localhost>
     [not found]   ` <20040823221028.GB4694@kroah.com>
2004-08-24 22:58     ` kernel 2.6.8 pwc patches and counterpatches Nemosoft Unv.
2004-08-24 23:04       ` Greg KH
2004-08-25  5:35         ` [linux-usb-devel] " Norbert Preining
2004-08-25  5:52           ` Greg KH [this message]
     [not found]         ` <200408260058.59490@smcc.demon.nl>
2004-08-26  7:18           ` Greg KH
2004-08-25 14:02       ` Simon Oosthoek
2004-08-26  9:22         ` [linux-usb-devel] " Tuukka Toivonen
2004-08-26 11:17           ` Oliver Neukum
2004-08-26 12:07             ` Tuukka Toivonen
2004-08-26  0:55       ` Rob van Nieuwkerk
2004-08-26  1:27         ` Rob van Nieuwkerk
2004-08-26  9:00       ` syrius.ml

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20040825055208.GB16147@kroah.com \
    --to=greg@kroah.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb-devel@lists.sourceforge.net \
    --cc=nemosoft@smcc.demon.nl \
    --cc=preining@logic.at \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.