[PATCH] nvidiafb: Reduce stack usage

Reduce stack usage of NVCommonSetup()

Signed-off-by: Antonino Daplas <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>

1 files changed, 23 insertions, 11 deletions

diff --git a/drivers/video/nvidia/nv_setup.c b/drivers/video/nvidia/nv_setup.c
index 2c024523c622..a18a9aebf05f 100644
--- a/drivers/video/nvidia/nv_setup.c
+++ b/drivers/video/nvidia/nv_setup.c
@@ -290,22 +290,29 @@ static void nv10GetConfig(struct nvidia_par *par)
         par->MaxVClockFreqKHz = par->twoStagePLL ? 400000 : 350000;
 }
 
-void NVCommonSetup(struct fb_info *info)
+int NVCommonSetup(struct fb_info *info)
 {
         struct nvidia_par *par = info->par;
-        struct fb_var_screeninfo var;
+        struct fb_var_screeninfo *var;
         u16 implementation = par->Chipset & 0x0ff0;
         u8 *edidA = NULL, *edidB = NULL;
-        struct fb_monspecs monitorA, monitorB;
+        struct fb_monspecs *monitorA, *monitorB;
         struct fb_monspecs *monA = NULL, *monB = NULL;
         int mobile = 0;
         int tvA = 0;
         int tvB = 0;
         int FlatPanel = -1;     /* really means the CRTC is slaved */
         int Television = 0;
+        int err = 0;
 
-        memset(&monitorA, 0, sizeof(struct fb_monspecs));
-        memset(&monitorB, 0, sizeof(struct fb_monspecs));
+        var = kzalloc(sizeof(struct fb_var_screeninfo), GFP_KERNEL);
+        monitorA = kzalloc(sizeof(struct fb_monspecs), GFP_KERNEL);
+        monitorB = kzalloc(sizeof(struct fb_monspecs), GFP_KERNEL);
+
+        if (!var || !monitorA || !monitorB) {
+                err = -ENOMEM;
+                goto done;
+        }
 
         par->PRAMIN = par->REGS + (0x00710000 / 4);
         par->PCRTC0 = par->REGS + (0x00600000 / 4);
@@ -407,9 +414,9 @@ void NVCommonSetup(struct fb_info *info)
                 par->CRTCnumber = 0;
                 if (nvidia_probe_i2c_connector(info, 1, &edidA))
                         nvidia_probe_of_connector(info, 1, &edidA);
-                if (edidA && !fb_parse_edid(edidA, &var)) {
+                if (edidA && !fb_parse_edid(edidA, var)) {
                         printk("nvidiafb: EDID found from BUS1\n");
-                        monA = &monitorA;
+                        monA = monitorA;
                         fb_edid_to_monspecs(edidA, monA);
                         FlatPanel = (monA->input & FB_DISP_DDI) ? 1 : 0;
 
@@ -495,17 +502,17 @@ void NVCommonSetup(struct fb_info *info)
 
                 if (nvidia_probe_i2c_connector(info, 1, &edidA))
                         nvidia_probe_of_connector(info, 1, &edidA);
-                if (edidA && !fb_parse_edid(edidA, &var)) {
+                if (edidA && !fb_parse_edid(edidA, var)) {
                         printk("nvidiafb: EDID found from BUS1\n");
-                        monA = &monitorA;
+                        monA = monitorA;
                         fb_edid_to_monspecs(edidA, monA);
                 }
 
                 if (nvidia_probe_i2c_connector(info, 2, &edidB))
                         nvidia_probe_of_connector(info, 2, &edidB);
-                if (edidB && !fb_parse_edid(edidB, &var)) {
+                if (edidB && !fb_parse_edid(edidB, var)) {
                         printk("nvidiafb: EDID found from BUS2\n");
-                        monB = &monitorB;
+                        monB = monitorB;
                         fb_edid_to_monspecs(edidB, monB);
                 }
 
@@ -640,4 +647,9 @@ void NVCommonSetup(struct fb_info *info)
 
         kfree(edidA);
         kfree(edidB);
+done:
+        kfree(var);
+        kfree(monitorA);
+        kfree(monitorB);
+        return err;
 }