diff options
author | Reza Arbab <arbab@linux.ibm.com> | 2019-10-22 16:22:21 -0500 |
---|---|---|
committer | Oliver O'Halloran <oohall@gmail.com> | 2019-11-04 10:52:47 +1100 |
commit | 91b4be52303e02960d72f60dba6fa110932c5ffd (patch) | |
tree | f693d4467c2a244beb839a622759ccbacb03f02f | |
parent | 66ab3cbd52fc64ecd4b0ee9e57d971abb9431b46 (diff) | |
download | skiboot-91b4be52303e02960d72f60dba6fa110932c5ffd.zip skiboot-91b4be52303e02960d72f60dba6fa110932c5ffd.tar.gz skiboot-91b4be52303e02960d72f60dba6fa110932c5ffd.tar.bz2 |
platforms/swift: Remove spurious error message
npu3_chip_possible_gpus() works by dividing the number of NVLink-mode
bricks by the number of bricks connecting a single GPU. In a system with
no GPUs, the latter value is unknown, so the function returns zero and
we trip a somewhat misleading error message.
The code afterward is safe to execute in any case, so there's no need to
return either. Remove the check entirely.
Signed-off-by: Reza Arbab <arbab@linux.ibm.com>
Signed-off-by: Oliver O'Halloran <oohall@gmail.com>
-rw-r--r-- | platforms/astbmc/swift.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/platforms/astbmc/swift.c b/platforms/astbmc/swift.c index ca612c3..a1040fb 100644 --- a/platforms/astbmc/swift.c +++ b/platforms/astbmc/swift.c @@ -63,11 +63,6 @@ static void swift_finalise_dt(bool is_reboot) if (is_reboot) return; - if (npu3_chip_possible_gpus() != 2) { - prlog(PR_ERR, "NPU: Unknown link topology detected\n"); - return; - } - /* Collect the first link we find for each GPU */ npu3_for_each_nvlink_npu(npu) { npu3_for_each_nvlink_dev(dev, npu) { |