Damien Zammit has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32237 )
Change subject: Add complete board no-blob/blob status ......................................................................
Patch Set 5:
(24 comments)
Thanks for helping with non-x86 blob status, I had no clue on those.
BL1 blob flag still undecided for Exynos. Currently set to 'n'
How should I handle the variants? Just noticed they have a board_info.txt as well.
PS: Sorry, don't know why all these people became reviewers, I didn't set them.
https://review.coreboot.org/#/c/32237/3//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32237/3//COMMIT_MSG@12 PS3, Line 12: that coreboot initialises
"that" being the CPU? it's not the CPU that is initialized
Done
https://review.coreboot.org/#/c/32237/3//COMMIT_MSG@12 PS3, Line 12: on the main cpu that coreboot initialises
This would be n for almost everything because of boot roms. I'd […]
Done
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv File Documentation/libre-boards.csv:
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv@23 PS2, Line 23: Needs VGA-ROM
Video output is not a requirement for coreboot...
Done
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv@45 PS2, Line 45: RAM init
FSP is not about raminit. It replaces nearly all coreboot […]
Done
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv@74 PS2, Line 74: blob for Braswell SOC
FSP
Done
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv@75 PS2, Line 75: blob for Icelake SOC
FSP
Done
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv@79 PS2, Line 79: Blob for Cannonlake SOC
FSP
Done
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv@83 PS2, Line 83: blob for GLK soc
FSP
Done
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv@88 PS2, Line 88: blob for Whiskeylake SOC
FSP
Done
https://review.coreboot.org/#/c/32237/2/Documentation/libre-boards.csv@105 PS2, Line 105: blob for Denverton SOC
FSP
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... File Documentation/mainboard-blob-status.csv:
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 3: cavium/cn8100_sff_evb,?,cavium
Y afaik
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 5: google/cheza,?,Qualcomm
It will definitely be an N at the end, if Y/N is the only granularity here.
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 7: google/foster,?,Nvidia Tegra210
IIRC, you need the MTC for higher DDR speeds, or maybe DDR4? […]
Ack
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 8: google/gale,?,Qualcomm
Right.
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 9: google/gru,?,RockChip rk3399
Right.
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 10: google/kukui,?,Mediatek mt8183 soc
This will load a blob running on a co-processor (eventually), but sounds like that doesn't count for […]
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 11: google/mistral,?,Qualcomm
Definitely N.
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 12: google/nyan,?,Nvidia Tegra124 : google/nyan_big,?,Nvidia Tegra124 : google/nyan_blaze,?,Nvidia Tegra124
SDRAM code is in soc/nvidia/tegra124/sdram.c, that's all there is. So yeah, this should be Y.
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 15: google/oak,?,Mediatek mt8173 soc
Right, if only blobs loaded by coreboot count, this is Y. […]
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 16: google/storm,?,Qualcomm
Right.
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 17: google/veyron,?,RockChip rk3288 : google/veyron_mickey,?,RockChip rk3288 : google/veyron_rialto,?,RockChip rk3288
Right.
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 20: opencellular/elgon,?,cavium
Y afaik
Done
https://review.coreboot.org/#/c/32237/3/Documentation/mainboard-blob-status.... PS3, Line 21: sifive/hifive-unleashed,?,Sifive soc
Short answer: Y […]
Done
https://review.coreboot.org/#/c/32237/4/Documentation/mainboard-blob-status.... File Documentation/mainboard-blob-status.csv:
https://review.coreboot.org/#/c/32237/4/Documentation/mainboard-blob-status.... PS4, Line 185: lenovo/t400,Y,,
VGA BIOS optional (needed for working display) for all Lenovos that have AMD or NVidia GPU
Not sure how to put that into a board_info.txt conditional on the presence of a PCI card...