On Tue, Dec 11, 2012 at 5:21 PM, Jesse Barnes <jbarnes at virtuousgeek.org> wrote: > drm_fb_helper_parse_command_line in drm_fb_helper.c is where we set the > flags based on the command line params. drm_fb_helper_initial_config > is what sets up the initial fbcon configuration; it might be best to > add a new command line param to control its behavior, rather than > trying to add some new flags to the connector that only apply to the fb > layer... I think a new connector->disable_fbdev flag or something which initial_config would check seems sensible. If you further overload ->force you need to check implications all over the place, and we've had a few funny bugs around connector forcing already (e.g. flip-flop when force fights against the poll helper, or unnecessary polling and similar madness). So better leave that as-is. Other option is to add a module option to the helper module to restrict to internal panels, i.e. connector type eDP || LVDS. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch
