From 343f439801225353201d8157b17e2eaa30da85ee Mon Sep 17 00:00:00 2001 From: Ivi Ballou Date: Wed, 25 Feb 2026 17:58:50 +0200 Subject: [PATCH] util/inteltool: set amb registers dumping error print to stdout Set the "Error: Dumping AMBs on this MCH is not (yet) supported." message to stdout. All other "dumping ... not (yet) supported" errors use stdout, which makes them usable with pagers like less. The current behavior prints the AMB dumping error in stderr, which breaks pagers. This change aims to fix this discrepancy. Change-Id: I502e9f8d5c71953e844bdc7174b3c7bd2987d00f Signed-off-by: Ivi Ballou Reviewed-on: https://review.coreboot.org/c/coreboot/+/91419 Reviewed-by: Paul Menzel Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons --- util/inteltool/amb.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/inteltool/amb.c b/util/inteltool/amb.c index 961c0abf36..b01ea4326b 100644 --- a/util/inteltool/amb.c +++ b/util/inteltool/amb.c @@ -431,7 +431,7 @@ int print_ambs(struct pci_dev *dev, struct pci_access *pacc) break; default: - fprintf(stderr, "Error: Dumping AMBs on this MCH is not (yet) supported.\n"); + printf("Error: Dumping AMBs on this MCH is not (yet) supported.\n"); return 1; }