[LinuxBIOS] [PATCH] Make GX1 video memory size configurable

[Juergen Beisert]

From: Juergen Beisert

This patch makes the reserved video memory on Geode GX1 based systems
configurable. This makes sense on systems with small memories when the VGA
feature is not used (CONFIG_VIDEO_MB = 0 in this case).
On Geode GX1 based systems the following amount of memory should be reserved
when VGA support is enabled:
 - 1MiB for VGA and SVGA resolutions
 - 2MiB for XGA resolution
 - 4MiB for SXGA resolution

Patch is against LinuxBIOSv2, revision of 2007-10-06.

Signed-off-by: Juergen Beisert

northbridge.c |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

---

Index: LinuxBIOSv2/src/northbridge/amd/gx1/northbridge.c
===================================================================
--- LinuxBIOSv2.orig/src/northbridge/amd/gx1/northbridge.c
+++ LinuxBIOSv2/src/northbridge/amd/gx1/northbridge.c
@@ -125,8 +125,6 @@ static uint32_t find_pci_tolm(struct bus
    return tolm;
 }

-#define FRAMEBUFFERK 4096
-
 static void pci_domain_set_resources(device_t dev)
 {
    device_t mc_dev;
@@ -153,7 +151,7 @@ static void pci_domain_set_resources(dev
    	tomk = ramreg << 10;

/* Sort out the framebuffer size */
-		tomk -= FRAMEBUFFERK;
+		tomk -= CONFIG_VIDEO_MB * 1024;
    	*bcdramtop = ((tomk << 10) - 1);
    	*mcgbaseadd = (tomk >> 9);

[Peter Stuge]

On Sat, Oct 06, 2007 at 10:12:37PM +0200, Juergen Beisert wrote:
> tomk -= FRAMEBUFFERK;
> tomk -= CONFIG_VIDEO_MB * 1024;

Does this option already exist? If so:

Acked-by: Peter Stuge

[Juergen Beisert]

On Sunday 07 October 2007 12:43, Peter Stuge wrote:
> On Sat, Oct 06, 2007 at 10:12:37PM +0200, Juergen Beisert wrote:
> > tomk -= FRAMEBUFFERK;
> > tomk -= CONFIG_VIDEO_MB * 1024;
>
> Does this option already exist? If so:

take a look into src/config/Options.lb, line 1013:

[...]
define CONFIG_VIDEO_MB
        default none
        export used
        comment "Integrated graphics with UMA has dynamic setup"
end
[...]

Or is this option to be used in a different way?

Juergen

[Uwe Hermann]

On Sun, Oct 07, 2007 at 02:28:49PM +0200, Juergen Beisert wrote:
> > Does this option already exist? If so:
>
> take a look into src/config/Options.lb, line 1013:
>
> [...]
> define CONFIG_VIDEO_MB
>         default none
>         export used
>         comment "Integrated graphics with UMA has dynamic setup"
> end
> [...]
>
> Or is this option to be used in a different way?

I don't think so.

r2840.

Uwe.