> -----Original Message-----
> From: Laxman Dewangan
> Sent: Saturday, October 06, 2012 8:48 PM
> To: broo...@opensource.wolfsonmicro.com; l...@ti.com;
> sa...@linux.intel.com
> Cc: Venu Byravarasu; axel....@gmail.com; linux-kernel@vger.kernel.org;
> Laxman Dewangan
> Subject: [PATCH 1/5] regulator: tps65090: rename driver name and regulator
> name
>
> To make the names proper and more appropriate:
> Rename the driver name from tps65090-regulator to tps65090-pmic.
> Rename the regulators from TPS65090_ID_* to TPS65090_REGULATOR_*
>
> Signed-off-by: Laxman Dewangan <ldewan...@nvidia.com>
> ---
> drivers/regulator/tps65090-regulator.c | 8 ++++----
> include/linux/regulator/tps65090-regulator.h | 20 ++++++++++----------
> 2 files changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/drivers/regulator/tps65090-regulator.c
> b/drivers/regulator/tps65090-regulator.c
> index 001ad55..96d59ad 100644
> --- a/drivers/regulator/tps65090-regulator.c
> +++ b/drivers/regulator/tps65090-regulator.c
> @@ -43,14 +43,14 @@ static struct regulator_ops tps65090_ops = {
>
> #define tps65090_REG(_id) \
> { \
> - .id = TPS65090_ID_##_id, \
> + .id = TPS65090_REGULATOR_##_id, \
> .desc = { \
> .name = tps65090_rails(_id), \
> - .id = TPS65090_ID_##_id, \
> + .id = TPS65090_REGULATOR_##_id, \
> .ops = &tps65090_ops, \
> .type = REGULATOR_VOLTAGE, \
> .owner = THIS_MODULE, \
> - .enable_reg = (TPS65090_ID_##_id) + 12, \
> + .enable_reg = (TPS65090_REGULATOR_##_id) + 12, \
> .enable_mask = BIT(0), \
> }, \
> }
> @@ -126,7 +126,7 @@ static int __devexit
> tps65090_regulator_remove(struct platform_device *pdev)
>
> static struct platform_driver tps65090_regulator_driver = {
> .driver = {
> - .name = "tps65090-regulator",
> + .name = "tps65090-pmic",
As the driver itself is regulator driver, why should we rename it to pmic?
> .owner = THIS_MODULE,
> },
> .probe = tps65090_regulator_probe,
> diff --git a/include/linux/regulator/tps65090-regulator.h
> b/include/linux/regulator/tps65090-regulator.h
> index 0fa04b6..5e27d4a 100644
> --- a/include/linux/regulator/tps65090-regulator.h
> +++ b/include/linux/regulator/tps65090-regulator.h
> @@ -24,16 +24,16 @@
> #define tps65090_rails(_name) "tps65090_"#_name
>
> enum {
> - TPS65090_ID_DCDC1,
> - TPS65090_ID_DCDC2,
> - TPS65090_ID_DCDC3,
> - TPS65090_ID_FET1,
> - TPS65090_ID_FET2,
> - TPS65090_ID_FET3,
> - TPS65090_ID_FET4,
> - TPS65090_ID_FET5,
> - TPS65090_ID_FET6,
> - TPS65090_ID_FET7,
> + TPS65090_REGULATOR_DCDC1,
> + TPS65090_REGULATOR_DCDC2,
> + TPS65090_REGULATOR_DCDC3,
> + TPS65090_REGULATOR_FET1,
> + TPS65090_REGULATOR_FET2,
> + TPS65090_REGULATOR_FET3,
> + TPS65090_REGULATOR_FET4,
> + TPS65090_REGULATOR_FET5,
> + TPS65090_REGULATOR_FET6,
> + TPS65090_REGULATOR_FET7,
> };
>
> /*
> --
> 1.7.1.1
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
