Re: [PATCH 5/6] regulator: anatop-regulator: make regulator-name using optionally

2017-04-12 Thread Dong Aisheng
On Tue, Apr 11, 2017 at 09:38:18PM +0100, Mark Brown wrote: > On Wed, Apr 12, 2017 at 09:58:46AM +0800, Dong Aisheng wrote: > > rdesc->name/regulator-name is optional according to standard regulator > > binding doc. Use it conditionally to avoid a kernel NULL point crash. > > It is optional in the

Re: [PATCH 5/6] regulator: anatop-regulator: make regulator-name using optionally

2017-04-11 Thread Mark Brown
On Wed, Apr 12, 2017 at 09:58:46AM +0800, Dong Aisheng wrote: > rdesc->name/regulator-name is optional according to standard regulator > binding doc. Use it conditionally to avoid a kernel NULL point crash. It is optional in the standard binding because it is used to override the name statically p

[PATCH 5/6] regulator: anatop-regulator: make regulator-name using optionally

2017-04-11 Thread Dong Aisheng
rdesc->name/regulator-name is optional according to standard regulator binding doc. Use it conditionally to avoid a kernel NULL point crash. Cc: Liam Girdwood Cc: Mark Brown Cc: Shawn Guo Cc: Sascha Hauer Cc: Robin Gong Signed-off-by: Dong Aisheng --- drivers/regulator/anatop-regulator.c |