Duncan Laurie has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40278 )
Change subject: mb/purism/librem_whl: Add new board Librem Mini (WHL-U) ......................................................................
Patch Set 10:
(1 comment)
Patch Set 9:
Patch Set 8: Code-Review+1
(2 comments)
This looks like a nice little device, do you happen to know if it can be powered from the type-c port?
sadly it's just a USB 3.1 Gen1 port with a type-C connector, doesn't offer any extra USB-C functionality (DP alt mode, TB, USB-PD). The NRE cost to add that was too high for the initial release
Makes sense, the power limitations of Type-C are a pain when you have a higher power CPU and no battery anyway.
https://review.coreboot.org/c/coreboot/+/40278/8/src/mainboard/purism/librem... File src/mainboard/purism/librem_whl/devicetree.cb:
https://review.coreboot.org/c/coreboot/+/40278/8/src/mainboard/purism/librem... PS8, Line 136: AcousticNoiseMitigation
good note. […]
These end up going to the VR over mailbox and I don't think there is any way to read it back out after boot unfortunately.
You could try copying some settings from the sarien board and see if affects the noise, the settings there are pretty conservative (i.e. only doing Fast/8 and not Fast/16)
You know you've gone too far when you get display flicker...
-
Duncan Laurie (Code Review)