On 09/17/2014 05:24 AM, Ingo Molnar wrote: > > Just to paint the bike shed a bit, this could be further > simplified to something like: > > ... > > ret = pmc_dbgfs_register(pmc, pdev); > if (ret) > iounmap(pmc->regmap); > > return ret; > > and then your patch will be perfect! :-) >
Yes, good idea. I sent a v3 with that implemented. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/