[Skiboot] [RFC PATCH] phb4: Make verbose EEH output less verbose

[Stewart Smith]

Russell Currey <ruscur at russell.cc> writes:
> On Tue, 2018-07-31 at 18:48 +1000, Oliver O'Halloran wrote:
>> Each PHB4 component has a error status register and a set of
>> component-specific registers that describe that error in more detail.
>> If the status register of a component has no bits set there's not
>> much
>> point in dumping the error detail registers.
>> 
>> This patch modifies the verbose EEH dumping code so that it only
>> dumps the error registers of a component if there are bits set in the
>> error status register of that component. This limits the output to
>> the
>> registers that are actually relevant to the error that occured which
>> makes interpreting the dumps slightly easier.
>> 
>> Signed-off-by: Oliver O'Halloran <oohall at gmail.com>
>> ---
>> I've been using this in my debug builds for a while, but I'm not sure
>> if it's a great idea since sometimes other people want to full dump.
>> It's probably fine...
>
> This is the same thing the kernel does, I don't see any reason why we
> wouldn't do it here.  The kernel also puts multiple registers on the
> same line to save space, this does require looking at the code to
> remember what order the regs are in but if you're decoding this stuff
> that's the least of your problems so *shrug*
>
> In any case,
>
> Acked-by: Russell Currey <ruscur at russell.cc>

I think this means I can make good on my joke^W promise to merge some of
Oliver's RFC patches at 4:59pm on a friday before he takes some time
off? :)

-- 
Stewart Smith
OPAL Architect, IBM.