[coreboot-gerrit] Change in coreboot[master]: soc/nvidia/tegra124: Assert divisor is non-zero

[Jacob Garber (Code Review)]

Jacob Garber has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/34572 )

Change subject: soc/nvidia/tegra124: Assert divisor is non-zero
......................................................................

soc/nvidia/tegra124: Assert divisor is non-zero

The logic for the calculation of plld.m is rather complicated, so do a
sanity check that it is non-zero before doing the division.

Change-Id: I60f49b8eed47a3de86713304bde7a4d3f3d935dd
Signed-off-by: Jacob Garber 
Found-by: Coverity CID 1260981
---
M src/soc/nvidia/tegra124/clock.c
1 file changed, 1 insertion(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/72/34572/1

diff --git a/src/soc/nvidia/tegra124/clock.c b/src/soc/nvidia/tegra124/clock.c
index 6877c04..8fb4b05 100644
--- a/src/soc/nvidia/tegra124/clock.c
+++ b/src/soc/nvidia/tegra124/clock.c
@@ -377,6 +377,7 @@
    	printk(BIOS_WARNING, "%s: Failed to match output frequency %u, "
    	       "best difference is %u.\n", __func__, frequency,
    	       best_diff);
+		assert(plld.m != 0);
    	rounded_rate = (ref / plld.m * plld.n) >> plld.p;
    }