All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Thomas Hellström" <thomas@shipmail.org>
To: Peter Zijlstra <peterz@infradead.org>
Cc: DRI <dri-devel@lists.sourceforge.net>,
	Linux Kernel list <linux-kernel@vger.kernel.org>
Subject: Re: DRM drivers with closed source user-space: WAS  [Patch 0/3] Resubmit VIA Chrome9 DRM via_chrome9 for upstream
Date: Mon, 20 Jul 2009 17:07:32 +0200	[thread overview]
Message-ID: <4A648834.8080203@shipmail.org> (raw)
In-Reply-To: <1248098773.15751.8908.camel@twins>

Peter Zijlstra wrote:
> On Mon, 2009-07-20 at 15:38 +0200, Thomas Hellström wrote:
>   
>> Politics:
>> It's true that sometimes some people don't like the code or what it 
>> does. But when this is the underlying cause of NAK-ing a driver I think 
>> it's very important that this is clearly stated, instead of inventing 
>> various random reasons that can easily be argued against. How should the 
>> driver writer otherwise get it right? Man-years might be spent fixing up 
>> drivers that will never get upstream anyway.
>>
>> I think it would help a lot of there was a documented set of driver 
>> features that were required and sufficient for a DRM driver to go 
>> upstream. It could look something like
>>
>>     * Kernel coding style obeyed. Passing checkpatch.
>>     
>
>       * fully functional GPL user-space driver.
>
> How can you argue that something as tailor made as a DRM interface can
> be used without it being a derived work?
>
> FWIW my full vote goes against allowing such thing to happen, and I
> think quite a lot of kernel people would agree with me.
>
> I would hope enough of of them would so that we can stop this from
> happening.
>
> Negative karma points to you for trying to chip away at the spirit of
>   

As stated before this was a suggestion to clarify the field for driver 
writers.

If the documented set of driver features required is fully open-source 
so be it. Just let people know.

/Thomas


> Linux.
>   




  parent reply	other threads:[~2009-07-20 15:07 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-20 13:38 DRM drivers with closed source user-space: WAS [Patch 0/3] Resubmit VIA Chrome9 DRM via_chrome9 for upstream Thomas Hellström
2009-07-20 13:58 ` Christoph Hellwig
2009-07-20 15:02   ` Thomas Hellström
2009-07-20 15:16     ` Alan Cox
2009-07-20 15:52       ` Matthew Garrett
2009-07-20 15:57         ` Christoph Hellwig
2009-07-20 16:02           ` Matthew Garrett
2009-07-20 16:37           ` Alan Cox
2009-07-20 23:28         ` Alan Cox
2009-07-20 23:33           ` Matthew Garrett
2009-07-20 19:13     ` Stephane Marchesin
2009-07-20 20:19       ` Thomas Hellström
2009-07-20 20:43         ` Dave Airlie
2009-07-20 21:40         ` Stephane Marchesin
2009-07-20 22:31           ` Dave Airlie
2009-07-20 14:06 ` Peter Zijlstra
2009-07-20 14:52   ` Alan Cox
2009-07-20 15:07   ` Thomas Hellström [this message]
2009-07-20 19:51   ` Dave Airlie
2009-07-20 14:09 ` Andrey Panin
2009-07-20 19:03   ` Krzysztof Halasa

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=4A648834.8080203@shipmail.org \
    --to=thomas@shipmail.org \
    --cc=dri-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=peterz@infradead.org \
    /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.