[PATCH 4/7] spi/pl022: move device disable to workqueue thread
Wolfram Sang
w.sang at pengutronix.de
Wed Dec 7 15:22:03 EST 2011
Hi Linus,
On Tue, Nov 22, 2011 at 10:28:44AM +0100, Linus Walleij wrote:
> On Tue, Nov 22, 2011 at 9:28 AM, Viresh Kumar <viresh.kumar at st.com> wrote:
> > On 11/22/2011 1:55 PM, Linus WALLEIJ wrote:
> >> if (list_empty(&pl022->queue) || !pl022->running) {
> >> + if (pl022->busy) {
> >> + pm_runtime_put(&pl022->adev->dev);
> >> + }
> >
> > We used to get warnings from checkpatch, for single line code inside {}.
> > Don't we get them anymore?
>
> It's driven by heuristics, I think it only warns on the first level since
> elsewhere you may want braces for readability.
>
> But I'll take it out when I add you Reviewed-by...
Does that imply you will send out a V3 of this series somewhen and I
don't need to add this right now to the for-grant branch I am currently
preparing?
Thanks,
Wolfram
--
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20111207/b856cd69/attachment.sig>
More information about the linux-arm-kernel
mailing list