[Intel-gfx] [PATCH 1/5] drm/i915: Store and print dmc firmware version

[Mika Kuoppala]

Damien Lespiau <damien.lespiau at intel.com> writes:

> On Thu, Oct 08, 2015 at 12:03:30PM +0100, Damien Lespiau wrote:
>> The DMC firmware version decoding was different in my patch and I'm sure
>> it worked then. Maye the header has changed :(
>
> By the way, if this is indeed the case, could you fix
> intel_firmware_decode as well?
>
>   http://cgit.freedesktop.org/xorg/app/intel-gpu-tools/tree/tools/intel_firmware_decode.c
>

Well this seems to be somewhat convoluted. The CSS header has a version
and then each binary package has their own. The fw version in binary
package seems to code only the minor part like in my patches.

I went through various firmware images and these seems to be in sync
currently so that version in CSS follows the version in dmc header,
albeit with different encoding.

There is not much info on bspec but I guess we should just
use the version provided in CSS header.

-Mika

> Thanks,
>
> -- 
> Damien