[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: rewrite soc_util

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41630 )

Change subject: soc/amd/picasso: rewrite soc_util
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/41630/7/src/soc/amd/picasso/soc_uti... 
File src/soc/amd/picasso/soc_util.c:

https://review.coreboot.org/c/coreboot/+/41630/7/src/soc/amd/picasso/soc_uti... 
PS7, Line 149: 		break;
> break is not useful after a goto or return

sure. I'll keep this one though so things won't possibly misbehave when checks or more SoC types with FT5 socket get added here