Re: [PATCH 1/9] regulator: tps65217: Enable suspend configuration

On Wednesday 22 June 2016 03:46 PM, Mark Brown wrote:

On Wed, Jun 22, 2016 at 03:44:02PM +0530, Keerthy wrote:

quoted

Hence saving it in a static array and using it later in the ops functions to
disable or enable regulator during suspend.

Why a static array and not part of the dynamically allocated driver
data?

Okay. That can be done.
I can introduce another integer pointer to struct tps65217 which 
currently holds the driver data.

I will allocate memory for TPS65217_NUM_REGULATOR strobes during 
regulator probe. Is this approach okay?