1. 21 Dec, 2018 5 commits
  2. 20 Dec, 2018 3 commits
  3. 19 Dec, 2018 3 commits
    • Andrei.Stefanescu@microchip.com's avatar
      regulator: mcp16502: code cleanup · 308144ce
      Andrei.Stefanescu@microchip.com authored
      This patch does the following:
      - align parameter with parenthesis
      - fix compile error
      
      If CONFIG_SUSPEND is not set the dummy pm_ops
      callbacks are named mcp16502_suspend and mcp16502_resume
      instead of mcp16502_suspend_noirq and mcp16502_resume_noirq.
      
      Excerpt from compile log (kbuild test robot):
      
      In file included from include/linux/device.h:23:0,
      from include/linux/gpio/driver.h:5,
      from include/asm-generic/gpio.h:13,
      from include/linux/gpio.h:62,
      from drivers/regulator/mcp16502.c:11:
      
      >> drivers/regulator/mcp16502.c:527:32: error: 'mcp16502_suspend_noirq'
      undeclared here (not in a function); did you mean 'mcp16502_suspend'?
      SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(mcp16502_suspend_noirq,
      
      >> drivers/regulator/mcp16502.c:528:10: error: 'mcp16502_resume_noirq'
      undeclared here (not in a function); did you mean 'mcp16502_suspend_noirq'?
      mcp16502_resume_noirq)
      
      vim +527 drivers/regulator/mcp16502.c
      
      524
      525	#ifdef CONFIG_PM
      526	static const struct dev_pm_ops mcp16502_pm_ops = {
      > 527		SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(mcp16502_suspend_noirq,
      > 528					     mcp16502_resume_noirq)
      529	};
      530	#endif
      531	static const struct i2c_device_id mcp16502_i2c_id[] = {
      532		{ "mcp16502", 0 },
      533		{ }
      534	};
      535	MODULE_DEVICE_TABLE(i2c, mcp16502_i2c_id);
      536
      Signed-off-by: default avatarAndrei Stefanescu <andrei.stefanescu@microchip.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      308144ce
    • Wei Yongjun's avatar
      regulator: act8945a-regulator: make symbol act8945a_pm static · 78e6fbf3
      Wei Yongjun authored
      Fixes the following sparse warning:
      
      drivers/regulator/act8945a-regulator.c:340:1: warning:
       symbol 'act8945a_pm' was not declared. Should it be static?
      
      Fixes: 7482d6ec ("regulator: act8945a-regulator: Implement PM functionalities")
      Signed-off-by: default avatarWei Yongjun <weiyongjun1@huawei.com>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      78e6fbf3
    • Kangjie Lu's avatar
      drivers/regulator: fix a missing check of return value · 966e927b
      Kangjie Lu authored
      If palmas_smps_read() fails, we should not use the read data in "reg"
      which may contain random value. The fix inserts a check for the return
      value of palmas_smps_read(): If it fails, we return the error code
      upstream and stop using "reg".
      Signed-off-by: default avatarKangjie Lu <kjlu@umn.edu>
      Signed-off-by: default avatarMark Brown <broonie@kernel.org>
      966e927b
  4. 17 Dec, 2018 2 commits
  5. 16 Dec, 2018 1 commit
  6. 14 Dec, 2018 21 commits
  7. 13 Dec, 2018 5 commits