Re: [RFC] [PATCH] quick_dump: A dump utility different than reg_dumper

[Daniel Vetter <daniel@ffwll.ch>]

On Fri, Sep 21, 2012 at 12:46:05PM -0700, Ben Widawsky wrote:
> intel_reg_dumper is cool. However, doing bit decoding and dealing with
> moving registers from generation to generation (and in some cases just
> steppings) becomes a bit of a burden over time. It's also not completely
> necessary when doing the not-so-uncommon diff against some other BIOS
> debug.
> 
> While I do not want this tool to replace intel_reg_dumper, I think it
> has it's uses. The target is particularly for power-on like situations
> where we want to easily add profiles with new registers, and not lose
> the old information. Doing this in the existing reg dumper can be a bit
> tedious and sometimes error prone. It also is a nice place to add
> registers which may or may not actually be useful. Ones which prove
> useful over time should get moved to reg dumper.
> 
> The tool provides a fast way to add new registers, and register profiles
> as we bring up new platforms, or try to share debug information. This is
> accomplished by defining python tuples in text files, and adding them to
> profiles, which are just a list of the files containing tuples.
> 
> I defined a few "base" files which is essentially the previous
> reg_dumper registers + some other interesting ones. These are
> automatically used unless the user supplies the "baseless" option. The
> base should be pretty decent for all recent generation chips with the
> exception of valleyview. On top of this, I provided some starter
> profiles: ivybridge, sandybridge, and valleyview.
> 
> NOTE: the valleyview profile is currently a work in progress.
> 
> So to get a quick sandybridge dump for instance:
> sudo ./quick_dump.py sandybridge
> 
> Feedback please.

I've wasted a few brain-cycles on the dump problem recently, too - pimping
intel_reg_dumper is rather obviously a pain. Two ideas:

- create a script that slurps in internal Bspec xml. The issue here is
  that I don't know how to handle regs with read-sideeffects (yep, we have
  those). Maybe manual blacklist. The upside is that this would hopefully
  allow auto-decoding of the bitfields (if we pimp the xml slurper
  sufficiently). Also, if we can get at the xml for the open-source Bspec,
  we could use this tool to auto-create a per-gen open-source reg slurper.
  Generally I fear that this approach is too messy and noisy and hence
  intel_reg_dumper pimping probably better.

- Another idea is to just dump entire register blocks for functional
  units with a header per block (e.g. "PLANE_A") and then offset : value
  hexvalue paris per line. That might be useful for quick&dirty comparison
  for bringup.

And a quick comment on your approach here: I'm not too sure whether the
file-base register block approach scales, respectively why exactly this is
better than frobbing the reg_dumper tool. Since that one has the concept
of register blocks already, too.

Cheers, Daniel

