* [PATCH] regulator: db8500-prcmu: small fixes
@ 2011-07-06 3:41 Axel Lin
2011-07-06 6:45 ` Mark Brown
0 siblings, 1 reply; 3+ messages in thread
From: Axel Lin @ 2011-07-06 3:41 UTC (permalink / raw)
To: linux-kernel; +Cc: Sundar Iyer, Bengt Jonsson, Liam Girdwood, Mark Brown
Small cleanups for better readability.
Signed-off-by: Axel Lin <axel.lin@gmail.com>
---
drivers/regulator/db8500-prcmu.c | 14 +++-----------
1 files changed, 3 insertions(+), 11 deletions(-)
diff --git a/drivers/regulator/db8500-prcmu.c b/drivers/regulator/db8500-prcmu.c
index e5f7b8f..2bb8f45 100644
--- a/drivers/regulator/db8500-prcmu.c
+++ b/drivers/regulator/db8500-prcmu.c
@@ -266,7 +266,7 @@ static struct regulator_ops db8500_regulator_switch_ops = {
* Regulator information
*/
static struct db8500_regulator_info
- db8500_regulator_info[DB8500_NUM_REGULATORS] = {
+db8500_regulator_info[DB8500_NUM_REGULATORS] = {
[DB8500_REGULATOR_VAPE] = {
.desc = {
.name = "db8500-vape",
@@ -492,11 +492,9 @@ static int __devinit db8500_regulator_probe(struct platform_device *pdev)
info->desc.name, err);
/* if failing, unregister all earlier regulators */
- i--;
- while (i >= 0) {
+ while (--i >= 0) {
info = &db8500_regulator_info[i];
regulator_unregister(info->rdev);
- i--;
}
return err;
}
@@ -536,13 +534,7 @@ static struct platform_driver db8500_regulator_driver = {
static int __init db8500_regulator_init(void)
{
- int ret;
-
- ret = platform_driver_register(&db8500_regulator_driver);
- if (ret < 0)
- return -ENODEV;
-
- return 0;
+ return platform_driver_register(&db8500_regulator_driver);
}
static void __exit db8500_regulator_exit(void)
--
1.7.4.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] regulator: db8500-prcmu: small fixes
2011-07-06 3:41 [PATCH] regulator: db8500-prcmu: small fixes Axel Lin
@ 2011-07-06 6:45 ` Mark Brown
2011-07-06 19:29 ` Liam Girdwood
0 siblings, 1 reply; 3+ messages in thread
From: Mark Brown @ 2011-07-06 6:45 UTC (permalink / raw)
To: Axel Lin; +Cc: linux-kernel, Sundar Iyer, Bengt Jonsson, Liam Girdwood
On Wed, Jul 06, 2011 at 11:41:12AM +0800, Axel Lin wrote:
> Small cleanups for better readability.
>
> Signed-off-by: Axel Lin <axel.lin@gmail.com>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] regulator: db8500-prcmu: small fixes
2011-07-06 6:45 ` Mark Brown
@ 2011-07-06 19:29 ` Liam Girdwood
0 siblings, 0 replies; 3+ messages in thread
From: Liam Girdwood @ 2011-07-06 19:29 UTC (permalink / raw)
To: Mark Brown
Cc: Axel Lin, linux-kernel@vger.kernel.org, Sundar Iyer,
Bengt Jonsson
On 06/07/11 07:45, Mark Brown wrote:
> On Wed, Jul 06, 2011 at 11:41:12AM +0800, Axel Lin wrote:
>> Small cleanups for better readability.
>>
>> Signed-off-by: Axel Lin <axel.lin@gmail.com>
>
> Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Applied.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2011-07-06 19:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-07-06 3:41 [PATCH] regulator: db8500-prcmu: small fixes Axel Lin
2011-07-06 6:45 ` Mark Brown
2011-07-06 19:29 ` Liam Girdwood
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox