Nico Huber uploaded patch set #3 to this change.

View Change

Increase range of our main Frequency_Type

Increase `Frequency_Type` to 1MHz .. 2.5Ghz. We used to adapt it
whenever necessary, e.g. when a new generation supported higher
frequencies. But that got a little annoying. So let's choose a
broader range that will hopefully lower the noise for some time.

1MHz is low enough for VGA resolution at 24p, and 2.5GHz is high
enough for 8K at 60p. That should give us some margin.

Some corner cases required refactoring of some calculations and
proofs. And, admittedly, the calculation in `HW.GFX.GMA.PCH.VGA`
can in theory overflow. We prove absence of runtime errors by
deferring the potential overflow to the modular `Word32` type.
This is not perfect, but overflow or not, we have to expect to
be called with a sane frequency anyway.

Change-Id: I3a159bc11ccadcaba8ad04e126e6feadfd46008e
Signed-off-by: Nico Huber <nico.h@gmx.de>
---
M common/hw-gfx-dp_info.adb
M common/hw-gfx-gma-pch-vga.adb
M common/hw-gfx.ads
3 files changed, 8 insertions(+), 25 deletions(-)

git pull ssh://review.coreboot.org:29418/libgfxinit refs/changes/50/35750/3

To view, visit change 35750. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libgfxinit
Gerrit-Branch: master
Gerrit-Change-Id: I3a159bc11ccadcaba8ad04e126e6feadfd46008e
Gerrit-Change-Number: 35750
Gerrit-PatchSet: 3
Gerrit-Owner: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur@aheymans.xyz>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-MessageType: newpatchset