[coreboot-gerrit] Change in coreboot[master]: [Not For Merge] nb/intel/i945: Remove unused #define REFRESH_*

Arthur Heymans (Code Review) gerrit at coreboot.org
Mon Mar 27 22:02:15 CEST 2017


Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19016 )

Change subject: [Not For Merge] nb/intel/i945: Remove unused #define REFRESH_*
......................................................................


Patch Set 1: Code-Review-1

This is less clear.
What might not be a bad idea is to simply store maximal tRFC instead of this encoded refresh, and program this reg accordingly.

-- 
To view, visit https://review.coreboot.org/19016
To unsubscribe, visit https://review.coreboot.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I70d5437face1a3ff4cacda4cb3f9a67d2ca4c403
Gerrit-PatchSet: 1
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Owner: HAOUAS Elyes <ehaouas at noos.fr>
Gerrit-Reviewer: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h at gmx.de>
Gerrit-HasComments: No



More information about the coreboot-gerrit mailing list