Re: [PATCH 0/7] intel_audio_dump: add support for Valleyview

[Daniel Vetter <daniel@ffwll.ch>]

On Tue, Mar 04, 2014 at 11:17:33AM -0500, mengdong.lin@intel.com wrote:
> From: Mengdong Lin <mengdong.lin@intel.com>
> 
> Layout of display and audio registers can be same for different Intel GPUs,
> so we want maximum code sharing when adding support for VLV.
> 
> This patch set:
> - support using base address plus an offset to dump registers
> - share common audio dump code for Ironlake, Haswell and Broadwell
> - add support for Valleyview, by reusing Ironlake code.
> - adjust code algnment
> 
> The patches are reviewed by Xiang Haihao.
> 
> Now OEM Baytrail machines are sent to OSV like Ubuntu, so we need this for
> customer support.

When you push feature work like this please also update the NEWS file with
a short entry. We use that for the i-g-t release announcements. I've done
this for now.

Thanks, Daniel
> 
> Mengdong Lin (7):
>   intel_audio_dump: support using base address plus an offset to dump
>     registers
>   intel_audio_dump: move definitions of transcoder/pipe/port/converter
>     earlier
>   intel_audio_dump: define IS_HASWELL_PLUS to cover Haswell and its
>     successors
>   intel_audio_dump: define get_num_pipes() to get number of pipes for a
>     device ID
>   intel_audio_dump: share common audio dump code for Ironlake, Haswell &
>     Broadwell
>   intel_audio_dump: adjust code alignment
>   intel_audio_dump: add support for Valleyview
> 
>  tools/intel_audio_dump.c | 2011 ++++++++++++++++++++++++----------------------
>  1 file changed, 1041 insertions(+), 970 deletions(-)
> 
> -- 
> 1.8.1.2
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch