Attention is currently required from: Hung-Te Lin, Jarried Lin.
Yu-Ping Wu has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/85717?usp=email )
Change subject: mb/google/rauru: Add support for getting storage id ......................................................................
Patch Set 1:
(3 comments)
File src/mainboard/google/rauru/boardid.c:
https://review.coreboot.org/c/coreboot/+/85717/comment/a7801f49_fbb8be88?usp... : PS1, Line 16: STORAGE_ID_HIGH_CHANNEL Do we need this if not used?
https://review.coreboot.org/c/coreboot/+/85717/comment/ee2eb5b8_d89f0505?usp... : PS1, Line 57: panel storage
File src/mainboard/google/rauru/storage.h:
https://review.coreboot.org/c/coreboot/+/85717/comment/8fea3406_75d39c83?usp... : PS1, Line 5:
Can we add another function `storage_type()` to return that enum? I think the UFS driver in mtk-fsp will need it, so that the mtk-fsp repo will have the same enum definition.
`storage_id()` can still return the ADC index.