viafb: improve pitch handling
[safe/jmp/linux-2.6] / drivers / video / via / viafbdev.c
index ba8ce61..e9c9f0e 100644 (file)
  */
 
 #include <linux/module.h>
+#include <linux/seq_file.h>
 #define _MASTER_FILE
 
 #include "global.h"
 
-static int MAX_CURS = 32;
 static struct fb_var_screeninfo default_var;
 static char *viafb_name = "Via";
 static u32 pseudo_pal[17];
@@ -32,14 +32,10 @@ static u32 pseudo_pal[17];
 /* video mode */
 static char *viafb_mode = "640x480";
 static char *viafb_mode1 = "640x480";
-static int viafb_resMode = VIA_RES_640X480;
 
 /* Added for specifying active devices.*/
 char *viafb_active_dev = "";
 
-/* Added for specifying video on devices.*/
-char *viafb_video_dev = "";
-
 /*Added for specify lcd output port*/
 char *viafb_lcd_port = "";
 char *viafb_dvi_port = "";
@@ -51,71 +47,20 @@ static void apply_second_mode_setting(struct fb_var_screeninfo
        *sec_var);
 static void retrieve_device_setting(struct viafb_ioctl_setting
        *setting_info);
-static void viafb_set_video_device(u32 video_dev_info);
-static void viafb_get_video_device(u32 *video_dev_info);
-
-/* Mode information */
-static const struct viafb_modeinfo viafb_modentry[] = {
-       {480, 640, VIA_RES_480X640, "480x640"},
-       {640, 480, VIA_RES_640X480, "640x480"},
-       {800, 480, VIA_RES_800X480, "800x480"},
-       {800, 600, VIA_RES_800X600, "800x600"},
-       {1024, 768, VIA_RES_1024X768, "1024x768"},
-       {1152, 864, VIA_RES_1152X864, "1152x864"},
-       {1280, 1024, VIA_RES_1280X1024, "1280x1024"},
-       {1600, 1200, VIA_RES_1600X1200, "1600x1200"},
-       {1440, 1050, VIA_RES_1440X1050, "1440x1050"},
-       {1280, 768, VIA_RES_1280X768, "1280x768"},
-       {1280, 800, VIA_RES_1280X800, "1280x800"},
-       {1280, 960, VIA_RES_1280X960, "1280x960"},
-       {1920, 1440, VIA_RES_1920X1440, "1920x1440"},
-       {848, 480, VIA_RES_848X480, "848x480"},
-       {1400, 1050, VIA_RES_1400X1050, "1400x1050"},
-       {720, 480, VIA_RES_720X480, "720x480"},
-       {720, 576, VIA_RES_720X576, "720x576"},
-       {1024, 512, VIA_RES_1024X512, "1024x512"},
-       {1024, 576, VIA_RES_1024X576, "1024x576"},
-       {1024, 600, VIA_RES_1024X600, "1024x600"},
-       {1280, 720, VIA_RES_1280X720, "1280x720"},
-       {1920, 1080, VIA_RES_1920X1080, "1920x1080"},
-       {1366, 768, VIA_RES_1368X768, "1368x768"},
-       {1680, 1050, VIA_RES_1680X1050, "1680x1050"},
-       {960, 600, VIA_RES_960X600, "960x600"},
-       {1000, 600, VIA_RES_1000X600, "1000x600"},
-       {1024, 576, VIA_RES_1024X576, "1024x576"},
-       {1024, 600, VIA_RES_1024X600, "1024x600"},
-       {1088, 612, VIA_RES_1088X612, "1088x612"},
-       {1152, 720, VIA_RES_1152X720, "1152x720"},
-       {1200, 720, VIA_RES_1200X720, "1200x720"},
-       {1280, 600, VIA_RES_1280X600, "1280x600"},
-       {1360, 768, VIA_RES_1360X768, "1360x768"},
-       {1440, 900, VIA_RES_1440X900, "1440x900"},
-       {1600, 900, VIA_RES_1600X900, "1600x900"},
-       {1600, 1024, VIA_RES_1600X1024, "1600x1024"},
-       {1792, 1344, VIA_RES_1792X1344, "1792x1344"},
-       {1856, 1392, VIA_RES_1856X1392, "1856x1392"},
-       {1920, 1200, VIA_RES_1920X1200, "1920x1200"},
-       {2048, 1536, VIA_RES_2048X1536, "2048x1536"},
-       {0, 0, VIA_RES_INVALID, "640x480"}
-};
 
 static struct fb_ops viafb_ops;
 
