Change in coreboot[master]: treewide: Kconfig: replace `def_bool n` by `bool`

[Martin Roth (Code Review)]

Attention is currently required from: Nico Huber, Michael Niewöhner, Werner Zeh.
Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/52809 )

Change subject: treewide: Kconfig: replace `def_bool n` by `bool`
......................................................................

Patch Set 3:

(1 comment)

Patchset:

PS3:
> > > It's actually the other way round. […]

Types cannot be overridden.  The type is only required in a single place, though it's typically included everywhere.

For the prompts with defaults, I moved the default to the end of the src/Kconfig file.  This is really the only spot where it's critical because the general menu is at the very top, ahead of the mainboard or chipset menus.

To me 'dev_bool n' is more than just "I don't care what the default is, so set it to n if nobody else sets it to y.  It's a signal saying that n SHOULD be the default option.  We lose that signal with this change.

Why is this change necessary?  You say that the def_bool n isn't needed, but is it hurting anything?  My thought is that it doesn't hurt anything, so let's just leave it as it is and not touch files all across the codebase.