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

From: Keerthy
Date: Wed Jun 22 2016 - 06:37:36 EST




On Wednesday 22 June 2016 03:46 PM, Mark Brown wrote:
On Wed, Jun 22, 2016 at 03:44:02PM +0530, Keerthy wrote:

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?