On Tue, Jan 07, 2014 at 02:49:50PM +0800, Shawn Guo wrote: > On Thu, Jan 02, 2014 at 09:28:19PM +0000, Russell King wrote: > > @@ -449,6 +458,24 @@ static int imx_drm_driver_load(struct drm_device *drm, unsigned long flags) > > } > > } > > > > + /* > > + * All components are now initialised, so setup the fb helper. > > + * The fb helper takes copies of key hardware information, so the > > + * crtcs/connectors/encoders must not change after this point. > > + */ > > +#if IS_ENABLED(CONFIG_DRM_IMX_FB_HELPER) > > + if (legacyfb_depth != 16 && legacyfb_depth != 32) { > > + dev_warn(drm->dev, "Invalid legacyfb_depth. Defaulting to 16bpp\n"); > > + legacyfb_depth = 16; > > + } > > + imxdrm->fbhelper = drm_fbdev_cma_init(drm, legacyfb_depth, > > + drm->mode_config.num_crtc, 4); > > s/4/MAX_CRTC > > imx-drm-core.c has the macro. Possible, but when moving code from one location to another it's best to move it with minimal changes. Even so, I've now changed this. -- FTTC broadband for 0.8mile line: 5.8Mbps down 500kbps up. Estimation in database were 13.1 to 19Mbit for a good line, about 7.5+ for a bad. Estimate before purchase was "up to 13.2Mbit". _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel