Message ID | 1545047086-30286-1-git-send-email-andrei.stefanescu@microchip.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | regulator: mcp16502: code cleanup | expand |
On Mon, Dec 17, 2018 at 11:44:59AM +0000, Andrei.Stefanescu@microchip.com wrote: > 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. Thanks for fixing this. > > Excerpt from compile log (kbuild test robot): There's no need to include the entire output from the kbuild robot in your commit log, especially when it's much larger than either the change or description of the change - it gets in the way of reading things as it makes it harder to pick out relevant information.
diff --git a/drivers/regulator/mcp16502.c b/drivers/regulator/mcp16502.c index 25d7b43..809a664 100644 --- a/drivers/regulator/mcp16502.c +++ b/drivers/regulator/mcp16502.c @@ -518,14 +518,14 @@ static int mcp16502_resume_noirq(struct device *dev) return 0; } #else /* !CONFIG_SUSPEND */ -#define mcp16502_suspend NULL -#define mcp16502_resume NULL +#define mcp16502_suspend_noirq NULL +#define mcp16502_resume_noirq NULL #endif /* !CONFIG_SUSPEND */ #ifdef CONFIG_PM static const struct dev_pm_ops mcp16502_pm_ops = { SET_NOIRQ_SYSTEM_SLEEP_PM_OPS(mcp16502_suspend_noirq, - mcp16502_resume_noirq) + mcp16502_resume_noirq) }; #endif static const struct i2c_device_id mcp16502_i2c_id[] = {