Re: [PATCH 20/32] video/msm_fb: use system_wq instead of dedicated workqueues

[Carl Vanderlip <carlv-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>]

On Tue, 2011-01-25 at 11:16 -0800, Daniel Walker wrote:
> On Tue, 2011-01-25 at 11:14 -0800, David Brown wrote:
> > On Tue, Jan 25 2011, Daniel Walker wrote:
> > 
> > > On Tue, 2011-01-25 at 14:45 +0100, Tejun Heo wrote:
> > >> On Mon, Jan 03, 2011 at 10:10:45AM -0800, Daniel Walker wrote:
> > >> > On Mon, 2011-01-03 at 18:06 +0100, Stanislaw Gruszka wrote:
> > >> > > On Mon, Jan 03, 2011 at 02:49:43PM +0100, Tejun Heo wrote:
> > >> > > > With cmwq, there's no reason to use separate workqueues.  Drop
> > >> > > > msmfb_info->resume_workqueue and use system_wq instead.
> > >> > > > 
> > >> > > > Signed-off-by: Tejun Heo <tj-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
> > >> > > > Cc: Stanislaw Gruszka <stf_xl-5tc4TXWwyLM@public.gmane.org>
> > >> > > I'm not the right person, CC according to MAINTAINERS
> > >> > 
> > >> > ARM/QUALCOMM MSM MACHINE SUPPORT
> > >> > M:      David Brown <davidb-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
> > >> > M:      Daniel Walker <dwalker-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
> > >> > M:      Bryan Huntsman <bryanh-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
> > >> 
> > >> Oops, sorry about that.  So how does the patch look?  Are you gonna
> > >> take it through msm tree or shall I take it?
> > >
> > > David ?
> > >
> > > Daniel
> > 
> > Is it difficult for you to test this change, Daniel?  I can pull it in
> > once we verify it still works.
> 
> Carl V. can test it, and review it I think ..
> 
> Daniel

Tested booting on MSM7X00A; appears to work fine (all I checked was that
it arrived at a buildroot login prompt).

Carl V.

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html