staging: sm750fb: use direct call to hw_sm750_output_setMode
authorMike Rapoport <mike.rapoport@gmail.com>
Fri, 23 Oct 2015 14:55:11 +0000 (17:55 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 25 Oct 2015 02:04:13 +0000 (19:04 -0700)
The proc_setMode method in lynx_output always uses the same
implementation and there is no point to use redirection rather than
direct function call.

Signed-off-by: Mike Rapoport <mike.rapoport@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/sm750fb/sm750.c
drivers/staging/sm750fb/sm750.h

index 24c4f39..cd20629 100644 (file)
@@ -386,7 +386,7 @@ static int lynxfb_ops_set_par(struct fb_info *info)
        }
        ret = hw_sm750_crtc_setMode(crtc, var, fix);
        if (!ret)
-               ret = output->proc_setMode(output, var, fix);
+               ret = hw_sm750_output_setMode(output, var, fix);
        return ret;
 }
 
@@ -674,8 +674,6 @@ static int sm750fb_set_drv(struct lynxfb_par *par)
        crtc->ypanstep = 1;
        crtc->ywrapstep = 0;
 
-       output->proc_setMode = hw_sm750_output_setMode;
-
        output->proc_setBLANK = (share->revid == SM750LE_REVISION_ID) ?
                                 hw_sm750le_setBLANK : hw_sm750_setBLANK;
        /* chip specific phase */
index 6f5e09d..bf905aa 100644 (file)
@@ -125,10 +125,6 @@ struct lynxfb_output {
        */
        void *priv;
 
-       int (*proc_setMode)(struct lynxfb_output*,
-                                               struct fb_var_screeninfo*,
-                                               struct fb_fix_screeninfo*);
-
        int (*proc_setBLANK)(struct lynxfb_output*, int);
 };