-static int viafb_update_fix(struct fb_fix_screeninfo *fix, struct fb_info *info)
-{
-       struct viafb_par *ppar;
-       ppar = info->par;
 
-       DEBUG_MSG(KERN_INFO "viafb_update_fix!\n");
-
-       fix->visual =
-           ppar->bpp == 8 ? FB_VISUAL_PSEUDOCOLOR : FB_VISUAL_TRUECOLOR;
-       fix->line_length = ppar->linelength;
+static void viafb_update_fix(struct fb_info *info)
+{
+       u32 bpp = info->var.bits_per_pixel;
 
-       return 0;
+       info->fix.visual =
+               bpp == 8 ? FB_VISUAL_PSEUDOCOLOR : FB_VISUAL_TRUECOLOR;
+       info->fix.line_length =
+               ((info->var.xres_virtual + 7) & ~7) * bpp / 8;
 }
 
-
 static void viafb_setup_fixinfo(struct fb_fix_screeninfo *fix,
        struct viafb_par *viaparinfo)
 {
@@ -124,8 +69,6 @@ static void viafb_setup_fixinfo(struct fb_fix_screeninfo *fix,
 
        fix->smem_start = viaparinfo->fbmem;
        fix->smem_len = viaparinfo->fbmem_free;
-       fix->mmio_start = viaparinfo->mmio_base;
-       fix->mmio_len = viaparinfo->mmio_len;
 
        fix->type = FB_TYPE_PACKED_PIXELS;
        fix->type_aux = 0;
@@ -148,28 +91,12 @@ static int viafb_release(struct fb_info *info, int user)
        return 0;
 }
 
-static void viafb_update_viafb_par(struct fb_info *info)
-{
-       struct viafb_par *ppar;
-
-       ppar = info->par;
-       ppar->bpp = info->var.bits_per_pixel;
-       ppar->linelength = ((info->var.xres_virtual + 7) & ~7) * ppar->bpp / 8;
-       ppar->hres = info->var.xres;
-       ppar->vres = info->var.yres;
-       ppar->xoffset = info->var.xoffset;
-       ppar->yoffset = info->var.yoffset;
-}
-
 static int viafb_check_var(struct fb_var_screeninfo *var,
        struct fb_info *info)
 {
        int vmode_index, htotal, vtotal;
-       struct viafb_par *ppar;
+       struct viafb_par *ppar = info->par;
        u32 long_refresh;
-       struct viafb_par *p_viafb_par;
-       ppar = info->par;
-
 
        DEBUG_MSG(KERN_INFO "viafb_check_var!\n");
        /* Sanity check */
@@ -177,7 +104,7 @@ static int viafb_check_var(struct fb_var_screeninfo *var,
        if (var->vmode & FB_VMODE_INTERLACED || var->vmode & FB_VMODE_DOUBLE)
                return -EINVAL;
 
-       vmode_index = viafb_get_mode_index(var->xres, var->yres, 0);
+       vmode_index = viafb_get_mode_index(var->xres, var->yres);
        if (vmode_index == VIA_RES_INVALID) {
                DEBUG_MSG(KERN_INFO
                          "viafb: Mode %dx%dx%d not supported!!\n",
@@ -214,33 +141,28 @@ static int viafb_check_var(struct fb_var_screeninfo *var,
        /* Adjust var according to our driver's own table */
        viafb_fill_var_timing_info(var, viafb_refresh, vmode_index);
 
-       /* This is indeed a patch for VT3353 */
-       if (!info->par)
-               return -1;
-       p_viafb_par = (struct viafb_par *)info->par;
-       if (p_viafb_par->chip_info->gfx_chip_name == UNICHROME_VX800)
-               var->accel_flags = 0;
-
        return 0;
 }
 
 static int viafb_set_par(struct fb_info *info)
 {
+       struct viafb_par *viapar = info->par;
        int vmode_index;
        int vmode_index1 = 0;
        DEBUG_MSG(KERN_INFO "viafb_set_par!\n");
 
+       viapar->depth = fb_get_color_depth(&info->var, &info->fix);
        viafb_update_device_setting(info->var.xres, info->var.yres,
                              info->var.bits_per_pixel, viafb_refresh, 0);
 
-       vmode_index = viafb_get_mode_index(info->var.xres, info->var.yres, 0);
+       vmode_index = viafb_get_mode_index(info->var.xres, info->var.yres);
 
        if (viafb_SAMM_ON == 1) {
                DEBUG_MSG(KERN_INFO
                "viafb_second_xres = %d, viafb_second_yres = %d, bpp = %d\n",
                          viafb_second_xres, viafb_second_yres, viafb_bpp1);
                vmode_index1 = viafb_get_mode_index(viafb_second_xres,
-                       viafb_second_yres, 1);
+                       viafb_second_yres);
                DEBUG_MSG(KERN_INFO "->viafb_SAMM_ON: index=%d\n",
                        vmode_index1);
 
@@ -253,21 +175,10 @@ static int viafb_set_par(struct fb_info *info)
                        info->var.bits_per_pixel, vmode_index1,
                        viafb_second_xres, viafb_second_yres, viafb_bpp1);
 
-               /*We should set memory offset according virtual_x */
-               /*Fix me:put this function into viafb_setmode */
-               viafb_memory_pitch_patch(info);
-
-               /* Update ***fb_par information */
-               viafb_update_viafb_par(info);
-
-               /* Update other fixed information */
-               viafb_update_fix(&info->fix, info);
+               viafb_update_fix(info);
                viafb_bpp = info->var.bits_per_pixel;
                /* Update viafb_accel, it is necessary to our 2D accelerate */
                viafb_accel = info->var.accel_flags;
-
-               if (viafb_accel)
-                       viafb_set_2d_color_depth(info->var.bits_per_pixel);
        }
 
        return 0;
@@ -504,12 +415,7 @@ static int viafb_pan_display(struct fb_var_screeninfo *var,
            var->bits_per_pixel / 16;
 
        DEBUG_MSG(KERN_INFO "\nviafb_pan_display,offset =%d ", offset);
-
-       viafb_write_reg_mask(0x48, 0x3d4, ((offset >> 24) & 0x3), 0x3);
-       viafb_write_reg_mask(0x34, 0x3d4, ((offset >> 16) & 0xff), 0xff);
-       viafb_write_reg_mask(0x0c, 0x3d4, ((offset >> 8) & 0xff), 0xff);
-       viafb_write_reg_mask(0x0d, 0x3d4, (offset & 0xff), 0xff);
-
+       viafb_set_primary_address(offset);
        return 0;
 }
 
@@ -546,23 +452,24 @@ static int viafb_blank(int blank_mode, struct fb_info *info)
 
 static int viafb_ioctl(struct fb_info *info, u_int cmd, u_long arg)
 {
-       struct viafb_ioctl_mode viamode;
-       struct viafb_ioctl_samm viasamm;
-       struct viafb_driver_version driver_version;
-       struct fb_var_screeninfo sec_var;
-       struct _panel_size_pos_info panel_pos_size_para;
+       union {
+               struct viafb_ioctl_mode viamode;
+               struct viafb_ioctl_samm viasamm;
+               struct viafb_driver_version driver_version;
+               struct fb_var_screeninfo sec_var;
+               struct _panel_size_pos_info panel_pos_size_para;
+               struct viafb_ioctl_setting viafb_setting;
+               struct device_t active_dev;
+       } u;
        u32 state_info = 0;
-       u32 viainfo_size = sizeof(struct viafb_ioctl_info);
        u32 *viafb_gamma_table;
        char driver_name[] = "viafb";
 
        u32 __user *argp = (u32 __user *) arg;
        u32 gpu32;
-       u32 video_dev_info = 0;
-       struct viafb_ioctl_setting viafb_setting = {};
-       struct device_t active_dev = {};
 
        DEBUG_MSG(KERN_INFO "viafb_ioctl: 0x%X !!\n", cmd);
+       memset(&u, 0, sizeof(u));
 
        switch (cmd) {
        case VIAFB_GET_CHIP_INFO:
@@ -571,7 +478,7 @@ static int viafb_ioctl(struct fb_info *info, u_int cmd, u_long arg)
                        return -EFAULT;
                break;
        case VIAFB_GET_INFO_SIZE:
-               return put_user(viainfo_size, argp);
+               return put_user((u32)sizeof(struct viafb_ioctl_info), argp);
        case VIAFB_GET_INFO:
                return viafb_ioctl_get_viafb_info(arg);
        case VIAFB_HOTPLUG:
@@ -584,60 +491,60 @@ static int viafb_ioctl(struct fb_info *info, u_int cmd, u_long arg)
                viafb_hotplug = (gpu32) ? 1 : 0;
                break;
        case VIAFB_GET_RESOLUTION:
-               viamode.xres = (u32) viafb_hotplug_Xres;
-               viamode.yres = (u32) viafb_hotplug_Yres;
-               viamode.refresh = (u32) viafb_hotplug_refresh;
-               viamode.bpp = (u32) viafb_hotplug_bpp;
+               u.viamode.xres = (u32) viafb_hotplug_Xres;
+               u.viamode.yres = (u32) viafb_hotplug_Yres;
+               u.viamode.refresh = (u32) viafb_hotplug_refresh;
+               u.viamode.bpp = (u32) viafb_hotplug_bpp;
                if (viafb_SAMM_ON == 1) {
-                       viamode.xres_sec = viafb_second_xres;
-                       viamode.yres_sec = viafb_second_yres;
-                       viamode.virtual_xres_sec = viafb_second_virtual_xres;
-                       viamode.virtual_yres_sec = viafb_second_virtual_yres;
-                       viamode.refresh_sec = viafb_refresh1;
-                       viamode.bpp_sec = viafb_bpp1;
+                       u.viamode.xres_sec = viafb_second_xres;
+                       u.viamode.yres_sec = viafb_second_yres;
+                       u.viamode.virtual_xres_sec = viafb_second_virtual_xres;
+                       u.viamode.virtual_yres_sec = viafb_second_virtual_yres;
+                       u.viamode.refresh_sec = viafb_refresh1;
+                       u.viamode.bpp_sec = viafb_bpp1;
                } else {
-                       viamode.xres_sec = 0;
-                       viamode.yres_sec = 0;
-                       viamode.virtual_xres_sec = 0;
-                       viamode.virtual_yres_sec = 0;
-                       viamode.refresh_sec = 0;
-                       viamode.bpp_sec = 0;
+                       u.viamode.xres_sec = 0;
+                       u.viamode.yres_sec = 0;
+                       u.viamode.virtual_xres_sec = 0;
+                       u.viamode.virtual_yres_sec = 0;
+                       u.viamode.refresh_sec = 0;
+                       u.viamode.bpp_sec = 0;
                }
-               if (copy_to_user(argp, &viamode, sizeof(viamode)))
+               if (copy_to_user(argp, &u.viamode, sizeof(u.viamode)))
                        return -EFAULT;
                break;
        case VIAFB_GET_SAMM_INFO:
-               viasamm.samm_status = viafb_SAMM_ON;
+               u.viasamm.samm_status = viafb_SAMM_ON;
 
                if (viafb_SAMM_ON == 1) {
                        if (viafb_dual_fb) {
-                               viasamm.size_prim = viaparinfo->fbmem_free;
-                               viasamm.size_sec = viaparinfo1->fbmem_free;
+                               u.viasamm.size_prim = viaparinfo->fbmem_free;
+                               u.viasamm.size_sec = viaparinfo1->fbmem_free;
                        } else {
                                if (viafb_second_size) {
-                                       viasamm.size_prim =
+                                       u.viasamm.size_prim =
                                            viaparinfo->fbmem_free -
                                            viafb_second_size * 1024 * 1024;
-                                       viasamm.size_sec =
+                                       u.viasamm.size_sec =
                                            viafb_second_size * 1024 * 1024;
                                } else {
-                                       viasamm.size_prim =
+                                       u.viasamm.size_prim =
                                            viaparinfo->fbmem_free >> 1;
-                                       viasamm.size_sec =
+                                       u.viasamm.size_sec =
                                            (viaparinfo->fbmem_free >> 1);
                                }
                        }
-                       viasamm.mem_base = viaparinfo->fbmem;
-                       viasamm.offset_sec = viafb_second_offset;
+                       u.viasamm.mem_base = viaparinfo->fbmem;
+                       u.viasamm.offset_sec = viafb_second_offset;
                } else {
-                       viasamm.size_prim =
+                       u.viasamm.size_prim =
                            viaparinfo->memsize - viaparinfo->fbmem_used;
-                       viasamm.size_sec = 0;
-                       viasamm.mem_base = viaparinfo->fbmem;
-                       viasamm.offset_sec = 0;
+                       u.viasamm.size_sec = 0;
+                       u.viasamm.mem_base = viaparinfo->fbmem;
+                       u.viasamm.offset_sec = 0;
                }
 
-               if (copy_to_user(argp, &viasamm, sizeof(viasamm)))
+               if (copy_to_user(argp, &u.viasamm, sizeof(u.viasamm)))
                        return -EFAULT;
 
                break;
@@ -662,74 +569,75 @@ static int viafb_ioctl(struct fb_info *info, u_int cmd, u_long arg)
                        viafb_lcd_disable();
                break;
        case VIAFB_SET_DEVICE:
-               if (copy_from_user(&active_dev, (void *)argp,
-                       sizeof(active_dev)))
+               if (copy_from_user(&u.active_dev, (void *)argp,
+                       sizeof(u.active_dev)))
                        return -EFAULT;
-               viafb_set_device(active_dev);
+               viafb_set_device(u.active_dev);
                viafb_set_par(info);
                break;
        case VIAFB_GET_DEVICE:
-               active_dev.crt = viafb_CRT_ON;
-               active_dev.dvi = viafb_DVI_ON;
-               active_dev.lcd = viafb_LCD_ON;
-               active_dev.samm = viafb_SAMM_ON;
-               active_dev.primary_dev = viafb_primary_dev;
+               u.active_dev.crt = viafb_CRT_ON;
+               u.active_dev.dvi = viafb_DVI_ON;
+               u.active_dev.lcd = viafb_LCD_ON;
+               u.active_dev.samm = viafb_SAMM_ON;
+               u.active_dev.primary_dev = viafb_primary_dev;
 
-               active_dev.lcd_dsp_cent = viafb_lcd_dsp_method;
-               active_dev.lcd_panel_id = viafb_lcd_panel_id;
-               active_dev.lcd_mode = viafb_lcd_mode;
+               u.active_dev.lcd_dsp_cent = viafb_lcd_dsp_method;
+               u.active_dev.lcd_panel_id = viafb_lcd_panel_id;
+               u.active_dev.lcd_mode = viafb_lcd_mode;
 
-               active_dev.xres = viafb_hotplug_Xres;
-               active_dev.yres = viafb_hotplug_Yres;
+               u.active_dev.xres = viafb_hotplug_Xres;
+               u.active_dev.yres = viafb_hotplug_Yres;
 
-               active_dev.xres1 = viafb_second_xres;
-               active_dev.yres1 = viafb_second_yres;
+               u.active_dev.xres1 = viafb_second_xres;
+               u.active_dev.yres1 = viafb_second_yres;
 
-               active_dev.bpp = viafb_bpp;
-               active_dev.bpp1 = viafb_bpp1;
-               active_dev.refresh = viafb_refresh;
-               active_dev.refresh1 = viafb_refresh1;
+               u.active_dev.bpp = viafb_bpp;
+               u.active_dev.bpp1 = viafb_bpp1;
+               u.active_dev.refresh = viafb_refresh;
+               u.active_dev.refresh1 = viafb_refresh1;
 
-               active_dev.epia_dvi = viafb_platform_epia_dvi;
-               active_dev.lcd_dual_edge = viafb_device_lcd_dualedge;
-               active_dev.bus_width = viafb_bus_width;
+               u.active_dev.epia_dvi = viafb_platform_epia_dvi;
+               u.active_dev.lcd_dual_edge = viafb_device_lcd_dualedge;
+               u.active_dev.bus_width = viafb_bus_width;
 
-               if (copy_to_user(argp, &active_dev, sizeof(active_dev)))
+               if (copy_to_user(argp, &u.active_dev, sizeof(u.active_dev)))
                        return -EFAULT;
                break;
 
        case VIAFB_GET_DRIVER_VERSION:
-               driver_version.iMajorNum = VERSION_MAJOR;
-               driver_version.iKernelNum = VERSION_KERNEL;
-               driver_version.iOSNum = VERSION_OS;
-               driver_version.iMinorNum = VERSION_MINOR;
+               u.driver_version.iMajorNum = VERSION_MAJOR;
+               u.driver_version.iKernelNum = VERSION_KERNEL;
+               u.driver_version.iOSNum = VERSION_OS;
+               u.driver_version.iMinorNum = VERSION_MINOR;
 
-               if (copy_to_user(argp, &driver_version,
-                       sizeof(driver_version)))
+               if (copy_to_user(argp, &u.driver_version,
+                       sizeof(u.driver_version)))
                        return -EFAULT;
 
                break;
 
        case VIAFB_SET_DEVICE_INFO:
-               if (copy_from_user(&viafb_setting,
-                       argp, sizeof(viafb_setting)))
+               if (copy_from_user(&u.viafb_setting,
+                       argp, sizeof(u.viafb_setting)))
                        return -EFAULT;
-               if (apply_device_setting(viafb_setting, info) < 0)
+               if (apply_device_setting(u.viafb_setting, info) < 0)
                        return -EINVAL;
 
                break;
 
        case VIAFB_SET_SECOND_MODE:
-               if (copy_from_user(&sec_var, argp, sizeof(sec_var)))
+               if (copy_from_user(&u.sec_var, argp, sizeof(u.sec_var)))
                        return -EFAULT;
-               apply_second_mode_setting(&sec_var);
+               apply_second_mode_setting(&u.sec_var);
                break;
 
        case VIAFB_GET_DEVICE_INFO:
 
-               retrieve_device_setting(&viafb_setting);
+               retrieve_device_setting(&u.viafb_setting);
 
-               if (copy_to_user(argp, &viafb_setting, sizeof(viafb_setting)))
+               if (copy_to_user(argp, &u.viafb_setting,
+                                sizeof(u.viafb_setting)))
                        return -EFAULT;
 
                break;
@@ -790,15 +698,6 @@ static int viafb_ioctl(struct fb_info *info, u_int cmd, u_long arg)
                if (put_user(state_info, argp))
                        return -EFAULT;
                break;
-       case VIAFB_SET_VIDEO_DEVICE:
-               get_user(video_dev_info, argp);
-               viafb_set_video_device(video_dev_info);
-               break;
-       case VIAFB_GET_VIDEO_DEVICE:
-               viafb_get_video_device(&video_dev_info);
-               if (put_user(video_dev_info, argp))
-                       return -EFAULT;
-               break;
        case VIAFB_SYNC_SURFACE:
                DEBUG_MSG(KERN_INFO "lobo VIAFB_SYNC_SURFACE\n");
                break;
@@ -806,51 +705,51 @@ static int viafb_ioctl(struct fb_info *info, u_int cmd, u_long arg)
                break;
 
        case VIAFB_GET_PANEL_MAX_SIZE:
-               if (copy_from_user
-                   (&panel_pos_size_para, argp, sizeof(panel_pos_size_para)))
+               if (copy_from_user(&u.panel_pos_size_para, argp,
+                                  sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
-               panel_pos_size_para.x = panel_pos_size_para.y = 0;
-               if (copy_to_user(argp, &panel_pos_size_para,
-                    sizeof(panel_pos_size_para)))
+               u.panel_pos_size_para.x = u.panel_pos_size_para.y = 0;
+               if (copy_to_user(argp, &u.panel_pos_size_para,
+                    sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
                break;
        case VIAFB_GET_PANEL_MAX_POSITION:
-               if (copy_from_user
-                   (&panel_pos_size_para, argp, sizeof(panel_pos_size_para)))
+               if (copy_from_user(&u.panel_pos_size_para, argp,
+                                  sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
-               panel_pos_size_para.x = panel_pos_size_para.y = 0;
-               if (copy_to_user(argp, &panel_pos_size_para,
-                    sizeof(panel_pos_size_para)))
+               u.panel_pos_size_para.x = u.panel_pos_size_para.y = 0;
+               if (copy_to_user(argp, &u.panel_pos_size_para,
+                                sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
                break;
 
        case VIAFB_GET_PANEL_POSITION:
-               if (copy_from_user
-                   (&panel_pos_size_para, argp, sizeof(panel_pos_size_para)))
+               if (copy_from_user(&u.panel_pos_size_para, argp,
+                                  sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
-               panel_pos_size_para.x = panel_pos_size_para.y = 0;
-               if (copy_to_user(argp, &panel_pos_size_para,
-                    sizeof(panel_pos_size_para)))
+               u.panel_pos_size_para.x = u.panel_pos_size_para.y = 0;
+               if (copy_to_user(argp, &u.panel_pos_size_para,
+                                sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
                break;
        case VIAFB_GET_PANEL_SIZE:
-               if (copy_from_user
-                   (&panel_pos_size_para, argp, sizeof(panel_pos_size_para)))
+               if (copy_from_user(&u.panel_pos_size_para, argp,
+                                  sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
-               panel_pos_size_para.x = panel_pos_size_para.y = 0;
-               if (copy_to_user(argp, &panel_pos_size_para,
-                    sizeof(panel_pos_size_para)))
+               u.panel_pos_size_para.x = u.panel_pos_size_para.y = 0;
+               if (copy_to_user(argp, &u.panel_pos_size_para,
+                                sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
                break;
 
        case VIAFB_SET_PANEL_POSITION:
-               if (copy_from_user
-                   (&panel_pos_size_para, argp, sizeof(panel_pos_size_para)))
+               if (copy_from_user(&u.panel_pos_size_para, argp,
+                                  sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
                break;
        case VIAFB_SET_PANEL_SIZE:
-               if (copy_from_user
-                   (&panel_pos_size_para, argp, sizeof(panel_pos_size_para)))
+               if (copy_from_user(&u.panel_pos_size_para, argp,
+                                  sizeof(u.panel_pos_size_para)))
                        return -EFAULT;
                break;
 
@@ -864,372 +763,232 @@ static int viafb_ioctl(struct fb_info *info, u_int cmd, u_long arg)
 static void viafb_fillrect(struct fb_info *info,
        const struct fb_fillrect *rect)
 {
-       u32 col = 0, rop = 0;
-       int pitch;
+       struct viafb_par *viapar = info->par;
+       u32 fg_color;
+       u8 rop;
 
-       if (!viafb_accel)
-               return cfb_fillrect(info, rect);
+       if (!viapar->shared->hw_bitblt) {
+               cfb_fillrect(info, rect);
+               return;
+       }
 
        if (!rect->width || !rect->height)
                return;
 
-       switch (rect->rop) {
-       case ROP_XOR:
+       if (info->fix.visual == FB_VISUAL_TRUECOLOR)
+               fg_color = ((u32 *)info->pseudo_palette)[rect->color];
+       else
+               fg_color = rect->color;
+
+       if (rect->rop == ROP_XOR)
                rop = 0x5A;
-               break;
-       case ROP_COPY:
-       default:
+       else
                rop = 0xF0;
-               break;
-       }
-
-       switch (info->var.bits_per_pixel) {
-       case 8:
-               col = rect->color;
-               break;
-       case 16:
-               col = ((u32 *) (info->pseudo_palette))[rect->color];
-               break;
-       case 32:
-               col = ((u32 *) (info->pseudo_palette))[rect->color];
-               break;
-       }
-
-       /* BitBlt Source Address */
-       MMIO_OUT32(VIA_REG_SRCPOS, 0x0);
-       /* Source Base Address */
-       MMIO_OUT32(VIA_REG_SRCBASE, 0x0);
-       /* Destination Base Address */
-       /*MMIO_OUT32(VIA_REG_DSTBASE, 0x0); */
-       MMIO_OUT32(VIA_REG_DSTBASE,
-                  ((u32) (info->screen_base) - (u32) viafb_FB_MM) >> 3);
-       /* Pitch */
-       pitch = (info->var.xres_virtual + 7) & ~7;
-       MMIO_OUT32(VIA_REG_PITCH,
-                  VIA_PITCH_ENABLE |
-                  (((pitch *
-                     info->var.bits_per_pixel >> 3) >> 3) |
-                     (((pitch * info->
-                     var.bits_per_pixel >> 3) >> 3) << 16)));
-       /* BitBlt Destination Address */
-       MMIO_OUT32(VIA_REG_DSTPOS, ((rect->dy << 16) | rect->dx));
-       /* Dimension: width & height */
-       MMIO_OUT32(VIA_REG_DIMENSION,
-                  (((rect->height - 1) << 16) | (rect->width - 1)));
-       /* Forground color or Destination color */
-       MMIO_OUT32(VIA_REG_FGCOLOR, col);
-       /* GE Command */
-       MMIO_OUT32(VIA_REG_GECMD, (0x01 | 0x2000 | (rop << 24)));
 
+       DEBUG_MSG(KERN_DEBUG "viafb 2D engine: fillrect\n");
+       if (viapar->shared->hw_bitblt(viapar->io_virt, VIA_BITBLT_FILL,
+               rect->width, rect->height, info->var.bits_per_pixel,
+               viapar->vram_addr, info->fix.line_length, rect->dx, rect->dy,
+               NULL, 0, 0, 0, 0, fg_color, 0, rop))
+               cfb_fillrect(info, rect);
 }
 
 static void viafb_copyarea(struct fb_info *info,
        const struct fb_copyarea *area)
 {
-       u32 dy = area->dy, sy = area->sy, direction = 0x0;
-       u32 sx = area->sx, dx = area->dx, width = area->width;
-       int pitch;
-
-       DEBUG_MSG(KERN_INFO "viafb_copyarea!!\n");
-
-       if (!viafb_accel)
-               return cfb_copyarea(info, area);
+       struct viafb_par *viapar = info->par;
 
-       if (!area->width || !area->height)
+       if (!viapar->shared->hw_bitblt) {
+               cfb_copyarea(info, area);
                return;
-
-       if (sy < dy) {
-               dy += area->height - 1;
-               sy += area->height - 1;
-               direction |= 0x4000;
        }
 
-       if (sx < dx) {
-               dx += width - 1;
-               sx += width - 1;
-               direction |= 0x8000;
-       }
-
-       /* Source Base Address */
-       /*MMIO_OUT32(VIA_REG_SRCBASE, 0x0); */
-       MMIO_OUT32(VIA_REG_SRCBASE,
-                  ((u32) (info->screen_base) - (u32) viafb_FB_MM) >> 3);
-       /* Destination Base Address */
-       /*MMIO_OUT32(VIA_REG_DSTBASE, 0x0); */
-       MMIO_OUT32(VIA_REG_DSTBASE,
-                  ((u32) (info->screen_base) - (u32) viafb_FB_MM) >> 3);
-       /* Pitch */
-       pitch = (info->var.xres_virtual + 7) & ~7;
-       /* VIA_PITCH_ENABLE can be omitted now. */
-       MMIO_OUT32(VIA_REG_PITCH,
-                  VIA_PITCH_ENABLE |
-                  (((pitch *
-                     info->var.bits_per_pixel >> 3) >> 3) | (((pitch *
-                                                               info->var.
-                                                               bits_per_pixel
-                                                               >> 3) >> 3)
-                                                             << 16)));
-       /* BitBlt Source Address */
-       MMIO_OUT32(VIA_REG_SRCPOS, ((sy << 16) | sx));
-       /* BitBlt Destination Address */
-       MMIO_OUT32(VIA_REG_DSTPOS, ((dy << 16) | dx));
-       /* Dimension: width & height */
-       MMIO_OUT32(VIA_REG_DIMENSION,
-                  (((area->height - 1) << 16) | (area->width - 1)));
-       /* GE Command */
-       MMIO_OUT32(VIA_REG_GECMD, (0x01 | direction | (0xCC << 24)));
+       if (!area->width || !area->height)
+               return;
 
+       DEBUG_MSG(KERN_DEBUG "viafb 2D engine: copyarea\n");
+       if (viapar->shared->hw_bitblt(viapar->io_virt, VIA_BITBLT_COLOR,
+               area->width, area->height, info->var.bits_per_pixel,
+               viapar->vram_addr, info->fix.line_length, area->dx, area->dy,
+               NULL, viapar->vram_addr, info->fix.line_length,
+               area->sx, area->sy, 0, 0, 0))
+               cfb_copyarea(info, area);
 }
 
 static void viafb_imageblit(struct fb_info *info,
        const struct fb_image *image)
 {
-       u32 size, bg_col = 0, fg_col = 0, *udata;
-       int i;
-       int pitch;
-
-       if (!viafb_accel)
-               return cfb_imageblit(info, image);
+       struct viafb_par *viapar = info->par;
+       u32 fg_color = 0, bg_color = 0;
+       u8 op;
 
-       udata = (u32 *) image->data;
-
-       switch (info->var.bits_per_pixel) {
-       case 8:
-               bg_col = image->bg_color;
-               fg_col = image->fg_color;
-               break;
-       case 16:
-               bg_col = ((u32 *) (info->pseudo_palette))[image->bg_color];
-               fg_col = ((u32 *) (info->pseudo_palette))[image->fg_color];
-               break;
-       case 32:
-               bg_col = ((u32 *) (info->pseudo_palette))[image->bg_color];
-               fg_col = ((u32 *) (info->pseudo_palette))[image->fg_color];
-               break;
-       }
-       size = image->width * image->height;
-
-       /* Source Base Address */
-       MMIO_OUT32(VIA_REG_SRCBASE, 0x0);
-       /* Destination Base Address */
-       /*MMIO_OUT32(VIA_REG_DSTBASE, 0x0); */
-       MMIO_OUT32(VIA_REG_DSTBASE,
-                  ((u32) (info->screen_base) - (u32) viafb_FB_MM) >> 3);
-       /* Pitch */
-       pitch = (info->var.xres_virtual + 7) & ~7;
-       MMIO_OUT32(VIA_REG_PITCH,
-                  VIA_PITCH_ENABLE |
-                  (((pitch *
-                     info->var.bits_per_pixel >> 3) >> 3) | (((pitch *
-                                                               info->var.
-                                                               bits_per_pixel
-                                                               >> 3) >> 3)
-                                                             << 16)));
-       /* BitBlt Source Address */
-       MMIO_OUT32(VIA_REG_SRCPOS, 0x0);
-       /* BitBlt Destination Address */
-       MMIO_OUT32(VIA_REG_DSTPOS, ((image->dy << 16) | image->dx));
-       /* Dimension: width & height */
-       MMIO_OUT32(VIA_REG_DIMENSION,
-                  (((image->height - 1) << 16) | (image->width - 1)));
-       /* fb color */
-       MMIO_OUT32(VIA_REG_FGCOLOR, fg_col);
-       /* bg color */
-       MMIO_OUT32(VIA_REG_BGCOLOR, bg_col);
-       /* GE Command */
-       MMIO_OUT32(VIA_REG_GECMD, 0xCC020142);
-
-       for (i = 0; i < size / 4; i++) {
-               MMIO_OUT32(VIA_MMIO_BLTBASE, *udata);
-               udata++;
+       if (!viapar->shared->hw_bitblt ||
+               (image->depth != 1 && image->depth != viapar->depth)) {
+               cfb_imageblit(info, image);
+               return;
        }
 
+       if (image->depth == 1) {
+               op = VIA_BITBLT_MONO;
+               if (info->fix.visual == FB_VISUAL_TRUECOLOR) {
+                       fg_color =
+                               ((u32 *)info->pseudo_palette)[image->fg_color];
+                       bg_color =
+                               ((u32 *)info->pseudo_palette)[image->bg_color];
+               } else {
+                       fg_color = image->fg_color;
+                       bg_color = image->bg_color;
+               }
+       } else
+               op = VIA_BITBLT_COLOR;
+
+       DEBUG_MSG(KERN_DEBUG "viafb 2D engine: imageblit\n");
+       if (viapar->shared->hw_bitblt(viapar->io_virt, op,
+               image->width, image->height, info->var.bits_per_pixel,
+               viapar->vram_addr, info->fix.line_length, image->dx, image->dy,
+               (u32 *)image->data, 0, 0, 0, 0, fg_color, bg_color, 0))
+               cfb_imageblit(info, image);
 }
 
 static int viafb_cursor(struct fb_info *info, struct fb_cursor *cursor)
 {
-       u8 data[CURSOR_SIZE / 8];
-       u32 data_bak[CURSOR_SIZE / 32];
-       u32 temp, xx, yy, bg_col = 0, fg_col = 0;
-       int size, i, j = 0;
-       static int hw_cursor;
-       struct viafb_par *p_viafb_par;
-
-       if (viafb_accel)
-               hw_cursor = 1;
-
-       if (!viafb_accel) {
-               if (hw_cursor) {
-                       viafb_show_hw_cursor(info, HW_Cursor_OFF);
-                       hw_cursor = 0;
-               }
-               return -ENODEV;
-       }
+       struct viafb_par *viapar = info->par;
+       u32 temp, xx, yy, bg_color = 0, fg_color = 0,
+               chip_name = viapar->shared->chip_info.gfx_chip_name;
+       int i, j = 0, cur_size = 64;
 
-       if ((((struct viafb_par *)(info->par))->iga_path == IGA2)
-           && (viaparinfo->chip_info->gfx_chip_name == UNICHROME_CLE266))
+       if (info->flags & FBINFO_HWACCEL_DISABLED || info != viafbinfo)
                return -ENODEV;
 
-       /* When duoview and using lcd , use soft cursor */
-       if (viafb_LCD_ON || ((struct viafb_par *)(info->par))->duoview)
+       if (chip_name == UNICHROME_CLE266 && viapar->iga_path == IGA2)
                return -ENODEV;
 
        viafb_show_hw_cursor(info, HW_Cursor_OFF);
-       viacursor = *cursor;
 
        if (cursor->set & FB_CUR_SETHOT) {
-               viacursor.hot = cursor->hot;
-               temp = ((viacursor.hot.x) << 16) + viacursor.hot.y;
-               MMIO_OUT32(VIA_REG_CURSOR_ORG, temp);
+               temp = (cursor->hot.x << 16) + cursor->hot.y;
+               writel(temp, viapar->io_virt + VIA_REG_CURSOR_ORG);
        }
 
        if (cursor->set & FB_CUR_SETPOS) {
-               viacursor.image.dx = cursor->image.dx;
-               viacursor.image.dy = cursor->image.dy;
                yy = cursor->image.dy - info->var.yoffset;
                xx = cursor->image.dx - info->var.xoffset;
                temp = yy & 0xFFFF;
                temp |= (xx << 16);
-               MMIO_OUT32(VIA_REG_CURSOR_POS, temp);
+               writel(temp, viapar->io_virt + VIA_REG_CURSOR_POS);
        }
 
-       if (cursor->set & FB_CUR_SETSIZE) {
-               temp = MMIO_IN32(VIA_REG_CURSOR_MODE);
+       if (cursor->image.width <= 32 && cursor->image.height <= 32)
+               cur_size = 32;
+       else if (cursor->image.width <= 64 && cursor->image.height <= 64)
+               cur_size = 64;
+       else {
+               printk(KERN_WARNING "viafb_cursor: The cursor is too large "
+                       "%dx%d", cursor->image.width, cursor->image.height);
+               return -ENXIO;
+       }
 
-               if ((cursor->image.width <= 32)
-                   && (cursor->image.height <= 32)) {
-                       MAX_CURS = 32;
+       if (cursor->set & FB_CUR_SETSIZE) {
+               temp = readl(viapar->io_virt + VIA_REG_CURSOR_MODE);
+               if (cur_size == 32)
                        temp |= 0x2;
-               } else if ((cursor->image.width <= 64)
-                          && (cursor->image.height <= 64)) {
-                       MAX_CURS = 64;
-                       temp &= 0xFFFFFFFD;
-               } else {
-                       DEBUG_MSG(KERN_INFO
-                       "The cursor image is biger than 64x64 bits...\n");
-                       return -ENXIO;
-               }
-               MMIO_OUT32(VIA_REG_CURSOR_MODE, temp);
+               else
+                       temp &= ~0x2;
 
-               viacursor.image.height = cursor->image.height;
-               viacursor.image.width = cursor->image.width;
+               writel(temp, viapar->io_virt + VIA_REG_CURSOR_MODE);
        }
 
        if (cursor->set & FB_CUR_SETCMAP) {
-               viacursor.image.fg_color = cursor->image.fg_color;
-               viacursor.image.bg_color = cursor->image.bg_color;
-
-               switch (info->var.bits_per_pixel) {
-               case 8:
-               case 16:
-               case 32:
-                       bg_col =
-                           (0xFF << 24) |
-                           (((info->cmap.red)[viacursor.image.bg_color] &
-                           0xFF00) << 8) |
-                           ((info->cmap.green)[viacursor.image.bg_color] &
-                           0xFF00) |
-                           (((info->cmap.blue)[viacursor.image.bg_color] &
-                           0xFF00) >> 8);
-                       fg_col =
-                           (0xFF << 24) |
-                           (((info->cmap.red)[viacursor.image.fg_color] &
-                           0xFF00) << 8) |
-                           ((info->cmap.green)[viacursor.image.fg_color] &
-                           0xFF00) |
-                           (((info->cmap.blue)[viacursor.image.fg_color] &
-                           0xFF00) >> 8);
-                       break;
-               default:
-                       return 0;
-               }
-
-               /* This is indeed a patch for VT3324/VT3353 */
-               if (!info->par)
-                       return 0;
-               p_viafb_par = (struct viafb_par *)info->par;
-
-               if ((p_viafb_par->chip_info->gfx_chip_name ==
-                       UNICHROME_CX700) ||
-                       ((p_viafb_par->chip_info->gfx_chip_name ==
-                       UNICHROME_VX800))) {
-                       bg_col =
-                           (((info->cmap.red)[viacursor.image.bg_color] &
-                           0xFFC0) << 14) |
-                           (((info->cmap.green)[viacursor.image.bg_color] &
-                           0xFFC0) << 4) |
-                           (((info->cmap.blue)[viacursor.image.bg_color] &
-                           0xFFC0) >> 6);
-                       fg_col =
-                           (((info->cmap.red)[viacursor.image.fg_color] &
-                           0xFFC0) << 14) |
-                           (((info->cmap.green)[viacursor.image.fg_color] &
-                           0xFFC0) << 4) |
-                           (((info->cmap.blue)[viacursor.image.fg_color] &
-                           0xFFC0) >> 6);
+               fg_color = cursor->image.fg_color;
+               bg_color = cursor->image.bg_color;
+               if (chip_name == UNICHROME_CX700 ||
+                       chip_name == UNICHROME_VX800) {
+                       fg_color =
+                               ((info->cmap.red[fg_color] & 0xFFC0) << 14) |
+                               ((info->cmap.green[fg_color] & 0xFFC0) << 4) |
+                               ((info->cmap.blue[fg_color] & 0xFFC0) >> 6);
+                       bg_color =
+                               ((info->cmap.red[bg_color] & 0xFFC0) << 14) |
+                               ((info->cmap.green[bg_color] & 0xFFC0) << 4) |
+                               ((info->cmap.blue[bg_color] & 0xFFC0) >> 6);
+               } else {
+                       fg_color =
+                               ((info->cmap.red[fg_color] & 0xFF00) << 8) |
+                               (info->cmap.green[fg_color] & 0xFF00) |
+                               ((info->cmap.blue[fg_color] & 0xFF00) >> 8);
+                       bg_color =
+                               ((info->cmap.red[bg_color] & 0xFF00) << 8) |
+                               (info->cmap.green[bg_color] & 0xFF00) |
+                               ((info->cmap.blue[bg_color] & 0xFF00) >> 8);
                }
 
-               MMIO_OUT32(VIA_REG_CURSOR_BG, bg_col);
-               MMIO_OUT32(VIA_REG_CURSOR_FG, fg_col);
+               writel(bg_color, viapar->io_virt + VIA_REG_CURSOR_BG);
+               writel(fg_color, viapar->io_virt + VIA_REG_CURSOR_FG);
        }
 
        if (cursor->set & FB_CUR_SETSHAPE) {
-               size =
-                   ((viacursor.image.width + 7) >> 3) *
-                   viacursor.image.height;
-
-               if (MAX_CURS == 32) {
-                       for (i = 0; i < (CURSOR_SIZE / 32); i++) {
-                               data_bak[i] = 0x0;
-                               data_bak[i + 1] = 0xFFFFFFFF;
+               struct {
+                       u8 data[CURSOR_SIZE];
+                       u32 bak[CURSOR_SIZE / 4];
+               } *cr_data = kzalloc(sizeof(*cr_data), GFP_ATOMIC);
+               int size = ((cursor->image.width + 7) >> 3) *
+                       cursor->image.height;
+
+               if (!cr_data)
+                       return -ENOMEM;
+
+               if (cur_size == 32) {
+                       for (i = 0; i < (CURSOR_SIZE / 4); i++) {
+                               cr_data->bak[i] = 0x0;
+                               cr_data->bak[i + 1] = 0xFFFFFFFF;
                                i += 1;
                        }
-               } else if (MAX_CURS == 64) {
-                       for (i = 0; i < (CURSOR_SIZE / 32); i++) {
-                               data_bak[i] = 0x0;
-                               data_bak[i + 1] = 0x0;
-                               data_bak[i + 2] = 0xFFFFFFFF;
-                               data_bak[i + 3] = 0xFFFFFFFF;
+               } else {
+                       for (i = 0; i < (CURSOR_SIZE / 4); i++) {
+                               cr_data->bak[i] = 0x0;
+                               cr_data->bak[i + 1] = 0x0;
+                               cr_data->bak[i + 2] = 0xFFFFFFFF;
+                               cr_data->bak[i + 3] = 0xFFFFFFFF;
                                i += 3;
                        }
                }
 
-               switch (viacursor.rop) {
+               switch (cursor->rop) {
                case ROP_XOR:
                        for (i = 0; i < size; i++)
-                               data[i] = viacursor.mask[i];
+                               cr_data->data[i] = cursor->mask[i];
                        break;
                case ROP_COPY:
 
                        for (i = 0; i < size; i++)
-                               data[i] = viacursor.mask[i];
+                               cr_data->data[i] = cursor->mask[i];
                        break;
                default:
                        break;
                }
 
-               if (MAX_CURS == 32) {
+               if (cur_size == 32) {
                        for (i = 0; i < size; i++) {
-                               data_bak[j] = (u32) data[i];
-                               data_bak[j + 1] = ~data_bak[j];
+                               cr_data->bak[j] = (u32) cr_data->data[i];
+                               cr_data->bak[j + 1] = ~cr_data->bak[j];
                                j += 2;
                        }
-               } else if (MAX_CURS == 64) {
+               } else {
                        for (i = 0; i < size; i++) {
-                               data_bak[j] = (u32) data[i];
-                               data_bak[j + 1] = 0x0;
-                               data_bak[j + 2] = ~data_bak[j];
-                               data_bak[j + 3] = ~data_bak[j + 1];
+                               cr_data->bak[j] = (u32) cr_data->data[i];
+                               cr_data->bak[j + 1] = 0x0;
+                               cr_data->bak[j + 2] = ~cr_data->bak[j];
+                               cr_data->bak[j + 3] = ~cr_data->bak[j + 1];
                                j += 4;
                        }
                }
 
-               memcpy(((struct viafb_par *)(info->par))->fbmem_virt +
-                      ((struct viafb_par *)(info->par))->cursor_start,
-                      data_bak, CURSOR_SIZE);
+               memcpy_toio(viafbinfo->screen_base + viapar->shared->
+                       cursor_vram_addr, cr_data->bak, CURSOR_SIZE);
+               kfree(cr_data);
        }
 
-       if (viacursor.enable)
+       if (cursor->enable)
                viafb_show_hw_cursor(info, HW_Cursor_ON);
 
        return 0;
@@ -1242,23 +1001,21 @@ static int viafb_sync(struct fb_info *info)
        return 0;
 }
 
-int viafb_get_mode_index(int hres, int vres, int flag)
+int viafb_get_mode_index(int hres, int vres)
 {
        u32 i;
        DEBUG_MSG(KERN_INFO "viafb_get_mode_index!\n");
 
-       for (i = 0; viafb_modentry[i].mode_index != VIA_RES_INVALID; i++)
-               if (viafb_modentry[i].xres == hres &&
-                       viafb_modentry[i].yres == vres)
+       for (i = 0; i < NUM_TOTAL_MODETABLE; i++)
+               if (CLE266Modes[i].mode_array &&
+                       CLE266Modes[i].crtc[0].crtc.hor_addr == hres &&
+                       CLE266Modes[i].crtc[0].crtc.ver_addr == vres)
                        break;
 
-       viafb_resMode = viafb_modentry[i].mode_index;
-       if (flag)
-               viafb_mode1 = viafb_modentry[i].mode_res;
-       else
-               viafb_mode = viafb_modentry[i].mode_res;
+       if (i == NUM_TOTAL_MODETABLE)
+               return VIA_RES_INVALID;
 
-       return viafb_resMode;
+       return CLE266Modes[i].ModeIndex;
 }
 
 static void check_available_device_to_enable(int device_id)
@@ -1362,36 +1119,11 @@ static void viafb_set_device(struct device_t active_dev)
                viafb_SAMM_ON = active_dev.samm;
        viafb_primary_dev = active_dev.primary_dev;
 
-       viafb_set_start_addr();
+       viafb_set_primary_address(0);
+       viafb_set_secondary_address(viafb_SAMM_ON ? viafb_second_offset : 0);
        viafb_set_iga_path();
 }
 
-static void viafb_set_video_device(u32 video_dev_info)
-{
-       viaparinfo->video_on_crt = STATE_OFF;
-       viaparinfo->video_on_dvi = STATE_OFF;
-       viaparinfo->video_on_lcd = STATE_OFF;
-
-       /* Check available device to enable: */
-       if ((video_dev_info & CRT_Device) == CRT_Device)
-               viaparinfo->video_on_crt = STATE_ON;
-       else if ((video_dev_info & DVI_Device) == DVI_Device)
-               viaparinfo->video_on_dvi = STATE_ON;
-       else if ((video_dev_info & LCD_Device) == LCD_Device)
-               viaparinfo->video_on_lcd = STATE_ON;
-}
-
-static void viafb_get_video_device(u32 *video_dev_info)
-{
-       *video_dev_info = None_Device;
-       if (viaparinfo->video_on_crt == STATE_ON)
-               *video_dev_info |= CRT_Device;
-       else if (viaparinfo->video_on_dvi == STATE_ON)
-               *video_dev_info |= DVI_Device;
-       else if (viaparinfo->video_on_lcd == STATE_ON)
-               *video_dev_info |= LCD_Device;
-}
-
 static int get_primary_device(void)
 {
        int primary_device = 0;
@@ -1433,18 +1165,6 @@ static int get_primary_device(void)
        return primary_device;
 }
 
-static u8 is_duoview(void)
-{
-       if (0 == viafb_SAMM_ON) {
-               if (viafb_LCD_ON + viafb_LCD2_ON +
-                       viafb_DVI_ON + viafb_CRT_ON == 2)
-                       return true;
-               return false;
-       } else {
-               return false;
-       }
-}
-
 static void apply_second_mode_setting(struct fb_var_screeninfo
        *sec_var)
 {
@@ -1546,14 +1266,13 @@ static int apply_device_setting(struct viafb_ioctl_setting setting_info,
                        if (viafb_SAMM_ON)
                                viafb_primary_dev = setting_info.primary_device;
 
-                       viafb_set_start_addr();
+                       viafb_set_primary_address(0);
+                       viafb_set_secondary_address(viafb_SAMM_ON ? viafb_second_offset : 0);
                        viafb_set_iga_path();
                }
                need_set_mode = 1;
        }
 
-       viaparinfo->duoview = is_duoview();
-
        if (!need_set_mode) {
                ;
        } else {
@@ -1576,18 +1295,6 @@ static void retrieve_device_setting(struct viafb_ioctl_setting
                setting_info->device_status |= LCD_Device;
        if (viafb_LCD2_ON == 1)
                setting_info->device_status |= LCD2_Device;
-       if ((viaparinfo->video_on_crt == 1) && (viafb_CRT_ON == 1)) {
-               setting_info->video_device_status =
-                       viaparinfo->crt_setting_info->iga_path;
-       } else if ((viaparinfo->video_on_dvi == 1) && (viafb_DVI_ON == 1)) {
-               setting_info->video_device_status =
-                       viaparinfo->tmds_setting_info->iga_path;
-       } else if ((viaparinfo->video_on_lcd == 1) && (viafb_LCD_ON == 1)) {
-               setting_info->video_device_status =
-                       viaparinfo->lvds_setting_info->iga_path;
-       } else {
-               setting_info->video_device_status = 0;
-       }
 
        setting_info->samm_status = viafb_SAMM_ON;
        setting_info->primary_device = get_primary_device();
@@ -1674,25 +1381,6 @@ static void parse_active_dev(void)
                viafb_CRT_ON = STATE_ON;
                viafb_SAMM_ON = STATE_OFF;
        }
-       viaparinfo->duoview = is_duoview();
-}
-
-static void parse_video_dev(void)
-{
-       viaparinfo->video_on_crt = STATE_OFF;
-       viaparinfo->video_on_dvi = STATE_OFF;
-       viaparinfo->video_on_lcd = STATE_OFF;
-
-       if (!strncmp(viafb_video_dev, "CRT", 3)) {
-               /* Video on CRT */
-               viaparinfo->video_on_crt = STATE_ON;
-       } else if (!strncmp(viafb_video_dev, "DVI", 3)) {
-               /* Video on DVI */
-               viaparinfo->video_on_dvi = STATE_ON;
-       } else if (!strncmp(viafb_video_dev, "LCD", 3)) {
-               /* Video on LCD */
-               viaparinfo->video_on_lcd = STATE_ON;
-       }
 }
 
 static int parse_port(char *opt_str, int *output_interface)
@@ -1741,10 +1429,8 @@ static void parse_dvi_port(void)
  * DVP1Driving, DFPHigh, DFPLow CR96,   SR2A[5], SR1B[1], SR2A[4], SR1E[2],
  * CR9B,    SR65,    CR97,    CR99
  */
-static int viafb_dvp0_proc_read(char *buf, char **start, off_t offset,
-int count, int *eof, void *data)
+static int viafb_dvp0_proc_show(struct seq_file *m, void *v)
 {
-       int len = 0;
        u8 dvp0_data_dri = 0, dvp0_clk_dri = 0, dvp0 = 0;
        dvp0_data_dri =
            (viafb_read_reg(VIASR, SR2A) & BIT5) >> 4 |
@@ -1753,13 +1439,17 @@ int count, int *eof, void *data)
            (viafb_read_reg(VIASR, SR2A) & BIT4) >> 3 |
            (viafb_read_reg(VIASR, SR1E) & BIT2) >> 2;
        dvp0 = viafb_read_reg(VIACR, CR96) & 0x0f;
-       len +=
-           sprintf(buf + len, "%x %x %x\n", dvp0, dvp0_data_dri, dvp0_clk_dri);
-       *eof = 1;               /*Inform kernel end of data */
-       return len;
+       seq_printf(m, "%x %x %x\n", dvp0, dvp0_data_dri, dvp0_clk_dri);
+       return 0;
 }
-static int viafb_dvp0_proc_write(struct file *file,
-       const char __user *buffer, unsigned long count, void *data)
+
+static int viafb_dvp0_proc_open(struct inode *inode, struct file *file)
+{
+       return single_open(file, viafb_dvp0_proc_show, NULL);
+}
+
+static ssize_t viafb_dvp0_proc_write(struct file *file,
+       const char __user *buffer, size_t count, loff_t *pos)
 {
        char buf[20], *value, *pbuf;
        u8 reg_val = 0;
@@ -1803,21 +1493,33 @@ static int viafb_dvp0_proc_write(struct file *file,
        }
        return count;
 }
-static int viafb_dvp1_proc_read(char *buf, char **start, off_t offset,
-       int count, int *eof, void *data)
+
+static const struct file_operations viafb_dvp0_proc_fops = {
+       .owner          = THIS_MODULE,
+       .open           = viafb_dvp0_proc_open,
+       .read           = seq_read,
+       .llseek         = seq_lseek,
+       .release        = single_release,
+       .write          = viafb_dvp0_proc_write,
+};
+
+static int viafb_dvp1_proc_show(struct seq_file *m, void *v)
 {
-       int len = 0;
        u8 dvp1 = 0, dvp1_data_dri = 0, dvp1_clk_dri = 0;
        dvp1 = viafb_read_reg(VIACR, CR9B) & 0x0f;
        dvp1_data_dri = (viafb_read_reg(VIASR, SR65) & 0x0c) >> 2;
        dvp1_clk_dri = viafb_read_reg(VIASR, SR65) & 0x03;
-       len +=
-           sprintf(buf + len, "%x %x %x\n", dvp1, dvp1_data_dri, dvp1_clk_dri);
-       *eof = 1;               /*Inform kernel end of data */
-       return len;
+       seq_printf(m, "%x %x %x\n", dvp1, dvp1_data_dri, dvp1_clk_dri);
+       return 0;
 }
-static int viafb_dvp1_proc_write(struct file *file,
-       const char __user *buffer, unsigned long count, void *data)
+
+static int viafb_dvp1_proc_open(struct inode *inode, struct file *file)
+{
+       return single_open(file, viafb_dvp1_proc_show, NULL);
+}
+
+static ssize_t viafb_dvp1_proc_write(struct file *file,
+       const char __user *buffer, size_t count, loff_t *pos)
 {
        char buf[20], *value, *pbuf;
        u8 reg_val = 0;
@@ -1856,18 +1558,30 @@ static int viafb_dvp1_proc_write(struct file *file,
        return count;
 }
 
-static int viafb_dfph_proc_read(char *buf, char **start, off_t offset,
-       int count, int *eof, void *data)
+static const struct file_operations viafb_dvp1_proc_fops = {
+       .owner          = THIS_MODULE,
+       .open           = viafb_dvp1_proc_open,
+       .read           = seq_read,
+       .llseek         = seq_lseek,
+       .release        = single_release,
+       .write          = viafb_dvp1_proc_write,
+};
+
+static int viafb_dfph_proc_show(struct seq_file *m, void *v)
 {
-       int len = 0;
        u8 dfp_high = 0;
        dfp_high = viafb_read_reg(VIACR, CR97) & 0x0f;
-       len += sprintf(buf + len, "%x\n", dfp_high);
-       *eof = 1;               /*Inform kernel end of data */
-       return len;
+       seq_printf(m, "%x\n", dfp_high);
+       return 0;
 }
-static int viafb_dfph_proc_write(struct file *file,
-       const char __user *buffer, unsigned long count, void *data)
+
+static int viafb_dfph_proc_open(struct inode *inode, struct file *file)
+{
+       return single_open(file, viafb_dfph_proc_show, NULL);
+}
+
+static ssize_t viafb_dfph_proc_write(struct file *file,
+       const char __user *buffer, size_t count, loff_t *pos)
 {
        char buf[20];
        u8 reg_val = 0;
@@ -1882,18 +1596,31 @@ static int viafb_dfph_proc_write(struct file *file,
        viafb_write_reg_mask(CR97, VIACR, reg_val, 0x0f);
        return count;
 }
-static int viafb_dfpl_proc_read(char *buf, char **start, off_t offset,
-       int count, int *eof, void *data)
+
+static const struct file_operations viafb_dfph_proc_fops = {
+       .owner          = THIS_MODULE,
+       .open           = viafb_dfph_proc_open,
+       .read           = seq_read,
+       .llseek         = seq_lseek,
+       .release        = single_release,
+       .write          = viafb_dfph_proc_write,
+};
+
+static int viafb_dfpl_proc_show(struct seq_file *m, void *v)
 {
-       int len = 0;
        u8 dfp_low = 0;
        dfp_low = viafb_read_reg(VIACR, CR99) & 0x0f;
-       len += sprintf(buf + len, "%x\n", dfp_low);
-       *eof = 1;               /*Inform kernel end of data */
-       return len;
+       seq_printf(m, "%x\n", dfp_low);
+       return 0;
 }
-static int viafb_dfpl_proc_write(struct file *file,
-       const char __user *buffer, unsigned long count, void *data)
+
+static int viafb_dfpl_proc_open(struct inode *inode, struct file *file)
+{
+       return single_open(file, viafb_dfpl_proc_show, NULL);
+}
+
+static ssize_t viafb_dfpl_proc_write(struct file *file,
+       const char __user *buffer, size_t count, loff_t *pos)
 {
        char buf[20];
        u8 reg_val = 0;
@@ -1908,10 +1635,18 @@ static int viafb_dfpl_proc_write(struct file *file,
        viafb_write_reg_mask(CR99, VIACR, reg_val, 0x0f);
        return count;
 }
-static int viafb_vt1636_proc_read(char *buf, char **start,
-       off_t offset, int count, int *eof, void *data)
+
+static const struct file_operations viafb_dfpl_proc_fops = {
+       .owner          = THIS_MODULE,
+       .open           = viafb_dfpl_proc_open,
+       .read           = seq_read,
+       .llseek         = seq_lseek,
+       .release        = single_release,
+       .write          = viafb_dfpl_proc_write,
+};
+
+static int viafb_vt1636_proc_show(struct seq_file *m, void *v)
 {
-       int len = 0;
        u8 vt1636_08 = 0, vt1636_09 = 0;
        switch (viaparinfo->chip_info->lvds_chip_info.lvds_chip_name) {
        case VT1636_LVDS:
@@ -1921,7 +1656,7 @@ static int viafb_vt1636_proc_read(char *buf, char **start,
                vt1636_09 =
                    viafb_gpio_i2c_read_lvds(viaparinfo->lvds_setting_info,
                    &viaparinfo->chip_info->lvds_chip_info, 0x09) & 0x1f;
-               len += sprintf(buf + len, "%x %x\n", vt1636_08, vt1636_09);
+               seq_printf(m, "%x %x\n", vt1636_08, vt1636_09);
                break;
        default:
                break;
@@ -1934,16 +1669,21 @@ static int viafb_vt1636_proc_read(char *buf, char **start,
                vt1636_09 =
                    viafb_gpio_i2c_read_lvds(viaparinfo->lvds_setting_info2,
                        &viaparinfo->chip_info->lvds_chip_info2, 0x09) & 0x1f;
-               len += sprintf(buf + len, " %x %x\n", vt1636_08, vt1636_09);
+               seq_printf(m, " %x %x\n", vt1636_08, vt1636_09);
                break;
        default:
                break;
        }
-       *eof = 1;               /*Inform kernel end of data */
-       return len;
+       return 0;
 }
-static int viafb_vt1636_proc_write(struct file *file,
-       const char __user *buffer, unsigned long count, void *data)
+
+static int viafb_vt1636_proc_open(struct inode *inode, struct file *file)
+{
+       return single_open(file, viafb_vt1636_proc_show, NULL);
+}
+
+static ssize_t viafb_vt1636_proc_write(struct file *file,
+       const char __user *buffer, size_t count, loff_t *pos)
 {
        char buf[30], *value, *pbuf;
        struct IODATA reg_val;
@@ -2032,44 +1772,27 @@ static int viafb_vt1636_proc_write(struct file *file,
        return count;
 }
 
-static void viafb_init_proc(struct proc_dir_entry *viafb_entry)
+static const struct file_operations viafb_vt1636_proc_fops = {
+       .owner          = THIS_MODULE,
+       .open           = viafb_vt1636_proc_open,
+       .read           = seq_read,
+       .llseek         = seq_lseek,
+       .release        = single_release,
+       .write          = viafb_vt1636_proc_write,
+};
+
+static void viafb_init_proc(struct proc_dir_entry **viafb_entry)
 {
-       struct proc_dir_entry *entry;
-       viafb_entry = proc_mkdir("viafb", NULL);
+       *viafb_entry = proc_mkdir("viafb", NULL);
        if (viafb_entry) {
-               entry = create_proc_entry("dvp0", 0, viafb_entry);
-               if (entry) {
-                       entry->owner = THIS_MODULE;
-                       entry->read_proc = viafb_dvp0_proc_read;
-                       entry->write_proc = viafb_dvp0_proc_write;
-               }
-               entry = create_proc_entry("dvp1", 0, viafb_entry);
-               if (entry) {
-                       entry->owner = THIS_MODULE;
-                       entry->read_proc = viafb_dvp1_proc_read;
-                       entry->write_proc = viafb_dvp1_proc_write;
-               }
-               entry = create_proc_entry("dfph", 0, viafb_entry);
-               if (entry) {
-                       entry->owner = THIS_MODULE;
-                       entry->read_proc = viafb_dfph_proc_read;
-                       entry->write_proc = viafb_dfph_proc_write;
-               }
-               entry = create_proc_entry("dfpl", 0, viafb_entry);
-               if (entry) {
-                       entry->owner = THIS_MODULE;
-                       entry->read_proc = viafb_dfpl_proc_read;
-                       entry->write_proc = viafb_dfpl_proc_write;
-               }
+               proc_create("dvp0", 0, *viafb_entry, &viafb_dvp0_proc_fops);
+               proc_create("dvp1", 0, *viafb_entry, &viafb_dvp1_proc_fops);
+               proc_create("dfph", 0, *viafb_entry, &viafb_dfph_proc_fops);
+               proc_create("dfpl", 0, *viafb_entry, &viafb_dfpl_proc_fops);
                if (VT1636_LVDS == viaparinfo->chip_info->lvds_chip_info.
                        lvds_chip_name || VT1636_LVDS ==
                    viaparinfo->chip_info->lvds_chip_info2.lvds_chip_name) {
-                       entry = create_proc_entry("vt1636", 0, viafb_entry);
-                       if (entry) {
-                               entry->owner = THIS_MODULE;
-                               entry->read_proc = viafb_vt1636_proc_read;
-                               entry->write_proc = viafb_vt1636_proc_write;
-                       }
+                       proc_create("vt1636", 0, *viafb_entry, &viafb_vt1636_proc_fops);
                }
 
        }
@@ -2083,53 +1806,44 @@ static void viafb_remove_proc(struct proc_dir_entry *viafb_entry)
        remove_proc_entry("dfpl", viafb_entry);
        remove_proc_entry("vt1636", viafb_entry);
        remove_proc_entry("vt1625", viafb_entry);
+       remove_proc_entry("viafb", NULL);
 }
 
 static int __devinit via_pci_probe(void)
 {
-       unsigned int default_xres, default_yres;
+       unsigned long default_xres, default_yres;
        char *tmpc, *tmpm;
        char *tmpc_sec, *tmpm_sec;
        int vmode_index;
-       u32 tmds_length, lvds_length, crt_length, chip_length, viafb_par_length;
+       u32 viafb_par_length;
 
        DEBUG_MSG(KERN_INFO "VIAFB PCI Probe!!\n");
 
        viafb_par_length = ALIGN(sizeof(struct viafb_par), BITS_PER_LONG/8);
-       tmds_length = ALIGN(sizeof(struct tmds_setting_information),
-               BITS_PER_LONG/8);
-       lvds_length = ALIGN(sizeof(struct lvds_setting_information),
-               BITS_PER_LONG/8);
-       crt_length = ALIGN(sizeof(struct lvds_setting_information),
-               BITS_PER_LONG/8);
-       chip_length = ALIGN(sizeof(struct chip_information), BITS_PER_LONG/8);
 
        /* Allocate fb_info and ***_par here, also including some other needed
         * variables
        */
-       viafbinfo = framebuffer_alloc(viafb_par_length + 2 * lvds_length +
-       tmds_length + crt_length + chip_length, NULL);
+       viafbinfo = framebuffer_alloc(viafb_par_length +
+               ALIGN(sizeof(struct viafb_shared), BITS_PER_LONG/8), NULL);
        if (!viafbinfo) {
                printk(KERN_ERR"Could not allocate memory for viafb_info.\n");
                return -ENODEV;
        }
 
        viaparinfo = (struct viafb_par *)viafbinfo->par;
-       viaparinfo->tmds_setting_info = (struct tmds_setting_information *)
-               ((u32)viaparinfo + viafb_par_length);
-       viaparinfo->lvds_setting_info = (struct lvds_setting_information *)
-               ((u32)viaparinfo->tmds_setting_info + tmds_length);
-       viaparinfo->lvds_setting_info2 = (struct lvds_setting_information *)
-               ((u32)viaparinfo->lvds_setting_info + lvds_length);
-       viaparinfo->crt_setting_info = (struct crt_setting_information *)
-               ((u32)viaparinfo->lvds_setting_info2 + lvds_length);
-       viaparinfo->chip_info = (struct chip_information *)
-               ((u32)viaparinfo->crt_setting_info + crt_length);
+       viaparinfo->shared = viafbinfo->par + viafb_par_length;
+       viaparinfo->vram_addr = 0;
+       viaparinfo->tmds_setting_info = &viaparinfo->shared->tmds_setting_info;
+       viaparinfo->lvds_setting_info = &viaparinfo->shared->lvds_setting_info;
+       viaparinfo->lvds_setting_info2 =
+               &viaparinfo->shared->lvds_setting_info2;
+       viaparinfo->crt_setting_info = &viaparinfo->shared->crt_setting_info;
+       viaparinfo->chip_info = &viaparinfo->shared->chip_info;
 
        if (viafb_dual_fb)
                viafb_SAMM_ON = 1;
        parse_active_dev();
-       parse_video_dev();
        parse_lcd_port();
        parse_dvi_port();
 
@@ -2144,18 +1858,21 @@ static int __devinit via_pci_probe(void)
        viafb_get_fb_info(&viaparinfo->fbmem, &viaparinfo->memsize);
        viaparinfo->fbmem_free = viaparinfo->memsize;
        viaparinfo->fbmem_used = 0;
-       viaparinfo->fbmem_virt = ioremap_nocache(viaparinfo->fbmem,
+       viafbinfo->screen_base = ioremap_nocache(viaparinfo->fbmem,
                viaparinfo->memsize);
-       viafbinfo->screen_base = (char *)viaparinfo->fbmem_virt;
-
-       if (!viaparinfo->fbmem_virt) {
+       if (!viafbinfo->screen_base) {
                printk(KERN_INFO "ioremap failed\n");
-               return -1;
+               return -ENOMEM;
        }
 
-       viafb_get_mmio_info(&viaparinfo->mmio_base, &viaparinfo->mmio_len);
-       viaparinfo->io_virt = ioremap_nocache(viaparinfo->mmio_base,
-               viaparinfo->mmio_len);
+       viafb_get_mmio_info(&viafbinfo->fix.mmio_start,
+               &viafbinfo->fix.mmio_len);
+       viaparinfo->io_virt = ioremap_nocache(viafbinfo->fix.mmio_start,
+               viafbinfo->fix.mmio_len);
+       if (!viaparinfo->io_virt) {
+               printk(KERN_WARNING "ioremap failed: hardware acceleration disabled\n");
+               viafb_accel = 0;
+       }
 
        viafbinfo->node = 0;
        viafbinfo->fbops = &viafb_ops;
@@ -2163,7 +1880,7 @@ static int __devinit via_pci_probe(void)
 
        viafbinfo->pseudo_palette = pseudo_pal;
        if (viafb_accel) {
-               viafb_init_accel();
+               viafb_init_accel(viaparinfo->shared);
                viafb_init_2d_engine();
                viafb_hw_cursor_init();
        }
@@ -2177,13 +1894,12 @@ static int __devinit via_pci_probe(void)
                        viafb_second_size * 1024 * 1024;
        }
 
-       viafb_FB_MM = viaparinfo->fbmem_virt;
        tmpm = viafb_mode;
        tmpc = strsep(&tmpm, "x");
-       strict_strtoul(tmpc, 0, (unsigned long *)&default_xres);
-       strict_strtoul(tmpm, 0, (unsigned long *)&default_yres);
+       strict_strtoul(tmpc, 0, &default_xres);
+       strict_strtoul(tmpm, 0, &default_yres);
 
-       vmode_index = viafb_get_mode_index(default_xres, default_yres, 0);
+       vmode_index = viafb_get_mode_index(default_xres, default_yres);
        DEBUG_MSG(KERN_INFO "0->index=%d\n", vmode_index);
 
        if (viafb_SAMM_ON == 1) {
@@ -2267,11 +1983,10 @@ static int __devinit via_pci_probe(void)
                }
                viaparinfo1 = viafbinfo1->par;
                memcpy(viaparinfo1, viaparinfo, viafb_par_length);
+               viaparinfo1->vram_addr = viafb_second_offset;
                viaparinfo1->memsize = viaparinfo->memsize -
                        viafb_second_offset;
                viaparinfo->memsize = viafb_second_offset;
-               viaparinfo1->fbmem_virt = viaparinfo->fbmem_virt +
-                       viafb_second_offset;
                viaparinfo1->fbmem = viaparinfo->fbmem + viafb_second_offset;
 
                viaparinfo1->fbmem_used = viaparinfo->fbmem_used;
@@ -2280,19 +1995,18 @@ static int __devinit via_pci_probe(void)
                viaparinfo->fbmem_free = viaparinfo->memsize;
                viaparinfo->fbmem_used = 0;
                if (viafb_accel) {
-                       viaparinfo1->cursor_start =
-                           viaparinfo->cursor_start - viafb_second_offset;
                        viaparinfo1->VQ_start = viaparinfo->VQ_start -
                                viafb_second_offset;
                        viaparinfo1->VQ_end = viaparinfo->VQ_end -
                                viafb_second_offset;
                }
 
+               viaparinfo->iga_path = IGA1;
+               viaparinfo1->iga_path = IGA2;
                memcpy(viafbinfo1, viafbinfo, sizeof(struct fb_info));
+               viafbinfo1->par = viaparinfo1;
                viafbinfo1->screen_base = viafbinfo->screen_base +
                        viafb_second_offset;
-               viafbinfo1->fix.smem_start = viaparinfo1->fbmem;
-               viafbinfo1->fix.smem_len = viaparinfo1->fbmem_free;
 
                default_var.xres = viafb_second_xres;
                default_var.yres = viafb_second_yres;
@@ -2314,15 +2028,17 @@ static int __devinit via_pci_probe(void)
                viafb_setup_fixinfo(&viafbinfo1->fix, viaparinfo1);
                viafb_check_var(&default_var, viafbinfo1);
                viafbinfo1->var = default_var;
-               viafb_update_viafb_par(viafbinfo);
-               viafb_update_fix(&viafbinfo1->fix, viafbinfo1);
+               viafb_update_fix(viafbinfo1);
+               viaparinfo1->depth = fb_get_color_depth(&viafbinfo1->var,
+                       &viafbinfo1->fix);
        }
 
        viafb_setup_fixinfo(&viafbinfo->fix, viaparinfo);
        viafb_check_var(&default_var, viafbinfo);
        viafbinfo->var = default_var;
-       viafb_update_viafb_par(viafbinfo);
-       viafb_update_fix(&viafbinfo->fix, viafbinfo);
+       viafb_update_fix(viafbinfo);
+       viaparinfo->depth = fb_get_color_depth(&viafbinfo->var,
+               &viafbinfo->fix);
        default_var.activate = FB_ACTIVATE_NOW;
        fb_alloc_cmap(&viafbinfo->cmap, 256, 0);
 
@@ -2344,7 +2060,7 @@ static int __devinit via_pci_probe(void)
                  viafbinfo->node, viafbinfo->fix.id, default_var.xres,
                  default_var.yres, default_var.bits_per_pixel);
 
-       viafb_init_proc(viaparinfo->proc_entry);
+       viafb_init_proc(&viaparinfo->shared->proc_entry);
        viafb_init_dac(IGA2);
        return 0;
 }
@@ -2356,7 +2072,7 @@ static void __devexit via_pci_remove(void)
        unregister_framebuffer(viafbinfo);
        if (viafb_dual_fb)
                unregister_framebuffer(viafbinfo1);
-       iounmap((void *)viaparinfo->fbmem_virt);
+       iounmap((void *)viafbinfo->screen_base);
        iounmap(viaparinfo->io_virt);
 
        viafb_delete_i2c_buss(viaparinfo);
@@ -2365,7 +2081,7 @@ static void __devexit via_pci_remove(void)
        if (viafb_dual_fb)
                framebuffer_release(viafbinfo1);
 
-       viafb_remove_proc(viaparinfo->proc_entry);
+       viafb_remove_proc(viaparinfo->shared->proc_entry);
 }
 
 #ifndef MODULE
@@ -2432,8 +2148,6 @@ static int __init viafb_setup(char *options)
                else if (!strncmp(this_opt, "viafb_lcd_mode=", 15))
                        strict_strtoul(this_opt + 15, 0,
                                (unsigned long *)&viafb_lcd_mode);
-               else if (!strncmp(this_opt, "viafb_video_dev=", 16))
-                       viafb_video_dev = kstrdup(this_opt + 16, GFP_KERNEL);
                else if (!strncmp(this_opt, "viafb_lcd_port=", 15))
                        viafb_lcd_port = kstrdup(this_opt + 15, GFP_KERNEL);
                else if (!strncmp(this_opt, "viafb_dvi_port=", 15))
@@ -2554,9 +2268,6 @@ module_param(viafb_lcd_mode, int, 0);
 MODULE_PARM_DESC(viafb_lcd_mode,
        "Set Flat Panel mode(Default=OPENLDI)");
 
-module_param(viafb_video_dev, charp, 0);
-MODULE_PARM_DESC(viafb_video_dev, "Specify video devices.");
-
 module_param(viafb_lcd_port, charp, 0);
 MODULE_PARM_DESC(viafb_lcd_port, "Specify LCD output port.");