Change in flashrom[master]: RFC: flashchips.c: merge GD25B128B/GD25Q128B and GD25Q127C/GD25Q128C

[Nikolai Artemiev (Code Review)]

Attention is currently required from: Nico Huber, Angel Pons.
Nikolai Artemiev has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/52883 )

Change subject: RFC: flashchips.c: merge GD25B128B/GD25Q128B and GD25Q127C/GD25Q128C
......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:
> Sorry, I'm not convinced. This looks like a workaround for something only downstream uses. […]

Losing FEATURE_QPI definitely isn't ideal. I'm looking into whether the 'B' chips also support QPI; if they do does then I think merging would be ok, but please let me know if there are other issues.

Having a way to ignore a list of chips would be useful too, though I think there is still benefit to merging chips that actually are identical, both for downstream and upstream users.