From patchwork Thu Aug 24 07:21:25 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sricharan Ramabadhran X-Patchwork-Id: 110873 Delivered-To: patch@linaro.org Received: by 10.37.128.210 with SMTP id c18csp8069765ybm; Thu, 24 Aug 2017 00:26:52 -0700 (PDT) X-Received: by 10.84.167.2 with SMTP id c2mr5995673plb.369.1503559612364; Thu, 24 Aug 2017 00:26:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1503559612; cv=none; d=google.com; s=arc-20160816; b=cwXWAQ6C/mtT9pQWG56MBl3qmV6c32oZ0WGxEWSoQSJNaZDcY5x2h52jwLBLcH2T+x 7FBFkAC8bgM/AV9XU5AFRMwpvtarVBy6bnIAKe9+58f/qtExX4xP3XiFK1rym2jyQvT2 SVU9zxsvL6iZ041h2h1m8Gv5dHNUI1FHKBix80LAqXQgKUZSQPm6vlDbQbSew4PKfu1M E9FNr/Z2mhmCN59Qc/ziQGLAEwqYXxcYB87mJUNU1VCFcbKBX2GNjziwHRWuequZFgQP aCkMKEIMuEmOADJZysQyr2OPuYV51Q2yO4mKj1GVNKF+AN6rlq6r8mQy7PuVaSDRcUyu as+w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:references:in-reply-to:message-id:date :subject:cc:to:from:dmarc-filter:dkim-signature:dkim-signature :arc-authentication-results; bh=DkxiTUb6NzVHhko/ekleMDchrUjss7xl/mMB25IYnv4=; b=VL7vHpvUmWemzhQzMbm59P5IePP/IY8/LDQfBqJJ2rhCmZeL7oyQ8NVKdnagq6izHZ n0jgbw46Y74MzQClCOew5hypb36z1qShEhg4aMCB4Xi6TFolCQtZltcQbtRqvgso9N1+ A3wBza/FpFKsKU8ut74Q2RHCSrNC8V2UQUo5E8qpwKX8XwOcuU8MNwuvFAjAeT+A4ZHY eLMuGXGT3xLFmxXChNaeNMPA9QumVuEwWt6CreF3Nms3o26UD0aKi/VtBvAFDADAoG9o hpF1bCImh6rqSJYJ8IPII/C+4mF0MtcnATXvKaqY/XSldHGP+zNchX1lzX/Wma+DXF4s 0YRA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@codeaurora.org header.s=default header.b=jG4jH0cR; dkim=pass header.i=@codeaurora.org header.s=default header.b=KJG7qk4M; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id y9si2373968pgq.143.2017.08.24.00.26.52; Thu, 24 Aug 2017 00:26:52 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@codeaurora.org header.s=default header.b=jG4jH0cR; dkim=pass header.i=@codeaurora.org header.s=default header.b=KJG7qk4M; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752255AbdHXH0t (ORCPT + 26 others); Thu, 24 Aug 2017 03:26:49 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:59604 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751329AbdHXHWG (ORCPT ); Thu, 24 Aug 2017 03:22:06 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id A56D060721; Thu, 24 Aug 2017 07:22:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1503559325; bh=zNFr/LDvETYl/VT/zuBHLdyNPvcrzswdBeKisZZGyPU=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=jG4jH0cRmjWggIJgGFl2fjHOpnGTutcahtGfDZFUeXdhP/giwyA4MXAB/dCOrB2JF k9ez2XAPX9whp2TQtU/k26eiO2lmQao6zMf1hWWZtvq9MGvqRxZBRSgxGnC3Es7y7c VAPZNfuZC4tyZdXPLhkm5eSdvN26o4DpKn0N6L9c= X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on pdx-caf-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.8 required=2.0 tests=ALL_TRUSTED,BAYES_00, DKIM_SIGNED, T_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.0 Received: from srichara-linux.qualcomm.com (blr-c-bdr-fw-01_globalnat_allzones-outside.qualcomm.com [103.229.19.19]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-SHA256 (128/128 bits)) (No client certificate requested) (Authenticated sender: sricharan@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 9B4EA603AF; Thu, 24 Aug 2017 07:22:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1503559324; bh=zNFr/LDvETYl/VT/zuBHLdyNPvcrzswdBeKisZZGyPU=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=KJG7qk4MYjSDtBs7TUUjuMAb45ZPkYcVM8d/8/0O6v7pz5wMC7nFE+KfGQ5UnwdZa OH0PtnAVOWRzo/a9wg1DiRD4cCQFpMANhh21mLk9BDRBTggJrktkKSUahqYMMaerBL C2JxQ2woDtD/5BrwjLnEp0YjZLY0DUlGvbbnHJdg= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 9B4EA603AF Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=sricharan@codeaurora.org From: Sricharan R To: ohad@wizery.com, bjorn.andersson@linaro.org, linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org Cc: sricharan@codeaurora.org Subject: [PATCH v2 03/20] rpmsg: glink: Split rpm_probe to reuse the common code Date: Thu, 24 Aug 2017 12:51:25 +0530 Message-Id: <1503559302-3744-4-git-send-email-sricharan@codeaurora.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1503559302-3744-1-git-send-email-sricharan@codeaurora.org> References: <1503559302-3744-1-git-send-email-sricharan@codeaurora.org> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Bjorn Andersson There is quite some code common in glink_rpm_probe that can reused for glink-smem based transport as well. So split the function and move the code to glink_native_probe that can be used later when we add the support for glink-smem based transport. Also reuse driver's remove as well. Signed-off-by: Bjorn Andersson Signed-off-by: Sricharan R --- drivers/rpmsg/qcom_glink_rpm.c | 85 ++++++++++++++++++++++++------------------ 1 file changed, 49 insertions(+), 36 deletions(-) -- QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation diff --git a/drivers/rpmsg/qcom_glink_rpm.c b/drivers/rpmsg/qcom_glink_rpm.c index 870ce32..5f0fa0d 100644 --- a/drivers/rpmsg/qcom_glink_rpm.c +++ b/drivers/rpmsg/qcom_glink_rpm.c @@ -19,6 +19,7 @@ #include #include #include +#include #include #include #include @@ -1155,29 +1156,21 @@ static int glink_rpm_parse_toc(struct device *dev, return -EINVAL; } -static int glink_rpm_probe(struct platform_device *pdev) +struct qcom_glink *qcom_glink_native_probe(struct device *dev, + struct qcom_glink_pipe *rx, + struct qcom_glink_pipe *tx) { - struct qcom_glink *glink; - struct glink_rpm_pipe *rx_pipe; - struct glink_rpm_pipe *tx_pipe; - struct device_node *np; - void __iomem *msg_ram; - size_t msg_ram_size; - struct device *dev = &pdev->dev; - struct resource r; int irq; int ret; + struct qcom_glink *glink; glink = devm_kzalloc(dev, sizeof(*glink), GFP_KERNEL); if (!glink) - return -ENOMEM; + return ERR_PTR(-ENOMEM); glink->dev = dev; - - rx_pipe = devm_kzalloc(&pdev->dev, sizeof(*rx_pipe), GFP_KERNEL); - tx_pipe = devm_kzalloc(&pdev->dev, sizeof(*tx_pipe), GFP_KERNEL); - if (!rx_pipe || !tx_pipe) - return -ENOMEM; + glink->tx_pipe = tx; + glink->rx_pipe = rx; mutex_init(&glink->tx_lock); spin_lock_init(&glink->rx_lock); @@ -1188,14 +1181,48 @@ static int glink_rpm_probe(struct platform_device *pdev) idr_init(&glink->lcids); idr_init(&glink->rcids); - glink->mbox_client.dev = &pdev->dev; + glink->mbox_client.dev = dev; glink->mbox_chan = mbox_request_channel(&glink->mbox_client, 0); if (IS_ERR(glink->mbox_chan)) { if (PTR_ERR(glink->mbox_chan) != -EPROBE_DEFER) - dev_err(&pdev->dev, "failed to acquire IPC channel\n"); - return PTR_ERR(glink->mbox_chan); + dev_err(dev, "failed to acquire IPC channel\n"); + return ERR_CAST(glink->mbox_chan); + } + + irq = of_irq_get(dev->of_node, 0); + ret = devm_request_irq(dev, irq, + qcom_glink_intr, + IRQF_NO_SUSPEND | IRQF_SHARED, + "glink-native", glink); + if (ret) { + dev_err(dev, "failed to request IRQ\n"); + return ERR_PTR(ret); } + ret = qcom_glink_send_version(glink); + if (ret) + return ERR_PTR(ret); + + return glink; +} + +static int glink_rpm_probe(struct platform_device *pdev) +{ + struct qcom_glink *glink; + struct glink_rpm_pipe *rx_pipe; + struct glink_rpm_pipe *tx_pipe; + struct device_node *np; + void __iomem *msg_ram; + size_t msg_ram_size; + struct device *dev = &pdev->dev; + struct resource r; + int ret; + + rx_pipe = devm_kzalloc(&pdev->dev, sizeof(*rx_pipe), GFP_KERNEL); + tx_pipe = devm_kzalloc(&pdev->dev, sizeof(*tx_pipe), GFP_KERNEL); + if (!rx_pipe || !tx_pipe) + return -ENOMEM; + np = of_parse_phandle(dev->of_node, "qcom,rpm-msg-ram", 0); ret = of_address_to_resource(np, 0, &r); of_node_put(np); @@ -1219,27 +1246,13 @@ static int glink_rpm_probe(struct platform_device *pdev) tx_pipe->native.avail = glink_rpm_tx_avail; tx_pipe->native.write = glink_rpm_tx_write; - glink->tx_pipe = &tx_pipe->native; - glink->rx_pipe = &rx_pipe->native; - writel(0, tx_pipe->head); writel(0, rx_pipe->tail); - irq = platform_get_irq(pdev, 0); - ret = devm_request_irq(dev, irq, - qcom_glink_intr, - IRQF_NO_SUSPEND | IRQF_SHARED, - "glink-rpm", glink); - if (ret) { - dev_err(dev, "Failed to request IRQ\n"); - return ret; - } - - glink->irq = irq; - - ret = qcom_glink_send_version(glink); - if (ret) - return ret; + glink = qcom_glink_native_probe(&pdev->dev, &rx_pipe->native, + &tx_pipe->native); + if (IS_ERR(glink)) + return PTR_ERR(glink); platform_set_drvdata(pdev, glink);