Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Marcel Holtmann <marcel@...>
Cc: David Woodhouse <dwmw2@...>, Sam Ravnborg <sam@...>, <linux-kernel@...>, <aoliva@...>, <alan@...>, Abhay Salunke <Abhay_Salunke@...>, <kay.sievers@...>, Takashi Iwai <tiwai@...>, Michael Buesch <mb@...>
Date: Sunday, May 25, 2008 - 9:40 am

> > Why are you so fixated on special-casing the single character '/'?

Why are you thinking of it as namespaces then? It isn't. The "firmware
key" lives in a flat namespace where each key is an arbitrary
nul-terminated string.


No, it doesn't.


There's no need to think of it that way. Look at a uevent now:

UEVENT[1211722721.323011] add      /devices/pci0001:10/0001:10:12.0/ssb0:0/=
firmware/ssb0:0 (firmware)
ACTION=3Dadd
DEVPATH=3D/devices/pci0001:10/0001:10:12.0/ssb0:0/firmware/ssb0:0
SUBSYSTEM=3Dfirmware
FIRMWARE=3Db43/b0g0bsinitvals5.fw
TIMEOUT=3D60
SEQNUM=3D1376


The "firmware key" is contained in the FIRMWARE environment variable. If
you want to allow loading multiple firmwares at the same time, you
wouldn't have to make the key part of the device name, you would only
have to add a unique ID to the firmware device name, say

=EF=BB=BF
UEVENT[1211722721.323011] add      /devices/pci0001:10/0001:10:12.0/ssb0:0/=
firmware/ssb0:0-1234 (firmware)
ACTION=3Dadd
DEVPATH=3D/devices/pci0001:10/0001:10:12.0/ssb0:0/firmware/ssb0:0-1234
SUBSYSTEM=3Dfirmware
FIRMWARE=3Db43/b0g0bsinitvals5.fw
TIMEOUT=3D60
SEQNUM=3D1376

johannes
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH 1/3] firmware: allow firmware files to be built i..., Lennart Sorensen, (Fri May 23, 12:38 pm)
[PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Fri May 23, 9:46 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sat May 24, 10:46 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sat May 24, 11:25 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sat May 24, 11:34 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sat May 24, 2:18 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 7:49 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Alexandre Oliva, (Sun May 25, 1:17 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:49 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Alexandre Oliva, (Sun May 25, 11:30 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Alexandre Oliva, (Sun May 25, 11:13 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Alexandre Oliva, (Mon May 26, 1:09 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 3:01 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 9:19 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:15 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:34 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:23 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:46 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 3:03 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sun May 25, 10:18 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 9:12 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Johannes Berg, (Sun May 25, 9:40 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:12 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:28 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 7:54 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sun May 25, 9:46 am)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sun May 25, 2:07 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Sat May 24, 3:23 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, Marcel Holtmann, (Sat May 24, 3:31 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Fri May 23, 2:07 pm)
Re: [PATCH 2/3] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Fri May 23, 1:13 pm)