Re: [PATCH RFC 30/46] imx-drm: remove separate imx-fbdev

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux