diff options
| author | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-04-11 09:19:12 (GMT) |
|---|---|---|
| committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2012-04-11 09:19:12 (GMT) |
| commit | 32c8fad438200b523ec44a49b40cb8ffdf715f7c (patch) | |
| tree | 5470d156542e081920e5ff468da3121918eca3e6 /drivers | |
| parent | 9eb0c4218aa444f863e7f54909351d5b4f0fac06 (diff) | |
| download | linux-32c8fad438200b523ec44a49b40cb8ffdf715f7c.tar.xz | |
regulator: Appease smatch in regulator_register()
We don't support missing configs at all so segfaulting isn't that bad
but since we've got checks in the code move the dereference after them.
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/regulator/core.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c index 4bb1f84..7943fd6 100644 --- a/drivers/regulator/core.c +++ b/drivers/regulator/core.c @@ -2833,13 +2833,15 @@ regulator_register(const struct regulator_desc *regulator_desc, const struct regulator_init_data *init_data; static atomic_t regulator_no = ATOMIC_INIT(0); struct regulator_dev *rdev; - struct device *dev = config->dev; + struct device *dev; int ret, i; const char *supply = NULL; if (regulator_desc == NULL || config == NULL) return ERR_PTR(-EINVAL); + dev = config->dev; + if (regulator_desc->name == NULL || regulator_desc->ops == NULL) return ERR_PTR(-EINVAL); |