* Goucem, Fabrice <f-goucem@xxxxxx> [100309 05:11]: > Hi Hari, > > Thanks for your comment. > What you're saying is perfectly true, however I find current way is a bit awkward and potentially dangerous if for any reason value of OMAP_DMA_SYNC_PACKET changes in the future, or if more constants are defined later on. > > I think proposed patch improves the readability of the code and makes it clearer to developers what DMA synchronisations are supported. In that case the patch subject is wrong, these features are already supported as Hari pointed out. Maybe update the patch subject and description accordingly? Tony -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html