drivers: soc: andes: Fix an error for print_detailed_cause() on imprecise exception (torvalds#101)

ben717-linux · CharlesWu465 · commit 825484e69ed1 · 2024-09-06T14:43:09.000+08:00
To print error messages for imprecise exception, extract the field sdcause in the CSR sdcause instead of reading the whole register. Signed-off-by: Ben Zong-You Xie <ben717@andestech.com> Reviewed-on: https://gitea.andestech.com/RD-SW/linux/pulls/101 Reviewed-by: Tim Shih-Ting OuYang <tim609@andestech.com> Reviewed-by: Leo Yu-Chi Liang <ycliang@andestech.com> Reviewed-by: Dylan Dai-Rong Jhong <dylan@andestech.com> Co-authored-by: Ben Zong-You Xie <ben717@andestech.com> Co-committed-by: Ben Zong-You Xie <ben717@andestech.com>
diff --git a/drivers/soc/andes/dcause.c b/drivers/soc/andes/dcause.c
@@ -45,6 +45,7 @@ void print_detailed_cause(long scause, unsigned long sdcause)
 		}
 	} else {
 		scause &= ~CAUSE_IRQ_FLAG;
+		sdcause &= SDCAUSE_DCAUSE_MASK;
 		switch (scause) {
 		case IRQ_IMPRECISE_ECC:
 			pr_info("The detailed trap cause: %s\n",
diff --git a/include/linux/soc/andes/dcause.h b/include/linux/soc/andes/dcause.h
@@ -10,6 +10,8 @@
 #define IRQ_BUS_RW_TRANS	0x111
 #define IRQ_PMOVI		0x112
 
+#define SDCAUSE_DCAUSE_MASK	0x1f
+
 static const char *const sdcause_fetch_access[] = {
 	"Reserved",
 	"ECC/Parity error",

Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,7 @@ void print_detailed_cause(long scause, unsigned long sdcause)`
`45`	`45`	`}`
`46`	`46`	`} else {`
`47`	`47`	`scause &= ~CAUSE_IRQ_FLAG;`
	`48`	`+ sdcause &= SDCAUSE_DCAUSE_MASK;`
`48`	`49`	`switch (scause) {`
`49`	`50`	`case IRQ_IMPRECISE_ECC:`
`50`	`51`	`pr_info("The detailed trap cause: %s\n",`