[Mesa-dev] [PATCH v3] *** Decode fields spanning across two DWords correctly ***

[Kenneth Graunke]

On Tuesday, September 20, 2016 3:59:27 PM PDT Sirisha Gandikota wrote:
> From: Sirisha Gandikota <Sirisha.Gandikota at intel.com>
> 
> The first version of aubinator did not take into account the fields
> spanning across 2 DWords. Hence fields like 64bit address/offset and
> int were not decoded correctly. This patch should fix that issue.
> 
> v2: Aptly renamed dw to qw in the method gen_field_iterator_next()
> and removed extra white space in the same method (Anuj)
> 
> v3: Change all instances of dw to qw (Anuj)
> 
> Tested on HSW, BDW, SKL aub files
> 
> 
> Sirisha Gandikota (1):
>   aubinator: Fix the decoding of values that span two Dwords
> 
>  src/intel/tools/decoder.c | 50 +++++++++++++++++++++++++++++++++++------------
>  1 file changed, 37 insertions(+), 13 deletions(-)

Hi Sirisha,

You don't need to send a cover letter (--compose) for single patches.
We generally only do that for patch series.

--Ken
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: This is a digitally signed message part.
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160920/dffc9385/attachment-0001.sig>