On 10/09/18 08:36, Boris Brezillon wrote:

quoted

quoted

--- /dev/null
+++ b/include/linux/nvmem-machine.h

This should go in nvmem-consumer.h, I don't think we should add header
files for each usecase. These are nvmem consumers so lets put them in
correct header file.

Actually no, it should go in nvmem-provider.h. Consumer should not be
allowed to define cells, only reference existing ones.The usecase can be provider and consumer.

In arm machine use case it is a consumer and it other use cases this can 
be a provider.

I agree with you, It makes more sense to be in nvmem-provider.h

thanks,
srini

Regards,

Boris