x86: sysreset: Set up LPC only after relocation
authorSimon Glass <sjg@chromium.org>
Tue, 9 May 2023 10:13:47 +0000 (18:13 +0800)
committerBin Meng <bmeng@tinylab.org>
Thu, 11 May 2023 02:25:29 +0000 (10:25 +0800)
Probing LPC can cause PCI enumeration to take place, which significantly
increases pre-relocation memory usage. Also, LPC is somtimes enabled
directly by SPL.

Adjust the logic to probe the LPC only after relocation. This allows
chromebook_link64 to start up without a much larger
CONFIG_SYS_MALLOC_F_LEN value.

Signed-off-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Bin Meng <bmeng.cn@gmail.com>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
drivers/sysreset/sysreset_x86.c

index 8042f39..4936fdb 100644 (file)
@@ -129,8 +129,13 @@ static int x86_sysreset_probe(struct udevice *dev)
 {
        struct x86_sysreset_plat *plat = dev_get_plat(dev);
 
-       /* Locate the PCH if there is one. It isn't essential */
-       uclass_first_device(UCLASS_PCH, &plat->pch);
+       /*
+        * Locate the PCH if there is one. It isn't essential. Avoid this before
+        * relocation as we shouldn't need reset then and it needs a lot of
+        * memory for PCI enumeration.
+        */
+       if (gd->flags & GD_FLG_RELOC)
+               uclass_first_device(UCLASS_PCH, &plat->pch);
 
        return 0;
 }