[Intel-gfx] [PATCH i-g-t] tools Add a intel_fw_dump tool

Damien Lespiau damien.lespiau at intel.com
Tue Jun 30 08:30:09 PDT 2015


On Tue, Jun 30, 2015 at 05:49:12PM +0300, Jani Nikula wrote:
> On Tue, 30 Jun 2015, Damien Lespiau <damien.lespiau at intel.com> wrote:
> > So we can inspect fw headers. Sample output:
> 
> Nitpick on the tool name, I'd like to make a distinction between tools
> that dump some blob from the driver/hardware/bios/whatever, and tools
> that decode those blobs. (See also the confusing naming with
> intel_bios_dumper vs. intel_bios_reader.)  I'd call this
> "intel_firmware_decode".

Sure, I can't say I care much, so glad someone does.

> Another nitpick is that I'd plan on using getopt from the beginning, and
> having a -f or --file option for the input.

Well, for that one, the file is the actual thing we act on, so I
wouldn't use an option, we always need it. Options can always come later
if needed.

-- 
Damien


More information about the Intel-gfx mailing list