Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29480 )
Change subject: mb/kontron/bsl6: Add new Skylake COMe module ......................................................................
Patch Set 30:
(1 comment)
https://review.coreboot.org/c/coreboot/+/29480/29/src/mainboard/kontron/bsl6... File src/mainboard/kontron/bsl6/Kconfig:
https://review.coreboot.org/c/coreboot/+/29480/29/src/mainboard/kontron/bsl6... PS29, Line 46: config CBFS_SIZE : hex : default 0x600000
It makes it more tedious to switch between vendor and coreboot, though...
In the best case, it shifts convenience from one use case to another. In the worst case, somebody bricks their machine. There is no guarantee that everybody has the same flash layout. It's likely for a laptop, yes, but less likely for some off-the-shelf board and much less likely in the embedded world. And for a COMe module, we don't even know the chip size, one could use a different one on the carrier board.
If you want, you can add this default for the Boxer26 variant, but not for the generic one. It's not hardware specific and we are not the board vendor, so we can't say that we know this.