diff options
| author | Maarten ter Huurne <maarten@treewalker.org> | 2016-03-17 15:05:06 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2016-03-21 18:52:07 +0100 |
| commit | e6e79fd9cee682b137779d2da3b379251927d99f (patch) | |
| tree | 3e004fa6fe4665a9176f6ca7c53ae44b51f8e199 /drivers/regulator/max8973-regulator.c | |
| parent | regulator: act8865: Remove redundant dev lookups (diff) | |
| download | linux-e6e79fd9cee682b137779d2da3b379251927d99f.tar.xz linux-e6e79fd9cee682b137779d2da3b379251927d99f.zip | |
regulator: act8865: Remove "too many regulators" error handler
The check would dereference pdata, which can be NULL in the non-DT
use case.
Nothing will break if pdata->num_regulators is larger than the number
of regulators that the driver defines: pdata->num_regulators is only
read in act8865_get_init_data() to iterate through pdata->regulators.
The error handler might have some value as a sanity check on the
platform data, but the platform data could be broken in many other
ways that are not checked for (unknown IDs, duplicate IDs), so I see
no reason to perform only this specific check.
Signed-off-by: Maarten ter Huurne <maarten@treewalker.org>
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'drivers/regulator/max8973-regulator.c')
0 files changed, 0 insertions, 0 deletions