int ast_astdp_read_edid(struct drm_device *dev, u8 *ediddata)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u8 i = 0, j = 0;
 
        /*
        u8 bDPTX = 0;
        u8 bDPExecute = 1;
 
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        // S3 come back, need more time to wait BMC ready.
        if (bPower)
                WaitCount = 300;
 
 void ast_dp_power_on_off(struct drm_device *dev, bool on)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        // Read and Turn off DP PHY sleep
        u8 bE3 = ast_get_index_reg_mask(ast, AST_IO_CRTC_PORT, 0xE3, AST_DP_VIDEO_ENABLE);
 
 
 void ast_dp_set_on_off(struct drm_device *dev, bool on)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u8 video_on_off = on;
 
        // Video On/Off
 
 void ast_dp_set_mode(struct drm_crtc *crtc, struct ast_vbios_mode_info *vbios_mode)
 {
-       struct ast_device *ast = to_ast_private(crtc->dev);
+       struct ast_device *ast = to_ast_device(crtc->dev);
 
        u32 ulRefreshRateIndex;
        u8 ModeIdx;
 
 
 static int ast_load_dp501_microcode(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        int ret;
 
        ret = request_firmware(&ast->dp501_fw, "ast_dp501_fw.bin", dev->dev);
 
 static bool ast_write_cmd(struct drm_device *dev, u8 data)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        int retry = 0;
        if (wait_nack(ast)) {
                send_nack(ast);
 
 static bool ast_write_data(struct drm_device *dev, u8 data)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
 
        if (wait_nack(ast)) {
                send_nack(ast);
 #if 0
 static bool ast_read_data(struct drm_device *dev, u8 *data)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u8 tmp;
 
        *data = 0;
 
 bool ast_backup_fw(struct drm_device *dev, u8 *addr, u32 size)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u32 i, data;
        u32 boot_address;
 
 
 static bool ast_launch_m68k(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u32 i, data, len = 0;
        u32 boot_address;
        u8 *fw_addr = NULL;
 
 bool ast_dp501_read_edid(struct drm_device *dev, u8 *ediddata)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u32 i, boot_address, offset, data;
        u32 *pEDIDidx;
 
 
 static bool ast_init_dvo(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u8 jreg;
        u32 data;
        ast_write32(ast, 0xf004, 0x1e6e0000);
 
 static void ast_init_analog(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u32 data;
 
        /*
 
 void ast_init_3rdtx(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u8 jreg;
 
        if (ast->chip == AST2300 || ast->chip == AST2400) {
 
        const struct firmware *dp501_fw;        /* dp501 fw */
 };
 
