[coreboot-gerrit] Change in coreboot[master]: soc/amd/cezanne: Turn off gpp clock request for disabled devices

[Felix Held (Code Review)]

Attention is currently required from: Robert Zieba, Jason Glenesk, Raul Rangel, Martin L Roth, Marshall Dawson, Paul Menzel, Fred Reitberger, Rob Barnes, Karthik Ramasubramanian.
Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61259 )

Change subject: soc/amd/cezanne: Turn off gpp clock request for disabled devices
......................................................................

Patch Set 35:

(2 comments)

File src/soc/amd/cezanne/fch.c:

https://review.coreboot.org/c/coreboot/+/61259/comment/8ad9318c_2f790a00 
PS34, Line 200:  && (id != 0x0000ffff) && (id != 0xffff0000)
> Done

ah, that's where those cases were from. even though i don't expect those to happen, it's a good idea to also check for those two odd combinations

File src/soc/amd/cezanne/fch.c:

https://review.coreboot.org/c/coreboot/+/61259/comment/fc2e9fb9_5de7dc0f 
PS35, Line 185: break
replace break with continue? that will processing the next dxio descriptors instead of just jumping out of the for loop and not looking at the remaining dxio descriptors