>> (1 comment)
>
> regarding the datasheet, it is 16bits.

The register read was already 16 bits. That has nothing to do with
the variable you store the value in.

by the way, there is a conflict with this change :
https://review.coreboot.org/#/c/coreboot/+/17645/
so maybe I have to keep only changes related to pineview and i945.

Doesn't matter. 17645 needs an update anyway.

View Change

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