Change in coreboot[master]: [WIP] drivers/mrc_cache: Drop get_write_protect_state() stubs

[Kyösti Mälkki (Code Review)]

Attention is currently required from: Jason Glenesk, Marshall Dawson, Tim Wawrzynczak, Julius Werner, Andrey Petrov, Alexander Couzens, Patrick Rudolph, Yu-Ping Wu, Felix Held.
Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58968 )

Change subject: [WIP] drivers/mrc_cache: Drop get_write_protect_state() stubs
......................................................................

Patch Set 3:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/58968/comment/57de5b7c_c2ec0c51 
PS3, Line 11: from soc/.
> I think MRC_SETTINGS_PROTECT is supposed to be a capability Kconfig (i.e. […]

I think there may be some cases where developers will want to overwrite the entire SPI flash image in a running system, from OS with flashrom.

File src/security/vboot/bootmode.c:

https://review.coreboot.org/c/coreboot/+/58968/comment/c48102e4_82e6e56b 
PS3, Line 65: int __weak get_write_protect_state(void)
> Moving this out of VBOOT_NO_BOARD_SUPPORT makes me uneasy because that makes it easier to accidental […]

Yeah, I thought checking write_protect_state() might be dropped entirely.

Should we keep it at all, I have some plans to avoid the accident of hitting the __weak functions unintentionaly.