-static inline struct ast_device *to_ast_private(struct drm_device *dev)
+static inline struct ast_device *to_ast_device(struct drm_device *dev)
 {
        return container_of(dev, struct ast_device, base);
 }
 
 static void ast_i2c_setsda(void *i2c_priv, int data)
 {
        struct ast_i2c_chan *i2c = i2c_priv;
-       struct ast_device *ast = to_ast_private(i2c->dev);
+       struct ast_device *ast = to_ast_device(i2c->dev);
        int i;
        u8 ujcrb7, jtemp;
 
 static void ast_i2c_setscl(void *i2c_priv, int clock)
 {
        struct ast_i2c_chan *i2c = i2c_priv;
-       struct ast_device *ast = to_ast_private(i2c->dev);
+       struct ast_device *ast = to_ast_device(i2c->dev);
        int i;
        u8 ujcrb7, jtemp;
 
 static int ast_i2c_getsda(void *i2c_priv)
 {
        struct ast_i2c_chan *i2c = i2c_priv;
-       struct ast_device *ast = to_ast_private(i2c->dev);
+       struct ast_device *ast = to_ast_device(i2c->dev);
        uint32_t val, val2, count, pass;
 
        count = 0;
 static int ast_i2c_getscl(void *i2c_priv)
 {
        struct ast_i2c_chan *i2c = i2c_priv;
-       struct ast_device *ast = to_ast_private(i2c->dev);
+       struct ast_device *ast = to_ast_device(i2c->dev);
        uint32_t val, val2, count, pass;
 
        count = 0;
 
 static void ast_detect_config_mode(struct drm_device *dev, u32 *scu_rev)
 {
        struct device_node *np = dev->dev->of_node;
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct pci_dev *pdev = to_pci_dev(dev->dev);
        uint32_t data, jregd0, jregd1;
 
 
 static int ast_detect_chip(struct drm_device *dev, bool *need_post)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct pci_dev *pdev = to_pci_dev(dev->dev);
        uint32_t jreg, scu_rev;
 
 static int ast_get_dram_info(struct drm_device *dev)
 {
        struct device_node *np = dev->dev->of_node;
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        uint32_t mcr_cfg, mcr_scu_mpll, mcr_scu_strap;
        uint32_t denum, num, div, ref_pll, dsel;
 
 
                                                   struct drm_atomic_state *state)
 {
        struct drm_device *dev = plane->dev;
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct drm_plane_state *plane_state = drm_atomic_get_new_plane_state(state, plane);
        struct drm_shadow_plane_state *shadow_plane_state = to_drm_shadow_plane_state(plane_state);
        struct drm_framebuffer *fb = plane_state->fb;
 static void ast_primary_plane_helper_atomic_enable(struct drm_plane *plane,
                                                   struct drm_atomic_state *state)
 {
-       struct ast_device *ast = to_ast_private(plane->dev);
+       struct ast_device *ast = to_ast_device(plane->dev);
        struct ast_plane *ast_plane = to_ast_plane(plane);
 
        /*
 static void ast_primary_plane_helper_atomic_disable(struct drm_plane *plane,
                                                    struct drm_atomic_state *state)
 {
-       struct ast_device *ast = to_ast_private(plane->dev);
+       struct ast_device *ast = to_ast_device(plane->dev);
 
        ast_set_index_reg_mask(ast, AST_IO_SEQ_PORT, 0x1, 0xdf, 0x20);
 }
        struct drm_shadow_plane_state *shadow_plane_state = to_drm_shadow_plane_state(plane_state);
        struct drm_framebuffer *fb = plane_state->fb;
        struct drm_plane_state *old_plane_state = drm_atomic_get_old_plane_state(state, plane);
-       struct ast_device *ast = to_ast_private(plane->dev);
+       struct ast_device *ast = to_ast_device(plane->dev);
        struct iosys_map src_map = shadow_plane_state->data[0];
        struct drm_rect damage;
        const u8 *src = src_map.vaddr; /* TODO: Use mapping abstraction properly */
 static void ast_cursor_plane_helper_atomic_disable(struct drm_plane *plane,
                                                   struct drm_atomic_state *state)
 {
-       struct ast_device *ast = to_ast_private(plane->dev);
+       struct ast_device *ast = to_ast_device(plane->dev);
 
        ast_set_cursor_enabled(ast, false);
 }
 
 static void ast_crtc_dpms(struct drm_crtc *crtc, int mode)
 {
-       struct ast_device *ast = to_ast_private(crtc->dev);
+       struct ast_device *ast = to_ast_device(crtc->dev);
        u8 ch = AST_DPMS_VSYNC_OFF | AST_DPMS_HSYNC_OFF;
        struct ast_crtc_state *ast_state;
        const struct drm_format_info *format;
 static enum drm_mode_status
 ast_crtc_helper_mode_valid(struct drm_crtc *crtc, const struct drm_display_mode *mode)
 {
-       struct ast_device *ast = to_ast_private(crtc->dev);
+       struct ast_device *ast = to_ast_device(crtc->dev);
        enum drm_mode_status status;
        uint32_t jtemp;
 
        struct drm_crtc_state *crtc_state = drm_atomic_get_new_crtc_state(state,
                                                                          crtc);
        struct drm_device *dev = crtc->dev;
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct ast_crtc_state *ast_crtc_state = to_ast_crtc_state(crtc_state);
        struct ast_vbios_mode_info *vbios_mode_info = &ast_crtc_state->vbios_mode_info;
 
 static void ast_crtc_helper_atomic_enable(struct drm_crtc *crtc, struct drm_atomic_state *state)
 {
        struct drm_device *dev = crtc->dev;
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct drm_crtc_state *crtc_state = drm_atomic_get_new_crtc_state(state, crtc);
        struct ast_crtc_state *ast_crtc_state = to_ast_crtc_state(crtc_state);
        struct ast_vbios_mode_info *vbios_mode_info =
 {
        struct drm_crtc_state *old_crtc_state = drm_atomic_get_old_crtc_state(state, crtc);
        struct drm_device *dev = crtc->dev;
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
 
        ast_crtc_dpms(crtc, DRM_MODE_DPMS_OFF);
 
 
 static int ast_crtc_init(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct drm_crtc *crtc = &ast->crtc;
        int ret;
 
 {
        struct ast_vga_connector *ast_vga_connector = to_ast_vga_connector(connector);
        struct drm_device *dev = connector->dev;
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct edid *edid;
        int count;
 
 {
        struct ast_sil164_connector *ast_sil164_connector = to_ast_sil164_connector(connector);
        struct drm_device *dev = connector->dev;
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct edid *edid;
        int count;
 
 
 static void ast_mode_config_helper_atomic_commit_tail(struct drm_atomic_state *state)
 {
-       struct ast_device *ast = to_ast_private(state->dev);
+       struct ast_device *ast = to_ast_device(state->dev);
 
        /*
         * Concurrent operations could possibly trigger a call to
                                                       const struct drm_display_mode *mode)
 {
        static const unsigned long max_bpp = 4; /* DRM_FORMAT_XRGB8888 */
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        unsigned long fbsize, fbpages, max_fbpages;
 
        max_fbpages = (ast->vram_fb_available) >> PAGE_SHIFT;
 
 
 void ast_enable_vga(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
 
        ast_io_write8(ast, AST_IO_VGA_ENABLE_PORT, 0x01);
        ast_io_write8(ast, AST_IO_MISC_PORT_WRITE, 0x01);
 
 void ast_enable_mmio(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
 
        ast_set_index_reg(ast, AST_IO_CRTC_PORT, 0xa1, 0x06);
 }
 
 bool ast_is_vga_enabled(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u8 ch;
 
        ch = ast_io_read8(ast, AST_IO_VGA_ENABLE_PORT);
 static void
 ast_set_def_ext_reg(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct pci_dev *pdev = to_pci_dev(dev->dev);
        u8 i, index, reg;
        const u8 *ext_reg_info;
 
 static void ast_init_dram_reg(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u8 j;
        u32 data, temp, i;
        const struct ast_dramstruct *dram_reg_info;
 
 void ast_post_gpu(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct pci_dev *pdev = to_pci_dev(dev->dev);
        u32 reg;
 
 
 static void ast_post_chip_2300(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        struct ast2300_dram_param param;
        u32 temp;
        u8 reg;
 
 void ast_post_chip_2500(struct drm_device *dev)
 {
-       struct ast_device *ast = to_ast_private(dev);
+       struct ast_device *ast = to_ast_device(dev);
        u32 temp;
        u8 reg;