[AMD Official Use Only]
The asic structure has a function callback "errmsg" which should be used for this.
Tom
________________________________________
From: Tuikov, Luben <Luben.Tuikov@xxxxxxx>
Sent: Wednesday, March 9, 2022 19:42
To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx
Cc: Tuikov, Luben; Deucher, Alexander; StDenis, Tom
Subject: [PATCH 1/4] umr: Print errors to stderr
Print the following error message,
Invalid gca config data header
to stderr, since printing it to stdout,
confuses parser scripts.
Also modify this message to be clearer. For instance,
Invalid or unknown GCA config data header version:4
Cc: Alex Deucher <Alexander.Deucher@xxxxxxx>
Cc: Tom StDenis <tom.stdenis@xxxxxxx>
Signed-off-by: Luben Tuikov <luben.tuikov@xxxxxxx>
---
src/lib/scan_config.c | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/lib/scan_config.c b/src/lib/scan_config.c
index c7f4e9d3c69faa..3503bbd92a104a 100644
--- a/src/lib/scan_config.c
+++ b/src/lib/scan_config.c
@@ -122,7 +122,7 @@ static uint64_t read_int_drm(int cardno, char *fname)
}
/**
- * umr_scan_config - Scan the debugfs confiruration data
+ * umr_scan_config - Scan the debugfs configuration data
*/
int umr_scan_config(struct umr_asic *asic, int xgmi_scan)
{
@@ -246,7 +246,9 @@ gca_config:
case 4: parse_rev4(asic, data, &r);
break;
default:
- printf("Invalid gca config data header\n");
+ fprintf(stderr,
+ "Invalid or unknown GCA config data header version:%d\n",
+ data[0]);
return -1;
}
base-commit: 818bb1e8f1b5a26fda0e3d7abfc1e68605ebad28
--
2.35.1.291.gdab1b7905d
