Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31810 )
Change subject: nb/amd/amdfam10: Remove redundant test if "BIOS_DEBUG" set ......................................................................
Patch Set 2:
You're changing the logic here, RES_DEBUG used to be 0 but now you enable all those prints unconditionally.
I'd recommend to leave in RES_DEBUG and just change all the 'if (IS_ENABLED(RES_DEBUG))' to a simple 'if (RES_DEBUG)'. This sort of stuff is useful to keep debug prints in that we don't want to enable by default, but want to have available easily for developers that want to enable some more log spam when they debug something.