On Wed, Jun 26, 2013 at 08:57:32PM +0300, ville.syrjala at linux.intel.com wrote: > From: Ville Syrj?l? <ville.syrjala at linux.intel.com> > > Repurpose the (currently unused) third element in the register > definition tuple to indicate the type of the register. 'DPIO' > is the only special register type for now. > > Signed-off-by: Ville Syrj?l? <ville.syrjala at linux.intel.com> > --- > tools/quick_dump/quick_dump.py | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/tools/quick_dump/quick_dump.py b/tools/quick_dump/quick_dump.py > index 6111b5d..390cafa 100755 > --- a/tools/quick_dump/quick_dump.py > +++ b/tools/quick_dump/quick_dump.py > @@ -13,7 +13,10 @@ def parse_file(file): > print('-' * 54) > for line in file: > register = ast.literal_eval(line) > - val = reg.read(register[1]) > + if register[2] == 'DPIO': > + val = reg.dpio_read(register[1]) > + else: > + val = reg.read(register[1]) > intreg = int(register[1], 16) > print('{0:#010x} | {1:<28} | {2:#010x}'.format(intreg, register[0], val)) > print('') I can't recall if you have push access, but I've pushed it. -- Ben Widawsky, Intel Open Source Technology Center