> 
> CC: Jesse Barnes <jbarnes@virtuousgeek.org>
> Signed-off-by: Ben Widawsky <ben@bwidawsk.net>
> ---
>  tools/quick_dump/base_display.txt   | 197 ++++++++++++++++++++++++++++++++++++
>  tools/quick_dump/base_interrupt.txt |  20 ++++
>  tools/quick_dump/base_other.txt     |   6 ++
>  tools/quick_dump/base_power.txt     |  21 ++++
>  tools/quick_dump/base_rings.txt     |  27 +++++
>  tools/quick_dump/gen6_other.txt     |   1 +
>  tools/quick_dump/gen7_other.txt     |   1 +
>  tools/quick_dump/ivybridge          |   1 +
>  tools/quick_dump/quick_dump.py      |  34 +++++++
>  tools/quick_dump/sandybridge        |   1 +
>  tools/quick_dump/valleyview         |   6 ++
>  tools/quick_dump/vlv_display.txt    | 197 ++++++++++++++++++++++++++++++++++++
>  12 files changed, 512 insertions(+)
>  create mode 100644 tools/quick_dump/base_display.txt
>  create mode 100644 tools/quick_dump/base_interrupt.txt
>  create mode 100644 tools/quick_dump/base_other.txt
>  create mode 100644 tools/quick_dump/base_power.txt
>  create mode 100644 tools/quick_dump/base_rings.txt
>  create mode 100644 tools/quick_dump/gen6_other.txt
>  create mode 100644 tools/quick_dump/gen7_other.txt
>  create mode 100644 tools/quick_dump/ivybridge
>  create mode 100755 tools/quick_dump/quick_dump.py
>  create mode 100644 tools/quick_dump/sandybridge
>  create mode 100644 tools/quick_dump/valleyview
>  create mode 100644 tools/quick_dump/vlv_display.txt
> 
> diff --git a/tools/quick_dump/base_display.txt b/tools/quick_dump/base_display.txt
> new file mode 100644
> index 0000000..8bead7f
> --- /dev/null
> +++ b/tools/quick_dump/base_display.txt
> @@ -0,0 +1,197 @@
> +('CPU_VGACNTRL', '0x00041000', '')
> +('PORT_DBG', '0x00042308', '')
> +('DIGITAL_PORT_HOTPLUG_CNTRL', '0x00044030', '')
> +('FDI_PLL_BIOS_0', '0x00046000', '')
> +('FDI_PLL_BIOS_1', '0x00046004', '')
> +('FDI_PLL_BIOS_2', '0x00046008', '')
> +('DISPLAY_PORT_PLL_BIOS_0', '0x0004600c', '')
> +('DISPLAY_PORT_PLL_BIOS_1', '0x00046010', '')
> +('DISPLAY_PORT_PLL_BIOS_2', '0x00046014', '')
> +('FDI_PLL_FREQ_CTL', '0x00046030', '')
> +('BLC_PWM_CPU_CTL2', '0x00048250', '')
> +('BLC_PWM_CPU_CTL', '0x00048254', '')
> +('HTOTAL_A', '0x00060000', '')
> +('HBLANK_A', '0x00060004', '')
> +('HSYNC_A', '0x00060008', '')
> +('VTOTAL_A', '0x0006000c', '')
> +('VBLANK_A', '0x00060010', '')
> +('VSYNC_A', '0x00060014', '')
> +('PIPEASRC', '0x0006001c', '')
> +('VSYNCSHIFT_A', '0x00060028', '')
> +('PIPEA_DATA_M1', '0x00060030', '')
> +('PIPEA_DATA_N1', '0x00060034', '')
> +('PIPEA_DATA_M2', '0x00060038', '')
> +('PIPEA_DATA_N2', '0x0006003c', '')
> +('PIPEA_LINK_M1', '0x00060040', '')
> +('PIPEA_LINK_N1', '0x00060044', '')
> +('PIPEA_LINK_M2', '0x00060048', '')
> +('PIPEA_LINK_N2', '0x0006004c', '')
> +('FDI_TXA_CTL', '0x00060100', '')
> +('HTOTAL_B', '0x00061000', '')
> +('HBLANK_B', '0x00061004', '')
> +('HSYNC_B', '0x00061008', '')
> +('VTOTAL_B', '0x0006100c', '')
> +('VBLANK_B', '0x00061010', '')
> +('VSYNC_B', '0x00061014', '')
> +('PIPEBSRC', '0x0006101c', '')
> +('VSYNCSHIFT_B', '0x00061028', '')
> +('PIPEB_DATA_M1', '0x00061030', '')
> +('PIPEB_DATA_N1', '0x00061034', '')
> +('PIPEB_DATA_M2', '0x00061038', '')
> +('PIPEB_DATA_N2', '0x0006103c', '')
> +('PIPEB_LINK_M1', '0x00061040', '')
> +('PIPEB_LINK_N1', '0x00061044', '')
> +('PIPEB_LINK_M2', '0x00061048', '')
> +('PIPEB_LINK_N2', '0x0006104c', '')
> +('FDI_TXB_CTL', '0x00061100', '')
> +('HTOTAL_C', '0x00062000', '')
> +('HBLANK_C', '0x00062004', '')
> +('HSYNC_C', '0x00062008', '')
> +('VTOTAL_C', '0x0006200c', '')
> +('VBLANK_C', '0x00062010', '')
> +('VSYNC_C', '0x00062014', '')
> +('PIPECSRC', '0x0006201c', '')
> +('VSYNCSHIFT_C', '0x00062028', '')
> +('PIPEC_DATA_M1', '0x00062030', '')
> +('PIPEC_DATA_N1', '0x00062034', '')
> +('PIPEC_DATA_M2', '0x00062038', '')
> +('PIPEC_DATA_N2', '0x0006203c', '')
> +('PIPEC_LINK_M1', '0x00062040', '')
> +('PIPEC_LINK_N1', '0x00062044', '')
> +('PIPEC_LINK_M2', '0x00062048', '')
> +('PIPEC_LINK_N2', '0x0006204c', '')
> +('FDI_TXC_CTL', '0x00062100', '')
> +('CPU_eDP_A', '0x00064000', '')
> +('PFA_WIN_POS', '0x00068070', '')
> +('PFA_WIN_SIZE', '0x00068074', '')
> +('PFA_CTL_1', '0x00068080', '')
> +('PFA_CTL_2', '0x00068084', '')
> +('PFA_CTL_3', '0x00068088', '')
> +('PFA_CTL_4', '0x00068090', '')
> +('PFB_WIN_POS', '0x00068870', '')
> +('PFB_WIN_SIZE', '0x00068874', '')
> +('PFB_CTL_1', '0x00068880', '')
> +('PFB_CTL_2', '0x00068884', '')
> +('PFB_CTL_3', '0x00068888', '')
> +('PFB_CTL_4', '0x00068890', '')
> +('PFC_WIN_POS', '0x00069070', '')
> +('PFC_WIN_SIZE', '0x00069074', '')
> +('PFC_CTL_1', '0x00069080', '')
> +('PFC_CTL_2', '0x00069084', '')
> +('PFC_CTL_3', '0x00069088', '')
> +('PFC_CTL_4', '0x00069090', '')
> +('PIPEACONF', '0x00070008', '')
> +('DSPACNTR', '0x00070180', '')
> +('DSPABASE', '0x00070184', '')
> +('DSPASTRIDE', '0x00070188', '')
> +('DSPASURF', '0x0007019c', '')
> +('DSPATILEOFF', '0x000701a4', '')
> +('PIPEBCONF', '0x00071008', '')
> +('DSPBCNTR', '0x00071180', '')
> +('DSPBBASE', '0x00071184', '')
> +('DSPBSTRIDE', '0x00071188', '')
> +('DSPBSURF', '0x0007119c', '')
> +('DSPBTILEOFF', '0x000711a4', '')
> +('PIPECCONF', '0x00072008', '')
> +('DSPCCNTR', '0x00072180', '')
> +('DSPCBASE', '0x00072184', '')
> +('DSPCSTRIDE', '0x00072188', '')
> +('DSPCSURF', '0x0007219c', '')
> +('DSPCTILEOFF', '0x000721a4', '')
> +('PCH_DPLL_A', '0x000c6014', '')
> +('PCH_DPLL_B', '0x000c6018', '')
> +('PCH_FPA0', '0x000c6040', '')
> +('PCH_FPA1', '0x000c6044', '')
> +('PCH_FPB0', '0x000c6048', '')
> +('PCH_FPB1', '0x000c604c', '')
> +('PCH_DREF_CONTROL', '0x000c6200', '')
> +('PCH_RAWCLK_FREQ', '0x000c6204', '')
> +('PCH_DPLL_TMR_CFG', '0x000c6208', '')
> +('PCH_SSC4_PARMS', '0x000c6210', '')
> +('PCH_SSC4_AUX_PARMS', '0x000c6214', '')
> +('PCH_DPLL_ANALOG_CTL', '0x000c6300', '')
> +('PCH_DPLL_SEL', '0x000c7000', '')
> +('PCH_PP_STATUS', '0x000c7200', '')
> +('PCH_PP_CONTROL', '0x000c7204', '')
> +('PCH_PP_ON_DELAYS', '0x000c7208', '')
> +('PCH_PP_OFF_DELAYS', '0x000c720c', '')
> +('PCH_PP_DIVISOR', '0x000c7210', '')
> +('BLC_PWM_PCH_CTL1', '0x000c8250', '')
> +('BLC_PWM_PCH_CTL2', '0x000c8254', '')
> +('TRANS_HTOTAL_A', '0x000e0000', '')
> +('TRANS_HBLANK_A', '0x000e0004', '')
> +('TRANS_HSYNC_A', '0x000e0008', '')
> +('TRANS_VTOTAL_A', '0x000e000c', '')
> +('TRANS_VBLANK_A', '0x000e0010', '')
> +('TRANS_VSYNC_A', '0x000e0014', '')
> +('TRANS_VSYNCSHIFT_A', '0x000e0028', '')
> +('TRANSA_DATA_M1', '0x000e0030', '')
> +('TRANSA_DATA_N1', '0x000e0034', '')
> +('TRANSA_DATA_M2', '0x000e0038', '')
> +('TRANSA_DATA_N2', '0x000e003c', '')
> +('TRANSA_DP_LINK_M1', '0x000e0040', '')
> +('TRANSA_DP_LINK_N1', '0x000e0044', '')
> +('TRANSA_DP_LINK_M2', '0x000e0048', '')
> +('TRANSA_DP_LINK_N2', '0x000e004c', '')
> +('TRANS_DP_CTL_A', '0x000e0300', '')
> +('TRANS_HTOTAL_B', '0x000e1000', '')
> +('TRANS_HBLANK_B', '0x000e1004', '')
> +('TRANS_HSYNC_B', '0x000e1008', '')
> +('TRANS_VTOTAL_B', '0x000e100c', '')
> +('TRANS_VBLANK_B', '0x000e1010', '')
> +('TRANS_VSYNC_B', '0x000e1014', '')
> +('TRANS_VSYNCSHIFT_B', '0x000e1028', '')
> +('TRANSB_DATA_M1', '0x000e1030', '')
> +('TRANSB_DATA_N1', '0x000e1034', '')
> +('TRANSB_DATA_M2', '0x000e1038', '')
> +('TRANSB_DATA_N2', '0x000e103c', '')
> +('TRANSB_DP_LINK_M1', '0x000e1040', '')
> +('TRANSB_DP_LINK_N1', '0x000e1044', '')
> +('TRANSB_DP_LINK_M2', '0x000e1048', '')
> +('TRANSB_DP_LINK_N2', '0x000e104c', '')
> +('PCH_ADPA', '0x000e1100', '')
> +('HDMIB', '0x000e1140', '')
> +('HDMIC', '0x000e1150', '')
> +('HDMID', '0x000e1160', '')
> +('PCH_LVDS', '0x000e1180', '')
> +('TRANS_DP_CTL_B', '0x000e1300', '')
> +('TRANS_HTOTAL_C', '0x000e2000', '')
> +('TRANS_HBLANK_C', '0x000e2004', '')
> +('TRANS_HSYNC_C', '0x000e2008', '')
> +('TRANS_VTOTAL_C', '0x000e200c', '')
> +('TRANS_VBLANK_C', '0x000e2010', '')
> +('TRANS_VSYNC_C', '0x000e2014', '')
> +('TRANS_VSYNCSHIFT_C', '0x000e2028', '')
> +('TRANSC_DATA_M1', '0x000e2030', '')
> +('TRANSC_DATA_N1', '0x000e2034', '')
> +('TRANSC_DATA_M2', '0x000e2038', '')
> +('TRANSC_DATA_N2', '0x000e203c', '')
> +('TRANSC_DP_LINK_M1', '0x000e2040', '')
> +('TRANSC_DP_LINK_N1', '0x000e2044', '')
> +('TRANSC_DP_LINK_M2', '0x000e2048', '')
> +('TRANSC_DP_LINK_N2', '0x000e204c', '')
> +('TRANS_DP_CTL_C', '0x000e2300', '')
> +('PCH_DP_B', '0x000e4100', '')
> +('PCH_DP_C', '0x000e4200', '')
> +('PCH_DP_D', '0x000e4300', '')
> +('TRANSACONF', '0x000f0008', '')
> +('FDI_RXA_CTL', '0x000f000c', '')
> +('FDI_RXA_MISC', '0x000f0010', '')
> +('FDI_RXA_IIR', '0x000f0014', '')
> +('FDI_RXA_IMR', '0x000f0018', '')
> +('FDI_RXA_TUSIZE1', '0x000f0030', '')
> +('FDI_RXA_TUSIZE2', '0x000f0038', '')
> +('TRANSBCONF', '0x000f1008', '')
> +('FDI_RXB_CTL', '0x000f100c', '')
> +('FDI_RXB_MISC', '0x000f1010', '')
> +('FDI_RXB_IIR', '0x000f1014', '')
> +('FDI_RXB_IMR', '0x000f1018', '')
> +('FDI_RXB_TUSIZE1', '0x000f1030', '')
> +('FDI_RXB_TUSIZE2', '0x000f1038', '')
> +('TRANSCCONF', '0x000f2008', '')
> +('FDI_RXC_CTL', '0x000f200c', '')
> +('FDI_RXC_MISC', '0x000f2010', '')
> +('FDI_RXC_TUSIZE1', '0x000f2030', '')
> +('FDI_RXC_TUSIZE2', '0x000f2038', '')
> +('FDI_PLL_CTL_1', '0x000fe000', '')
> +('FDI_PLL_CTL_2', '0x000fe004', '')
> diff --git a/tools/quick_dump/base_interrupt.txt b/tools/quick_dump/base_interrupt.txt
> new file mode 100644
> index 0000000..df4244b
> --- /dev/null
> +++ b/tools/quick_dump/base_interrupt.txt
> @@ -0,0 +1,20 @@
> +('GEN6_PMINTRMSK', '0x0000a168', '')
> +('DEISR', '0x00044000', '')
> +('DEIMR', '0x00044004', '')
> +('DEIIR', '0x00044008', '')
> +('DEIER', '0x0004400c', '')
> +('GTISR', '0x00044010', '')
> +('GTIMR', '0x00044014', '')
> +('GTIIR', '0x00044018', '')
> +('GTIER', '0x0004401c', '')
> +('GEN6_PMISR', '0x00044020', '')
> +('GEN6_PMIMR', '0x00044024', '')
> +('GEN6_PMIIR', '0x00044028', '')
> +('GEN6_PMIER', '0x0004402c', '')
> +('SDEISR', '0x000c4000', '')
> +('SDEIMR', '0x000c4004', '')
> +('SDEIIR', '0x000c4008', '')
> +('SDEIER', '0x000c400c', '')
> +('RENDER_IMR', '0x000020a8', '')
> +('BSD_IMR', '0x000120a8', '')
> +('BLT_IMR', '0x000220a8', '')
> diff --git a/tools/quick_dump/base_other.txt b/tools/quick_dump/base_other.txt
> new file mode 100644
> index 0000000..24fefdf
> --- /dev/null
> +++ b/tools/quick_dump/base_other.txt
> @@ -0,0 +1,6 @@
> +('PGETBL_CTL', '0x00002020', '')
> +('MI_MODE', '0x0000209c', '')
> +('ERROR_GEN6', '0x000040a0', '')
> +('RENDER_HWSTAM', '0x00002098', '')
> +('GEN6_BSD_HWSTAM', '0x00012098', '')
> +('GEN6_BLITTER_HWSTAM', '0x00022098', '')
> diff --git a/tools/quick_dump/base_power.txt b/tools/quick_dump/base_power.txt
> new file mode 100644
> index 0000000..4a142e5
> --- /dev/null
> +++ b/tools/quick_dump/base_power.txt
> @@ -0,0 +1,21 @@
> +('GEN6_RPNSWREQ', '0x0000a008', '')
> +('GEN6_RC_VIDEO_FREQ', '0x0000a00c', '')
> +('GEN6_RP_DOWN_TIMEOUT', '0x0000a010', '')
> +('GEN6_RP_INTERRUPT_LIMITS', '0x0000a014', '')
> +('GEN6_RP_CONTROL', '0x0000a024', '')
> +('GEN6_RP_UP_THRESHOLD', '0x0000a02c', '')
> +('GEN6_RP_UP_EI', '0x0000a068', '')
> +('GEN6_RP_DOWN_EI', '0x0000a06c', '')
> +('GEN6_RP_IDLE_HYSTERSIS', '0x0000a070', '')
> +('GEN6_RC_CONTROL', '0x0000a090', '')
> +('GEN6_RC_STATE', '0x0000a094', '')
> +('GEN6_RC1_WAKE_RATE_LIMIT', '0x0000a098', '')
> +('GEN6_RC6_WAKE_RATE_LIMIT', '0x0000a09c', '')
> +('GEN6_RC_EVALUATION_INTERVAL', '0x0000a0a8', '')
> +('GEN6_RC_IDLE_HYSTERSIS', '0x0000a0ac', '')
> +('GEN6_RC_SLEEP', '0x0000a0b0', '')
> +('GEN6_RC1e_THRESHOLD', '0x0000a0b4', '')
> +('GEN6_RC6_THRESHOLD', '0x0000a0b8', '')
> +('RC6_RESIDENCY_TIME', '0x00138108', '')
> +('RC6p_RESIDENCY_TIME', '0x0013810c', '')
> +('RC6pp_RESIDENCY_TIME', '0x00138110', '')
> diff --git a/tools/quick_dump/base_rings.txt b/tools/quick_dump/base_rings.txt
> new file mode 100644
> index 0000000..d791172
> --- /dev/null
> +++ b/tools/quick_dump/base_rings.txt
> @@ -0,0 +1,27 @@
> +('RENDER_INSTPM', '0x20c0', '')
> +('BSD_INSTPM', '0x120c0', '')
> +('BLT_INSTPM', '0x220c0', '')
> +('RENDER_RING_HEAD', '0x2034', '')
> +('BSD_RING_HEAD', '0x12034', '')
> +('BLT_RING_HEAD', '0x22034', '')
> +('RENDER_IPEIR', '0x2064', '')
> +('BSD_IPEIR', '0x12064', '')
> +('BLT_IPEIR', '0x22064', '')
> +('RENDER_IPEHR', '0x2068', '')
> +('BSD_IPEHR', '0x12068', '')
> +('BLT_IPEHR', '0x22068', '')
> +('RENDER_INSTDONE', '0x206c', '')
> +('BSD_INSTDONE', '0x1206c', '')
> +('BLT_INSTDONE', '0x2206c', '')
> +('RENDER_INSTPS', '0x2070', '')
> +('BSD_INSTPS', '0x12070', '')
> +('BLT_INSTPS', '0x22070', '')
> +('RENDER_RING_ACTHD', '0x2074', '')
> +('BSD_RING_ACTHD', '0x12074', '')
> +('BLT_RING_ACTHD', '0x22074', '')
> +('RENDER_FADDR', '0x2078', '')
> +('BSD_FADDR', '0x12078', '')
> +('BLT_FADDR', '0x22078', '')
> +('RENDER_RING_TAIL', '0x2030', '')
> +('BSD_RING_TAIL', '0x12030', '')
> +('BLT_RING_TAIL', '0x22030', '')
> diff --git a/tools/quick_dump/gen6_other.txt b/tools/quick_dump/gen6_other.txt
> new file mode 100644
> index 0000000..9aa6539
> --- /dev/null
> +++ b/tools/quick_dump/gen6_other.txt
> @@ -0,0 +1 @@
> +('GFX_MODE', '0x00002520', '')
> diff --git a/tools/quick_dump/gen7_other.txt b/tools/quick_dump/gen7_other.txt
> new file mode 100644
> index 0000000..b6313a4
> --- /dev/null
> +++ b/tools/quick_dump/gen7_other.txt
> @@ -0,0 +1 @@
> +('GFX_MODE_GEN7', '0x0000229c', '')
> diff --git a/tools/quick_dump/ivybridge b/tools/quick_dump/ivybridge
> new file mode 100644
> index 0000000..4637b42
> --- /dev/null
> +++ b/tools/quick_dump/ivybridge
> @@ -0,0 +1 @@
> +gen7_other.txt
> diff --git a/tools/quick_dump/quick_dump.py b/tools/quick_dump/quick_dump.py
> new file mode 100755
> index 0000000..c26f69a
> --- /dev/null
> +++ b/tools/quick_dump/quick_dump.py
> @@ -0,0 +1,34 @@
> +#!/usr/bin/env python3
> +
> +import argparse
> +import os
> +import sys
> +import ast
> +import subprocess
> +
> +def parse_file(file):
> +	for line in file:
> +		register = ast.literal_eval(line)
> +		value = subprocess.check_output(["../intel_reg_read", register[1]])
> +		value = value.decode('UTF-8') # convert the byte array to string
> +		value = value.rstrip() #dump the newline
> +		value = value.split(':') #output is 'addr : offset'
> +		print(value[0], "(", register[0], ")", value[1])
> +
> +
> +parser = argparse.ArgumentParser(description='Dumb register dumper.')
> +parser.add_argument('-b', '--baseless', action='store_true', default=True, help='baseless mode, ignore files starting with base_')
> +parser.add_argument('profile', nargs='?', type=argparse.FileType('r'), default=None)
> +args = parser.parse_args()
> +
> +#parse anything named base_ these are assumed to apply for all gens.
> +if args.baseless == False:
> +	for root, dirs, files in os.walk('.'):
> +		for name in files:
> +			if name.startswith(("base_")):
> +				file = open(name.rstrip(), 'r')
> +				parse_file(file)
> +
> +for extra in args.profile:
> +	extra_file = open(extra.rstrip(), 'r')
> +	parse_file(extra_file)
> diff --git a/tools/quick_dump/sandybridge b/tools/quick_dump/sandybridge
> new file mode 100644
> index 0000000..6ece0fd
> --- /dev/null
> +++ b/tools/quick_dump/sandybridge
> @@ -0,0 +1 @@
> +gen6_other.txt
> diff --git a/tools/quick_dump/valleyview b/tools/quick_dump/valleyview
> new file mode 100644
> index 0000000..9323100
> --- /dev/null
> +++ b/tools/quick_dump/valleyview
> @@ -0,0 +1,6 @@
> +vlv_display.txt
> +base_interrupt.txt
> +base_other.txt
> +base_power.txt
> +base_rings.txt
> +ivybridge
> diff --git a/tools/quick_dump/vlv_display.txt b/tools/quick_dump/vlv_display.txt
> new file mode 100644
> index 0000000..e66f11a
> --- /dev/null
> +++ b/tools/quick_dump/vlv_display.txt
> @@ -0,0 +1,197 @@
> +('CPU_VGACNTRL', '0x001c1000', '')
> +('PORT_DBG', '0x001c2308', '')
> +('DIGITAL_PORT_HOTPLUG_CNTRL', '0x001c4030', '')
> +('FDI_PLL_BIOS_0', '0x001c6000', '')
> +('FDI_PLL_BIOS_1', '0x001c6004', '')
> +('FDI_PLL_BIOS_2', '0x001c6008', '')
> +('DISPLAY_PORT_PLL_BIOS_0', '0x001c600c', '')
> +('DISPLAY_PORT_PLL_BIOS_1', '0x001c6010', '')
> +('DISPLAY_PORT_PLL_BIOS_2', '0x001c6014', '')
> +('FDI_PLL_FREQ_CTL', '0x001c6030', '')
> +('BLC_PWM_CPU_CTL2', '0x001c8250', '')
> +('BLC_PWM_CPU_CTL', '0x001c8254', '')
> +('HTOTAL_A', '0x001e0000', '')
> +('HBLANK_A', '0x001e0004', '')
> +('HSYNC_A', '0x001e0008', '')
> +('VTOTAL_A', '0x001e000c', '')
> +('VBLANK_A', '0x001e0010', '')
> +('VSYNC_A', '0x001e0014', '')
> +('PIPEASRC', '0x001e001c', '')
> +('VSYNCSHIFT_A', '0x001e0028', '')
> +('PIPEA_DATA_M1', '0x001e0030', '')
> +('PIPEA_DATA_N1', '0x001e0034', '')
> +('PIPEA_DATA_M2', '0x001e0038', '')
> +('PIPEA_DATA_N2', '0x001e003c', '')
> +('PIPEA_LINK_M1', '0x001e0040', '')
> +('PIPEA_LINK_N1', '0x001e0044', '')
> +('PIPEA_LINK_M2', '0x001e0048', '')
> +('PIPEA_LINK_N2', '0x001e004c', '')
> +('FDI_TXA_CTL', '0x001e0100', '')
> +('HTOTAL_B', '0x001e1000', '')
> +('HBLANK_B', '0x001e1004', '')
> +('HSYNC_B', '0x001e1008', '')
> +('VTOTAL_B', '0x001e100c', '')
> +('VBLANK_B', '0x001e1010', '')
> +('VSYNC_B', '0x001e1014', '')
> +('PIPEBSRC', '0x001e101c', '')
> +('VSYNCSHIFT_B', '0x001e1028', '')
> +('PIPEB_DATA_M1', '0x001e1030', '')
> +('PIPEB_DATA_N1', '0x001e1034', '')
> +('PIPEB_DATA_M2', '0x001e1038', '')
> +('PIPEB_DATA_N2', '0x001e103c', '')
> +('PIPEB_LINK_M1', '0x001e1040', '')
> +('PIPEB_LINK_N1', '0x001e1044', '')
> +('PIPEB_LINK_M2', '0x001e1048', '')
> +('PIPEB_LINK_N2', '0x001e104c', '')
> +('FDI_TXB_CTL', '0x001e1100', '')
> +('HTOTAL_C', '0x001e2000', '')
> +('HBLANK_C', '0x001e2004', '')
> +('HSYNC_C', '0x001e2008', '')
> +('VTOTAL_C', '0x001e200c', '')
> +('VBLANK_C', '0x001e2010', '')
> +('VSYNC_C', '0x001e2014', '')
> +('PIPECSRC', '0x001e201c', '')
> +('VSYNCSHIFT_C', '0x001e2028', '')
> +('PIPEC_DATA_M1', '0x001e2030', '')
> +('PIPEC_DATA_N1', '0x001e2034', '')
> +('PIPEC_DATA_M2', '0x001e2038', '')
> +('PIPEC_DATA_N2', '0x001e203c', '')
> +('PIPEC_LINK_M1', '0x001e2040', '')
> +('PIPEC_LINK_N1', '0x001e2044', '')
> +('PIPEC_LINK_M2', '0x001e2048', '')
> +('PIPEC_LINK_N2', '0x001e204c', '')
> +('FDI_TXC_CTL', '0x001e2100', '')
> +('CPU_eDP_A', '0x001e4000', '')
> +('PFA_WIN_POS', '0x001e8070', '')
> +('PFA_WIN_SIZE', '0x001e8074', '')
> +('PFA_CTL_1', '0x001e8080', '')
> +('PFA_CTL_2', '0x001e8084', '')
> +('PFA_CTL_3', '0x001e8088', '')
> +('PFA_CTL_4', '0x001e8090', '')
> +('PFB_WIN_POS', '0x001e8870', '')
> +('PFB_WIN_SIZE', '0x001e8874', '')
> +('PFB_CTL_1', '0x001e8880', '')
> +('PFB_CTL_2', '0x001e8884', '')
> +('PFB_CTL_3', '0x001e8888', '')
> +('PFB_CTL_4', '0x001e8890', '')
> +('PFC_WIN_POS', '0x001e9070', '')
> +('PFC_WIN_SIZE', '0x001e9074', '')
> +('PFC_CTL_1', '0x001e9080', '')
> +('PFC_CTL_2', '0x001e9084', '')
> +('PFC_CTL_3', '0x001e9088', '')
> +('PFC_CTL_4', '0x001e9090', '')
> +('PIPEACONF', '0x001f0008', '')
> +('DSPACNTR', '0x001f0180', '')
> +('DSPABASE', '0x001f0184', '')
> +('DSPASTRIDE', '0x001f0188', '')
> +('DSPASURF', '0x001f019c', '')
> +('DSPATILEOFF', '0x001f01a4', '')
> +('PIPEBCONF', '0x001f1008', '')
> +('DSPBCNTR', '0x001f1180', '')
> +('DSPBBASE', '0x001f1184', '')
> +('DSPBSTRIDE', '0x001f1188', '')
> +('DSPBSURF', '0x001f119c', '')
> +('DSPBTILEOFF', '0x001f11a4', '')
> +('PIPECCONF', '0x001f2008', '')
> +('DSPCCNTR', '0x001f2180', '')
> +('DSPCBASE', '0x001f2184', '')
> +('DSPCSTRIDE', '0x001f2188', '')
> +('DSPCSURF', '0x001f219c', '')
> +('DSPCTILEOFF', '0x001f21a4', '')
> +('PCH_DPLL_A', '0x00246014', '')
> +('PCH_DPLL_B', '0x00246018', '')
> +('PCH_FPA0', '0x00246040', '')
> +('PCH_FPA1', '0x00246044', '')
> +('PCH_FPB0', '0x00246048', '')
> +('PCH_FPB1', '0x0024604c', '')
> +('PCH_DREF_CONTROL', '0x00246200', '')
> +('PCH_RAWCLK_FREQ', '0x00246204', '')
> +('PCH_DPLL_TMR_CFG', '0x00246208', '')
> +('PCH_SSC4_PARMS', '0x00246210', '')
> +('PCH_SSC4_AUX_PARMS', '0x00246214', '')
> +('PCH_DPLL_ANALOG_CTL', '0x00246300', '')
> +('PCH_DPLL_SEL', '0x00247000', '')
> +('PCH_PP_STATUS', '0x00247200', '')
> +('PCH_PP_CONTROL', '0x00247204', '')
> +('PCH_PP_ON_DELAYS', '0x00247208', '')
> +('PCH_PP_OFF_DELAYS', '0x0024720c', '')
> +('PCH_PP_DIVISOR', '0x00247210', '')
> +('BLC_PWM_PCH_CTL1', '0x00248250', '')
> +('BLC_PWM_PCH_CTL2', '0x00248254', '')
> +('TRANS_HTOTAL_A', '0x00260000', '')
> +('TRANS_HBLANK_A', '0x00260004', '')
> +('TRANS_HSYNC_A', '0x00260008', '')
> +('TRANS_VTOTAL_A', '0x0026000c', '')
> +('TRANS_VBLANK_A', '0x00260010', '')
> +('TRANS_VSYNC_A', '0x00260014', '')
> +('TRANS_VSYNCSHIFT_A', '0x00260028', '')
> +('TRANSA_DATA_M1', '0x00260030', '')
> +('TRANSA_DATA_N1', '0x00260034', '')
> +('TRANSA_DATA_M2', '0x00260038', '')
> +('TRANSA_DATA_N2', '0x0026003c', '')
> +('TRANSA_DP_LINK_M1', '0x00260040', '')
> +('TRANSA_DP_LINK_N1', '0x00260044', '')
> +('TRANSA_DP_LINK_M2', '0x00260048', '')
> +('TRANSA_DP_LINK_N2', '0x0026004c', '')
> +('TRANS_DP_CTL_A', '0x00260300', '')
> +('TRANS_HTOTAL_B', '0x00261000', '')
> +('TRANS_HBLANK_B', '0x00261004', '')
> +('TRANS_HSYNC_B', '0x00261008', '')
> +('TRANS_VTOTAL_B', '0x0026100c', '')
> +('TRANS_VBLANK_B', '0x00261010', '')
> +('TRANS_VSYNC_B', '0x00261014', '')
> +('TRANS_VSYNCSHIFT_B', '0x00261028', '')
> +('TRANSB_DATA_M1', '0x00261030', '')
> +('TRANSB_DATA_N1', '0x00261034', '')
> +('TRANSB_DATA_M2', '0x00261038', '')
> +('TRANSB_DATA_N2', '0x0026103c', '')
> +('TRANSB_DP_LINK_M1', '0x00261040', '')
> +('TRANSB_DP_LINK_N1', '0x00261044', '')
> +('TRANSB_DP_LINK_M2', '0x00261048', '')
> +('TRANSB_DP_LINK_N2', '0x0026104c', '')
> +('PCH_ADPA', '0x00261100', '')
> +('HDMIB', '0x00261140', '')
> +('HDMIC', '0x00261150', '')
> +('HDMID', '0x00261160', '')
> +('PCH_LVDS', '0x00261180', '')
> +('TRANS_DP_CTL_B', '0x00261300', '')
> +('TRANS_HTOTAL_C', '0x00262000', '')
> +('TRANS_HBLANK_C', '0x00262004', '')
> +('TRANS_HSYNC_C', '0x00262008', '')
> +('TRANS_VTOTAL_C', '0x0026200c', '')
> +('TRANS_VBLANK_C', '0x00262010', '')
> +('TRANS_VSYNC_C', '0x00262014', '')
> +('TRANS_VSYNCSHIFT_C', '0x00262028', '')
> +('TRANSC_DATA_M1', '0x00262030', '')
> +('TRANSC_DATA_N1', '0x00262034', '')
> +('TRANSC_DATA_M2', '0x00262038', '')
> +('TRANSC_DATA_N2', '0x0026203c', '')
> +('TRANSC_DP_LINK_M1', '0x00262040', '')
> +('TRANSC_DP_LINK_N1', '0x00262044', '')
> +('TRANSC_DP_LINK_M2', '0x00262048', '')
> +('TRANSC_DP_LINK_N2', '0x0026204c', '')
> +('TRANS_DP_CTL_C', '0x00262300', '')
> +('PCH_DP_B', '0x00264100', '')
> +('PCH_DP_C', '0x00264200', '')
> +('PCH_DP_D', '0x00264300', '')
> +('TRANSACONF', '0x00270008', '')
> +('FDI_RXA_CTL', '0x0027000c', '')
> +('FDI_RXA_MISC', '0x00270010', '')
> +('FDI_RXA_IIR', '0x00270014', '')
> +('FDI_RXA_IMR', '0x00270018', '')
> +('FDI_RXA_TUSIZE1', '0x00270030', '')
> +('FDI_RXA_TUSIZE2', '0x00270038', '')
> +('TRANSBCONF', '0x00271008', '')
> +('FDI_RXB_CTL', '0x0027100c', '')
> +('FDI_RXB_MISC', '0x00271010', '')
> +('FDI_RXB_IIR', '0x00271014', '')
> +('FDI_RXB_IMR', '0x00271018', '')
> +('FDI_RXB_TUSIZE1', '0x00271030', '')
> +('FDI_RXB_TUSIZE2', '0x00271038', '')
> +('TRANSCCONF', '0x00272008', '')
> +('FDI_RXC_CTL', '0x0027200c', '')
> +('FDI_RXC_MISC', '0x00272010', '')
> +('FDI_RXC_TUSIZE1', '0x00272030', '')
> +('FDI_RXC_TUSIZE2', '0x00272038', '')
> +('FDI_PLL_CTL_1', '0x0027e000', '')
> +('FDI_PLL_CTL_2', '0x0027e004', '')
> -- 
> 1.7.12
> 
> _______________________________________________
> 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