On Fri, 3 Mar 2017 23:21:29 +0100 Richard Weinberger wrote: > Am 03.03.2017 um 15:11 schrieb Boris Brezillon: > >> And add a list of successfully added notifiers, along with their > >> data pointer, to the MTD device. That's simple and would also remove > >> the need for notifier to have a private list of their instances as I > >> had to do here. > > > > And then you're abusing the notifier concept. As said earlier, a > > notifier is not necessarily using the device, and thus, don't > > necessarily need private data. > > It's not only about what is the simplest solution for your use case, > > but also what other users want/need. > > Yes, please don't use the mtd_notifier. > I strongly vote to embed the nvmem pointer into struct mtd_info. Ok, I'll do that. However it mean it will have to stays in drivers/mtd as it then become part of the MTD core. Alban