Register with kernel poweroff handler instead of setting pm_power_off
directly.
Cc: Haavard Skinnemoen hskinnem...@gmail.com
Cc: Hans-Christian Egtvedt egtv...@samfundet.no
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
arch/avr32/boards/atngw100/mrmt.c | 2 +-
1 file changed, 1
Drivers should not call pm_power_off directly; it is not guaranteed
to be non-NULL. Call kernel_power_off instead.
Cc: Jean Delvare jdelv...@suse.de
Reviewed-by: Jean Delvare jdelv...@suse.de
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
drivers/hwmon/ab8500.c | 5 +++--
1 file changed, 3
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with high priority value of 192 to reflect that
the original code overwrites existing poweroff handlers.
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo Molnar mi...@redhat.com
Cc: H. Peter Anvin
Replace mach_power_off with pm_power_off to simplify the subsequent
move of pm_power_off to generic code.
Cc: Geert Uytterhoeven ge...@linux-m68k.org
Cc: Greg Ungerer g...@uclinux.org
Cc: Joshua Thompson fun...@jurai.org
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
arch/m68k/emu/natfeat.c
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Make twl4030_power_off static as it is only called from the twl4030-power
driver.
Cc:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with high priority value of 192 to reflect that
the original code overwrites existing poweroff handlers.
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo Molnar mi...@redhat.com
Cc: H. Peter Anvin
Various drivers implement architecture and/or device specific means to
remove power from the system. For the most part, those drivers set the
global variable pm_power_off to point to a function within the driver.
This mechanism has a number of drawbacks. Typically only one scheme
to remove
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with lower priority of 64 to reflect that the call
is expected to be replaced at some point.
Cc: Tony Luck tony.l...@intel.com
Cc: Fenghua Yu fenghua...@intel.com
Signed-off-by: Guenter Roeck
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Cc: Samuel Ortiz sa...@linux.intel.com
Cc: Lee Jones lee.jo...@linaro.org
Signed-off-by:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Cc: Samuel Ortiz sa...@linux.intel.com
Cc: Lee Jones lee.jo...@linaro.org
Signed-off-by:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Select proprity 0 since the code does not really poweroff
the system.
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
drivers/power/reset/msm-poweroff.c | 13 +++--
1 file changed, 11 insertions(+), 2
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Cc: Samuel Ortiz sa...@linux.intel.com
Cc: Lee Jones lee.jo...@linaro.org
Signed-off-by:
Register with kernel poweroff handler instead of setting pm_power_off
directly.
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
drivers/power/reset/vexpress-poweroff.c | 19 +--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a high priority value of 192 to reflect that
the original code overwrites pm_power_off unconditionally.
Register poweroff handler after the ipmi system is ready, and unregister
it prior to cleanup. This
Register with kernel poweroff handler instead of setting pm_power_off
directly.
Cc: Rusty Russell ru...@rustcorp.com.au
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo Molnar mi...@redhat.com
Cc: H. Peter Anvin h...@zytor.com
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
Register with kernel poweroff handler instead of seting pm_power_off
directly. Register as poweroff handler of last resort since the driver
does not really power off the system but executes a restart.
Cc: Sebastian Reichel s...@kernel.org
Cc: Dmitry Eremin-Solenikov dbarysh...@gmail.com
Cc:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Cc: Lee Jones lee.jo...@linaro.org
Cc: Samuel Ortiz sa...@linux.intel.com
Signed-off-by:
No users of pm_power_off are left, so it is safe to remove the function.
Cc: Rafael J. Wysocki r...@rjwysocki.net
Cc: Pavel Machek pa...@ucw.cz
Cc: Len Brown len.br...@intel.com
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
include/linux/pm.h | 1 -
Register with kernel poweroff handler instead of setting pm_power_off
directly.
Cc: Geert Uytterhoeven ge...@linux-m68k.org
Cc: Joshua Thompson fun...@jurai.org
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
arch/m68k/emu/natfeat.c | 2 +-
arch/m68k/mac/config.c | 2 +-
2 files changed, 2
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Cc: Lee Jones lee.jo...@linaro.org
Cc: Samuel Ortiz sa...@linux.intel.com
Signed-off-by:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Cc: Linus Walleij linus.wall...@linaro.org
Cc: Lee Jones lee.jo...@linaro.org
Cc: Samuel
Use have_kernel_poweroff() to determine if the kernel is able
to power off the system.
Cc: Santosh Shilimkar santosh.shilim...@ti.com
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
drivers/memory/emif.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Register with kernel poweroff handler instead of setting pm_power_off
directly.
Signed-off-by: Guenter Roeck li...@roeck-us.net
---
arch/sh/boards/board-sh7785lcr.c | 2 +-
arch/sh/boards/board-urquell.c | 2 +-
arch/sh/boards/mach-highlander/setup.c | 2 +-
Poweroff handlers may now be installed with register_poweroff_handler.
Use the new API function have_kernel_poweroff to determine if a poweroff
handler has been installed.
Cc: Rafael J. Wysocki r...@rjwysocki.net
Cc: Pavel Machek pa...@ucw.cz
Cc: Len Brown len.br...@intel.com
Signed-off-by:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo Molnar mi...@redhat.com
Cc: H. Peter Anvin
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with high priority value of 192 to reflect that
the driver explicitly overrides existing poweroff handlers.
Cc: Rafael J. Wysocki r...@rjwysocki.net
Cc: Len Brown l...@kernel.org
Signed-off-by: Guenter Roeck
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with high priority value of 192 to reflect that
the driver explicitly wants to override default poweroff handlers.
Cc: Thomas Gleixner t...@linutronix.de
Cc: Ingo Molnar mi...@redhat.com
Cc: H. Peter Anvin
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with default priority value of 128 since we don't know
any better.
Cc: Julian Andres Klode j...@jak-linux.org
Cc: Marc Dietrich marvi...@gmx.de
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Signed-off-by:
On Mon, 06 Oct 2014, Guenter Roeck wrote:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Make twl4030_power_off static as it is only
On Mon, 06 Oct 2014, Guenter Roeck wrote:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Cc: Linus Walleij linus.wall...@linaro.org
On 07/10/2014 07:28, Guenter Roeck :
Register with kernel poweroff handler instead of setting pm_power_off
directly. Always use register_poweroff_handler_simple as there is no
indication that more than one poweroff handler is registered.
If the poweroff handler only resets the system or puts
Mark functions static that are not used or declared outside of lo.c.
Signed-off-by: Thomas Gummerer t.gumme...@gmail.com
---
drivers/staging/lustre/lnet/lnet/lo.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/lustre/lnet/lnet/lo.c
y各种定价模@式的砍价策3略.xls
Description: Binary data
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Tue, Oct 7, 2014 at 7:28 AM, Guenter Roeck li...@roeck-us.net wrote:
Replace mach_power_off with pm_power_off to simplify the subsequent
move of pm_power_off to generic code.
Thanks!
Cc: Geert Uytterhoeven ge...@linux-m68k.org
Cc: Greg Ungerer g...@uclinux.org
Cc: Joshua Thompson
On Tue, Oct 7, 2014 at 7:28 AM, Guenter Roeck li...@roeck-us.net wrote:
pm_power_off is defined for all architectures. Move it to common code.
Have all architectures call do_kernel_poweroff instead of pm_power_off.
Some architectures point pm_power_off to machine_power_off. For those,
call
On Tue, Oct 7, 2014 at 7:28 AM, Guenter Roeck li...@roeck-us.net wrote:
Register with kernel poweroff handler instead of setting pm_power_off
directly.
Cc: Geert Uytterhoeven ge...@linux-m68k.org
Cc: Joshua Thompson fun...@jurai.org
Signed-off-by: Guenter Roeck li...@roeck-us.net
As someone
checkpatch.pk tiny fix
get rid of 2 warnings and 2 errors for hal/fw.c
Signed-off-by: Igor Bogomazov yg...@ygrex.ru
Cc: Greg Kroah-Hartman gre...@linuxfoundation.org
Cc: navin patidar navin.pati...@gmail.com
Cc: Stephen Rothwell s...@canb.auug.org.au
---
linux-next 3.17.0
diff --git
On Tue, Oct 07, 2014 at 06:28:08AM +0100, Guenter Roeck wrote:
pm_power_off is an implementation detail. Replace it with a more generic
description of the driver's functionality.
Cc: Rob Herring robh...@kernel.org
Cc: Pawel Moll pawel.m...@arm.com
Cc: Mark Rutland mark.rutl...@arm.com
On Tue, Oct 07, 2014 at 06:28:07AM +0100, Guenter Roeck wrote:
Devicetree bindings are supposed to be operating system independent
and should thus not describe how a specific functionality is implemented
in Linux.
Cc: Rob Herring robh...@kernel.org
Cc: Pawel Moll pawel.m...@arm.com
Cc:
On Tue, Oct 07, 2014 at 06:28:09AM +0100, Guenter Roeck wrote:
Replace reference to pm_power_off (which is an implementation detail)
and replace it with a more generic description of the driver's functionality.
Cc: Rob Herring robh...@kernel.org
Cc: Pawel Moll pawel.m...@arm.com
Cc: Mark
fixed several 'line over 80 characters' in places where it can be done
without changing/refactoring code
Signed-off-by: Aleh Suprunovich b...@ahlamon.org
---
drivers/staging/octeon-usb/octeon-hcd.c | 25 +
1 file changed, 17 insertions(+), 8 deletions(-)
diff --git
Am 07.10.2014 07:28, schrieb Guenter Roeck:
arch/um/kernel/reboot.c| 2 --
Acked-by: Richard Weinberger rich...@nod.at
Thanks,
//richard
___
devel mailing list
de...@linuxdriverproject.org
Fix Possible unnecessary KERN_INFO checkpatch.pl warning
Signed-off-by: Dzmitry Sledneu dzmitry.sled...@gmail.com
---
drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c
Fix Possible unnecessary 'out of memory' message checkpatch.pl warning
Signed-off-by: Dzmitry Sledneu dzmitry.sled...@gmail.com
---
drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c
This patch documents deprecated vendor name of device isl29028 for
compatibility with older kernels.
Signed-off-by: Darshana Padmadas darshanapadma...@gmail.com
---
Documentation/devicetree/bindings/vendor-prefixes.txt | 1 +
1 file changed, 1 insertion(+)
diff --git
Patchset documents correct and the deprecated vendor-prefix found by checkpatch
warning and also documents information of device isl29028 for compatibility.
Patchset also includes corrected vendor-prefix and device name in compatible
property for files with checkpatch warning of undocumented
This patch adds the correct vendor-prefix for device isl29028 and
maintains deprecated vendor-prefix found by checkpatch warning
for older kernel releases.
Signed-off-by: Darshana Padmadas darshanapadma...@gmail.com
---
drivers/staging/iio/light/isl29028.c | 3 ++-
1 file changed, 2
This patch adds the correct vendor-prefix listed in
Documentation/devicetree/bindings/vendor-prefixes.txt
with the device name to compatible property.
Signed-off-by: Darshana Padmadas darshanapadma...@gmail.com
---
arch/arm/boot/dts/tegra30-cardhu.dtsi | 2 +-
1 file changed, 1 insertion(+), 1
This patch documents the device isl29028 with its vendor-prefix. Undocumented
deprecated vendor-prefix
found by checkpatch also documented for compatibility reasons.
Signed-off-by: Darshana Padmadas darshanapadma...@gmail.com
---
Documentation/devicetree/bindings/i2c/trivial-devices.txt | 2 ++
Fix Possible unnecessary KERN_INFO checkpatch.pl warning
Signed-off-by: Dzmitry Sledneu dzmitry.sled...@gmail.com
---
drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c
Fix Possible unnecessary 'out of memory' message checkpatch.pl warning
Signed-off-by: Dzmitry Sledneu dzmitry.sled...@gmail.com
---
drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c
Hi Darshana,
On Tue, Oct 07, 2014 at 04:19:26PM +0100, Darshana Padmadas wrote:
This patch documents the device isl29028 with its vendor-prefix. Undocumented
deprecated vendor-prefix
found by checkpatch also documented for compatibility reasons.
Signed-off-by: Darshana Padmadas
On Tue, 2014-10-07 at 17:23 +0200, Dzmitry Sledneu wrote:
Fix Possible unnecessary KERN_INFO checkpatch.pl warning
[]
diff --git a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c
b/drivers/staging/gs_fpgaboot/gs_fpgaboot.c
[]
@@ -58,7 +58,7 @@ static void datadump(char *msg, void *m, int n)
From: Joe Perches j...@perches.com
Use print_hex_dump_bytes instead of pr_info
Signed-off-by: Dzmitry Sledneu dzmitry.sled...@gmail.com
Reviewed-by: Insop Song insop.s...@gainspeed.com
---
drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 25 ++---
1 file changed, 2
Fix Possible unnecessary 'out of memory' message checkpatch.pl warning
Signed-off-by: Dzmitry Sledneu dzmitry.sled...@gmail.com
---
drivers/staging/gs_fpgaboot/gs_fpgaboot.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/gs_fpgaboot/gs_fpgaboot.c
On Tue, Oct 07, 2014 at 12:02:19PM +0100, Mark Rutland wrote:
On Tue, Oct 07, 2014 at 06:28:09AM +0100, Guenter Roeck wrote:
Replace reference to pm_power_off (which is an implementation detail)
and replace it with a more generic description of the driver's
functionality.
Cc: Rob
On Mon, Oct 06, 2014 at 10:28:23PM -0700, Guenter Roeck wrote:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with a low priority value of 64 to reflect that
the original code only sets pm_power_off if it was not already set.
Other changes:
Drop
On Mon, Oct 06, 2014 at 10:28:25PM -0700, Guenter Roeck wrote:
Register with kernel poweroff handler instead of setting pm_power_off
directly. Register with default priority value of 128 to reflect that
the original code generates an error if another poweroff handler has
already been
On Mon, Oct 06, 2014 at 10:28:22PM -0700, Guenter Roeck wrote:
Register with kernel poweroff handler instead of seting pm_power_off
directly. Register as poweroff handler of last resort since the driver
does not really power off the system but executes a restart.
I would not say last resort,
- Original Message -
From: Colin Cross ccr...@android.com
To: pint...@samsung.com
Cc: Laura Abbott lau...@codeaurora.org; Heesub Shin
heesub.s...@samsung.com; a...@linux-foundation.org
a...@linux-foundation.org; gre...@linuxfoundation.org
gre...@linuxfoundation.org;
On Mon, Oct 06, 2014 at 10:28:08PM -0700, Guenter Roeck wrote:
pm_power_off is an implementation detail. Replace it with a more generic
description of the driver's functionality.
Cc: Rob Herring robh...@kernel.org
Cc: Pawel Moll pawel.m...@arm.com
Cc: Mark Rutland mark.rutl...@arm.com
On Mon, Oct 06, 2014 at 10:28:09PM -0700, Guenter Roeck wrote:
Replace reference to pm_power_off (which is an implementation detail)
and replace it with a more generic description of the driver's functionality.
Acked-by: Andrew Lunn and...@lunn.ch
Thanks
Andrew
Cc: Rob Herring
On Tue, Oct 7, 2014 at 9:07 AM, PINTU KUMAR pint...@samsung.com wrote:
- Original Message -
From: Colin Cross ccr...@android.com
To: pint...@samsung.com
Cc: Laura Abbott lau...@codeaurora.org; Heesub Shin
heesub.s...@samsung.com; a...@linux-foundation.org
a...@linux-foundation.org;
On 10/07/14 00:28, Guenter Roeck wrote:
Devicetree bindings are supposed to be operating system independent
and should thus not describe how a specific functionality is implemented
in Linux.
So your argument is that linux/Documentation/devicetree/bindings should
not be specific to Linux.
On Tue, Oct 7, 2014 at 10:19 AM, Darshana Padmadas
darshanapadma...@gmail.com wrote:
This patch adds the correct vendor-prefix listed in
Documentation/devicetree/bindings/vendor-prefixes.txt
with the device name to compatible property.
It would be nice if the subject and msg gave some clue as
On Tue, Oct 07, 2014 at 11:21:11AM -0500, Rob Landley wrote:
On 10/07/14 00:28, Guenter Roeck wrote:
Devicetree bindings are supposed to be operating system independent
and should thus not describe how a specific functionality is implemented
in Linux.
So your argument is that
On Tue, Oct 07, 2014 at 05:21:11PM +0100, Rob Landley wrote:
On 10/07/14 00:28, Guenter Roeck wrote:
Devicetree bindings are supposed to be operating system independent
and should thus not describe how a specific functionality is implemented
in Linux.
So your argument is that
On 10/07/2014 09:31 AM, Guenter Roeck wrote:
On Tue, Oct 07, 2014 at 11:21:11AM -0500, Rob Landley wrote:
On 10/07/14 00:28, Guenter Roeck wrote:
Devicetree bindings are supposed to be operating system independent
and should thus not describe how a specific functionality is implemented
in
-Original Message-
From: Christoph Hellwig [mailto:h...@infradead.org]
Sent: Thursday, September 25, 2014 6:47 AM
To: KY Srinivasan
Cc: h...@suse.de; linux-ker...@vger.kernel.org;
de...@linuxdriverproject.org; oher...@suse.com;
jbottom...@parallels.com; linux-s...@vger.kernel.org
On Monday, October 06, 2014 10:28:05 PM Guenter Roeck wrote:
Poweroff handlers may now be installed with register_poweroff_handler.
Use the new API function have_kernel_poweroff to determine if a poweroff
handler has been installed.
Cc: Rafael J. Wysocki r...@rjwysocki.net
Cc: Pavel Machek
On Monday, October 06, 2014 10:28:46 PM Guenter Roeck wrote:
No users of pm_power_off are left, so it is safe to remove the function.
Cc: Rafael J. Wysocki r...@rjwysocki.net
Cc: Pavel Machek pa...@ucw.cz
Cc: Len Brown len.br...@intel.com
Signed-off-by: Guenter Roeck li...@roeck-us.net
ACK
This patch fixes two unnecessary else conditions that were found by
checkpatch.pl.
Signed-off-by: Serguey Parkhomovsky sergueyparkhomov...@gmail.com
---
drivers/staging/rtl8712/rtl8712_efuse.c | 41 +
1 file changed, 21 insertions(+), 20 deletions(-)
diff --git
From: Fabio Estevam fabio.este...@freescale.com
There is no need to forward declare 'struct imx_drm_crtc' because the
imx-drm.h header file already includes it.
Signed-off-by: Fabio Estevam fabio.este...@freescale.com
---
drivers/staging/imx-drm/imx-drm-core.c | 2 --
1 file changed, 2
On Tue, 2014-10-07 at 16:48 -0700, Serguey Parkhomovsky wrote:
This patch fixes two unnecessary else conditions that were found by
checkpatch.pl.
[]
diff --git a/drivers/staging/rtl8712/rtl8712_efuse.c
b/drivers/staging/rtl8712/rtl8712_efuse.c
[]
@@ -414,19 +414,19 @@ u8
-Original Message-
From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf
Of Dexuan Cui
Sent: Friday, September 26, 2014 12:52 PM
To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-
de...@linuxdriverproject.org; o...@aepfle.de;
75 matches
Mail list logo