[coreboot-gerrit] Change in ...coreboot[master]: sb/nvidia/ck804: Prevent uninitialized reads

[Jacob Garber (Code Review)]

Jacob Garber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33459 )

Change subject: sb/nvidia/ck804: Prevent uninitialized reads
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/33459/1/src/southbridge/nvidia/ck804/early_s... 
File src/southbridge/nvidia/ck804/early_setup_car.c:

https://review.coreboot.org/#/c/33459/1/src/southbridge/nvidia/ck804/early_s... 
PS1, Line 369: 	ck804_early_set_port(ck804_num, busn, io_base);
             : 	ck804_early_setup(ck804_num, busn, io_base);
             : 	ck804_early_clear_port(ck804_num, busn, io_base);
> This will still blow up if ck804_num is zero.

Those use for loops to iterate over the initialized elements, so if ck804_num is zero they actually don't do anything (since there aren't any initialized elements to do anything with)