Hi, On 06/08/14 15:52, Daniel Vetter wrote: > In drm/i915 we want to get at the video= cmdline modes even when we > don't have fbdev support enabled, so that users can always override > the kernel's initial mode selection. > > But that gives us a direct depency upon the parsing code in the fbdev > subsystem. Since it's so little code just extract these 2 functions > and always build them in. > > Whiel at it fix the checkpatch fail in this code. > > v2: Also move fb_mode_option. Spotted by the kbuild. > > v3: Review from Geert: > - Keep the old copyright notice from fb_mem.c, although I have no > idea what exactly applies. > - Only compile this when needed. > > Cc: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > Cc: Plagniol-Villard <plagnioj@xxxxxxxxxxxx> > Cc: Tomi Valkeinen <tomi.valkeinen@xxxxxx> > Cc: linux-fbdev@xxxxxxxxxxxxxxx > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxx> > > -- > > I prefer if we can merge this through drm-next since we'll use it > there in follow-up patches. > -Daniel > --- > drivers/video/fbdev/Kconfig | 4 ++ > drivers/video/fbdev/core/Makefile | 1 + > drivers/video/fbdev/core/fb_cmdline.c | 110 ++++++++++++++++++++++++++++++++++ > drivers/video/fbdev/core/fbmem.c | 92 ---------------------------- > drivers/video/fbdev/core/modedb.c | 3 - > 5 files changed, 115 insertions(+), 95 deletions(-) > create mode 100644 drivers/video/fbdev/core/fb_cmdline.c Sorry for late response. Looks fine for me, and I'm fine merging it via drm-next. Acked-by: Tomi Valkeinen <tomi.valkeinen@xxxxxx> Tomi
Attachment:
signature.asc
Description: OpenPGP digital signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
