regulator: Remove unneeded include of linux/delay.h from regulator drivers

PATCH Applied Axel Lin 2 participants 2 messages Apr 18, 2012 – Apr 23, 2012

Subsystems (3)

OMAP2+ support Voltage and current regulator framework Samsung multifunction PMIC device drivers

lore

Share & tools

lore

Axel Lin

author

2012-04-18 00:52 UTC #0 lore raw

PATCH 1/1 regulator: Remove unneeded include of linux/delay.h from regulator drivers

All the drivers that need delay for the regulator voltage output voltage to stabilize after being enabled or after being set to a new value has been converted to implement enable_time and set_voltage_time_sel callbacks. Then regulator core will take care of the necessary delay.

For the drivers that don't need the delay, don't need to include linux/delay.h. This patch removes the unneeded include of linux/delay.h in regulator drivers.

Signed-off-by Axel Lin <[email protected]>

---
 drivers/regulator/aat2870-regulator.c  |    1 -
 drivers/regulator/ab3100.c             |    1 -
 drivers/regulator/fixed.c              |    1 -
 drivers/regulator/gpio-regulator.c     |    1 -
 drivers/regulator/isl6271a-regulator.c |    1 -
 drivers/regulator/max8997.c            |    1 -
 drivers/regulator/max8998.c            |    1 -
 drivers/regulator/rc5t583-regulator.c  |    1 -
 drivers/regulator/s5m8767.c            |    1 -
 drivers/regulator/tps62360-regulator.c |    1 -
 drivers/regulator/tps65023-regulator.c |    1 -
 drivers/regulator/tps6507x-regulator.c |    1 -
 drivers/regulator/tps65090-regulator.c |    1 -
 drivers/regulator/tps65910-regulator.c |    1 -
 drivers/regulator/tps65912-regulator.c |    1 -
 drivers/regulator/twl-regulator.c      |    1 -
 16 files changed, 0 insertions(+), 16 deletions(-)

drivers/regulator/aat2870-regulator.c +0 -1

index 7f000d6..06776ca 100644
@@ -24,7 +24,6 @@
 #include <linux/err.h>
 #include <linux/module.h>
 #include <linux/slab.h>
-#include <linux/delay.h>
 #include <linux/platform_device.h>
 #include <linux/regulator/driver.h>
 #include <linux/regulator/machine.h>

drivers/regulator/ab3100.c +0 -1

index ce61925..e2062ad 100644
@@ -13,7 +13,6 @@
 #include <linux/kernel.h>
 #include <linux/init.h>
 #include <linux/err.h>
-#include <linux/delay.h>
 #include <linux/platform_device.h>
 #include <linux/regulator/driver.h>
 #include <linux/mfd/abx500.h>

drivers/regulator/fixed.c +0 -1

index b47b005..bd437c1 100644
@@ -25,7 +25,6 @@
 #include <linux/regulator/driver.h>
 #include <linux/regulator/fixed.h>
 #include <linux/gpio.h>
-#include <linux/delay.h>
 #include <linux/slab.h>
 #include <linux/of.h>
 #include <linux/of_gpio.h>

drivers/regulator/gpio-regulator.c +0 -1

index f93b06b..9997d7a 100644
@@ -30,7 +30,6 @@
 #include <linux/regulator/machine.h>
 #include <linux/regulator/gpio-regulator.h>
 #include <linux/gpio.h>
