[coreboot-gerrit] Change in coreboot[master]: intel/e7505,i82801dx: Remove wrapper spd_read_byte()

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38235 )

Change subject: intel/e7505,i82801dx: Remove wrapper spd_read_byte()
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38235/4/src/northbridge/intel/e7505... 
File src/northbridge/intel/e7505/raminit.c:

https://review.coreboot.org/c/coreboot/+/38235/4/src/northbridge/intel/e7505... 
PS4, Line 1286: 		    smbus_read_byte(dimm_socket_address,
> Even then, not replacing the function call would be worse (the code would rot)

I have some follow-up on the topic, I'll touch this then.

There is enough CAR to hold the entire SPDs instead, no need to read them individually.