On Friday 02 January 2015 17:02:21 Rickard Strandqvist wrote: > 2015-01-02 10:46 GMT+01:00 Arnd Bergmann <arnd@xxxxxxxx>: > > On Thursday 01 January 2015 18:06:34 Rickard Strandqvist wrote: > > > > You have correctly spotted the only place that references > > voltdm_for_each_pwrdm > > and changed the comment, but you failed to notice that it no longer makes > > sense to call voltdm_add_pwrdm or keep the voltdm->pwrdm_list pointer > > at all, so this function, and the associated list heads can all be removed > > as well. > > > > Arnd > > > > > Hi Arnd > > Thanks for your reply. > > I see what you mean, but you are aware that voltdm_add_pwrdm() also be > called from: > arch/arm/mach-omap2/powerdomain.c in _pwrdm_register() Yes, that call needs to be removed as well of course. > I have also removde the voltdm_for_each() Ok, sounds good. That list of course has to stay though, so removing _voltdm_register would be bad ;-) Arnd -- 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