uvesafb_open had checked the par->vbe_state_size, so we don't need to check it again in uvesafb_vbe_state_save, this patch just can reduce a few lines of code. Signed-off-by: Wang YanQing <udknight@xxxxxxxxx> --- drivers/video/uvesafb.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/video/uvesafb.c b/drivers/video/uvesafb.c index 2f8f82d..b064a3e 100644 --- a/drivers/video/uvesafb.c +++ b/drivers/video/uvesafb.c @@ -357,9 +357,6 @@ static u8 *uvesafb_vbe_state_save(struct uvesafb_par *par) u8 *state; int err; - if (!par->vbe_state_size) - return NULL; - state = kmalloc(par->vbe_state_size, GFP_KERNEL); if (!state) return ERR_PTR(-ENOMEM); -- 1.7.11.1.116.g8228a23 -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html