Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36165 )
Change subject: drivers/intel/fsp2_0: Move Debug options to "Debugging" ......................................................................
Patch Set 6:
(4 comments)
https://review.coreboot.org/c/coreboot/+/36165/3/src/Kconfig File src/Kconfig:
https://review.coreboot.org/c/coreboot/+/36165/3/src/Kconfig@727 PS3, Line 727: source "src/cpu/*/Kconfig.debug"
Maybe change these to Kconfig.cpu_debug or something so we can add them from anywhere? […]
Done
https://review.coreboot.org/c/coreboot/+/36165/3/src/Kconfig@728 PS3, Line 728: drivers
Ok, the 'src/' is actually needed here. […]
Done
https://review.coreboot.org/c/coreboot/+/36165/3/src/Kconfig@728 PS3, Line 728: source "drivers/intel/*/Kconfig.debug"
Put another comment above this? I don't think it counts as CPU Debug Settings.
Done
https://review.coreboot.org/c/coreboot/+/36165/3/src/drivers/intel/fsp2_0/Kc... File src/drivers/intel/fsp2_0/Kconfig.debug:
https://review.coreboot.org/c/coreboot/+/36165/3/src/drivers/intel/fsp2_0/Kc... PS3, Line 2:
Put this inside a menu?
Done, kind of. It has a heading now.