summaryrefslogtreecommitdiff
path: root/src/southbridge
diff options
context:
space:
mode:
authorJacob Garber <jgarber1@ualberta.ca>2019-06-13 16:34:54 -0600
committerPatrick Georgi <pgeorgi@google.com>2019-07-10 18:23:11 +0000
commit15947182fde54bbc3c4faba780a515d8b63bce7f (patch)
treeda4e73da53074a621ec94aa7361ea5f1587a16d1 /src/southbridge
parent967f862e47e26cd07b6b43eda6e1445f58e8989b (diff)
downloadcoreboot-15947182fde54bbc3c4faba780a515d8b63bce7f.tar.xz
sb/nvidia/ck804: Remove old debugging code
These printk() statements were added in commit cdc526e582 (southbridge/nvidia/ck804: Fix boot hang on ASUS KFSN4-DRE w/ K8 CPU) when debugging another issue. They have undefined reads if ck804_num is 0 and aren't needed anymore, so drop them. Change-Id: I80b775370ac6485958948f0bff4510755a6cd2b8 Signed-off-by: Jacob Garber <jgarber1@ualberta.ca> Found-by: Coverity CID 137058{1,3} Reviewed-on: https://review.coreboot.org/c/coreboot/+/33459 Reviewed-by: Patrick Georgi <pgeorgi@google.com> Reviewed-by: Angel Pons <th3fanbus@gmail.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/southbridge')
-rw-r--r--src/southbridge/nvidia/ck804/early_setup_car.c3
1 files changed, 0 insertions, 3 deletions
diff --git a/src/southbridge/nvidia/ck804/early_setup_car.c b/src/southbridge/nvidia/ck804/early_setup_car.c
index 156c38745b..e7a06f9dc7 100644
--- a/src/southbridge/nvidia/ck804/early_setup_car.c
+++ b/src/southbridge/nvidia/ck804/early_setup_car.c
@@ -355,11 +355,8 @@ static int ck804_early_setup_x(void)
}
}
- printk(BIOS_DEBUG, "ck804_early_set_port(%d, %d, %d)\n", ck804_num, busn[0], io_base[0]);
ck804_early_set_port(ck804_num, busn, io_base);
- printk(BIOS_DEBUG, "ck804_early_setup(%d, %d, %d)\n", ck804_num, busn[0], io_base[0]);
ck804_early_setup(ck804_num, busn, io_base);
- printk(BIOS_DEBUG, "ck804_early_clear_port(%d, %d, %d)\n", ck804_num, busn[0], io_base[0]);
ck804_early_clear_port(ck804_num, busn, io_base);
return set_ht_link_ck804(4);