6 comments:
This code was intended for
merlinfalcon version, but as there are some legal issues (documentation)
blocking its merge to coreboot, it'll be released for 00670F00,
For a while I physically had both versions of padmelon. […]
Why not simply say this is for the 670F00 version? As I mentioned before, just describe what this patch does.
Partially from Marc's work and partially from AMD's gardenia.
Please note it somewhere. Preferably in the commit message.
Patch Set #7, Line 15: actual
filler word. doesn't add any information here.
Patch Set #7, Line 15: a SIO, fintek f81803a
through a Fintek F81803A SIO.
Patch Set #7, Line 18: Both versions tested and boot to Linux using SeaBIOS
I'm confused. Does this support all versions of Family 15h Models 60h-6Fh? Just that the blobs aren't present? If that's the case, then reword the body of the commit message to remove all the stuff about "This code was intended for merlinfalcon version, but as there are some legal issues (documentation) blocking its merge to coreboot, it'll be released for 00670F00, being replaced later with merlinfalcon when the binaries can be merged to coreboot (PSP, video and AGESA)."
File src/mainboard/amd/padmelon/gpio.c:
onst struct soc_amd_gpio gpio_set_stage_reset[] = {
/* NFC PU */
PAD_GPO(GPIO_64, HIGH),
/* PCIe presence detect */
PAD_GPO(GPIO_64, HIGH),
/* MUX for Power Express Eval */
PAD_GPI(GPIO_116, PULL_DOWN),
/* SD power */
PAD_GPO(GPIO_64, HIGH),
};
const struct soc_amd_gpio gpio_set_stage_ram[] = {
/* BT radio disable */
PAD_GPO(GPIO_14, HIGH),
/* NFC wake */
PAD_GPO(GPIO_65, HIGH),
/* Webcam */
PAD_GPO(GPIO_66, HIGH),
/* GPS sleep */
PAD_GPO(GPIO_70, HIGH),
}
As good as I could make based on Marc's work. I have not double check against schematic though.
Please update correctly, or note that this patch is work in progress.
To view, visit change 33993. To unsubscribe, or for help writing mail filters, visit settings.