Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34572 )
Change subject: soc/nvidia/tegra124: Assert divisor is non-zero ......................................................................
Patch Set 1: Code-Review+2
(1 comment)
https://review.coreboot.org/c/coreboot/+/34572/1/src/soc/nvidia/tegra124/clo... File src/soc/nvidia/tegra124/clock.c:
https://review.coreboot.org/c/coreboot/+/34572/1/src/soc/nvidia/tegra124/clo... PS1, Line 380: assert(plld.m != 0);
Why only m and not n? Seems pretty much the same situation in both cases here.
Oh wait, I missed how n is a multiplier again (that formula could use some more parenthesis...). Ignore this then.