Angel Pons has uploaded this change for review.

View Change

sb/intel/lynxpoint: Program PM registers directly

Perform the same operations as the RCBA reg script did, but directly
writing the corresponding registers. Some of these operations could be
simplified, but it is not done on this commit to ease verification.

Change-Id: I4c3177ab14ca9bfa2e8d11c27fb249850183eee5
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
---
M src/southbridge/intel/lynxpoint/lpc.c
1 file changed, 45 insertions(+), 50 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/98/43098/1
diff --git a/src/southbridge/intel/lynxpoint/lpc.c b/src/southbridge/intel/lynxpoint/lpc.c
index f486434..26bd30f 100644
--- a/src/southbridge/intel/lynxpoint/lpc.c
+++ b/src/southbridge/intel/lynxpoint/lpc.c
@@ -276,55 +276,6 @@
printk(BIOS_DEBUG, "LynxPoint PM init\n");
}

-const struct rcba_config_instruction lpt_lp_pm_rcba[] = {
- RCBA_RMW_REG_32(0x232c, ~1, 0x00000000),
- RCBA_RMW_REG_32(0x1100, ~0xc000, 0xc000),
- RCBA_RMW_REG_32(0x1100, ~0, 0x00000100),
- RCBA_RMW_REG_32(0x1100, ~0, 0x0000003f),
- RCBA_RMW_REG_32(0x2320, ~0x60, 0x10),
- RCBA_RMW_REG_32(0x3314, 0, 0x00012fff),
- RCBA_RMW_REG_32(0x3318, 0, 0x0dcf0400),
- RCBA_RMW_REG_32(0x3324, 0, 0x04000000),
- RCBA_RMW_REG_32(0x3368, 0, 0x00041400),
- RCBA_RMW_REG_32(0x3388, 0, 0x3f8ddbff),
- RCBA_RMW_REG_32(0x33ac, 0, 0x00007001),
- RCBA_RMW_REG_32(0x33b0, 0, 0x00181900),
- RCBA_RMW_REG_32(0x33c0, 0, 0x00060A00),
- RCBA_RMW_REG_32(0x33d0, 0, 0x06200840),
- RCBA_RMW_REG_32(0x3a28, 0, 0x01010101),
- RCBA_RMW_REG_32(0x3a2c, 0, 0x04040404),
- RCBA_RMW_REG_32(0x2b1c, 0, 0x03808033),
- RCBA_RMW_REG_32(0x2b34, 0, 0x80000009),
- RCBA_RMW_REG_32(0x3348, 0, 0x022ddfff),
- RCBA_RMW_REG_32(0x334c, 0, 0x00000001),
- RCBA_RMW_REG_32(0x3358, 0, 0x0001c000),
- RCBA_RMW_REG_32(0x3380, 0, 0x3f8ddbff),
- RCBA_RMW_REG_32(0x3384, 0, 0x0001c7e1),
- RCBA_RMW_REG_32(0x338c, 0, 0x0001c7e1),
- RCBA_RMW_REG_32(0x3398, 0, 0x0001c000),
- RCBA_RMW_REG_32(0x33a8, 0, 0x00181900),
- RCBA_RMW_REG_32(0x33dc, 0, 0x00080000),
- RCBA_RMW_REG_32(0x33e0, 0, 0x00000001),
- RCBA_RMW_REG_32(0x3a20, 0, 0x00000404),
- RCBA_RMW_REG_32(0x3a24, 0, 0x01010101),
- RCBA_RMW_REG_32(0x3a30, 0, 0x01010101),
- RCBA_RMW_REG_32(0x0410, ~0, 0x00000003),
- RCBA_RMW_REG_32(0x2618, ~0, 0x08000000),
- RCBA_RMW_REG_32(0x2300, ~0, 0x00000002),
- RCBA_RMW_REG_32(0x2600, ~0, 0x00000008),
- RCBA_RMW_REG_32(0x33b4, 0, 0x00007001),
- RCBA_RMW_REG_32(0x3350, 0, 0x022ddfff),
- RCBA_RMW_REG_32(0x3354, 0, 0x00000001),
- RCBA_RMW_REG_32(0x33d4, ~0, 0x08000000), /* Power Optimizer */
- RCBA_RMW_REG_32(0x33c8, ~0, 0x00000080), /* Power Optimizer */
- RCBA_RMW_REG_32(0x2b10, 0, 0x0000883c), /* Power Optimizer */
- RCBA_RMW_REG_32(0x2b14, 0, 0x1e0a4616), /* Power Optimizer */
- RCBA_RMW_REG_32(0x2b24, 0, 0x40000005), /* Power Optimizer */
- RCBA_RMW_REG_32(0x2b20, 0, 0x0005db01), /* Power Optimizer */
- RCBA_RMW_REG_32(0x3a80, 0, 0x05145005),
- RCBA_END_CONFIG
-};
-
/* LynxPoint LP PCH Power Management init */
static void lpt_lp_pm_init(struct device *dev)
{
@@ -335,7 +286,51 @@

pci_write_config8(dev, 0xa9, 0x46);

- pch_config_rcba(lpt_lp_pm_rcba);
+ RCBA32_AND_OR(0x232c, ~1, 0x00000000);
+ RCBA32_AND_OR(0x1100, ~0xc000, 0xc000);
+ RCBA32_AND_OR(0x1100, ~0, 0x00000100);
+ RCBA32_AND_OR(0x1100, ~0, 0x0000003f);
+ RCBA32_AND_OR(0x2320, ~0x60, 0x10);
+ RCBA32_AND_OR(0x3314, 0, 0x00012fff);
+ RCBA32_AND_OR(0x3318, 0, 0x0dcf0400);
+ RCBA32_AND_OR(0x3324, 0, 0x04000000);
+ RCBA32_AND_OR(0x3368, 0, 0x00041400);
+ RCBA32_AND_OR(0x3388, 0, 0x3f8ddbff);
+ RCBA32_AND_OR(0x33ac, 0, 0x00007001);
+ RCBA32_AND_OR(0x33b0, 0, 0x00181900);
+ RCBA32_AND_OR(0x33c0, 0, 0x00060A00);
+ RCBA32_AND_OR(0x33d0, 0, 0x06200840);
+ RCBA32_AND_OR(0x3a28, 0, 0x01010101);
+ RCBA32_AND_OR(0x3a2c, 0, 0x04040404);
+ RCBA32_AND_OR(0x2b1c, 0, 0x03808033);
+ RCBA32_AND_OR(0x2b34, 0, 0x80000009);
+ RCBA32_AND_OR(0x3348, 0, 0x022ddfff);
+ RCBA32_AND_OR(0x334c, 0, 0x00000001);
+ RCBA32_AND_OR(0x3358, 0, 0x0001c000);
+ RCBA32_AND_OR(0x3380, 0, 0x3f8ddbff);
+ RCBA32_AND_OR(0x3384, 0, 0x0001c7e1);
+ RCBA32_AND_OR(0x338c, 0, 0x0001c7e1);
+ RCBA32_AND_OR(0x3398, 0, 0x0001c000);
+ RCBA32_AND_OR(0x33a8, 0, 0x00181900);
+ RCBA32_AND_OR(0x33dc, 0, 0x00080000);
+ RCBA32_AND_OR(0x33e0, 0, 0x00000001);
+ RCBA32_AND_OR(0x3a20, 0, 0x00000404);
+ RCBA32_AND_OR(0x3a24, 0, 0x01010101);
+ RCBA32_AND_OR(0x3a30, 0, 0x01010101);
+ RCBA32_AND_OR(0x0410, ~0, 0x00000003);
+ RCBA32_AND_OR(0x2618, ~0, 0x08000000);
+ RCBA32_AND_OR(0x2300, ~0, 0x00000002);
+ RCBA32_AND_OR(0x2600, ~0, 0x00000008);
+ RCBA32_AND_OR(0x33b4, 0, 0x00007001);
+ RCBA32_AND_OR(0x3350, 0, 0x022ddfff);
+ RCBA32_AND_OR(0x3354, 0, 0x00000001);
+ RCBA32_AND_OR(0x33d4, ~0, 0x08000000); /* Power Optimizer */
+ RCBA32_AND_OR(0x33c8, ~0, 0x00000080); /* Power Optimizer */
+ RCBA32_AND_OR(0x2b10, 0, 0x0000883c); /* Power Optimizer */
+ RCBA32_AND_OR(0x2b14, 0, 0x1e0a4616); /* Power Optimizer */
+ RCBA32_AND_OR(0x2b24, 0, 0x40000005); /* Power Optimizer */
+ RCBA32_AND_OR(0x2b20, 0, 0x0005db01); /* Power Optimizer */
+ RCBA32_AND_OR(0x3a80, 0, 0x05145005);

pci_write_config32(dev, 0xac,
pci_read_config32(dev, 0xac) | (1 << 21));

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

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I4c3177ab14ca9bfa2e8d11c27fb249850183eee5
Gerrit-Change-Number: 43098
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org>
Gerrit-MessageType: newchange