[PATCH 0/8] Fix some bugs/races in imx-drm

Russell King - ARM Linux linux at arm.linux.org.uk
Tue Dec 17 19:09:56 UTC 2013


On Tue, Dec 17, 2013 at 08:05:41AM -0800, Greg Kroah-Hartman wrote:
> On Tue, Dec 17, 2013 at 03:17:02PM +0000, Russell King - ARM Linux wrote:
> > On Tue, Dec 17, 2013 at 01:04:20PM +0800, Shawn Guo wrote:
> > > On Mon, Dec 16, 2013 at 12:38:23PM +0000, Russell King - ARM Linux wrote:
> > > > Russell King (8):
> > > >       imx-drm: imx-drm-core: fix error cleanup path for imx_drm_add_crtc()
> > > >       imx-drm: imx-drm-core: fix DRM cleanup paths
> > > >       imx-drm: fix missing symbol exports
> > > 
> > > Except the little doubt I replied on this one, for the whole series:
> > > 
> > > Acked-by: Shawn Guo <shawn.guo at linaro.org>
> > > Tested-by: Shawn Guo <shawn.guo at linaro.org>
> > > 
> > > >       imx-drm: ipu-v3: fix potential CRTC device registration race
> > > >       imx-drm: imx-tve: don't call sleeping functions beneath enable_lock spinlo
> > > >       imx-drm: imx-drm-core: use defined constant for number of CRTCs.
> > > >       imx-drm: imx-drm-core: make imx_drm_crtc_register() safer
> > > >       imx-drm: imx-drm-core: improve safety of imx_drm_add_crtc()
> > 
> > Greg,
> > 
> > Do you want me to re-send them with these acks/tested-by's added, or are
> > you happy to take them as-is?
> 
> I can take these as-is.

Thanks.

> > I also have some further patches to send (I think three) which apply on
> > top of these but are not fixes (so aren't -rc material), more cleanups.
> 
> Ok, feel free to send them, I can take them.

They'll be along shortly.  Diffstat looks smaller - just one file:

 drivers/staging/imx-drm/imx-drm-core.c |   55 +++++++++++--------------------
 1 files changed, 20 insertions(+), 35 deletions(-)

Russell King (3):
      imx-drm: imx-drm-core: use the crtc drm device for vblank
      imx-drm: imx-drm-core: avoid going the long route round for drm_device
      imx-drm: imx-drm-core: merge imx_drm_crtc_register() into imx_drm_add_crtc()



More information about the devel mailing list