Chandramouli Narayanan <[EMAIL PROTECTED]> writes: > + > +static int __init efifb_init(void) > +{ > + int ret; > + > + if (!efi_enabled) > + return -ENODEV; > +
Please remove this efi_enabled check it appears redundant with the check immediately below. > + if (screen_info.orig_video_isVGA != VIDEO_TYPE_EFI) > + return -ENODEV; > + > + > + ret = platform_driver_register(&efifb_driver); > + > + if (!ret) { > + ret = platform_device_register(&efifb_device); > + if (ret) > + platform_driver_unregister(&efifb_driver); > + } > + return ret; > +} > +module_init(efifb_init); Eric - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/