netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Matthew Garrett <mjg59@srcf.ucam.org>
To: Paulius Zaleckas <paulius.zaleckas@gmail.com>
Cc: Elly Jones <ellyjones@google.com>,
	netdev@vger.kernel.org, dbrownell@users.sourceforge.net,
	jglasgow@google.com, msb@google.com, olofj@google.com
Subject: Re: [PATCH] Add Qualcomm Gobi 2000 driver.
Date: Fri, 1 Oct 2010 14:50:52 +0100	[thread overview]
Message-ID: <20101001135052.GA4209@srcf.ucam.org> (raw)
In-Reply-To: <4CA5E40D.4080507@gmail.com>

On Fri, Oct 01, 2010 at 04:37:17PM +0300, Paulius Zaleckas wrote:
> On 09/28/2010 08:10 PM, Elly Jones wrote:
>> From: Elizabeth Jones<ellyjones@google.com>
>>
>> This driver is a rewrite of the original Qualcomm GPL driver, released as part
>> of Qualcomm's "Code Aurora" initiative. The driver has been transformed into
>> Linux kernel style and made to use kernel APIs where appropriate; some bugs have
>> also been fixed. Note that the device in question requires firmware and a
>> firmware loader; the latter has been written by mjg (see
>> http://www.codon.org.uk/~mjg59/gobi_loader/).
>
> Why not use already existing in kernel firmware upload API?

Because chosing the correct firmware requires making a runtime policy 
decision, the firmware is of variable size (anywhere from 12MB to 16MB) 
and it needs to be reencoded with something resembling PPP framing 
before it gets dumped to the device. It's certainly possible to do it 
in-kernel, but it'd be rather a lot more code and allocating that much 
unswappable space sounds like something that would make some usecases 
impossible.

-- 
Matthew Garrett | mjg59@srcf.ucam.org

      reply	other threads:[~2010-10-01 13:51 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-28 17:10 [PATCH] Add Qualcomm Gobi 2000 driver Elly Jones
2010-09-28 17:16 ` Ben Hutchings
2010-09-28 18:11 ` Joe Perches
2010-09-28 21:23 ` Dan Williams
2010-10-04 15:03   ` Elly Jones
2010-09-29  2:39 ` [PATCH 0/2] qcusbnet: Cleanups Joe Perches
2010-09-29  2:39   ` [PATCH 1/2] drivers/net/usb/qcusbnet: Checkpatch cleanups Joe Perches
2010-09-29  3:09     ` [PATCH 3] drivers/net/usb/qcusbnet: Spelling and extern int debug cleanups Joe Perches
2010-10-01 13:26     ` [PATCH 1/2] drivers/net/usb/qcusbnet: Checkpatch cleanups Paulius Zaleckas
2010-10-01 13:41       ` Joe Perches
2010-10-01 13:54         ` Paulius Zaleckas
2010-09-29  2:39   ` [PATCH 2/2] drivers/net/usb/qcusbnet/qmidevice: Remove printhex, use print_hex_dump Joe Perches
2010-10-01  0:33   ` [PATCH 0/2] qcusbnet: Cleanups David Miller
2010-10-01  0:45     ` Joe Perches
2010-10-01 14:42       ` Elly Jones
2010-10-01 15:56         ` Joe Perches
2010-10-01 18:29         ` David Brownell
2010-10-01 13:37 ` [PATCH] Add Qualcomm Gobi 2000 driver Paulius Zaleckas
2010-10-01 13:50   ` Matthew Garrett [this message]

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=20101001135052.GA4209@srcf.ucam.org \
    --to=mjg59@srcf.ucam.org \
    --cc=dbrownell@users.sourceforge.net \
    --cc=ellyjones@google.com \
    --cc=jglasgow@google.com \
    --cc=msb@google.com \
    --cc=netdev@vger.kernel.org \
    --cc=olofj@google.com \
    --cc=paulius.zaleckas@gmail.com \
    /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;
as well as URLs for NNTP newsgroup(s).