[coreboot-gerrit] Change in coreboot[master]: util/spd_tools: Encode SDRAM min cycle time (TCKMinPs)

[Reka Norman (Code Review)]

Attention is currently required from: Tim Wawrzynczak, Nick Vaccaro, Karthik Ramasubramanian.
Reka Norman has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62387 )

Change subject: util/spd_tools: Encode SDRAM min cycle time (TCKMinPs)
......................................................................

Patch Set 2: Code-Review+1

(3 comments)

Patchset:

PS2: 
(+1 only because I don't have +2 permissions yet)

File util/spd_tools/src/spd_gen/lp5.go:

https://review.coreboot.org/c/coreboot/+/62387/comment/4eedc969_0c02dab0 
PS1, Line 587: if ok == false || tCKMinPs == 0 {
             : 		return LP5GetDefaultTCKMinPs(memAttribs)
             : 	}
> I defined the default speed to TCKMinPs mapping inside the earlier table. […]

Thanks! This is nicer than what I suggested :)

File util/spd_tools/src/spd_gen/lp5.go:

https://review.coreboot.org/c/coreboot/+/62387/comment/df9f7606_6f4095e0 
PS2, Line 573: Println
nit: Println() doesn't support format specifiers. Can you change it to Printf() and add a \n