[PATCH] regulator: core: fix build warning

[PATCH] regulator: core: fix build warning

[Sudip Mukherjee]

We were getting warning about:
warning: 'regulator_lock_supply' defined but not used
warning: 'regulator_unlock_supply' defined but not used

It appears that there is no user of regulator_lock_supply() and
regulator_unlock_supply().
Commit 9f01cd4a915e ("regulator: core: introduce function to lock
regulators and its supplies") says there will be future users, but as of
now there is no user and is creating build warnings.

Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org>
---
 drivers/regulator/core.c | 39 ---------------------------------------
 1 file changed, 39 deletions(-)

diff --git a/drivers/regulator/core.c b/drivers/regulator/core.c
index 7896ef5..3ba7131 100644
--- a/drivers/regulator/core.c
+++ b/drivers/regulator/core.c
@@ -133,45 +133,6 @@ static bool have_full_constraints(void)
 }
 
 /**
- * regulator_lock_supply - lock a regulator and its supplies
- * @rdev:         regulator source
- */
-static void regulator_lock_supply(struct regulator_dev *rdev)
-{
-	struct regulator *supply;
-	int i = 0;
-
-	while (1) {
-		mutex_lock_nested(&rdev->mutex, i++);
-		supply = rdev->supply;
-
-		if (!rdev->supply)
-			return;
-
-		rdev = supply->rdev;
-	}
-}
-
-/**
- * regulator_unlock_supply - unlock a regulator and its supplies
- * @rdev:         regulator source
- */
-static void regulator_unlock_supply(struct regulator_dev *rdev)
-{
-	struct regulator *supply;
-
-	while (1) {
-		mutex_unlock(&rdev->mutex);
-		supply = rdev->supply;
-
-		if (!rdev->supply)
-			return;
-
-		rdev = supply->rdev;
-	}
-}
-
-/**
  * of_get_regulator - get a regulator device node based on supply name
  * @dev: Device pointer for the consumer (of regulator) device
  * @supply: regulator supply name
-- 
1.9.1

Re: [PATCH] regulator: core: fix build warning

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 659 bytes --]

On Wed, Oct 07, 2015 at 05:24:14PM +0530, Sudip Mukherjee wrote:
> We were getting warning about:
> warning: 'regulator_lock_supply' defined but not used
> warning: 'regulator_unlock_supply' defined but not used
> 
> It appears that there is no user of regulator_lock_supply() and
> regulator_unlock_supply().
> Commit 9f01cd4a915e ("regulator: core: introduce function to lock
> regulators and its supplies") says there will be future users, but as of
> now there is no user and is creating build warnings.

No, there are other parts of the series in review - the earlier parts
are merged now so that we don't have to constantly re-review them.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]