Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1261237

Re: [PATCH] regulator: Use regulator_lock_supply() for get_voltage() too

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi Mark,

[auto build test WARNING on regulator/for-next -- if it's inappropriate base, please suggest rules for selecting the more suitable base]

url:    https://github.com/0day-ci/linux/commits/Mark-Brown/regulator-Use-regulator_lock_supply-for-get_voltage-too/20151103-144509
config: x86_64-randconfig-x013-11022153 (attached as .config)
reproduce:
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All warnings (new ones prefixed by >>):

   drivers/regulator/core.c: In function '_regulator_get_voltage':
>> drivers/regulator/core.c:3113:32: warning: passing argument 1 of '_regulator_get_voltage' from incompatible pointer type [-Wincompatible-pointer-types]
      ret = _regulator_get_voltage(rdev->supply);
                                   ^
   drivers/regulator/core.c:3097:12: note: expected 'struct regulator_dev *' but argument is of type 'struct regulator *'
    static int _regulator_get_voltage(struct regulator_dev *rdev)
               ^
   drivers/regulator/core.c: In function 'regulator_get_voltage':
>> drivers/regulator/core.c:3140:26: warning: passing argument 1 of 'regulator_unlock_supply' from incompatible pointer type [-Wincompatible-pointer-types]
     regulator_unlock_supply(&regulator->rdev->mutex);
                             ^
   drivers/regulator/core.c:159:13: note: expected 'struct regulator_dev *' but argument is of type 'struct mutex *'
    static void regulator_unlock_supply(struct regulator_dev *rdev)
                ^

vim +/_regulator_get_voltage +3113 drivers/regulator/core.c

  3107			ret = rdev->desc->ops->get_voltage(rdev);
  3108		} else if (rdev->desc->ops->list_voltage) {
  3109			ret = rdev->desc->ops->list_voltage(rdev, 0);
  3110		} else if (rdev->desc->fixed_uV && (rdev->desc->n_voltages == 1)) {
  3111			ret = rdev->desc->fixed_uV;
  3112		} else if (rdev->supply) {
> 3113			ret = _regulator_get_voltage(rdev->supply);
  3114		} else {
  3115			return -EINVAL;
  3116		}
  3117	
  3118		if (ret < 0)
  3119			return ret;
  3120		return ret - rdev->constraints->uV_offset;
  3121	}
  3122	
  3123	/**
  3124	 * regulator_get_voltage - get regulator output voltage
  3125	 * @regulator: regulator source
  3126	 *
  3127	 * This returns the current regulator voltage in uV.
  3128	 *
  3129	 * NOTE: If the regulator is disabled it will return the voltage value. This
  3130	 * function should not be used to determine regulator state.
  3131	 */
  3132	int regulator_get_voltage(struct regulator *regulator)
  3133	{
  3134		int ret;
  3135	
  3136		regulator_lock_supply(regulator->rdev);
  3137	
  3138		ret = _regulator_get_voltage(regulator->rdev);
  3139	
> 3140		regulator_unlock_supply(&regulator->rdev->mutex);
  3141	
  3142		return ret;
  3143	}

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH] regulator: Use regulator_lock_supply() for get_voltage() too Mark Brown <broonie@kernel.org> - 2015-11-03 07:20 +0100
  Re: [PATCH] regulator: Use regulator_lock_supply() for get_voltage()  too kbuild test robot <lkp@intel.com> - 2015-11-03 08:10 +0100

csiph-web