mirror of
https://github.com/rd-stuffs/msm-4.14.git
synced 2025-02-20 11:45:48 +08:00
regulator: da905{2,5}: Remove unnecessary array check
commit 5a7d7d0f9f791b1e13f26dbbb07c86482912ad62 upstream. Clang warns that the address of a pointer will always evaluated as true in a boolean context: drivers/regulator/da9052-regulator.c:423:22: warning: address of array 'pdata->regulators' will always evaluate to 'true' [-Wpointer-bool-conversion] if (pdata && pdata->regulators) { ~~ ~~~~~~~^~~~~~~~~~ drivers/regulator/da9055-regulator.c:615:22: warning: address of array 'pdata->regulators' will always evaluate to 'true' [-Wpointer-bool-conversion] if (pdata && pdata->regulators) { ~~ ~~~~~~~^~~~~~~~~~ Link: https://github.com/ClangBuiltLinux/linux/issues/142 Signed-off-by: Nathan Chancellor <natechancellor@gmail.com> Signed-off-by: Mark Brown <broonie@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
46406aef7c
commit
c5158fb609
@ -421,7 +421,7 @@ static int da9052_regulator_probe(struct platform_device *pdev)
|
||||
config.dev = &pdev->dev;
|
||||
config.driver_data = regulator;
|
||||
config.regmap = da9052->regmap;
|
||||
if (pdata && pdata->regulators) {
|
||||
if (pdata) {
|
||||
config.init_data = pdata->regulators[cell->id];
|
||||
} else {
|
||||
#ifdef CONFIG_OF
|
||||
|
@ -612,7 +612,7 @@ static int da9055_regulator_probe(struct platform_device *pdev)
|
||||
config.driver_data = regulator;
|
||||
config.regmap = da9055->regmap;
|
||||
|
||||
if (pdata && pdata->regulators) {
|
||||
if (pdata) {
|
||||
config.init_data = pdata->regulators[pdev->id];
|
||||
} else {
|
||||
ret = da9055_regulator_dt_init(pdev, regulator, &config,
|
||||
|
Loading…
x
Reference in New Issue
Block a user