diff options
author | Muhammad Falak R Wani <falakreyaz@gmail.com> | 2016-05-19 16:39:28 +0530 |
---|---|---|
committer | Sebastian Reichel <sre@kernel.org> | 2016-05-29 20:38:02 +0200 |
commit | b32bd7e7d5c1c04bb351420c09217f38dad8b8f1 (patch) | |
tree | 1b3d68ffa8d660d97ed89bf19e3c4b0c50541829 /drivers/hsi | |
parent | f6004b7bc651c11591d544ce3fd60d7bf7d71ccd (diff) | |
download | op-kernel-dev-b32bd7e7d5c1c04bb351420c09217f38dad8b8f1.zip op-kernel-dev-b32bd7e7d5c1c04bb351420c09217f38dad8b8f1.tar.gz |
hsi: use kmemdup
Use kmemdup when some other buffer is immediately copied into allocated
region. It replaces call to allocation followed by memcpy, by a single
call to kmemdup.
Signed-off-by: Muhammad Falak R Wani <falakreyaz@gmail.com>
Signed-off-by: Sebastian Reichel <sre@kernel.org>
Diffstat (limited to 'drivers/hsi')
-rw-r--r-- | drivers/hsi/hsi.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/hsi/hsi.c b/drivers/hsi/hsi.c index df380d5..d7ce07a 100644 --- a/drivers/hsi/hsi.c +++ b/drivers/hsi/hsi.c @@ -90,19 +90,19 @@ struct hsi_client *hsi_new_client(struct hsi_port *port, cl->tx_cfg = info->tx_cfg; if (cl->tx_cfg.channels) { size = cl->tx_cfg.num_channels * sizeof(*cl->tx_cfg.channels); - cl->tx_cfg.channels = kzalloc(size , GFP_KERNEL); + cl->tx_cfg.channels = kmemdup(info->tx_cfg.channels, size, + GFP_KERNEL); if (!cl->tx_cfg.channels) goto err_tx; - memcpy(cl->tx_cfg.channels, info->tx_cfg.channels, size); } cl->rx_cfg = info->rx_cfg; if (cl->rx_cfg.channels) { size = cl->rx_cfg.num_channels * sizeof(*cl->rx_cfg.channels); - cl->rx_cfg.channels = kzalloc(size , GFP_KERNEL); + cl->rx_cfg.channels = kmemdup(info->rx_cfg.channels, size, + GFP_KERNEL); if (!cl->rx_cfg.channels) goto err_rx; - memcpy(cl->rx_cfg.channels, info->rx_cfg.channels, size); } cl->device.bus = &hsi_bus_type; |