Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp1022584ybk; Wed, 13 May 2020 20:57:58 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxjESrzAyZK3y4wolL1HmYWG0OFWuOGCVZ9FQgVUyWuOVrLwNnTVVO2qmNadhXfeUUgXNZ9 X-Received: by 2002:a05:6402:1bd9:: with SMTP id ch25mr2162544edb.15.1589428678190; Wed, 13 May 2020 20:57:58 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1589428678; cv=none; d=google.com; s=arc-20160816; b=PCCe+tjPG8zWbBmjC/MsJ13rGBSs1pyKudG3s1/zZFQ1nr7ZMKMruDLZK/s1KBKvj5 ctjisgffUEh+jsOWMHwd2/jK19LuwZQs5Y62Vn2Dn+T8pdLQR4sS7BqEhT4rjaCA+u42 2hEBnxDxu1qEGsN2NzooQd+h5lnFmy29DNqAeVA7IEkTVcpCZcSCUbrVgf1lwK77MqcM /5xU2jM7TTPYjHM1rxaQzZ4Ye2F+CbZeKNFr1OJiKSn+t5RpCgjvfmZQkAqT5FXZDxWx qrywup89Z8O7Y3TgwXWKox8hdTCh5qnhZy1D9Xeb4iOgPIRnXNUVr9o7V7RGJXAf+yzf Tt8A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from; bh=IpXb6gzflaM3Yd5OZYlGyr0sNjendRPClTzs3G9i4oo=; b=qLZc3jtbdzcs3xmXo/mmDGCOr7QWXYD8ODS2Ax5WpqP95ewvfFDB15f0ZW/YJGoj+n IpT92Su5/JnbK+W0jzICQE5uf5vu8Yyf7DjrtrOM2GGBiMT9hI57jsXL/WV9lPh8e/ay BN75V+9vezxCpqsH4QzYhIGufWOjeOLkQdH8PCWZkIT8hwgCGXC1azegWF3JmHRjP9kW J2oDSWTN3pWCIepl4FFqMp2I+SpqA6Z/8ivzbAXW6/9XOwxZVHsGWm3AIDwQ4gPLRRLM rDuWl0kYZpq37RCyBx31GQsZN8RYa4jSr3Lxko0klrm9DxNTozEZioP6OIaTWp2nHpcz V6YQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id n17si1030380ejd.437.2020.05.13.20.57.34; Wed, 13 May 2020 20:57:58 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725967AbgEND4M (ORCPT + 99 others); Wed, 13 May 2020 23:56:12 -0400 Received: from m17616.mail.qiye.163.com ([59.111.176.16]:50153 "EHLO m17616.mail.qiye.163.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725931AbgEND4L (ORCPT ); Wed, 13 May 2020 23:56:11 -0400 Received: from ubuntu.localdomain (unknown [58.251.74.227]) by m17616.mail.qiye.163.com (Hmail) with ESMTPA id BE95D107CAA; Thu, 14 May 2020 11:56:06 +0800 (CST) From: Wang Wenhu To: Alex Elder , "David S. Miller" , netdev@vger.kernel.org, linux-kernel@vger.kernel.org Cc: kernel@vivo.com, Wang Wenhu Subject: [PATCH] drivers: ipa: use devm_kzalloc for simplicity Date: Wed, 13 May 2020 20:55:20 -0700 Message-Id: <20200514035520.2162-1-wenhu.wang@vivo.com> X-Mailer: git-send-email 2.17.1 X-HM-Spam-Status: e1kfGhgUHx5ZQUtXWQgYFAkeWUFZT1VNS0pCQkJMTkJNTE9LTFlXWShZQU hPN1dZLVlBSVdZCQ4XHghZQVk1NCk2OjckKS43PlkG X-HM-Sender-Digest: e1kMHhlZQR0aFwgeV1kSHx4VD1lBWUc6MRw6Cww6EzgrQjAJSjc*Hx8T FDYKCklVSlVKTkNCT0lDTk1CSkpNVTMWGhIXVQweFRMOVQwaFRw7DRINFFUYFBZFWVdZEgtZQVlO Q1VJTkpVTE9VSUlMWVdZCAFZQUhITks3Bg++ X-HM-Tid: 0a7211533bcc9374kuwsbe95d107caa Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Make a substitution of kzalloc with devm_kzalloc to simplify the ipa_probe() process. Signed-off-by: Wang Wenhu Cc: Alex Elder --- drivers/net/ipa/ipa_clock.c | 7 ++----- drivers/net/ipa/ipa_main.c | 7 ++----- 2 files changed, 4 insertions(+), 10 deletions(-) diff --git a/drivers/net/ipa/ipa_clock.c b/drivers/net/ipa/ipa_clock.c index 374491ea11cf..ddbd687fe64b 100644 --- a/drivers/net/ipa/ipa_clock.c +++ b/drivers/net/ipa/ipa_clock.c @@ -276,7 +276,7 @@ struct ipa_clock *ipa_clock_init(struct device *dev) goto err_clk_put; } - clock = kzalloc(sizeof(*clock), GFP_KERNEL); + clock = devm_kzalloc(dev, sizeof(*clock), GFP_KERNEL); if (!clock) { ret = -ENOMEM; goto err_clk_put; @@ -285,15 +285,13 @@ struct ipa_clock *ipa_clock_init(struct device *dev) ret = ipa_interconnect_init(clock, dev); if (ret) - goto err_kfree; + goto err_clk_put; mutex_init(&clock->mutex); atomic_set(&clock->count, 0); return clock; -err_kfree: - kfree(clock); err_clk_put: clk_put(clk); @@ -308,6 +306,5 @@ void ipa_clock_exit(struct ipa_clock *clock) WARN_ON(atomic_read(&clock->count) != 0); mutex_destroy(&clock->mutex); ipa_interconnect_exit(clock); - kfree(clock); clk_put(clk); } diff --git a/drivers/net/ipa/ipa_main.c b/drivers/net/ipa/ipa_main.c index 28998dcce3d2..b7b348b863f7 100644 --- a/drivers/net/ipa/ipa_main.c +++ b/drivers/net/ipa/ipa_main.c @@ -760,7 +760,7 @@ static int ipa_probe(struct platform_device *pdev) } /* Allocate and initialize the IPA structure */ - ipa = kzalloc(sizeof(*ipa), GFP_KERNEL); + ipa = devm_kzalloc(dev, sizeof(*ipa), GFP_KERNEL); if (!ipa) { ret = -ENOMEM; goto err_wakeup_source_unregister; @@ -776,7 +776,7 @@ static int ipa_probe(struct platform_device *pdev) ret = ipa_reg_init(ipa); if (ret) - goto err_kfree_ipa; + goto err_wakeup_source_unregister; ret = ipa_mem_init(ipa, data->mem_count, data->mem_data); if (ret) @@ -848,8 +848,6 @@ static int ipa_probe(struct platform_device *pdev) ipa_mem_exit(ipa); err_reg_exit: ipa_reg_exit(ipa); -err_kfree_ipa: - kfree(ipa); err_wakeup_source_unregister: wakeup_source_unregister(wakeup_source); err_clock_exit: @@ -885,7 +883,6 @@ static int ipa_remove(struct platform_device *pdev) gsi_exit(&ipa->gsi); ipa_mem_exit(ipa); ipa_reg_exit(ipa); - kfree(ipa); wakeup_source_unregister(wakeup_source); ipa_clock_exit(clock); rproc_put(rproc); -- 2.17.1