Re: [PATCH] cxl: Dump PSL_FIR1/2 registers on PSL9 error irq

2017-10-09 Thread Vaibhav Jain
Fred just pointed out that there is no register named FIR2 on PSL9 instead we have a FIR_MASK register in its place. So I will re-spin this patch with the correct register name. Thanks everyone for the review and to Fred for pointing out the issue. -- Vaibhav Jain

Re: [PATCH] cxl: Dump PSL_FIR1/2 registers on PSL9 error irq

2017-09-07 Thread Andrew Donnellan
LGTM Acked-by: Andrew Donnellan On 07/09/17 22:13, Vaibhav Jain wrote: For PSL9 currently we aren't dumping the PSL FIR1/2 registers when a PSL error interrupt is triggered. Contents of these registers are useful in debugging AFU issues. This patch fixes issue

Re: [PATCH] cxl: Dump PSL_FIR1/2 registers on PSL9 error irq

2017-09-07 Thread christophe lombard
Le 07/09/2017 à 14:13, Vaibhav Jain a écrit : For PSL9 currently we aren't dumping the PSL FIR1/2 registers when a PSL error interrupt is triggered. Contents of these registers are useful in debugging AFU issues. This patch fixes issue by updating the cxl_native_err_irq_dump_regs() to dump

[PATCH] cxl: Dump PSL_FIR1/2 registers on PSL9 error irq

2017-09-07 Thread Vaibhav Jain
For PSL9 currently we aren't dumping the PSL FIR1/2 registers when a PSL error interrupt is triggered. Contents of these registers are useful in debugging AFU issues. This patch fixes issue by updating the cxl_native_err_irq_dump_regs() to dump these regs on PSL error interrupt thereby bringing