[coreboot-gerrit] [S] Change in coreboot[main]: mb/intel/archercity_crb: Deselect OCP drivers

[Martin L Roth (Code Review)]

Attention is currently required from: Angel Pons, Jincheng Li, Lean Sheng Tan, Patrick Rudolph, Shuo Liu.

Martin L Roth has posted comments on this change by Patrick Rudolph. ( https://review.coreboot.org/c/coreboot/+/82169?usp=email )

Change subject: mb/intel/archercity_crb: Deselect OCP drivers
......................................................................

Patch Set 1:

(1 comment)

File src/mainboard/intel/archercity_crb/Kconfig:

https://review.coreboot.org/c/coreboot/+/82169/comment/d02647cc_f9c17fde?usp... :
PS1, Line 34: config MAINBOARD_OCP_COMPABILITY
> Can we name it as below? […]

Those names seem reasonable, but as Angel mentions, maybe the easier way is just to add a config file that enables this to the coreboot/configs folder. Feel free to reach out to me on google chat if you have any questions about that - gaumless@gmail.

Honestly, I'm not convinced that variants are the way to go. It's not a different board, just a different config.