On Thu, Apr 17, 2014 at 02:01:32PM -0600, Shuah Khan wrote: > Operating on the lock should be atomic, which is what devres_update() > is doing. It can be simplified as follows by holding devres_lock > in devm_token_lock(). > > spin_lock_irqsave(&dev->devres_lock, flags); > if (tkn_ptr->status == TOKEN_DEVRES_FREE) > tkn_ptr->status = TOKEN_DEVRES_BUSY; > spin_unlock_irqrestore(&dev->devres_lock, flags); > > Is this in-line with what you have in mind?
How is that different from tkn_ptr->status = TOKEN_DEVRES_BUSY? -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html