gma500: rework register stuff sanely
authorAlan Cox <alan@linux.intel.com>
Thu, 8 Mar 2012 16:02:05 +0000 (16:02 +0000)
committerDave Airlie <airlied@redhat.com>
Sat, 10 Mar 2012 13:05:44 +0000 (13:05 +0000)
Rework registers handling to prepare for Medfield.

Signed-off-by: Alan Cox <alan@linux.intel.com>
[split out from a single big patch]
Signed-off-by: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
drivers/gpu/drm/gma500/cdv_intel_display.c
drivers/gpu/drm/gma500/oaktrail_device.c
drivers/gpu/drm/gma500/oaktrail_hdmi.c
drivers/gpu/drm/gma500/psb_device.c
drivers/gpu/drm/gma500/psb_drv.h
drivers/gpu/drm/gma500/psb_intel_display.c

index dc9e246768da54ff1546b4cc668790c850590c20..67debf024d9c1df817add54b0a74ac2bcc38d917 100644 (file)
@@ -968,7 +968,7 @@ void cdv_intel_crtc_load_lut(struct drm_crtc *crtc)
                gma_power_end(dev);
        } else {
                for (i = 0; i < 256; i++) {
-                       dev_priv->regs.save_palette_a[i] =
+                       dev_priv->regs.psb.save_palette_a[i] =
                                  ((psb_intel_crtc->lut_r[i] +
                                  psb_intel_crtc->lut_adj[i]) << 16) |
                                  ((psb_intel_crtc->lut_g[i] +
@@ -1338,19 +1338,20 @@ static int cdv_intel_crtc_clock_get(struct drm_device *dev,
                gma_power_end(dev);
        } else {
                dpll = (pipe == 0) ?
-                       dev_priv->regs.saveDPLL_A : dev_priv->regs.saveDPLL_B;
+                       dev_priv->regs.psb.saveDPLL_A :
+                       dev_priv->regs.psb.saveDPLL_B;
 
                if ((dpll & DISPLAY_RATE_SELECT_FPA1) == 0)
                        fp = (pipe == 0) ?
-                               dev_priv->regs.saveFPA0 :
-                               dev_priv->regs.saveFPB0;
+                               dev_priv->regs.psb.saveFPA0 :
+                               dev_priv->regs.psb.saveFPB0;
                else
                        fp = (pipe == 0) ?
-                               dev_priv->regs.saveFPA1 :
-                               dev_priv->regs.saveFPB1;
+                               dev_priv->regs.psb.saveFPA1 :
+                               dev_priv->regs.psb.saveFPB1;
 
                is_lvds = (pipe == 1) &&
-                               (dev_priv->regs.saveLVDS & LVDS_PORT_EN);
+                               (dev_priv->regs.psb.saveLVDS & LVDS_PORT_EN);
        }
 
        clock.m1 = (fp & FP_M1_DIV_MASK) >> FP_M1_DIV_SHIFT;
@@ -1420,17 +1421,17 @@ struct drm_display_mode *cdv_intel_crtc_mode_get(struct drm_device *dev,
                gma_power_end(dev);
        } else {
                htot = (pipe == 0) ?
-                       dev_priv->regs.saveHTOTAL_A :
-                       dev_priv->regs.saveHTOTAL_B;
+                       dev_priv->regs.psb.saveHTOTAL_A :
+                       dev_priv->regs.psb.saveHTOTAL_B;
                hsync = (pipe == 0) ?
-                       dev_priv->regs.saveHSYNC_A :
-                       dev_priv->regs.saveHSYNC_B;
+                       dev_priv->regs.psb.saveHSYNC_A :
+                       dev_priv->regs.psb.saveHSYNC_B;
                vtot = (pipe == 0) ?
-                       dev_priv->regs.saveVTOTAL_A :
-                       dev_priv->regs.saveVTOTAL_B;
+                       dev_priv->regs.psb.saveVTOTAL_A :
+                       dev_priv->regs.psb.saveVTOTAL_B;
                vsync = (pipe == 0) ?
-                       dev_priv->regs.saveVSYNC_A :
-                       dev_priv->regs.saveVSYNC_B;
+                       dev_priv->regs.psb.saveVSYNC_A :
+                       dev_priv->regs.psb.saveVSYNC_B;
        }
 
        mode = kzalloc(sizeof(*mode), GFP_KERNEL);
index 22def104bb4ae9637760a568c96aa9b2748b30ec..2b79fbb0d137b66ba4185ea7d6b07a78717d704c 100644 (file)
@@ -186,82 +186,82 @@ int oaktrail_backlight_init(struct drm_device *dev)
 static int oaktrail_save_display_registers(struct drm_device *dev)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
-       struct psb_state *regs = &dev_priv->regs;
+       struct psb_save_area *regs = &dev_priv->regs;
        int i;
        u32 pp_stat;
 
        /* Display arbitration control + watermarks */
-       regs->saveDSPARB = PSB_RVDC32(DSPARB);
-       regs->saveDSPFW1 = PSB_RVDC32(DSPFW1);
-       regs->saveDSPFW2 = PSB_RVDC32(DSPFW2);
-       regs->saveDSPFW3 = PSB_RVDC32(DSPFW3);
-       regs->saveDSPFW4 = PSB_RVDC32(DSPFW4);
-       regs->saveDSPFW5 = PSB_RVDC32(DSPFW5);
-       regs->saveDSPFW6 = PSB_RVDC32(DSPFW6);
-       regs->saveCHICKENBIT = PSB_RVDC32(DSPCHICKENBIT);
+       regs->psb.saveDSPARB = PSB_RVDC32(DSPARB);
+       regs->psb.saveDSPFW1 = PSB_RVDC32(DSPFW1);
+       regs->psb.saveDSPFW2 = PSB_RVDC32(DSPFW2);
+       regs->psb.saveDSPFW3 = PSB_RVDC32(DSPFW3);
+       regs->psb.saveDSPFW4 = PSB_RVDC32(DSPFW4);
+       regs->psb.saveDSPFW5 = PSB_RVDC32(DSPFW5);
+       regs->psb.saveDSPFW6 = PSB_RVDC32(DSPFW6);
+       regs->psb.saveCHICKENBIT = PSB_RVDC32(DSPCHICKENBIT);
 
        /* Pipe & plane A info */
-       regs->savePIPEACONF = PSB_RVDC32(PIPEACONF);
-       regs->savePIPEASRC = PSB_RVDC32(PIPEASRC);
-       regs->saveFPA0 = PSB_RVDC32(MRST_FPA0);
-       regs->saveFPA1 = PSB_RVDC32(MRST_FPA1);
-       regs->saveDPLL_A = PSB_RVDC32(MRST_DPLL_A);
-       regs->saveHTOTAL_A = PSB_RVDC32(HTOTAL_A);
-       regs->saveHBLANK_A = PSB_RVDC32(HBLANK_A);
-       regs->saveHSYNC_A = PSB_RVDC32(HSYNC_A);
-       regs->saveVTOTAL_A = PSB_RVDC32(VTOTAL_A);
-       regs->saveVBLANK_A = PSB_RVDC32(VBLANK_A);
-       regs->saveVSYNC_A = PSB_RVDC32(VSYNC_A);
-       regs->saveBCLRPAT_A = PSB_RVDC32(BCLRPAT_A);
-       regs->saveDSPACNTR = PSB_RVDC32(DSPACNTR);
-       regs->saveDSPASTRIDE = PSB_RVDC32(DSPASTRIDE);
-       regs->saveDSPAADDR = PSB_RVDC32(DSPABASE);
-       regs->saveDSPASURF = PSB_RVDC32(DSPASURF);
-       regs->saveDSPALINOFF = PSB_RVDC32(DSPALINOFF);
-       regs->saveDSPATILEOFF = PSB_RVDC32(DSPATILEOFF);
+       regs->psb.savePIPEACONF = PSB_RVDC32(PIPEACONF);
+       regs->psb.savePIPEASRC = PSB_RVDC32(PIPEASRC);
+       regs->psb.saveFPA0 = PSB_RVDC32(MRST_FPA0);
+       regs->psb.saveFPA1 = PSB_RVDC32(MRST_FPA1);
+       regs->psb.saveDPLL_A = PSB_RVDC32(MRST_DPLL_A);
+       regs->psb.saveHTOTAL_A = PSB_RVDC32(HTOTAL_A);
+       regs->psb.saveHBLANK_A = PSB_RVDC32(HBLANK_A);
+       regs->psb.saveHSYNC_A = PSB_RVDC32(HSYNC_A);
+       regs->psb.saveVTOTAL_A = PSB_RVDC32(VTOTAL_A);
+       regs->psb.saveVBLANK_A = PSB_RVDC32(VBLANK_A);
+       regs->psb.saveVSYNC_A = PSB_RVDC32(VSYNC_A);
+       regs->psb.saveBCLRPAT_A = PSB_RVDC32(BCLRPAT_A);
+       regs->psb.saveDSPACNTR = PSB_RVDC32(DSPACNTR);
+       regs->psb.saveDSPASTRIDE = PSB_RVDC32(DSPASTRIDE);
+       regs->psb.saveDSPAADDR = PSB_RVDC32(DSPABASE);
+       regs->psb.saveDSPASURF = PSB_RVDC32(DSPASURF);
+       regs->psb.saveDSPALINOFF = PSB_RVDC32(DSPALINOFF);
+       regs->psb.saveDSPATILEOFF = PSB_RVDC32(DSPATILEOFF);
 
        /* Save cursor regs */
-       regs->saveDSPACURSOR_CTRL = PSB_RVDC32(CURACNTR);
-       regs->saveDSPACURSOR_BASE = PSB_RVDC32(CURABASE);
-       regs->saveDSPACURSOR_POS = PSB_RVDC32(CURAPOS);
+       regs->psb.saveDSPACURSOR_CTRL = PSB_RVDC32(CURACNTR);
+       regs->psb.saveDSPACURSOR_BASE = PSB_RVDC32(CURABASE);
+       regs->psb.saveDSPACURSOR_POS = PSB_RVDC32(CURAPOS);
 
        /* Save palette (gamma) */
        for (i = 0; i < 256; i++)
-               regs->save_palette_a[i] = PSB_RVDC32(PALETTE_A + (i << 2));
+               regs->psb.save_palette_a[i] = PSB_RVDC32(PALETTE_A + (i << 2));
 
        if (dev_priv->hdmi_priv)
                oaktrail_hdmi_save(dev);
 
        /* Save performance state */
-       regs->savePERF_MODE = PSB_RVDC32(MRST_PERF_MODE);
+       regs->psb.savePERF_MODE = PSB_RVDC32(MRST_PERF_MODE);
 
        /* LVDS state */
-       regs->savePP_CONTROL = PSB_RVDC32(PP_CONTROL);
-       regs->savePFIT_PGM_RATIOS = PSB_RVDC32(PFIT_PGM_RATIOS);
-       regs->savePFIT_AUTO_RATIOS = PSB_RVDC32(PFIT_AUTO_RATIOS);
+       regs->psb.savePP_CONTROL = PSB_RVDC32(PP_CONTROL);
+       regs->psb.savePFIT_PGM_RATIOS = PSB_RVDC32(PFIT_PGM_RATIOS);
+       regs->psb.savePFIT_AUTO_RATIOS = PSB_RVDC32(PFIT_AUTO_RATIOS);
        regs->saveBLC_PWM_CTL = PSB_RVDC32(BLC_PWM_CTL);
        regs->saveBLC_PWM_CTL2 = PSB_RVDC32(BLC_PWM_CTL2);
-       regs->saveLVDS = PSB_RVDC32(LVDS);
-       regs->savePFIT_CONTROL = PSB_RVDC32(PFIT_CONTROL);
-       regs->savePP_ON_DELAYS = PSB_RVDC32(LVDSPP_ON);
-       regs->savePP_OFF_DELAYS = PSB_RVDC32(LVDSPP_OFF);
-       regs->savePP_DIVISOR = PSB_RVDC32(PP_CYCLE);
+       regs->psb.saveLVDS = PSB_RVDC32(LVDS);
+       regs->psb.savePFIT_CONTROL = PSB_RVDC32(PFIT_CONTROL);
+       regs->psb.savePP_ON_DELAYS = PSB_RVDC32(LVDSPP_ON);
+       regs->psb.savePP_OFF_DELAYS = PSB_RVDC32(LVDSPP_OFF);
+       regs->psb.savePP_DIVISOR = PSB_RVDC32(PP_CYCLE);
 
        /* HW overlay */
-       regs->saveOV_OVADD = PSB_RVDC32(OV_OVADD);
-       regs->saveOV_OGAMC0 = PSB_RVDC32(OV_OGAMC0);
-       regs->saveOV_OGAMC1 = PSB_RVDC32(OV_OGAMC1);
-       regs->saveOV_OGAMC2 = PSB_RVDC32(OV_OGAMC2);
-       regs->saveOV_OGAMC3 = PSB_RVDC32(OV_OGAMC3);
-       regs->saveOV_OGAMC4 = PSB_RVDC32(OV_OGAMC4);
-       regs->saveOV_OGAMC5 = PSB_RVDC32(OV_OGAMC5);
+       regs->psb.saveOV_OVADD = PSB_RVDC32(OV_OVADD);
+       regs->psb.saveOV_OGAMC0 = PSB_RVDC32(OV_OGAMC0);
+       regs->psb.saveOV_OGAMC1 = PSB_RVDC32(OV_OGAMC1);
+       regs->psb.saveOV_OGAMC2 = PSB_RVDC32(OV_OGAMC2);
+       regs->psb.saveOV_OGAMC3 = PSB_RVDC32(OV_OGAMC3);
+       regs->psb.saveOV_OGAMC4 = PSB_RVDC32(OV_OGAMC4);
+       regs->psb.saveOV_OGAMC5 = PSB_RVDC32(OV_OGAMC5);
 
        /* DPST registers */
-       regs->saveHISTOGRAM_INT_CONTROL_REG =
+       regs->psb.saveHISTOGRAM_INT_CONTROL_REG =
                                        PSB_RVDC32(HISTOGRAM_INT_CONTROL);
-       regs->saveHISTOGRAM_LOGIC_CONTROL_REG =
+       regs->psb.saveHISTOGRAM_LOGIC_CONTROL_REG =
                                        PSB_RVDC32(HISTOGRAM_LOGIC_CONTROL);
-       regs->savePWM_CONTROL_LOGIC = PSB_RVDC32(PWM_CONTROL_LOGIC);
+       regs->psb.savePWM_CONTROL_LOGIC = PSB_RVDC32(PWM_CONTROL_LOGIC);
 
        if (dev_priv->iLVDS_enable) {
                /* Shut down the panel */
@@ -299,80 +299,80 @@ static int oaktrail_save_display_registers(struct drm_device *dev)
 static int oaktrail_restore_display_registers(struct drm_device *dev)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
-       struct psb_state *regs = &dev_priv->regs;
+       struct psb_save_area *regs = &dev_priv->regs;
        u32 pp_stat;
        int i;
 
        /* Display arbitration + watermarks */
-       PSB_WVDC32(regs->saveDSPARB, DSPARB);
-       PSB_WVDC32(regs->saveDSPFW1, DSPFW1);
-       PSB_WVDC32(regs->saveDSPFW2, DSPFW2);
-       PSB_WVDC32(regs->saveDSPFW3, DSPFW3);
-       PSB_WVDC32(regs->saveDSPFW4, DSPFW4);
-       PSB_WVDC32(regs->saveDSPFW5, DSPFW5);
-       PSB_WVDC32(regs->saveDSPFW6, DSPFW6);
-       PSB_WVDC32(regs->saveCHICKENBIT, DSPCHICKENBIT);
+       PSB_WVDC32(regs->psb.saveDSPARB, DSPARB);
+       PSB_WVDC32(regs->psb.saveDSPFW1, DSPFW1);
+       PSB_WVDC32(regs->psb.saveDSPFW2, DSPFW2);
+       PSB_WVDC32(regs->psb.saveDSPFW3, DSPFW3);
+       PSB_WVDC32(regs->psb.saveDSPFW4, DSPFW4);
+       PSB_WVDC32(regs->psb.saveDSPFW5, DSPFW5);
+       PSB_WVDC32(regs->psb.saveDSPFW6, DSPFW6);
+       PSB_WVDC32(regs->psb.saveCHICKENBIT, DSPCHICKENBIT);
 
        /* Make sure VGA plane is off. it initializes to on after reset!*/
        PSB_WVDC32(0x80000000, VGACNTRL);
 
        /* set the plls */
-       PSB_WVDC32(regs->saveFPA0, MRST_FPA0);
-       PSB_WVDC32(regs->saveFPA1, MRST_FPA1);
+       PSB_WVDC32(regs->psb.saveFPA0, MRST_FPA0);
+       PSB_WVDC32(regs->psb.saveFPA1, MRST_FPA1);
 
        /* Actually enable it */
-       PSB_WVDC32(regs->saveDPLL_A, MRST_DPLL_A);
+       PSB_WVDC32(regs->psb.saveDPLL_A, MRST_DPLL_A);
        DRM_UDELAY(150);
 
        /* Restore mode */
-       PSB_WVDC32(regs->saveHTOTAL_A, HTOTAL_A);
-       PSB_WVDC32(regs->saveHBLANK_A, HBLANK_A);
-       PSB_WVDC32(regs->saveHSYNC_A, HSYNC_A);
-       PSB_WVDC32(regs->saveVTOTAL_A, VTOTAL_A);
-       PSB_WVDC32(regs->saveVBLANK_A, VBLANK_A);
-       PSB_WVDC32(regs->saveVSYNC_A, VSYNC_A);
-       PSB_WVDC32(regs->savePIPEASRC, PIPEASRC);
-       PSB_WVDC32(regs->saveBCLRPAT_A, BCLRPAT_A);
+       PSB_WVDC32(regs->psb.saveHTOTAL_A, HTOTAL_A);
+       PSB_WVDC32(regs->psb.saveHBLANK_A, HBLANK_A);
+       PSB_WVDC32(regs->psb.saveHSYNC_A, HSYNC_A);
+       PSB_WVDC32(regs->psb.saveVTOTAL_A, VTOTAL_A);
+       PSB_WVDC32(regs->psb.saveVBLANK_A, VBLANK_A);
+       PSB_WVDC32(regs->psb.saveVSYNC_A, VSYNC_A);
+       PSB_WVDC32(regs->psb.savePIPEASRC, PIPEASRC);
+       PSB_WVDC32(regs->psb.saveBCLRPAT_A, BCLRPAT_A);
 
        /* Restore performance mode*/
-       PSB_WVDC32(regs->savePERF_MODE, MRST_PERF_MODE);
+       PSB_WVDC32(regs->psb.savePERF_MODE, MRST_PERF_MODE);
 
        /* Enable the pipe*/
        if (dev_priv->iLVDS_enable)
-               PSB_WVDC32(regs->savePIPEACONF, PIPEACONF);
+               PSB_WVDC32(regs->psb.savePIPEACONF, PIPEACONF);
 
        /* Set up the plane*/
-       PSB_WVDC32(regs->saveDSPALINOFF, DSPALINOFF);
-       PSB_WVDC32(regs->saveDSPASTRIDE, DSPASTRIDE);
-       PSB_WVDC32(regs->saveDSPATILEOFF, DSPATILEOFF);
+       PSB_WVDC32(regs->psb.saveDSPALINOFF, DSPALINOFF);
+       PSB_WVDC32(regs->psb.saveDSPASTRIDE, DSPASTRIDE);
+       PSB_WVDC32(regs->psb.saveDSPATILEOFF, DSPATILEOFF);
 
        /* Enable the plane */
-       PSB_WVDC32(regs->saveDSPACNTR, DSPACNTR);
-       PSB_WVDC32(regs->saveDSPASURF, DSPASURF);
+       PSB_WVDC32(regs->psb.saveDSPACNTR, DSPACNTR);
+       PSB_WVDC32(regs->psb.saveDSPASURF, DSPASURF);
 
        /* Enable Cursor A */
-       PSB_WVDC32(regs->saveDSPACURSOR_CTRL, CURACNTR);
-       PSB_WVDC32(regs->saveDSPACURSOR_POS, CURAPOS);
-       PSB_WVDC32(regs->saveDSPACURSOR_BASE, CURABASE);
+       PSB_WVDC32(regs->psb.saveDSPACURSOR_CTRL, CURACNTR);
+       PSB_WVDC32(regs->psb.saveDSPACURSOR_POS, CURAPOS);
+       PSB_WVDC32(regs->psb.saveDSPACURSOR_BASE, CURABASE);
 
        /* Restore palette (gamma) */
        for (i = 0; i < 256; i++)
-               PSB_WVDC32(regs->save_palette_a[i], PALETTE_A + (i << 2));
+               PSB_WVDC32(regs->psb.save_palette_a[i], PALETTE_A + (i << 2));
 
        if (dev_priv->hdmi_priv)
                oaktrail_hdmi_restore(dev);
 
        if (dev_priv->iLVDS_enable) {
                PSB_WVDC32(regs->saveBLC_PWM_CTL2, BLC_PWM_CTL2);
-               PSB_WVDC32(regs->saveLVDS, LVDS); /*port 61180h*/
-               PSB_WVDC32(regs->savePFIT_CONTROL, PFIT_CONTROL);
-               PSB_WVDC32(regs->savePFIT_PGM_RATIOS, PFIT_PGM_RATIOS);
-               PSB_WVDC32(regs->savePFIT_AUTO_RATIOS, PFIT_AUTO_RATIOS);
+               PSB_WVDC32(regs->psb.saveLVDS, LVDS); /*port 61180h*/
+               PSB_WVDC32(regs->psb.savePFIT_CONTROL, PFIT_CONTROL);
+               PSB_WVDC32(regs->psb.savePFIT_PGM_RATIOS, PFIT_PGM_RATIOS);
+               PSB_WVDC32(regs->psb.savePFIT_AUTO_RATIOS, PFIT_AUTO_RATIOS);
                PSB_WVDC32(regs->saveBLC_PWM_CTL, BLC_PWM_CTL);
-               PSB_WVDC32(regs->savePP_ON_DELAYS, LVDSPP_ON);
-               PSB_WVDC32(regs->savePP_OFF_DELAYS, LVDSPP_OFF);
-               PSB_WVDC32(regs->savePP_DIVISOR, PP_CYCLE);
-               PSB_WVDC32(regs->savePP_CONTROL, PP_CONTROL);
+               PSB_WVDC32(regs->psb.savePP_ON_DELAYS, LVDSPP_ON);
+               PSB_WVDC32(regs->psb.savePP_OFF_DELAYS, LVDSPP_OFF);
+               PSB_WVDC32(regs->psb.savePP_DIVISOR, PP_CYCLE);
+               PSB_WVDC32(regs->psb.savePP_CONTROL, PP_CONTROL);
        }
 
        /* Wait for cycle delay */
@@ -386,20 +386,20 @@ static int oaktrail_restore_display_registers(struct drm_device *dev)
        } while (pp_stat & 0x10000000);
 
        /* Restore HW overlay */
-       PSB_WVDC32(regs->saveOV_OVADD, OV_OVADD);
-       PSB_WVDC32(regs->saveOV_OGAMC0, OV_OGAMC0);
-       PSB_WVDC32(regs->saveOV_OGAMC1, OV_OGAMC1);
-       PSB_WVDC32(regs->saveOV_OGAMC2, OV_OGAMC2);
-       PSB_WVDC32(regs->saveOV_OGAMC3, OV_OGAMC3);
-       PSB_WVDC32(regs->saveOV_OGAMC4, OV_OGAMC4);
-       PSB_WVDC32(regs->saveOV_OGAMC5, OV_OGAMC5);
+       PSB_WVDC32(regs->psb.saveOV_OVADD, OV_OVADD);
+       PSB_WVDC32(regs->psb.saveOV_OGAMC0, OV_OGAMC0);
+       PSB_WVDC32(regs->psb.saveOV_OGAMC1, OV_OGAMC1);
+       PSB_WVDC32(regs->psb.saveOV_OGAMC2, OV_OGAMC2);
+       PSB_WVDC32(regs->psb.saveOV_OGAMC3, OV_OGAMC3);
+       PSB_WVDC32(regs->psb.saveOV_OGAMC4, OV_OGAMC4);
+       PSB_WVDC32(regs->psb.saveOV_OGAMC5, OV_OGAMC5);
 
        /* DPST registers */
-       PSB_WVDC32(regs->saveHISTOGRAM_INT_CONTROL_REG,
+       PSB_WVDC32(regs->psb.saveHISTOGRAM_INT_CONTROL_REG,
                                                HISTOGRAM_INT_CONTROL);
-       PSB_WVDC32(regs->saveHISTOGRAM_LOGIC_CONTROL_REG,
+       PSB_WVDC32(regs->psb.saveHISTOGRAM_LOGIC_CONTROL_REG,
                                                HISTOGRAM_LOGIC_CONTROL);
-       PSB_WVDC32(regs->savePWM_CONTROL_LOGIC, PWM_CONTROL_LOGIC);
+       PSB_WVDC32(regs->psb.savePWM_CONTROL_LOGIC, PWM_CONTROL_LOGIC);
 
        return 0;
 }
index 5a2de014f237fd941b8c4a464070986d3612f46a..e4c73c680c990622e2b13a5ec5bf92875452bb6f 100644 (file)
@@ -766,7 +766,7 @@ void oaktrail_hdmi_save(struct drm_device *dev)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
        struct oaktrail_hdmi_dev *hdmi_dev = dev_priv->hdmi_priv;
-       struct psb_state *regs = &dev_priv->regs;
+       struct psb_state *regs = &dev_priv->regs.psb;
        int i;
 
        /* dpll */
@@ -818,7 +818,7 @@ void oaktrail_hdmi_restore(struct drm_device *dev)
 {
        struct drm_psb_private *dev_priv = dev->dev_private;
        struct oaktrail_hdmi_dev *hdmi_dev = dev_priv->hdmi_priv;
-       struct psb_state *regs = &dev_priv->regs;
+       struct psb_state *regs = &dev_priv->regs.psb;
        int i;
 
        /* dpll */
index 0d370e7094b8c93ebebe5df6a11b3dadbe2c12ec..95d163e4f1f465e5c6d44ee502f4d98e858e1f4b 100644 (file)
@@ -177,7 +177,7 @@ static int psb_save_display_registers(struct drm_device *dev)
        struct drm_psb_private *dev_priv = dev->dev_private;
        struct drm_crtc *crtc;
        struct drm_connector *connector;
-       struct psb_state *regs = &dev_priv->regs;
+       struct psb_state *regs = &dev_priv->regs.psb;
 
        /* Display arbitration control + watermarks */
        regs->saveDSPARB = PSB_RVDC32(DSPARB);
@@ -214,7 +214,7 @@ static int psb_restore_display_registers(struct drm_device *dev)
        struct drm_psb_private *dev_priv = dev->dev_private;
        struct drm_crtc *crtc;
        struct drm_connector *connector;
-       struct psb_state *regs = &dev_priv->regs;
+       struct psb_state *regs = &dev_priv->regs.psb;
 
        /* Display arbitration + watermarks */
        PSB_WVDC32(regs->saveDSPARB, DSPARB);
index dee07e0d7c399dad917a0deec19f514f2a33ed19..3c0bf7be27380bb0c3c39107afc860a3e6897875 100644 (file)
@@ -337,8 +337,6 @@ struct psb_state {
        uint32_t savePFIT_CONTROL;
        uint32_t savePaletteA[256];
        uint32_t savePaletteB[256];
-       uint32_t saveBLC_PWM_CTL2;
-       uint32_t saveBLC_PWM_CTL;
        uint32_t saveCLOCKGATING;
        uint32_t saveDSPARB;
        uint32_t saveDSPATILEOFF;
@@ -350,8 +348,6 @@ struct psb_state {
        uint32_t savePP_ON_DELAYS;
        uint32_t savePP_OFF_DELAYS;
        uint32_t savePP_DIVISOR;
-       uint32_t saveBSM;
-       uint32_t saveVBT;
        uint32_t saveBCLRPAT_A;
        uint32_t saveBCLRPAT_B;
        uint32_t saveDSPALINOFF;
@@ -393,6 +389,16 @@ struct psb_state {
        uint32_t savePWM_CONTROL_LOGIC;
 };
 
+struct psb_save_area {
+       uint32_t saveBSM;
+       uint32_t saveVBT;
+       union {
+               struct psb_state psb;
+       };
+       uint32_t saveBLC_PWM_CTL2;
+       uint32_t saveBLC_PWM_CTL;
+};
+
 struct psb_ops;
 
 #define PSB_NUM_PIPE           3
@@ -520,7 +526,9 @@ struct drm_psb_private {
        /*
         * Register state
         */
-        struct psb_state regs;
+
+       struct psb_save_area regs;
+
        /* MSI reg save */
        uint32_t msi_addr;
        uint32_t msi_data;
index 4ba1ae8b5e30732ad8560f6e14eb245415cac535..eed91fdcf36e99c53d23ad7312a78af4c166dfff 100644 (file)
@@ -845,7 +845,7 @@ void psb_intel_crtc_load_lut(struct drm_crtc *crtc)
                gma_power_end(dev);
        } else {
                for (i = 0; i < 256; i++) {
-                       dev_priv->regs.save_palette_a[i] =
+                       dev_priv->regs.psb.save_palette_a[i] =
                                  ((psb_intel_crtc->lut_r[i] +
                                  psb_intel_crtc->lut_adj[i]) << 16) |
                                  ((psb_intel_crtc->lut_g[i] +
@@ -1141,18 +1141,19 @@ static int psb_intel_crtc_clock_get(struct drm_device *dev,
                gma_power_end(dev);
        } else {
                dpll = (pipe == 0) ?
-                       dev_priv->regs.saveDPLL_A : dev_priv->regs.saveDPLL_B;
+                       dev_priv->regs.psb.saveDPLL_A :
+                       dev_priv->regs.psb.saveDPLL_B;
 
                if ((dpll & DISPLAY_RATE_SELECT_FPA1) == 0)
                        fp = (pipe == 0) ?
-                               dev_priv->regs.saveFPA0 :
-                               dev_priv->regs.saveFPB0;
+                               dev_priv->regs.psb.saveFPA0 :
+                               dev_priv->regs.psb.saveFPB0;
                else
                        fp = (pipe == 0) ?
-                               dev_priv->regs.saveFPA1 :
-                               dev_priv->regs.saveFPB1;
+                               dev_priv->regs.psb.saveFPA1 :
+                               dev_priv->regs.psb.saveFPB1;
 
-               is_lvds = (pipe == 1) && (dev_priv->regs.saveLVDS &
+               is_lvds = (pipe == 1) && (dev_priv->regs.psb.saveLVDS &
                                                                LVDS_PORT_EN);
        }
 
@@ -1219,17 +1220,17 @@ struct drm_display_mode *psb_intel_crtc_mode_get(struct drm_device *dev,
                gma_power_end(dev);
        } else {
                htot = (pipe == 0) ?
-                       dev_priv->regs.saveHTOTAL_A :
-                       dev_priv->regs.saveHTOTAL_B;
+                       dev_priv->regs.psb.saveHTOTAL_A :
+                       dev_priv->regs.psb.saveHTOTAL_B;
                hsync = (pipe == 0) ?
-                       dev_priv->regs.saveHSYNC_A :
-                       dev_priv->regs.saveHSYNC_B;
+                       dev_priv->regs.psb.saveHSYNC_A :
+                       dev_priv->regs.psb.saveHSYNC_B;
                vtot = (pipe == 0) ?
-                       dev_priv->regs.saveVTOTAL_A :
-                       dev_priv->regs.saveVTOTAL_B;
+                       dev_priv->regs.psb.saveVTOTAL_A :
+                       dev_priv->regs.psb.saveVTOTAL_B;
                vsync = (pipe == 0) ?
-                       dev_priv->regs.saveVSYNC_A :
-                       dev_priv->regs.saveVSYNC_B;
+                       dev_priv->regs.psb.saveVSYNC_A :
+                       dev_priv->regs.psb.saveVSYNC_B;
        }
 
        mode = kzalloc(sizeof(*mode), GFP_KERNEL);