diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-09-04 20:31:11 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-09-04 20:31:11 -0400 |
commit | 27703bb4a66df49ff16b44b864d307d2eb71774c (patch) | |
tree | 356ce249a0af9775b3291a93fc56e466ffc5d1f8 /drivers/acpi/acpi_pad.c | |
parent | 6f3bc58d84e9e4c4ed8584e020365f996a26a153 (diff) | |
parent | ad151d544475df40ee0d18be7d5c945e29c1b545 (diff) |
Merge tag 'PTR_RET-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/rusty/linux
Pull PTR_RET() removal patches from Rusty Russell:
"PTR_RET() is a weird name, and led to some confusing usage. We ended
up with PTR_ERR_OR_ZERO(), and replacing or fixing all the usages.
This has been sitting in linux-next for a whole cycle"
[ There are still some PTR_RET users scattered about, with some of them
possibly being new, but most of them existing in Rusty's tree too. We
have that
#define PTR_RET(p) PTR_ERR_OR_ZERO(p)
thing in <linux/err.h>, so they continue to work for now - Linus ]
* tag 'PTR_RET-for-linus' of git://git.kernel.org/pub/scm/linux/kernel/git/rusty/linux:
GFS2: Replace PTR_RET with PTR_ERR_OR_ZERO
Btrfs: volume: Replace PTR_RET with PTR_ERR_OR_ZERO
drm/cma: Replace PTR_RET with PTR_ERR_OR_ZERO
sh_veu: Replace PTR_RET with PTR_ERR_OR_ZERO
dma-buf: Replace PTR_RET with PTR_ERR_OR_ZERO
drivers/rtc: Replace PTR_RET with PTR_ERR_OR_ZERO
mm/oom_kill: remove weird use of ERR_PTR()/PTR_ERR().
staging/zcache: don't use PTR_RET().
remoteproc: don't use PTR_RET().
pinctrl: don't use PTR_RET().
acpi: Replace weird use of PTR_RET.
s390: Replace weird use of PTR_RET.
PTR_RET is now PTR_ERR_OR_ZERO(): Replace most.
PTR_RET is now PTR_ERR_OR_ZERO
Diffstat (limited to 'drivers/acpi/acpi_pad.c')
-rw-r--r-- | drivers/acpi/acpi_pad.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/drivers/acpi/acpi_pad.c b/drivers/acpi/acpi_pad.c index 6230637054c6..fc6008fbce35 100644 --- a/drivers/acpi/acpi_pad.c +++ b/drivers/acpi/acpi_pad.c | |||
@@ -231,16 +231,19 @@ static struct task_struct *ps_tsks[NR_CPUS]; | |||
231 | static unsigned int ps_tsk_num; | 231 | static unsigned int ps_tsk_num; |
232 | static int create_power_saving_task(void) | 232 | static int create_power_saving_task(void) |
233 | { | 233 | { |
234 | int rc = -ENOMEM; | 234 | int rc; |
235 | 235 | ||
236 | ps_tsks[ps_tsk_num] = kthread_run(power_saving_thread, | 236 | ps_tsks[ps_tsk_num] = kthread_run(power_saving_thread, |
237 | (void *)(unsigned long)ps_tsk_num, | 237 | (void *)(unsigned long)ps_tsk_num, |
238 | "acpi_pad/%d", ps_tsk_num); | 238 | "acpi_pad/%d", ps_tsk_num); |
239 | rc = PTR_RET(ps_tsks[ps_tsk_num]); | 239 | |
240 | if (!rc) | 240 | if (IS_ERR(ps_tsks[ps_tsk_num])) { |
241 | ps_tsk_num++; | 241 | rc = PTR_ERR(ps_tsks[ps_tsk_num]); |
242 | else | ||
243 | ps_tsks[ps_tsk_num] = NULL; | 242 | ps_tsks[ps_tsk_num] = NULL; |
243 | } else { | ||
244 | rc = 0; | ||
245 | ps_tsk_num++; | ||
246 | } | ||
244 | 247 | ||
245 | return rc; | 248 | return rc; |
246 | } | 249 | } |