Attention is currently required from: Angel Pons.
Subrata Banik has posted comments on this change by Subrata Banik. ( https://review.coreboot.org/c/coreboot/+/83519?usp=email )
Change subject: device/pci_ids: Remove unused Intel UFS device IDs ......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/83519/comment/f2d61c97_372f24f7?usp... : PS1, Line 10: devices from `pci_ids.h` as they appear to be unused in the codebase.
I know UFS on ADL is "meh": IIRC, it's only POR on some OSes (Linux, but not Windows). Do you know if things have changed w.r.t. UFS and POR?
good question, bt I can only comment for CrOS where UFS is POR but the UFS communication is only limited to payload. Additionally, we don't have any PCI driver need for UFS in coreboot hence those PCI IDs for UFS may not relevant for us.
-
Subrata Banik (Code Review)