When U-Boot did not do the MP init, we don't get an actual CPU number
here. Skip printing it in that case.

Signed-off-by: Simon Glass <s...@chromium.org>
---

Changes in v2:
- Don't show an invalid CPU number on error

 cmd/x86/mtrr.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/cmd/x86/mtrr.c b/cmd/x86/mtrr.c
index b213a942fde4..ff4be6b7bf5b 100644
--- a/cmd/x86/mtrr.c
+++ b/cmd/x86/mtrr.c
@@ -145,10 +145,12 @@ static int do_mtrr(struct cmd_tbl *cmdtp, int flag, int 
argc,
                for (; i >= 0; i = mp_next_cpu(cpu_select, i)) {
                        if (!first)
                                printf("\n");
-                       printf("CPU %d:\n", i);
+                       if (i < MP_SELECT_ALL)
+                               printf("CPU %d:\n", i);
                        ret = do_mtrr_list(reg_count, i);
                        if (ret) {
-                               printf("Failed to read CPU %d (err=%d)\n", i,
+                               printf("Failed to read CPU %s (err=%d)\n",
+                                      i < MP_SELECT_ALL ? simple_itoa(i) : "",
                                       ret);
                                return CMD_RET_FAILURE;
                        }
-- 
2.40.0.348.gf938b09366-goog

Reply via email to