From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: Marcel Holtmann <marcel@holtmann.org>
Cc: David Woodhouse <dwmw2@infradead.org>,
Oliver Neukum <oliver@neukum.org>,
linux-kernel@vger.kernel.org, akpm@linux-foundation.org,
Michael Zappe <zapman@interlan.net>,
Stephane Alnet <stephane@u-picardie.fr>,
Brad Hards <bhards@bigpond.net.au>
Subject: Re: [PATCH] kaweth: use request_firmware()
Date: Sat, 31 May 2008 13:35:47 +0100 [thread overview]
Message-ID: <20080531133547.10b5d7f6@core> (raw)
In-Reply-To: <7D3DC891-CC9A-4CBD-8DCF-E684A2FB36B2@holtmann.org>
> I was not late to the party. I was the first one using
> request_firmware() and always keeping an eye on drivers using it and
> making sure using it correctly. However drivers sneaked in without
> that I realized it. Mainly because I don't have that hardware and
> reviewing every single driver obviously never happened.
Nothing in the documentation says that it can't have a heirarchical name,
so naturally lots of people did it. They didn't sneak in - they improved
on the interface and evolved it. They did it in a way that didn't upset
anything else (which is why you didn't notice).
No sneaking involved.
> I repeatedly said that no hierarchical namespace should be used (and
> this was a couple of years ago) and now since some developers misuse
> it, it makes it right.
It means by consensus the developer community decided that a heirarchical
namespace was useful. And that is the party you've arrived late for.
Alan
next prev parent reply other threads:[~2008-05-31 12:50 UTC|newest]
Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-25 10:23 [PATCHv2 00/28] Allow built-in firmware to be accessed by request_firmware() David Woodhouse
2008-05-25 10:23 ` [PATCHv2 01/28] libertas: treat firmware data as const David Woodhouse
2008-05-25 13:06 ` Alan Cox
2008-05-25 10:23 ` [PATCHv2 02/28] bluetooth: " David Woodhouse
2008-05-25 13:07 ` Alan Cox
2008-05-25 10:23 ` [PATCHv2 03/28] cyclades: " David Woodhouse
2008-05-25 13:07 ` Alan Cox
2008-05-25 10:23 ` [PATCHv2 04/28] cx25840: " David Woodhouse
2008-05-25 10:51 ` Hans Verkuil
2008-05-25 14:36 ` David Woodhouse
2008-05-25 14:56 ` David Woodhouse
2008-05-25 15:04 ` Hans Verkuil
2008-05-25 15:37 ` Mike Isely
2008-05-25 15:20 ` Mike Isely
2008-05-25 16:16 ` Tyler Trafford
2008-05-25 10:23 ` [PATCHv2 05/28] myri10ge: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 06/28] vx222: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 07/28] riptide: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 08/28] pcxhr: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 09/28] vx: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 10/28] ueagle-atm: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 11/28] cxacru: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 12/28] aic94xx: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 13/28] zd1201: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 14/28] rt2x00: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 15/28] p54: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 16/28] atmel: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 17/28] irda-usb: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 18/28] cxgb3: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 19/28] bt8xx: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 20/28] ttusb-dec: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 21/28] dvb frontends: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 22/28] cxusb: " David Woodhouse
2008-05-25 14:14 ` Michael Krufky
2008-05-25 14:27 ` David Woodhouse
2008-05-25 15:02 ` David Woodhouse
2008-05-25 10:23 ` [PATCHv2 23/28] gp8psk: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 24/28] tuners: " David Woodhouse
2008-05-25 10:23 ` [PATCHv2 25/28] firmware: make fw->data const David Woodhouse
2008-05-25 10:23 ` [PATCHv2 26/28] firmware: allow firmware files to be built into kernel image David Woodhouse
2008-05-30 5:08 ` Rusty Russell
2008-05-31 8:39 ` David Woodhouse
2008-05-25 10:23 ` [PATCHv2 27/28] firmware: Add CONFIG_BUILTIN_FIRMWARE option David Woodhouse
2008-05-25 17:00 ` [PATCHv3 " David Woodhouse
2008-05-26 8:58 ` David Woodhouse
2008-05-28 11:46 ` [PATCHv4 " David Woodhouse
2008-05-29 8:29 ` [PATCH 29/28] firmware: Add 'firmware_install' make target David Woodhouse
2008-05-25 10:23 ` [PATCHv2 28/28] firmware: convert korg1212 driver to use firmware loader exclusively David Woodhouse
2008-05-25 13:11 ` Alan Cox
2008-05-27 18:33 ` Yinghai Lu
2008-05-27 21:24 ` David Woodhouse
2008-05-29 9:09 ` [PATCHv2 #ERROR!/28] maestro3: treat firmware data as const David Woodhouse
2008-05-29 10:49 ` [PATCHv2 30/28] firmware: convert maestro3 driver to use firmware loader exclusively David Woodhouse
2008-05-29 12:21 ` [PATCH N/28] ymfpci: treat firmware data as const David Woodhouse
2008-05-29 12:23 ` [PATCH N/28] firmware: convert ymfpci driver to use firmware loader exclusively David Woodhouse
2008-05-29 13:47 ` [PATCH] smctr: use request_firmware() David Woodhouse
2008-05-29 19:30 ` maximilian attems
2008-05-29 21:02 ` David Woodhouse
2008-05-29 14:20 ` [PATCH] kaweth: " David Woodhouse
2008-05-30 13:55 ` Oliver Neukum
2008-05-30 15:16 ` David Woodhouse
2008-05-30 15:26 ` Marcel Holtmann
2008-05-30 15:48 ` Alan Cox
2008-05-30 21:02 ` Oliver Neukum
2008-05-31 0:00 ` Alexandre Oliva
2008-05-31 6:51 ` Marcel Holtmann
2008-05-31 12:35 ` Alan Cox [this message]
2008-05-29 17:49 ` [PATCH] ttusb-budget: " David Woodhouse
2008-06-13 19:23 ` Mauro Carvalho Chehab
2008-05-30 11:45 ` [PATCH] keyspan: " David Woodhouse
2008-05-30 13:54 ` Hugh Blemings
2008-05-30 12:19 ` [PATCH] keyspan_pda: " David Woodhouse
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=20080531133547.10b5d7f6@core \
--to=alan@lxorguk.ukuu.org.uk \
--cc=akpm@linux-foundation.org \
--cc=bhards@bigpond.net.au \
--cc=dwmw2@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=marcel@holtmann.org \
--cc=oliver@neukum.org \
--cc=stephane@u-picardie.fr \
--cc=zapman@interlan.net \
/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