Thomas has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48769 )
Change subject: mb/yanling: Add Yanling YL-KBR6L mainboard + doc ......................................................................
Patch Set 1:
(1 comment)
Patch Set 1:
I have got some new information. Protectli is going to ship these platforms as their product. I got the hardware with 8550U CPU and will help to fix remaining things. We will have to change the platform name and vendor accordingly.
Hey Michal,
That is good news. How are we going to proceed? Are you taking over? Should I change the platform name and vendor? The new models as variants in vault_kbl or create a new one, e.g. vault_kbr?
Or abandon this patch and you provide your fixed one?
https://review.coreboot.org/c/coreboot/+/48769/1/Documentation/mainboard/yan... File Documentation/mainboard/yanling/yl-kbr6l.md:
https://review.coreboot.org/c/coreboot/+/48769/1/Documentation/mainboard/yan... PS1, Line 3: Protectli FW6
Your right, I'll do that.
Maybe to clarify: link to the coreboot doc page or to their homepage? And if to their coreboot doc page: I don't know if relative links work so I would have to add an absolute link... which will break if the directory structure changes.