-#include <linux/delay.h>
 #include <linux/slab.h>
 
 struct gpio_regulator_data {

drivers/regulator/isl6271a-regulator.c +0 -1

index 60044c3..5caaf1f 100644
@@ -22,7 +22,6 @@
 #include <linux/platform_device.h>
 #include <linux/regulator/driver.h>
 #include <linux/i2c.h>
-#include <linux/delay.h>
 #include <linux/slab.h>
 
 #define	ISL6271A_VOLTAGE_MIN	850000

drivers/regulator/max8997.c +0 -1

index f4ab2e3..01172c2 100644
@@ -22,7 +22,6 @@
  */
 
 #include <linux/bug.h>
-#include <linux/delay.h>
 #include <linux/err.h>
 #include <linux/gpio.h>
 #include <linux/slab.h>

drivers/regulator/max8998.c +0 -1

index 4ca22f9..052ee35 100644
@@ -28,7 +28,6 @@
 #include <linux/slab.h>
 #include <linux/interrupt.h>
 #include <linux/mutex.h>
-#include <linux/delay.h>
 #include <linux/platform_device.h>
 #include <linux/regulator/driver.h>
 #include <linux/mfd/max8998.h>

drivers/regulator/rc5t583-regulator.c +0 -1

index 81a74d1..f35dc527 100644
@@ -23,7 +23,6 @@
  */
 
 #include <linux/module.h>
-#include <linux/delay.h>
 #include <linux/init.h>
 #include <linux/slab.h>
 #include <linux/err.h>

drivers/regulator/s5m8767.c +0 -1

index cb53187..a31d358 100644
@@ -12,7 +12,6 @@
  */
 
 #include <linux/bug.h>
-#include <linux/delay.h>
 #include <linux/err.h>
 #include <linux/gpio.h>
 #include <linux/slab.h>

drivers/regulator/tps62360-regulator.c +0 -1

index 0657c98..20fef1d 100644
@@ -32,7 +32,6 @@
 #include <linux/regulator/tps62360.h>
 #include <linux/gpio.h>
 #include <linux/i2c.h>
-#include <linux/delay.h>
 #include <linux/slab.h>
 #include <linux/regmap.h>

drivers/regulator/tps65023-regulator.c +0 -1

index a1fd656..f841bd0 100644
@@ -23,7 +23,6 @@
 #include <linux/regulator/driver.h>
 #include <linux/regulator/machine.h>
 #include <linux/i2c.h>
-#include <linux/delay.h>
 #include <linux/slab.h>
 #include <linux/regmap.h>

drivers/regulator/tps6507x-regulator.c +0 -1

index 8e432004b..9dfc98b 100644
@@ -23,7 +23,6 @@
 #include <linux/regulator/driver.h>
 #include <linux/regulator/machine.h>
 #include <linux/regulator/tps6507x.h>
-#include <linux/delay.h>
 #include <linux/slab.h>
 #include <linux/mfd/tps6507x.h>

drivers/regulator/tps65090-regulator.c +0 -1

index 299237d..c44b448 100644
@@ -17,7 +17,6 @@
  */
 
 #include <linux/module.h>
-#include <linux/delay.h>
 #include <linux/init.h>
 #include <linux/slab.h>
 #include <linux/err.h>

drivers/regulator/tps65910-regulator.c +0 -1

index fe66a2c..3d9c54f 100644
@@ -20,7 +20,6 @@
 #include <linux/platform_device.h>
 #include <linux/regulator/driver.h>
 #include <linux/regulator/machine.h>
-#include <linux/delay.h>
 #include <linux/slab.h>
 #include <linux/gpio.h>
 #include <linux/mfd/tps65910.h>

drivers/regulator/tps65912-regulator.c +0 -1

index af335dd..7312e36 100644
@@ -20,7 +20,6 @@
 #include <linux/platform_device.h>
 #include <linux/regulator/driver.h>
 #include <linux/regulator/machine.h>
-#include <linux/delay.h>
 #include <linux/slab.h>
 #include <linux/gpio.h>
 #include <linux/mfd/tps65912.h>

drivers/regulator/twl-regulator.c +0 -1

index 7385b27..fffacb4 100644
@@ -12,7 +12,6 @@
 #include <linux/module.h>
 #include <linux/init.h>
 #include <linux/err.h>
-#include <linux/delay.h>
 #include <linux/platform_device.h>
 #include <linux/of.h>
 #include <linux/of_device.h>

Signature · 1 line

1.7.5.4

Mark Brown

Applied

2012-04-23 12:24 UTC #1 lore raw

Quoted 4 lines · Axel Lin

All the drivers that need delay for the regulator voltage output voltage to stabilize after being enabled or after being set to a new value has been converted to implement enable_time and set_voltage_time_sel callbacks. Then regulator core will take care of the necessary delay.

Applied, thanks.