[coreboot-gerrit] Change in coreboot[master]: mb/asrock/g41m-vs3-r2: Allow to set the fixed duty cycle from rtc nvram

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/28714 )

Change subject: mb/asrock/g41m-vs3-r2: Allow to set the fixed duty cycle from rtc nvram
......................................................................


Patch Set 3:

This patch adds superio-specific things to the mainboard directly; I'd strongly prefer if the code was added to the superio support. Have a look at the w83697hf or it8718f code and see how that's done there


-- 
To view, visit https://review.coreboot.org/28714
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifb5bbb578182e88f4f7ba4e0e47404241b1a1962
Gerrit-Change-Number: 28714
Gerrit-PatchSet: 3
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Angel Pons <th3fanbus at gmail.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-CC: Felix Held <felix-coreboot at felixheld.de>
Gerrit-Comment-Date: Sat, 13 Oct 2018 12:35:46 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181013/809b2ddc/attachment.html>