diff options
| author | Chaehyun Lim <chaehyun.lim@gmail.com> | 2016-02-16 08:39:49 (GMT) |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-02-20 00:55:33 (GMT) |
| commit | b713ce0e498130c2c6f7d0f38410a98ddbb80a32 (patch) | |
| tree | 10294d577ec242b563b6fc3cbff253e032fe2928 | |
| parent | 23411f37ca886053cd212f9056af97a3b5c9054e (diff) | |
| download | linux-b713ce0e498130c2c6f7d0f38410a98ddbb80a32.tar.xz | |
staging: wilc1000: handle_set_mac_address: remove debug message of kmalloc failure
There is no need to print debug message when kmalloc is failed. This
message is redundant. The code already show us that kmalloc is failed.
The brace of if statement is remove as well due to have a single
statement.
Signed-off-by: Chaehyun Lim <chaehyun.lim@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/staging/wilc1000/host_interface.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging/wilc1000/host_interface.c index f4ca365..c6a9e52 100644 --- a/drivers/staging/wilc1000/host_interface.c +++ b/drivers/staging/wilc1000/host_interface.c @@ -431,10 +431,9 @@ static s32 handle_set_mac_address(struct wilc_vif *vif, struct wid wid; u8 *mac_buf = kmalloc(ETH_ALEN, GFP_KERNEL); - if (!mac_buf) { - PRINT_ER("No buffer to send mac address\n"); + if (!mac_buf) return -EFAULT; - } + memcpy(mac_buf, set_mac_addr->mac_addr, ETH_ALEN); wid.id = (u16)WID_MAC_ADDR; |