From patchwork Wed Aug 16 17:18:56 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sricharan Ramabadhran X-Patchwork-Id: 110268 Delivered-To: patch@linaro.org Received: by 10.140.95.78 with SMTP id h72csp1071949qge; Wed, 16 Aug 2017 10:24:50 -0700 (PDT) X-Received: by 10.84.132.44 with SMTP id 41mr2634506ple.194.1502904290037; Wed, 16 Aug 2017 10:24:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1502904290; cv=none; d=google.com; s=arc-20160816; b=GX9Wm83gzV7SbeggS9iIE+hc7buWPb0trP1LXsENkdGByuC2uJov737ipmuDIkAS22 Izt8DNnszh1dr3bJXCcmhHailpgI9dMkrujvmQ+CFssj3dAGYEZfhFNT46D/WKikhr79 bebdxT/YJ9pi/0IWh/d+IynXB/pYhnhnVHQn0yfYc0Rx1ODLkh0xhr4ebWRYx+0QUtkC 9ZmtDtq7jPTmaKNZzNv+Xt5JF3WS2Hvw0vEuDnFSalH83oP3NsGpsN+8EpWLAtA2QUIw X50AIsyeTS/0Jut2qShQfE5gxTpICwysotyX4jvTZnpqrV4wlZBnadVEflvv3poJxObF XwGw== 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=ED3pMe1JjwOhKeBKfZ+PYOblZvbTuDpsnjPmyFIYRhNwn2Q9dRdoCJpQw+pUJ3RsJ8 B80SIzF7aPFpEnbVObtu/9MgDqiEz1jqvgs/eyRvSRvOfR4KGHOQEAOqHjhe1RbSZk9Y AMsMcY8cvP15rh7Y3iIq+Yl22AUFmZs535rYBoLba3zk57ZTEEEQY4pj0c4C68zWQIeh MOTxcgQNOLcfOikk+yVGebebVwcFXrnYClBwz3t4Feq2YsIR1aORercfX4rltyi6FO1h py7a0WbeV6akkuwmNPSdFkxbwbY8qcLTprPcv4lgg9SCweta0A2z9JrikYx/Qb453gxW BTZQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@codeaurora.org header.s=default header.b=N2lMjqLS; dkim=pass header.i=@codeaurora.org header.s=default header.b=UjNNFLZI; 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 m9si746575pgs.644.2017.08.16.10.24.49; Wed, 16 Aug 2017 10:24:50 -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=N2lMjqLS; dkim=pass header.i=@codeaurora.org header.s=default header.b=UjNNFLZI; 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 S1752749AbdHPRYr (ORCPT + 26 others); Wed, 16 Aug 2017 13:24:47 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:51218 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752229AbdHPRTv (ORCPT ); Wed, 16 Aug 2017 13:19:51 -0400 Received: by smtp.codeaurora.org (Postfix, from userid 1000) id 6D93C60766; Wed, 16 Aug 2017 17:19:38 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1502903990; bh=zNFr/LDvETYl/VT/zuBHLdyNPvcrzswdBeKisZZGyPU=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=N2lMjqLSPUU1qEeXAklH4mX5v4dr0Re1fJ/H/QAoIwi9AaXIM8RYbEtWahbHgyhqp J9hhetZpJR76fl755eZmbt7+Wi8hFpQDj+pM0m4HAfa3O7u1wJMDgrvg0adYxpYcLA 0u3RTPYmuU11wVV04w/oFGVzBBBha6bjT9JcR/S8= 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 7606160727; Wed, 16 Aug 2017 17:19:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1502903977; bh=zNFr/LDvETYl/VT/zuBHLdyNPvcrzswdBeKisZZGyPU=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=UjNNFLZI1IAYBov7y62stXdnPGDQmuzXTbsX3+LL+EYtfI9chiPnMT9CSDLuKj+5O +SoTV7zFXTjia1C+Wuw4dODftW5cXQDZRcUswa+OEWxnE7wUiu6A1B6TmBLhYCdTdE JcVJ+APvztmaTT7duVXgo90z+XOx59E3nYI45FHU= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 7606160727 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 03/18] rpmsg: glink: Split rpm_probe to reuse the common code Date: Wed, 16 Aug 2017 22:48:56 +0530 Message-Id: <1502903951-5403-4-git-send-email-sricharan@codeaurora.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1502903951-5403-1-git-send-email-sricharan@codeaurora.org> References: <1502903951-5403-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);