Re: [PATCH v2 11/16] nvmem: add support for cell info

From: Srinivas Kandagatla
Date: Mon Sep 10 2018 - 04:53:56 EST




On 10/09/18 08:36, Boris Brezillon wrote:
--- /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