Re: [PATCH 00/18] firmware: moving drivers to request_firmware()

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: David Woodhouse
Date: Saturday, May 31, 2008 - 2:22 pm

On Thu, 2008-06-05 at 17:01 -0400, Jeff Garzik wrote:

We don't have them in that form right now. Of the firmware blobs I've
encountered so far -- even the ones which were in a file on their own --
none of them are in binary form; they're _all_ in some ASCII
representation which can be processed with 'diff'. That includes char
arrays, arrays of larger integers which need endian-awareness, 'hex
record' structures, and probably a bunch of other abominations I have
yet to encounter as I work through them.

None of them have just been binary files in the source tree.

I do not believe that taking the existing ascii-source representation
and turning it into binary blobs in the source tree is something which
will be unanimously welcomed. But if you prove me wrong, it's trivial to
switch to doing it that way. For example:
	objcopy -Iihex -Obinary firmware/tr_smctr.bin{.ihex,}
	git-rm firmware/smctr.bin.ihex
	git-add firmware/smctr.bin
	git-commit

I'm going to be making a 'shadow' tree containing the result of running
'make firmware_install', just as I have a tree for exported headers.
That tree _can_ have the raw binaries¹. But I don't think it's
appropriate while the firmware is still in the kernel source tree.

-- 
dwmw2

¹ And I plan to make another tree which pulls from that but also includes
  'distributable' firmware blobs, which the owners wouldn't put into the
  kernel tree because of the GPL requirement that would imply.

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

Messages in current thread:
[PATCH 02/18] firmware: Add CONFIG_BUILTIN_FIRMWARE option, David Woodhouse, (Fri May 23, 5:58 am)
[PATCH 03/18] firmware: Add 'firmware_install' make target, David Woodhouse, (Thu May 29, 1:01 am)
[PATCH 07/18] smctr: use request_firmware(), David Woodhouse, (Thu May 29, 6:39 am)
[PATCH 08/18] kaweth: use request_firmware(), David Woodhouse, (Thu May 29, 7:17 am)
[PATCH 09/18] ttusb-budget: use request_firmware(), David Woodhouse, (Thu May 29, 9:50 am)
[PATCH 10/18] ihex.h: binary representation of ihex records, David Woodhouse, (Fri May 30, 3:57 am)
[PATCH 14/18] keyspan_pda: use request_firmware(), David Woodhouse, (Fri May 30, 5:15 am)
[PATCH 17/18] ti_usb_3410_5052: use request_firmware(), David Woodhouse, (Fri May 30, 8:49 am)
[PATCH 18/18] whiteheat: use request_firmware(), David Woodhouse, (Fri May 30, 3:35 pm)
Re: [PATCH 00/18] firmware: moving drivers to request_firm ..., David Woodhouse, (Sat May 31, 2:22 pm)
Re: [PATCH 00/18] firmware: moving drivers to request_firm ..., Arkadiusz Miskiewicz, (Thu Jun 5, 3:07 am)