* [PATCH 1/2] ARM: kirkwood: cleanup lsxl board includes @ 2012-10-20 23:34 Michael Walle 2012-10-20 23:34 ` [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl Michael Walle 2012-10-22 3:06 ` [PATCH 1/2] ARM: kirkwood: cleanup lsxl board includes Jason Cooper 0 siblings, 2 replies; 7+ messages in thread From: Michael Walle @ 2012-10-20 23:34 UTC (permalink / raw) To: linux-arm-kernel Remove unneeded includes. These are leftovers from platform device registrations. Signed-off-by: Michael Walle <michael@walle.cc> --- arch/arm/mach-kirkwood/board-lsxl.c | 8 -------- 1 files changed, 0 insertions(+), 8 deletions(-) diff --git a/arch/arm/mach-kirkwood/board-lsxl.c b/arch/arm/mach-kirkwood/board-lsxl.c index 83d8975..64e5cc6 100644 --- a/arch/arm/mach-kirkwood/board-lsxl.c +++ b/arch/arm/mach-kirkwood/board-lsxl.c @@ -14,17 +14,9 @@ #include <linux/kernel.h> #include <linux/init.h> #include <linux/platform_device.h> -#include <linux/mtd/partitions.h> -#include <linux/ata_platform.h> -#include <linux/spi/flash.h> -#include <linux/spi/spi.h> #include <linux/mv643xx_eth.h> #include <linux/gpio.h> #include <linux/gpio-fan.h> -#include <linux/input.h> -#include <asm/mach-types.h> -#include <asm/mach/arch.h> -#include <mach/kirkwood.h> #include "common.h" #include "mpp.h" -- 1.7.2.5 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl 2012-10-20 23:34 [PATCH 1/2] ARM: kirkwood: cleanup lsxl board includes Michael Walle @ 2012-10-20 23:34 ` Michael Walle 2012-10-21 6:55 ` Andrew Lunn 2012-10-22 3:07 ` Jason Cooper 2012-10-22 3:06 ` [PATCH 1/2] ARM: kirkwood: cleanup lsxl board includes Jason Cooper 1 sibling, 2 replies; 7+ messages in thread From: Michael Walle @ 2012-10-20 23:34 UTC (permalink / raw) To: linux-arm-kernel Remove board specific gpio-fan driver registration. Moved into device tree. Signed-off-by: Michael Walle <michael@walle.cc> --- arch/arm/boot/dts/kirkwood-lsxl.dtsi | 11 ++++++++ arch/arm/mach-kirkwood/board-lsxl.c | 47 ---------------------------------- 2 files changed, 11 insertions(+), 47 deletions(-) diff --git a/arch/arm/boot/dts/kirkwood-lsxl.dtsi b/arch/arm/boot/dts/kirkwood-lsxl.dtsi index 8fea375..798e60e 100644 --- a/arch/arm/boot/dts/kirkwood-lsxl.dtsi +++ b/arch/arm/boot/dts/kirkwood-lsxl.dtsi @@ -94,4 +94,15 @@ gpios = <&gpio1 16 1>; }; }; + + gpio_fan { + compatible = "gpio-fan"; + gpios = <&gpio0 19 1 + &gpio0 18 1>; + gpio-fan,speed-map = <0 3 + 1500 2 + 3250 1 + 5000 0>; + alarm-gpios = <&gpio1 8 0>; + }; }; diff --git a/arch/arm/mach-kirkwood/board-lsxl.c b/arch/arm/mach-kirkwood/board-lsxl.c index 64e5cc6..7e18cad 100644 --- a/arch/arm/mach-kirkwood/board-lsxl.c +++ b/arch/arm/mach-kirkwood/board-lsxl.c @@ -16,7 +16,6 @@ #include <linux/platform_device.h> #include <linux/mv643xx_eth.h> #include <linux/gpio.h> -#include <linux/gpio-fan.h> #include "common.h" #include "mpp.h" @@ -45,51 +44,6 @@ static unsigned int lsxl_mpp_config[] __initdata = { 0 }; -#define LSXL_GPIO_FAN_HIGH 18 -#define LSXL_GPIO_FAN_LOW 19 -#define LSXL_GPIO_FAN_LOCK 40 - -static struct gpio_fan_alarm lsxl_alarm = { - .gpio = LSXL_GPIO_FAN_LOCK, -}; - -static struct gpio_fan_speed lsxl_speeds[] = { - { - .rpm = 0, - .ctrl_val = 3, - }, { - .rpm = 1500, - .ctrl_val = 1, - }, { - .rpm = 3250, - .ctrl_val = 2, - }, { - .rpm = 5000, - .ctrl_val = 0, - } -}; - -static int lsxl_gpio_list[] = { - LSXL_GPIO_FAN_HIGH, LSXL_GPIO_FAN_LOW, -}; - -static struct gpio_fan_platform_data lsxl_fan_data = { - .num_ctrl = ARRAY_SIZE(lsxl_gpio_list), - .ctrl = lsxl_gpio_list, - .alarm = &lsxl_alarm, - .num_speed = ARRAY_SIZE(lsxl_speeds), - .speed = lsxl_speeds, -}; - -static struct platform_device lsxl_fan_device = { - .name = "gpio-fan", - .id = -1, - .num_resources = 0, - .dev = { - .platform_data = &lsxl_fan_data, - }, -}; - /* * On the LS-XHL/LS-CHLv2, the shutdown process is following: * - Userland monitors key events until the power switch goes to off position @@ -120,7 +74,6 @@ void __init lsxl_init(void) kirkwood_ehci_init(); kirkwood_ge00_init(&lsxl_ge00_data); kirkwood_ge01_init(&lsxl_ge01_data); - platform_device_register(&lsxl_fan_device); /* register power-off method */ pm_power_off = lsxl_power_off; -- 1.7.2.5 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl 2012-10-20 23:34 ` [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl Michael Walle @ 2012-10-21 6:55 ` Andrew Lunn 2012-10-21 13:59 ` Michael Walle 2012-10-22 3:07 ` Jason Cooper 1 sibling, 1 reply; 7+ messages in thread From: Andrew Lunn @ 2012-10-21 6:55 UTC (permalink / raw) To: linux-arm-kernel On Sun, Oct 21, 2012 at 01:34:54AM +0200, Michael Walle wrote: > Remove board specific gpio-fan driver registration. Moved into device tree. > > Signed-off-by: Michael Walle <michael@walle.cc> > --- > arch/arm/boot/dts/kirkwood-lsxl.dtsi | 11 ++++++++ > arch/arm/mach-kirkwood/board-lsxl.c | 47 ---------------------------------- > 2 files changed, 11 insertions(+), 47 deletions(-) > > diff --git a/arch/arm/boot/dts/kirkwood-lsxl.dtsi b/arch/arm/boot/dts/kirkwood-lsxl.dtsi > index 8fea375..798e60e 100644 > --- a/arch/arm/boot/dts/kirkwood-lsxl.dtsi > +++ b/arch/arm/boot/dts/kirkwood-lsxl.dtsi > @@ -94,4 +94,15 @@ > gpios = <&gpio1 16 1>; > }; > }; > + > + gpio_fan { > + compatible = "gpio-fan"; > + gpios = <&gpio0 19 1 > + &gpio0 18 1>; > + gpio-fan,speed-map = <0 3 > + 1500 2 > + 3250 1 > + 5000 0>; > + alarm-gpios = <&gpio1 8 0>; > + }; > }; > diff --git a/arch/arm/mach-kirkwood/board-lsxl.c b/arch/arm/mach-kirkwood/board-lsxl.c > index 64e5cc6..7e18cad 100644 > --- a/arch/arm/mach-kirkwood/board-lsxl.c > +++ b/arch/arm/mach-kirkwood/board-lsxl.c > @@ -16,7 +16,6 @@ > #include <linux/platform_device.h> > #include <linux/mv643xx_eth.h> > #include <linux/gpio.h> > -#include <linux/gpio-fan.h> > #include "common.h" > #include "mpp.h" Hi Michael You might also be able to remove platform_device.h. Otherwise, it looks good. Andrew ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl 2012-10-21 6:55 ` Andrew Lunn @ 2012-10-21 13:59 ` Michael Walle 2012-10-21 17:59 ` Andrew Lunn 0 siblings, 1 reply; 7+ messages in thread From: Michael Walle @ 2012-10-21 13:59 UTC (permalink / raw) To: linux-arm-kernel Am Sonntag 21 Oktober 2012, 08:55:52 schrieb Andrew Lunn: > On Sun, Oct 21, 2012 at 01:34:54AM +0200, Michael Walle wrote: > > Remove board specific gpio-fan driver registration. Moved into device > > tree. > > > > Signed-off-by: Michael Walle <michael@walle.cc> > > --- > > > > arch/arm/boot/dts/kirkwood-lsxl.dtsi | 11 ++++++++ > > arch/arm/mach-kirkwood/board-lsxl.c | 47 > > ---------------------------------- 2 files changed, 11 insertions(+), > > 47 deletions(-) > > > > diff --git a/arch/arm/boot/dts/kirkwood-lsxl.dtsi > > b/arch/arm/boot/dts/kirkwood-lsxl.dtsi index 8fea375..798e60e 100644 > > --- a/arch/arm/boot/dts/kirkwood-lsxl.dtsi > > +++ b/arch/arm/boot/dts/kirkwood-lsxl.dtsi > > @@ -94,4 +94,15 @@ > > > > gpios = <&gpio1 16 1>; > > > > }; > > > > }; > > > > + > > + gpio_fan { > > + compatible = "gpio-fan"; > > + gpios = <&gpio0 19 1 > > + &gpio0 18 1>; > > + gpio-fan,speed-map = <0 3 > > + 1500 2 > > + 3250 1 > > + 5000 0>; > > + alarm-gpios = <&gpio1 8 0>; > > + }; > > > > }; > > > > diff --git a/arch/arm/mach-kirkwood/board-lsxl.c > > b/arch/arm/mach-kirkwood/board-lsxl.c index 64e5cc6..7e18cad 100644 > > --- a/arch/arm/mach-kirkwood/board-lsxl.c > > +++ b/arch/arm/mach-kirkwood/board-lsxl.c > > @@ -16,7 +16,6 @@ > > > > #include <linux/platform_device.h> > > #include <linux/mv643xx_eth.h> > > #include <linux/gpio.h> > > > > -#include <linux/gpio-fan.h> > > > > #include "common.h" > > #include "mpp.h" > > Hi Michael > > You might also be able to remove platform_device.h. Mh, pm_power_off, although defined in linux/pm.h, is somehow pulled with this include. So either we leave it that way (looking at other boards only a few has included pm.h) or i'll post a third patch which replaces that include. -- Michael ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl 2012-10-21 13:59 ` Michael Walle @ 2012-10-21 17:59 ` Andrew Lunn 0 siblings, 0 replies; 7+ messages in thread From: Andrew Lunn @ 2012-10-21 17:59 UTC (permalink / raw) To: linux-arm-kernel > > Hi Michael > > > > You might also be able to remove platform_device.h. > > Mh, pm_power_off, although defined in linux/pm.h, is somehow pulled with this > include. So either we leave it that way (looking at other boards only a few > has included pm.h) or i'll post a third patch which replaces that include. O.K, leave it as it is. I hope to look at a DT solution for pm_power_off sometime soon. Andrew ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl 2012-10-20 23:34 ` [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl Michael Walle 2012-10-21 6:55 ` Andrew Lunn @ 2012-10-22 3:07 ` Jason Cooper 1 sibling, 0 replies; 7+ messages in thread From: Jason Cooper @ 2012-10-22 3:07 UTC (permalink / raw) To: linux-arm-kernel On Sun, Oct 21, 2012 at 01:34:54AM +0200, Michael Walle wrote: > Remove board specific gpio-fan driver registration. Moved into device tree. > > Signed-off-by: Michael Walle <michael@walle.cc> > --- > arch/arm/boot/dts/kirkwood-lsxl.dtsi | 11 ++++++++ > arch/arm/mach-kirkwood/board-lsxl.c | 47 ---------------------------------- > 2 files changed, 11 insertions(+), 47 deletions(-) Applied to: git://git.infradead.org/users/jcooper/linux.git kirkwood/boards thx, Jason. ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] ARM: kirkwood: cleanup lsxl board includes 2012-10-20 23:34 [PATCH 1/2] ARM: kirkwood: cleanup lsxl board includes Michael Walle 2012-10-20 23:34 ` [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl Michael Walle @ 2012-10-22 3:06 ` Jason Cooper 1 sibling, 0 replies; 7+ messages in thread From: Jason Cooper @ 2012-10-22 3:06 UTC (permalink / raw) To: linux-arm-kernel On Sun, Oct 21, 2012 at 01:34:53AM +0200, Michael Walle wrote: > Remove unneeded includes. These are leftovers from platform device > registrations. > > Signed-off-by: Michael Walle <michael@walle.cc> > --- > arch/arm/mach-kirkwood/board-lsxl.c | 8 -------- > 1 files changed, 0 insertions(+), 8 deletions(-) Applied to: git://git.infradead.org/users/jcooper/linux.git kirkwood/cleanup thx, Jason. ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-10-22 3:07 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-10-20 23:34 [PATCH 1/2] ARM: kirkwood: cleanup lsxl board includes Michael Walle 2012-10-20 23:34 ` [PATCH 2/2] ARM: kirkwood: use gpio-fan DT binding on lsxl Michael Walle 2012-10-21 6:55 ` Andrew Lunn 2012-10-21 13:59 ` Michael Walle 2012-10-21 17:59 ` Andrew Lunn 2012-10-22 3:07 ` Jason Cooper 2012-10-22 3:06 ` [PATCH 1/2] ARM: kirkwood: cleanup lsxl board includes Jason Cooper
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).