[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp: Move function debug macros

[Marc Jones (Code Review)]

Marc Jones has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46093 )

Change subject: soc/intel/xeon_sp: Move function debug macros
......................................................................

Patch Set 5:

(2 comments)

These macros don't do anything by default. They need to be enabled in a local config. This shouldn't effect TIMELESS builds.

https://review.coreboot.org/c/coreboot/+/46093/5/src/Kconfig 
File src/Kconfig:

https://review.coreboot.org/c/coreboot/+/46093/5/src/Kconfig@1132 
PS5, Line 1132: config HAVE_DEBUG_FUNC
              : 	def_bool n
              :
> This was from a previous comment from Angle. https://review.coreboot. […]

Ack

https://review.coreboot.org/c/coreboot/+/46093/5/src/include/device/device.h 
File src/include/device/device.h:

https://review.coreboot.org/c/coreboot/+/46093/5/src/include/device/device.h... 
PS5, Line 282: #if CONFIG(DEBUG_FUNC)
> This is some pretty generic stuff. […]

Ack