Attention is currently required from: Erik van den Bogaert, Felix Singer, Frans Hendriks, Maxim Polyakov.
Harrie Paijmans has posted comments on this change by Harrie Paijmans. ( https://review.coreboot.org/c/coreboot/+/87198?usp=email )
Change subject: superio/fintek: Add support for f81966d
......................................................................
Patch Set 4:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/87198/comment/f8fddc09_ed496ca4?usp... :
PS2, Line 16: TEST
Please add information about this in the commit message TEST=.... […]
changed to verified with 'superiotool -de'
File src/superio/fintek/f81966d/f81966d_hwm.c:
https://review.coreboot.org/c/coreboot/+/87198/comment/e3439610_c3c6e9b6?usp... :
PS3, Line 27: pnp_enter_conf_mode(dev);
There is no need to set the configuration mode for HWM, doesn't break initialization, it's just not […]
Yes the indexed writes don't need unlocking. So can be safely removed.
Verified the change with 'superiotool -de' and writing non default 'hwm_amd_tsi_addr' functions as expected.
--
To view, visit
https://review.coreboot.org/c/coreboot/+/87198?usp=email
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ibe3987b6e15eb07b92d7f5a7de2bd511de85e2f7
Gerrit-Change-Number: 87198
Gerrit-PatchSet: 4
Gerrit-Owner: Harrie Paijmans
hpaijmans@eltan.com
Gerrit-Reviewer: Erik van den Bogaert
ebogaert@eltan.com
Gerrit-Reviewer: Felix Singer
service+coreboot-gerrit@felixsinger.de
Gerrit-Reviewer: Frans Hendriks
fhendriks@eltan.com
Gerrit-Reviewer: Maxim Polyakov
max.senia.poliak@gmail.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Attention: Erik van den Bogaert
ebogaert@eltan.com
Gerrit-Attention: Felix Singer
service+coreboot-gerrit@felixsinger.de
Gerrit-Attention: Frans Hendriks
fhendriks@eltan.com
Gerrit-Attention: Maxim Polyakov
max.senia.poliak@gmail.com
Gerrit-Comment-Date: Fri, 11 Apr 2025 08:59:59 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Maxim Polyakov
max.senia.poliak@gmail.com
Comment-In-Reply-To: Harrie Paijmans
hpaijmans@eltan.com