diff options
| author | Linus Torvalds <torvalds@linux-foundation.org> | 2017-11-15 13:14:11 -0500 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-11-15 13:14:11 -0500 |
| commit | 9682b3dea22190a6fd449d157e3175b0e748684d (patch) | |
| tree | 1b4d01168142fbb006e16b40e669de9737eb69ee /drivers/misc/ibmasm | |
| parent | 20df15783a44a289aaa8c8f83b3f715f9040c9c2 (diff) | |
| parent | 1972d6c0c86dd8620ff9f2811ce875135ffb5405 (diff) | |
Merge branch 'for-linus' of ssh://gitolite.kernel.org/pub/scm/linux/kernel/git/jikos/trivial
Pull trivial tree updates from Jiri Kosina:
"The usual rocket-science from trivial tree for 4.15"
* 'for-linus' of ssh://gitolite.kernel.org/pub/scm/linux/kernel/git/jikos/trivial:
MAINTAINERS: relinquish kconfig
MAINTAINERS: Update my email address
treewide: Fix typos in Kconfig
kfifo: Fix comments
init/Kconfig: Fix module signing document location
misc: ibmasm: Return error on error path
HID: logitech-hidpp: fix mistake in printk, "feeback" -> "feedback"
MAINTAINERS: Correct path to uDraw PS3 driver
tracing: Fix doc mistakes in trace sample
tracing: Kconfig text fixes for CONFIG_HWLAT_TRACER
MIPS: Alchemy: Remove reverted CONFIG_NETLINK_MMAP from db1xxx_defconfig
mm/huge_memory.c: fixup grammar in comment
lib/xz: Add fall-through comments to a switch statement
Diffstat (limited to 'drivers/misc/ibmasm')
| -rw-r--r-- | drivers/misc/ibmasm/event.c | 2 | ||||
| -rw-r--r-- | drivers/misc/ibmasm/module.c | 6 |
2 files changed, 5 insertions, 3 deletions
diff --git a/drivers/misc/ibmasm/event.c b/drivers/misc/ibmasm/event.c index 8e540f4e9d52..7e33025b4854 100644 --- a/drivers/misc/ibmasm/event.c +++ b/drivers/misc/ibmasm/event.c | |||
| @@ -155,7 +155,7 @@ int ibmasm_event_buffer_init(struct service_processor *sp) | |||
| 155 | 155 | ||
| 156 | buffer = kmalloc(sizeof(struct event_buffer), GFP_KERNEL); | 156 | buffer = kmalloc(sizeof(struct event_buffer), GFP_KERNEL); |
| 157 | if (!buffer) | 157 | if (!buffer) |
| 158 | return 1; | 158 | return -ENOMEM; |
| 159 | 159 | ||
| 160 | buffer->next_index = 0; | 160 | buffer->next_index = 0; |
| 161 | buffer->next_serial_number = 1; | 161 | buffer->next_serial_number = 1; |
diff --git a/drivers/misc/ibmasm/module.c b/drivers/misc/ibmasm/module.c index c5a456b0a564..e914b8c80943 100644 --- a/drivers/misc/ibmasm/module.c +++ b/drivers/misc/ibmasm/module.c | |||
| @@ -94,12 +94,14 @@ static int ibmasm_init_one(struct pci_dev *pdev, const struct pci_device_id *id) | |||
| 94 | snprintf(sp->dirname, IBMASM_NAME_SIZE, "%d", sp->number); | 94 | snprintf(sp->dirname, IBMASM_NAME_SIZE, "%d", sp->number); |
| 95 | snprintf(sp->devname, IBMASM_NAME_SIZE, "%s%d", DRIVER_NAME, sp->number); | 95 | snprintf(sp->devname, IBMASM_NAME_SIZE, "%s%d", DRIVER_NAME, sp->number); |
| 96 | 96 | ||
| 97 | if (ibmasm_event_buffer_init(sp)) { | 97 | result = ibmasm_event_buffer_init(sp); |
| 98 | if (result) { | ||
| 98 | dev_err(sp->dev, "Failed to allocate event buffer\n"); | 99 | dev_err(sp->dev, "Failed to allocate event buffer\n"); |
| 99 | goto error_eventbuffer; | 100 | goto error_eventbuffer; |
| 100 | } | 101 | } |
| 101 | 102 | ||
| 102 | if (ibmasm_heartbeat_init(sp)) { | 103 | result = ibmasm_heartbeat_init(sp); |
| 104 | if (result) { | ||
| 103 | dev_err(sp->dev, "Failed to allocate heartbeat command\n"); | 105 | dev_err(sp->dev, "Failed to allocate heartbeat command\n"); |
| 104 | goto error_heartbeat; | 106 | goto error_heartbeat; |
| 105 | } | 107 | } |
