[coreboot-gerrit] Change in coreboot[master]: drivers/genesyslogic/gl9763e: Add driver for Genesys Logic GL9763E

[Ben Chuang (Code Review)]

Ben Chuang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43751 )

Change subject: drivers/genesyslogic/gl9763e: Add driver for Genesys Logic GL9763E
......................................................................

Patch Set 6:

(1 comment)
> Patch Set 6:
>
> (1 comment)
>
> What platform is this being targeted to?

A variant which is based on Volteer2.

https://review.coreboot.org/c/coreboot/+/43751/6/src/drivers/genesyslogic/gl... 
File src/drivers/genesyslogic/gl9763e/gl9763e.c:

https://review.coreboot.org/c/coreboot/+/43751/6/src/drivers/genesyslogic/gl... 
PS6, Line 23: 	pci_update_config32(dev, VHS, ~VHS_REV_MASK, VHS_REV_R);
> Where are the ASPM controls? And what does each of these lines do?

ASPM control is on Line 20.

L:18, set VHS to be writable.
L:19, set single AXI request.
L:20, L0s support is disabled.
L:21, set SSC to 30000 ppm.
L:22, enable SSC.
L:23, set VHS to read-only.