On Wed, Sep 11, 2013 at 9:19 AM, Mark Brown broonie@kernel.org wrote:
From: Mark Brown broonie@linaro.org
Cut down on the size of core.c a bit more and ensure that the devres versions of things don't do too much peering inside the internals of the APIs they wrap.
Signed-off-by: Mark Brown broonie@linaro.org
drivers/regulator/Makefile | 2 +- drivers/regulator/core.c | 252 +------------------------------------------ drivers/regulator/devres.c | 252 +++++++++++++++++++++++++++++++++++++++++++ drivers/regulator/internal.h | 38 +++++++ 4 files changed, 292 insertions(+), 252 deletions(-) create mode 100644 drivers/regulator/devres.c create mode 100644 drivers/regulator/internal.h diff --git a/drivers/regulator/internal.h b/drivers/regulator/internal.h new file mode 100644 index 0000000..84bbda1 --- /dev/null +++ b/drivers/regulator/internal.h @@ -0,0 +1,38 @@ +/*
- internal.h -- Voltage/Current Regulator framework internal code
- Copyright 2007, 2008 Wolfson Microelectronics PLC.
- Copyright 2008 SlimLogic Ltd.
- Author: Liam Girdwood lrg@slimlogic.co.uk
- This program is free software; you can redistribute it and/or modify it
- under the terms of the GNU General Public License as published by the
- Free Software Foundation; either version 2 of the License, or (at your
- option) any later version.
- */
+#ifndef __REGULATOR_INTERNAL_H +#define __REGULATOR_INTERNAL_H
+/*
- struct regulator
- One for each consumer device.
- */
+struct regulator {
struct device *dev;
struct list_head list;
unsigned int always_on:1;
unsigned int bypass:1;
int uA_load;
int min_uV;
int max_uV;
char *supply_name;
struct device_attribute dev_attr;
struct regulator_dev *rdev;
struct dentry *debugfs;
+};
don't you need any forward declaration here nor any include (like for list_head and device_attribute)?
-- Thiago