public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <gregkh@suse.de>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: linux-kernel@vger.kernel.org
Subject: Re: qcserial comes up in unhelpful custom mode?
Date: Fri, 3 Apr 2009 08:57:04 -0700	[thread overview]
Message-ID: <20090403155704.GA26189@suse.de> (raw)
In-Reply-To: <20090403151232.GA21657@srcf.ucam.org>

On Fri, Apr 03, 2009 at 04:12:32PM +0100, Matthew Garrett wrote:
> On Fri, Apr 03, 2009 at 08:03:01AM -0700, Greg KH wrote:
> > On Fri, Apr 03, 2009 at 02:57:19PM +0100, Matthew Garrett wrote:
> > > So it's a driver that's useless without a closed source firmware loader, 
> > > either in the form of a Linux userspace application that can't be 
> > > distributed or in the form of Windows?
> > 
> > Hm, I originally thought that as well, but I have a laptop here that
> > doesn't seem to need the firmware code at all (I deleted it and rebooted
> > from poweroff and everything worked fine.)
> > 
> > So it seems it depends on the hardware platform.
> 
> Hmm. What's the USB ID of that one?

USB_DEVICE(0x05c6, 0x9211);

> > > I thought we'd decided that drivers weren't going to be merged in that
> > > case (poulsbow, the original 3945 driver)
> > 
> > We have lots of drivers that rely on firmware that is not in the kernel
> > to work properly, I see this as just the same thing.
> 
> It's not that it relies on firmware - I can get my hands on the firmware 
> easily enough. The issue is that I have no way of loading it without 
> using non-free code, which I think puts it in the same category as 
> Poulsbo and ipw3945 rather than, say, b43.

No, both of those are quite different, they interact with the closed
source bits directly from the kernel, which isn't allowed.  This just
needs the firmware loaded into the device to work at all, much like an
"empty" b43 device would be.

> > And again, I have been assured that the code is going to be distributed
> > soon, and that others are already using the driver successfully.
> 
> Everything I've been able to find suggests that the only people using it 
> are booting Windows first, so your one seems to be the oddity. On the 
> other hand, I'm kind of surprised that we're merging drivers on the 
> promise that code to use them will be available "soon". It seems to end 
> up giving the vendor credit that they don't deserve as yet.

As I did not realize the hardware was shipping yet (my machine is a
pre-production unit), I thought the firmware issue would be resolved by
the time shipments happened.

We want manufacturers to ship early and often, this was just doing that.

Oh, and it was not giving the vendor any credit at all, their original
driver was one of the worse things I had seen in a long time (200 or so
lines, over 300 checkpatch errors, a new record.)  I rewrote it from
scratch because of the mess involved.

thanks,

greg k-h

  reply	other threads:[~2009-04-03 16:15 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-01 20:37 qcserial comes up in unhelpful custom mode? Matthew Garrett
2009-04-03  2:56 ` Greg KH
2009-04-03 13:57   ` Matthew Garrett
2009-04-03 15:03     ` Greg KH
2009-04-03 15:12       ` Matthew Garrett
2009-04-03 15:57         ` Greg KH [this message]
2009-04-03 16:06           ` Matthew Garrett
2009-04-07 22:41             ` Greg KH
2009-04-03 15:12       ` Greg KH
2009-04-03 15:15         ` Matthew Garrett
2009-04-03 16:45         ` Matthew Garrett
2009-04-04 15:27           ` qcserial/gobi firmware loader Matthew Garrett
2009-04-04 16:24             ` [PATCH] qcserial: Add extra device IDs Matthew Garrett
2009-04-04 17:51             ` qcserial/gobi firmware loader Kay Sievers
2009-04-04 17:54               ` Matthew Garrett
2009-04-04 18:14             ` Alexander Shumakovitch
2009-04-04 18:37               ` Matthew Garrett

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=20090403155704.GA26189@suse.de \
    --to=gregkh@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjg59@srcf.ucam.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox