From patchwork Sat Apr 28 02:51:58 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nipun Gupta X-Patchwork-Id: 906024 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-tegra-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=nxp.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=nxp.com header.i=@nxp.com header.b="YPodo4ke"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 40XwMm1844z9s06 for ; Sat, 28 Apr 2018 12:52:52 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759547AbeD1Cwk (ORCPT ); Fri, 27 Apr 2018 22:52:40 -0400 Received: from mail-db5eur01on0052.outbound.protection.outlook.com ([104.47.2.52]:34640 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1759319AbeD1Cwe (ORCPT ); Fri, 27 Apr 2018 22:52:34 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=zuCLuozJK5yxXZpwmnfsGgcdx9+JnQww1/uwu5A4ZC0=; b=YPodo4keg9w1a6qmQG3NlonWiOzKsYp+x6fNY2aeUmbOosAYbH2lJ32YhpcHjSht8m/y0NLR8x54y5momsPubDJW0fZeBfuoslaPHIHP/5/Cj3BaZ8tPKCsEEgdqbn9XHv1ExR/BQfPYTVEzy0XY4uY/sX1q8hPaFCMOCNoUKkE= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=nipun.gupta@nxp.com; Received: from b27504-OptiPlex-790.ap.freescale.net (14.142.187.166) by HE1PR0401MB2425.eurprd04.prod.outlook.com (2603:10a6:3:25::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.715.18; Sat, 28 Apr 2018 02:52:21 +0000 From: Nipun Gupta To: robin.murphy@arm.com, hch@lst.de, linux@armlinux.org.uk, gregkh@linuxfoundation.org, m.szyprowski@samsung.com Cc: bhelgaas@google.com, zajec5@gmail.com, andy.gross@linaro.org, david.brown@linaro.org, dan.j.williams@intel.com, vinod.koul@intel.com, thierry.reding@gmail.com, robh+dt@kernel.org, frowand.list@gmail.com, jarkko.sakkinen@linux.intel.com, rafael.j.wysocki@intel.com, dmitry.torokhov@gmail.com, johan@kernel.org, msuchanek@suse.de, linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org, linux-wireless@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, dmaengine@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-tegra@vger.kernel.org, devicetree@vger.kernel.org, linux-pci@vger.kernel.org, bharat.bhushan@nxp.com, leoyang.li@nxp.com, Nipun Gupta Subject: [PATCH v4 1/2] dma-mapping: move dma configuration to bus infrastructure Date: Sat, 28 Apr 2018 08:21:58 +0530 Message-Id: <1524883919-29484-1-git-send-email-nipun.gupta@nxp.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1520868292-2479-1-git-send-email-nipun.gupta@nxp.com> References: <1520868292-2479-1-git-send-email-nipun.gupta@nxp.com> MIME-Version: 1.0 X-Originating-IP: [14.142.187.166] X-ClientProxiedBy: PN1PR01CA0071.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c00:1::11) To HE1PR0401MB2425.eurprd04.prod.outlook.com (2603:10a6:3:25::22) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(48565401081)(5600026)(4534165)(7168020)(4627221)(201703031133081)(201702281549075)(2017052603328)(7153060)(7193020); SRVR:HE1PR0401MB2425; X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 3:PqmsH8qqwsXero3+XISRDGI8+mfBQWi7+O4QUYV9vbyGyBvKZ1CAw5hjoeOrvU4kBmqNgSqYWGebhnWOZPK/Gx40EA8Ob0mvzlsMf0TDq1ACHHHh1N/G6/kOjCzX8pniiUNV1p5kDWMhK5DXeGjCD5D9BpRz03RQJWzJCUnl40phpfYIFoP3+s+GVjhUHNX9pA9JtEVrEfzvfUv6/tXWCWIIJCgsmXW+xK7Ig5fu+sqK8TRaFFzdu8nt2hxvBwMp; 25:KHuP4B6l8Z+7aEbPFOVJKDiUoNvqUlACjp8AotlzBvadhlrCkmtZvcwj/+hMqRYotzSh2ywrGEhubZITsdWvLezr2zwY0Q7E2yOSa56+UR0t0Sng6ap8pDMQ2hnYIQwX++BLkvoNyLD318xBMSaDQGfhlZdawrzPpKcgtQWWfnAQsRdWtc9tAmCOIztJKYycpyO4gh7Gpxf7oMBfDbTw4dD5o80oD88ftvqcHb0wUavNdo8kUkxpn49nF55abFNYbx/puHPXhiTxdldjwiamjcev4jcdnqTIBLM2dqQK4wSr9w1pzYoamlzHlPo0vN+Y7yfZ/OucBBDgsT2bpADv3w==; 31:sHILoEhuDRjXSXPKFwxZDrRoXt//GNM85NkoCQT/PIgBt58Rr5+kBJmOneZO3fc1NY3Q2/I8tMMTQM8q0lVKb7yvMXKGzGZadvir+xo46mg86qTFZKIPFmhgJhtIZqUG2AVPdhEl2WzE2GWqWD5zIWdvK71GDR34qzbd3GxUx+onkN8PHcwDYSoup7o7m7MIs/p5UMO7ISrHYDdOmSiuslYZEVK1WjgGqK7tbb8vgwk= X-MS-TrafficTypeDiagnostic: HE1PR0401MB2425: X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 20:9aYsNcs7TmYb0LrxcLzFs3skx1q6v7BiNnO4LNWUzEjr6BPkqk/yLDHdm9/ddHzFDZ02qxgA8bVXlQ6nMaC3blF+k0mQLCo/f0qx2vyqwXFdbDp1A9Bi/IxyUGZKbzHVeomCfuWl6IrbZvzf8rNzpUOIYS0w0S0VljYbwdNtHhuTR92Z/0R9f0NXNz5sXCZGc+wdbq+aS2KM+Xj0dncPkO+Yy7jFAkT2Qo7so6hNXalPcEjXuMbCkMNQyUL0ol1N+E5Gneg/27sKIfNH0CX3ObCCC7SKj0d9gl6keVbC+EFTzW5jDNZayplWfAzhLWPaHkk756JVDds2OXV1RQnPBqUyeP9Lb1FXTYEPzPGlpyKdWzoZEQTkq05rLk1vJErHZQr9drMOfcGHdG3AYcvkB40dOI4yAjyZBVSWdfORc9vJDiHMGErfWTKnTNi8ojRN+MewQe7op/R3/meoXHgnQAxxuuWXom49vad45smZqn3TNzFvZ4NZoDalZXQeCSvr; 4:te+Y6AlJJTnUBW2BofJ3OEA4PT1T91cKK437aDxfVh09GK9vA4KTAQ+b62pWve+KLTv16Kzr6y34DcCuDXPYUQulx2IHqPYTwJkqx3eDMsLn0gc+E7GHVCDYGp55MM6Cykwns6YzVUtV505+oOka+of0PhqyF4chZMphvGDZZ3POkHqQYj5FEhtnMIGgn3304hk2fLCqDEcQauJb9+UxE4O0roB25+eSyZxwCmYS/noa0huIIy64zdZCOm32kGkMiaJt/qjYAhpVl/1yApHbxyOhhkTKaCCUH1SsrtbD1Y0bSM5EEFC46fwKqpHnOzF6CjQujCjX3xB+jZk+lWSnB9oK4UecdVFuvhE6OQ4Z2SrlS1GhLk9ZMATj+QwyLp0aQpp2gPlstfaOyoNvhv9WwRn4/umO6H7HXq3aZ+T4PyE= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(185117386973197)(211936372134217)(153496737603132)(17755550239193); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3002001)(10201501046)(93006095)(93001095)(3231254)(944501410)(52105095)(6055026)(6041310)(20161123564045)(20161123562045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(6072148)(201708071742011); SRVR:HE1PR0401MB2425; BCL:0; PCL:0; RULEID:; SRVR:HE1PR0401MB2425; X-Forefront-PRVS: 0656A4403B X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(366004)(39380400002)(396003)(376002)(346002)(39860400002)(199004)(189003)(26005)(16526019)(39060400002)(5009440100003)(86362001)(486006)(105586002)(956004)(2616005)(11346002)(476003)(8676002)(186003)(305945005)(6666003)(478600001)(446003)(25786009)(81166006)(8936002)(44832011)(81156014)(4326008)(106356001)(7736002)(76176011)(53936002)(97736004)(16586007)(316002)(50226002)(50466002)(59450400001)(36756003)(6486002)(48376002)(55236004)(966005)(6506007)(386003)(5660300001)(6512007)(66066001)(7416002)(5890100001)(3846002)(47776003)(2906002)(6116002)(68736007)(52116002)(51416003)(6306002)(110426005)(357404004); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR0401MB2425; H:b27504-OptiPlex-790.ap.freescale.net; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:3; Received-SPF: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 23:32LTw2NImcBHB/lwhwWUNHXTwOMjh8CuUysbjjisenwDiu0f8h25qi9iGkTdk4Sw55LgaqPLI96cjb0iDC8Xc8Dxe3vkdTgTbbInR4tuobOox8NLACbN55++g83vPVKHSfrl9ifoZl4KGBZs+DRAHSkmYkyEP0jBhJ+pgO3+qj62b5EOeHhwgwJQyGTte/CcsuwJcP7kRB8OQ8OFz9qJ0eLMyCZ63FFM/4UJE7of9mu1g7KbwZqj93PNdPT+XFv/V86I7dUrkqzvgzHunw8LAL+cTildgj04GPzZks5qY2LIbmN/ozwQsW8Np1wCm2ecrQS43Bl1B/TasHqFJxbLdeBAiPUqJkfG4vV4GxbJw9ZmQwLzE7WYjry6e2jsJXskqh1fbD1tn+CAobYmihNBsB3lPCVaE5Ubl/GoaiUXeeom0M4z/GZixaPUtHe+cxzBk4uV+asxVSyFP3YfTCFf7j2nkTXnUkzU/01tPd/QUs+3wyp0sJ6OaJP/P9Blv20VAEjm6k7SvfPOFKkrVp2/E9ClGSZ38MnVlW5ZkchdcRN9r7Ol3/vY/7CuSzEhBzZiNmSJgM6UIKfZRgYkWDEOiH5YFy19LsE1icPXB1PF/+R/8967c4IjsNEsG00geuQy4Lx6v4EqFxxdtlgv8WQyOnzch3kfQfdTRZo+jsBKpVj4mux0DtiOhaTyArkxW2EyUr6V/YDM5KCmr+tmvU1Z6XM5YjKHCenLTDAkPqkwm0KNNU8s5Hy3lSv5nC0ncuRuSwX7TkWbypShZYjzvevAdsr6G/4n9bTqzA1a8y8guLCBShFs2c+AtpVjaUGB7zUd5f2rcrMs/UA4iwIkQI1xUBuuu9nyniC1L1SiVdHlU4Trrt2tMeV4PaW0vFcbXwacuxZ50xY6D7Kw2PsXz0dtqytlstMGp0EYrGmEP7F8A3KaNE6pjCzepH8kUVbV1RoO6VtsUz6J8bDPa6yBrw4MqiE/TawvZ2eiNCMxRVUXHGMFGoBKBBHv1Di8Ee7HGWFGVZyVGbWf3pSyBH1VqIiFhFOJuCtvEnw0teLoyOiVEd2MQThp16e2Ar8RtCqzeeZCpzJsDpfOnUjInkEEPDluZGXv1j16P5y/Txb3VKELoAS5ME6ZWQ7IQq7Jokihg6QapU3i8r+gBuxDk4oDZYFWZgGv67TM/SmGn48i4HOTBVTdtJFFYWLdPgiZL4M8uWv+f/OIuIGUOsLDo2FnZm3/AVYUZEQpPJCX/9TsCxMb839JAmpcOCZwUNT9tr13Kuf9fF/pLQVaYmpVlpygiWKW+qMTlNlb5D2ed2Zf+QEeqTi8hec5SPh+SHoDUlrs05tpG/GGVgNReTUakZmjvWubQR/uF4fEWTAQbT3KeXGDkm/iIGKVo1dmcbwBDKFFCUryt2Rnu69a5ss4eyRy/+cs4A== X-Microsoft-Antispam-Message-Info: Fh6ibvQJ/oSb/T8+iPAtWwe3yEPCKXTIeulrrRQwvcP2U9fMLzpqaTurp46dRC8DjI3M99CzjE/1/yUaKi+1AEJ4b8KtaR0V1O8qAdVEPXE9U0eD/M2uhnmO5g9KkWEZngaKh5NJrAlFkXStvqxxmNLjffD1+mgp99OVAXAFGCRtf8ynmI4Nl7GJ7403SWlO X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 6:P0mGYVMOVkFSUNL2i2eKpd+RH8p23K8w68F/QdT59HQXVO/p4CcboLQXgv+JZXiuioZUip3pe2903D4DVO3i+qbvoMBb6FoBpq3SfPnD92QQQMrSiiz3a5lH2NxHggHejrB5gdlhrzLgG87D41VqOxTtuAN7cbv/fesl2vloZwZdl/ppAZVa8DL1clDFqACjtUhVhQ1Evt07IjklDEbJOO2T26xvak+Yd+B0Z7HQLKlaBmoDL0/sE8AavWCS6iDJ0KywcHYNSfsOfD1KA7p/OtUPwI9JrEubE8hYBdwCLN+gOeOto+uopoQ3K3XO6SHDWH/PmqUjf85cgqRYgGOtAEwbEb9rt2utURzILV1cr61H/9I6ZeFL1ZJWHtLCiGzotNw2IQOBbT0uBWXP0HJ2bUJbecyq/xHI2KhU827NlpCw7GBSldrH8R5+RKnRcO2GN3dgbkBfU75GPTnxzq/Rvg==; 5:zVkN0sQSHmd+YH/w4zadHcTyE4nKWnjbKuoNMUbZknfjCTrVI571qZU9Phpfxc8KZVYDLOT59FjJAZNvTuKuz/0CrV3CwO5ZnVcUDNpvmpNLw4iZK0S9U11bEhuain/Pveeip50+zBr1H741d9R12U0uRxom4J22g6dWhzGkB1Q=; 24:flzfkHqem5T9DurVIXE5LK+gqYBAvAeXec9E5nQkJRzMhj+5tJKG0shiO5vg5Gu9Ydx9rfTW9cVnMvttYBCKc7P/mKK9/Db4RPLh3J+w+8s= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 7:oa1/lBXFRMFe6tjm680HPkPQTV0/t4c+k8DyybJhLndk3/MA3fdvksYcmfCm5UDmZT7NFhD0amitIw/tDjvUdtz/3H1QhiYiNtajPaXnsOfxeuQznXsYF7JpylQRLQ6fy3Ep25DAtoHwm/mKiwBMZiLUjSsWxbc7b+AJOhiM97Kr/ui/9lQ8T4hzQ/dHZccYDHP3Ro501PMKH801oXHVSQRcAlHHtLiJ9Hzrx9FfAZWe8coJiQDuakehZ6A4HWE2 X-MS-Office365-Filtering-Correlation-Id: 4087e252-248c-498b-c171-08d5acb313a4 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Apr 2018 02:52:21.4293 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 4087e252-248c-498b-c171-08d5acb313a4 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0401MB2425 Sender: linux-tegra-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org It is bus specific aspect to map a given device on the bus and relevant firmware description of its DMA configuration. So, this change introduces 'dma_configure' as bus callback giving flexibility to busses for implementing its own dma configuration function. The change eases the addition of new busses w.r.t. adding the dma configuration functionality. This patch also updates the PCI, Platform, ACPI and host1x bus to use new introduced callbacks. Suggested-by: Christoph Hellwig Signed-off-by: Nipun Gupta Reviewed-by: Greg Kroah-Hartman Acked-by: Bjorn Helgaas # PCI parts Acked-by: Thierry Reding --- - The patches are based on the comments on: https://patchwork.kernel.org/patch/10259087/ Changes in v2: - Do not have dma_deconfigure callback - Have '/dma_common_configure/' API to provide a common DMA configuration which can be used by busses if it suits them. - Platform and ACPI bus to use '/dma_common_configure/' in '/dma_configure/' callback. - Updated commit message - Updated pci_dma_configure API with changes suggested by Robin Changes in v3: - Move dma_common_configure() within platform_dma_configure() and reuse platofrm_dma_configure() for AMBA bus too - Declare 'attr' in pci_dma_configure() inside the else statement where it is used. Changes in v4: - Rebased on top of 4.17-rc2 drivers/amba/bus.c | 4 ++++ drivers/base/dma-mapping.c | 31 ++++--------------------------- drivers/base/platform.c | 17 +++++++++++++++++ drivers/gpu/host1x/bus.c | 8 ++++++++ drivers/pci/pci-driver.c | 32 ++++++++++++++++++++++++++++++++ include/linux/device.h | 4 ++++ include/linux/platform_device.h | 2 ++ 7 files changed, 71 insertions(+), 27 deletions(-) diff --git a/drivers/amba/bus.c b/drivers/amba/bus.c index 594c228..867dc2b 100644 --- a/drivers/amba/bus.c +++ b/drivers/amba/bus.c @@ -20,6 +20,7 @@ #include #include #include +#include #include @@ -188,12 +189,15 @@ static int amba_pm_runtime_resume(struct device *dev) /* * Primecells are part of the Advanced Microcontroller Bus Architecture, * so we call the bus "amba". + * DMA configuration for platform and AMBA bus is same. So here we reuse + * platform's DMA config routine. */ struct bus_type amba_bustype = { .name = "amba", .dev_groups = amba_dev_groups, .match = amba_match, .uevent = amba_uevent, + .dma_configure = platform_dma_configure, .pm = &amba_pm, .force_dma = true, }; diff --git a/drivers/base/dma-mapping.c b/drivers/base/dma-mapping.c index d82566d..f831a58 100644 --- a/drivers/base/dma-mapping.c +++ b/drivers/base/dma-mapping.c @@ -329,36 +329,13 @@ void dma_common_free_remap(void *cpu_addr, size_t size, unsigned long vm_flags) #endif /* - * Common configuration to enable DMA API use for a device + * enables DMA API use for a device */ -#include - int dma_configure(struct device *dev) { - struct device *bridge = NULL, *dma_dev = dev; - enum dev_dma_attr attr; - int ret = 0; - - if (dev_is_pci(dev)) { - bridge = pci_get_host_bridge_device(to_pci_dev(dev)); - dma_dev = bridge; - if (IS_ENABLED(CONFIG_OF) && dma_dev->parent && - dma_dev->parent->of_node) - dma_dev = dma_dev->parent; - } - - if (dma_dev->of_node) { - ret = of_dma_configure(dev, dma_dev->of_node); - } else if (has_acpi_companion(dma_dev)) { - attr = acpi_get_dma_attr(to_acpi_device_node(dma_dev->fwnode)); - if (attr != DEV_DMA_NOT_SUPPORTED) - ret = acpi_dma_configure(dev, attr); - } - - if (bridge) - pci_put_host_bridge_device(bridge); - - return ret; + if (dev->bus->dma_configure) + return dev->bus->dma_configure(dev); + return 0; } void dma_deconfigure(struct device *dev) diff --git a/drivers/base/platform.c b/drivers/base/platform.c index 8075ddc..638d42e 100644 --- a/drivers/base/platform.c +++ b/drivers/base/platform.c @@ -1130,6 +1130,22 @@ int platform_pm_restore(struct device *dev) #endif /* CONFIG_HIBERNATE_CALLBACKS */ +int platform_dma_configure(struct device *dev) +{ + enum dev_dma_attr attr; + int ret = 0; + + if (dev->of_node) { + ret = of_dma_configure(dev, dev->of_node); + } else if (has_acpi_companion(dev)) { + attr = acpi_get_dma_attr(to_acpi_device_node(dev->fwnode)); + if (attr != DEV_DMA_NOT_SUPPORTED) + ret = acpi_dma_configure(dev, attr); + } + + return ret; +} + static const struct dev_pm_ops platform_dev_pm_ops = { .runtime_suspend = pm_generic_runtime_suspend, .runtime_resume = pm_generic_runtime_resume, @@ -1141,6 +1157,7 @@ struct bus_type platform_bus_type = { .dev_groups = platform_dev_groups, .match = platform_match, .uevent = platform_uevent, + .dma_configure = platform_dma_configure, .pm = &platform_dev_pm_ops, .force_dma = true, }; diff --git a/drivers/gpu/host1x/bus.c b/drivers/gpu/host1x/bus.c index 88a3558..a9ec99d 100644 --- a/drivers/gpu/host1x/bus.c +++ b/drivers/gpu/host1x/bus.c @@ -314,6 +314,13 @@ static int host1x_device_match(struct device *dev, struct device_driver *drv) return strcmp(dev_name(dev), drv->name) == 0; } +static int host1x_dma_configure(struct device *dev) +{ + if (dev->of_node) + return of_dma_configure(dev, dev->of_node); + return 0; +} + static const struct dev_pm_ops host1x_device_pm_ops = { .suspend = pm_generic_suspend, .resume = pm_generic_resume, @@ -326,6 +333,7 @@ static int host1x_device_match(struct device *dev, struct device_driver *drv) struct bus_type host1x_bus_type = { .name = "host1x", .match = host1x_device_match, + .dma_configure = host1x_dma_configure, .pm = &host1x_device_pm_ops, .force_dma = true, }; diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c index b9a1311..ba8c6b4 100644 --- a/drivers/pci/pci-driver.c +++ b/drivers/pci/pci-driver.c @@ -16,6 +16,8 @@ #include #include #include +#include +#include #include "pci.h" #include "pcie/portdrv.h" @@ -1577,6 +1579,35 @@ static int pci_bus_num_vf(struct device *dev) return pci_num_vf(to_pci_dev(dev)); } +/** + * pci_dma_configure - Setup DMA configuration + * @dev: ptr to dev structure + * + * Function to update PCI devices's DMA configuration using the same + * info from the OF node or ACPI node of host bridge's parent (if any). + */ +static int pci_dma_configure(struct device *dev) +{ + struct device *bridge; + int ret = 0; + + bridge = pci_get_host_bridge_device(to_pci_dev(dev)); + + if (IS_ENABLED(CONFIG_OF) && bridge->parent && + bridge->parent->of_node) { + ret = of_dma_configure(dev, bridge->parent->of_node); + } else if (has_acpi_companion(bridge)) { + struct acpi_device *adev = to_acpi_device_node(bridge->fwnode); + enum dev_dma_attr attr = acpi_get_dma_attr(adev); + + if (attr != DEV_DMA_NOT_SUPPORTED) + ret = acpi_dma_configure(dev, attr); + } + + pci_put_host_bridge_device(bridge); + return ret; +} + struct bus_type pci_bus_type = { .name = "pci", .match = pci_bus_match, @@ -1589,6 +1620,7 @@ struct bus_type pci_bus_type = { .drv_groups = pci_drv_groups, .pm = PCI_PM_OPS_PTR, .num_vf = pci_bus_num_vf, + .dma_configure = pci_dma_configure, .force_dma = true, }; EXPORT_SYMBOL(pci_bus_type); diff --git a/include/linux/device.h b/include/linux/device.h index 0059b99..607c7eb 100644 --- a/include/linux/device.h +++ b/include/linux/device.h @@ -88,6 +88,8 @@ extern int __must_check bus_create_file(struct bus_type *, * @resume: Called to bring a device on this bus out of sleep mode. * @num_vf: Called to find out how many virtual functions a device on this * bus supports. + * @dma_configure: Called to setup DMA configuration on a device on + this bus. * @pm: Power management operations of this bus, callback the specific * device driver's pm-ops. * @iommu_ops: IOMMU specific operations for this bus, used to attach IOMMU @@ -130,6 +132,8 @@ struct bus_type { int (*num_vf)(struct device *dev); + int (*dma_configure)(struct device *dev); + const struct dev_pm_ops *pm; const struct iommu_ops *iommu_ops; diff --git a/include/linux/platform_device.h b/include/linux/platform_device.h index 49f634d..3097c94 100644 --- a/include/linux/platform_device.h +++ b/include/linux/platform_device.h @@ -356,6 +356,8 @@ static inline char *early_platform_driver_setup_func(void) \ #define platform_pm_restore NULL #endif +extern int platform_dma_configure(struct device *dev); + #ifdef CONFIG_PM_SLEEP #define USE_PLATFORM_PM_SLEEP_OPS \ .suspend = platform_pm_suspend, \ From patchwork Sat Apr 28 02:51:59 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Nipun Gupta X-Patchwork-Id: 906025 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-tegra-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dmarc=fail (p=none dis=none) header.from=nxp.com Authentication-Results: ozlabs.org; dkim=fail reason="signature verification failed" (1024-bit key; unprotected) header.d=nxp.com header.i=@nxp.com header.b="EPRRKxhB"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 40XwMr2Yymz9s06 for ; Sat, 28 Apr 2018 12:52:56 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932841AbeD1Cwt (ORCPT ); Fri, 27 Apr 2018 22:52:49 -0400 Received: from mail-db5eur01on0089.outbound.protection.outlook.com ([104.47.2.89]:17744 "EHLO EUR01-DB5-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1759485AbeD1Cwl (ORCPT ); Fri, 27 Apr 2018 22:52:41 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=+w2qmxxQVRe1f3ycUSORXheLvhJJRyD+gMSa0ADmtaI=; b=EPRRKxhB+zfE5ON4/jAxrHHZr0Z596ckiVDf4eWYIL6vivPAGYidfRpOnfXk69rTcLAlRY6fBj5g8wBV5R5m2v6cAv9M6LTPOxG/nxUxA+YKLRI4gXj/tKsHxvMpmCGS47zHclL+ESSoEhijwOJFiKZiGLG00Woyna/0xYvFqLE= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=nipun.gupta@nxp.com; Received: from b27504-OptiPlex-790.ap.freescale.net (14.142.187.166) by HE1PR0401MB2425.eurprd04.prod.outlook.com (2603:10a6:3:25::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.715.18; Sat, 28 Apr 2018 02:52:29 +0000 From: Nipun Gupta To: robin.murphy@arm.com, hch@lst.de, linux@armlinux.org.uk, gregkh@linuxfoundation.org, m.szyprowski@samsung.com Cc: bhelgaas@google.com, zajec5@gmail.com, andy.gross@linaro.org, david.brown@linaro.org, dan.j.williams@intel.com, vinod.koul@intel.com, thierry.reding@gmail.com, robh+dt@kernel.org, frowand.list@gmail.com, jarkko.sakkinen@linux.intel.com, rafael.j.wysocki@intel.com, dmitry.torokhov@gmail.com, johan@kernel.org, msuchanek@suse.de, linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org, linux-wireless@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-soc@vger.kernel.org, dmaengine@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-tegra@vger.kernel.org, devicetree@vger.kernel.org, linux-pci@vger.kernel.org, bharat.bhushan@nxp.com, leoyang.li@nxp.com, Nipun Gupta Subject: [PATCH v4 2/2] drivers: remove force dma flag from buses Date: Sat, 28 Apr 2018 08:21:59 +0530 Message-Id: <1524883919-29484-2-git-send-email-nipun.gupta@nxp.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1524883919-29484-1-git-send-email-nipun.gupta@nxp.com> References: <1520868292-2479-1-git-send-email-nipun.gupta@nxp.com> <1524883919-29484-1-git-send-email-nipun.gupta@nxp.com> MIME-Version: 1.0 X-Originating-IP: [14.142.187.166] X-ClientProxiedBy: PN1PR01CA0071.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c00:1::11) To HE1PR0401MB2425.eurprd04.prod.outlook.com (2603:10a6:3:25::22) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(7020095)(4652020)(48565401081)(5600026)(4534165)(7168020)(4627221)(201703031133081)(201702281549075)(2017052603328)(7153060)(7193020); SRVR:HE1PR0401MB2425; X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 3:12ME/HbqGs6Of5PRfk7JiuWnzUNSe+LgVqO2acmJb4eRlZXaAxfmMlvmIMkrHZ1bE16uAP3j9quurNLzfEvfQH0LeSPesJ0mYw/UNZBEFoorzlZiVR5jXDX2R3bdZywSbDDKYee0c8+9UywNVW1yPYRPhACFaNhFnKz1a4r4c8/WvoO5gKjYmleMK2p+hSbet6a5Cj2aHD0VP3qIU6Y1TzOAXU6dvxNLIChPHtcIgqvOha+XeJPKCdHj/EY9fIOo; 25:cFiQzW5ZMMomzM8yuCMS928ZYzQgE5QvuybldL7ShjO5Gca7f6KYUqOUiSlnVj0qFRYhdWgp4F+ffY8QzDhK4HNNRJ4VhBtEE8mltJFIig8EbcfcN5o64arPdH5Oj7CPNbd8Z2kcklnRIAyrLvWocB1igkELWQz4shz4Hx+GAhneUdkbxXJW0o61E7McNlUhODxKm5uSUwlgv9/dgdsuWNN8gcNwNYT+z9Oj79AH41drsxVSZaGOwIE2W6zvx4THtEDA1NFJjjveNgZZ6d+rMT78oil5Mex6ZbdvlyQEtB16YNcHqfuPh6JJzHi49txnWyOryEhs7EeU4AoanRfrZw==; 31:3OuddoQVO59GW4VZO2tazJp/y7aUURrgSgr77ehs4jAtbumyE76r6zr4siN4OZp4TCn6m+UcsS8XwmI+keF/nHFXQtvlPSKdFcmlNhYgVqxZxI0EMYdTouDMu/ChkZswDFE7qHb3WQD5InnGsvLC/g9FjhwLC1M1KzI7MZo28CB8jTipo9kAAulILDQiqx7YJedFwTWCsmpzeLefxMf8CE+58wT3E6Zjuik7r5TXdvs= X-MS-TrafficTypeDiagnostic: HE1PR0401MB2425: X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 20:xIj2rPcFCEpE0V4eZfBjcwbjuR2I+LGC7TmBqnVPlbgJsdbGbR0h23LDEn6DXnF0MWP/ak252hb3EoHQ+J8BoG+D93diyCDfW+k6iWXoZvGpdC+oBnbo/YT9phkYrItiD7rROIDDaYgxHnFZDzOcfSMzyPzkvHGy1UOyUNoz1cbJRh6XI348PkW+bd2/Ou671QJT7tgKaiIgqkTP8MWGPsnL0SfIY6ecrats6AwtbhjL25weSohHx0sDezFwpuNGkJo2+YBeYcWsUPuFExBqvbZVpGFFcD2sO7nwCZFiwawlhBgkbojDn8MACd1we+gBGIOmToUubw+CKCcmlcDPqlvaMEmnI0uPXUUNPkP+DVqhoAP39aLgo4fCHFsKpv96sCDL+ZCJdYv5rpTm69uELya4ZpfbZ2GKLKvAP94a5IFdNIG0caMz9ZWHQyiJfdtLATx9AyJowih2DzAsbx5cexBz2cMhUVQACpM8GcktX2qr4xqCTY+AQVLJzwtaUjKP; 4:BkZ45Fz6XhIB8+MMrkn+Ngi7StRfJLxsXE125BgqZl8F8y71dM8Y9UnmbslfFPMV3zupi2GEJNTahGlLiJ98uUypC+63gcf+zK2oq29GloGqWgMXGpb6BrfUcEY3c/sTU2ZcbZmOJZzIj4SgHj5Lq9eHjUVHRQX3Cg+lLmxGn/0K3c7zRHKLkNYZREfAq1mOjUDMTJy3KADov8ZysYIvlyOGV1oc2FeNcFuctbhmNXRh0y5ZWcxxPMkBVqWAiQf1dEZcUzYZvJBMeolQ49j1AVi317X8r0tWS4ovz4mE24h0COwQqACmYMj8d5Bsrg/E+6P6V1o8QM+utEvhK7o5CwYk2iglSKPNBGrnVB6RPeMLEQHbkQ89GFwXuyNDWn1q X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(185117386973197)(211936372134217)(153496737603132); X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3002001)(10201501046)(93006095)(93001095)(3231254)(944501410)(52105095)(6055026)(6041310)(20161123564045)(20161123562045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(6072148)(201708071742011); SRVR:HE1PR0401MB2425; BCL:0; PCL:0; RULEID:; SRVR:HE1PR0401MB2425; X-Forefront-PRVS: 0656A4403B X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(366004)(39380400002)(396003)(376002)(346002)(39860400002)(199004)(189003)(26005)(16526019)(39060400002)(5009440100003)(86362001)(486006)(105586002)(956004)(2616005)(11346002)(476003)(8676002)(575784001)(186003)(305945005)(6666003)(478600001)(446003)(25786009)(81166006)(8936002)(44832011)(81156014)(4326008)(106356001)(7736002)(76176011)(53936002)(97736004)(16586007)(316002)(50226002)(50466002)(59450400001)(36756003)(6486002)(48376002)(55236004)(6506007)(386003)(5660300001)(6512007)(66066001)(7416002)(3846002)(47776003)(2906002)(6116002)(68736007)(52116002)(51416003)(110426005)(357404004); DIR:OUT; SFP:1101; SCL:1; SRVR:HE1PR0401MB2425; H:b27504-OptiPlex-790.ap.freescale.net; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1; Received-SPF: None (protection.outlook.com: nxp.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 23:w5KxoBazi/TgzvvdwGe7Fc2a2lkAEQuvg4UKda7Ue/8xuan//LSfJnsYv0EoRSRMOObo1c0akHjDj3ABfbRW997/nFCHq0szktg4JH1k+gEv2tRqZiNn8291tV+OtdPLcjsZX71+nV7EMkTCaajChCCUIoVBJ1Y8V1NRzUBX1hsJIwtvwECqYiuYsziSZO4nYcKimjMjnQMFD5fqge8VgCmHYba7BbDvjx2xxB+aQ0g3ZZWK2eKFAEG4+YXf3BzNbX4BeEG2Jg3Zab29hgdooWtX+5psUWXYVaOizXJjMUggxiZlKhxDlY4eqeYcrCckRPY+g1CqDGPWcS2Afq2bJy3ekgTqkp0Ma+5zCnwv/aARqY+lxNFkgoRJyrOwrz50j7W8L7IaAmBU7bit8EZ9zjs84BvXd3JrBYMoTHu1XkBH/L+19BOesphG+/k9qKEQwJ/9E4yI1e/V8aiVkiyiP0faPg2ySPoW10xdOpEbN/nUvz4lImkxsXEh9J9qgVUxG4m0Z2Q+BXybyNlJQVh6v8OrHoxm8SUTFDRCrY+fKfE+M8IdMCXd4CbDKcdXqEG6brupClXZPBxinBBiGA+1wmFCNe9A3VEVP9ALd2QtZfpVfqeN00ZsmdxVZa5sngGlQmGXq4gFZLEH84ueGKg+udrCVrBWAs+icF49zmpdJwsZyNix9fbUrtPcm+hf/4/VHESTYAJx7si6OUpyIotpnAZnjFFECNkZxszDLIhxgLJmCW7TWNw13j8XWhZjVWu3EAG5PDL/B+sRzUlMSt6/ZnKeQxau7pe8Ygf7MbNdfRx5ixP+vlwNRemIV8+7YKw7S6mR6bjL21SpjiKgwOWEExrVbubc4L/J97Hd6rvggXfevyqrDADSEHnJrCDmCymnMfQHMjv6vNQg7iffp16paIl9hL9ViweXHP8J0vEIdAvbmi/CWLyCsDFW5Sku+CqjeOPvruJD2YOQnAl/GJ3ftoXkC3JfaGuJdZtVHQNeEJVm+nv1xo97SyjfY3n1TZj/ATF+ohCUiLuxmrF1pXuo2dbfleUT/UOqnfer9i8aYzwUBG1XLNZBeO4rqyqhkEDpt94JS1AcT4boI1Qn+ztc2YZfCNb81ucHXQm6cM4LFIiJQgH9rjLERE5TSRG2qZSv43Omr4c1EuZstt5tLOh4rXfeIJjfvi49jFwQSCXFX+Jlzf/3tlhcLjEKTZvFyLYZPxIYtbtzClAWG1+A9N7kCI13wVoKnTiZRPO/h2Zgu/ZTaDBEON/2am1hEPg0rRLWsgQE0iIxOMQ3Lzb/eUUBsuCi9PMRsYtTzHO43cwXKrOIDWWVcb+uOvC4kvzAb2FMdTUBzUlyXWd4Mnb6BjA1hKdNambJ4lZDE48rPps0Mtg= X-Microsoft-Antispam-Message-Info: EiiKYz0b9WOsmOxII5Dgp16nDOK+/IC1onIoMMvR91BU93gww/lnqgg9/x3NmJVYBBN/fU2nBMwucMtOEzByJqcQJv8Z21ZgEcYe0MRYzgGLdzU+paJJmszfgK3tAYUWcTE75l2u8/BSQY0ELJqp18iWTOTQYdtdLU4nUb2zYATLhh2G+XI03nzJiBu3W+d1 X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 6:iiuJolBeFkqWLCV0/FjeBy4PVRo+IQMYquq6kNEioQhzvLu/lgdZXurK+OnYd6Uo6m2oc9qyGNRGs4DSPp5+MDxnZeuz1/+o/RFoeXPFNg4duTYBrmCNmGByfwSwKoAKLXPlrxNyLI6S8ic87+PyO+X7c3sSPvpZSL+m6+NnZFB+X58c6r5znx3b2R3GbhfB00cp0cDW+6Du5/VFmJ3zkgZvLPUQfU+witSyqskyJSdZ8fjNIkPelHYduSabIIVETdypZQj1G7j3E4VYl9j+ipuHoQ79OOhO0DdQyGcYM6ARjdPwjLsXMV0JCa1cbMNwuTvUH+HPqbeDDRTEcZNnkZzRIILnxsjHZatGRpRNTPNDFdAo5cfwaZWI6AqSKBz4UefBnDw8T4u+xzdSGMJ5EQ9MrcDO/vQgYBn3scDs3dv3yueGxOX+DpQmanzt6mbOZmxjk2aeiCTsonjNIqRAww==; 5:/tRThzWvGRS/BghvEXcihJZ2M3TbCUROZYSFnZH23nXMd8Jct23sx/T/nmnoZh4WzZMrv92a6L4xUZObDLKEtM9OFnojcpNwds0BTS8+XnMdzG1NsdMTjn9szxiuvPjTNnaZvYoO5w4m7FfDpc7WIXl2X+aopJE3ZRK7dJAiBS4=; 24:IvvJ4eidfuqBjDms5BP8WxCOuAaZ2AGtMuM8EnZ6grIQ/tldra1jxkdEb8+mwcdlaGFx6tnwMiblEGRKmSk9jetTjn6TvTDo2ZdKHyBCIAM= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1; HE1PR0401MB2425; 7:Fh1N8mV6NavkbaLdxj3nPgYImmOaHosv6TFSPmveulx0SqXot6UHY1AlVo+8GunuYIpBahbaakcT3Dm1prtf2mZNRfD4yvgGukSwPqcmqKFbSP3XYA7S6XdMLVAtenIcLdgcXnMtPtMw2aYWO/MQSrUnoUenokCHIn2IXHiaiaurwnF8rLT2LIz6FLrGXPexnLnfpoblEXfLvwomQ3F0RtkDkYxKIPIUxPlU+bryHE0xvPyLpZLL3LhZpZpGiumH X-MS-Office365-Filtering-Correlation-Id: 14b22090-3ea4-4398-d72a-08d5acb318a9 X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Apr 2018 02:52:29.8356 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 14b22090-3ea4-4398-d72a-08d5acb318a9 X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0401MB2425 Sender: linux-tegra-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org With each bus implementing its own DMA configuration callback, there is no need for bus to explicitly have force_dma in its global structure. This patch modifies of_dma_configure API to accept an input parameter which specifies if implicit DMA configuration is required even when it is not described by the firmware. Signed-off-by: Nipun Gupta Acked-by: Bjorn Helgaas # PCI parts Reviewed-by: Rob Herring Acked-by: Vinod Koul --- Changes in v2: - This is a new change suggested by Robin and Christoph and is added to the series. Changes in v3: - Rebase and changes corresponding to the changes in patch 1/2 Changes in v4: - Rebased on top of 4.17-rc2 drivers/amba/bus.c | 1 - drivers/base/platform.c | 3 +-- drivers/bcma/main.c | 2 +- drivers/dma/qcom/hidma_mgmt.c | 2 +- drivers/gpu/host1x/bus.c | 5 ++--- drivers/of/device.c | 6 ++++-- drivers/of/of_reserved_mem.c | 2 +- drivers/pci/pci-driver.c | 3 +-- include/linux/device.h | 4 ---- include/linux/of_device.h | 8 ++++++-- 10 files changed, 17 insertions(+), 19 deletions(-) diff --git a/drivers/amba/bus.c b/drivers/amba/bus.c index 867dc2b..abe73c4 100644 --- a/drivers/amba/bus.c +++ b/drivers/amba/bus.c @@ -199,7 +199,6 @@ struct bus_type amba_bustype = { .uevent = amba_uevent, .dma_configure = platform_dma_configure, .pm = &amba_pm, - .force_dma = true, }; static int __init amba_init(void) diff --git a/drivers/base/platform.c b/drivers/base/platform.c index 638d42e..c0ff1e7 100644 --- a/drivers/base/platform.c +++ b/drivers/base/platform.c @@ -1136,7 +1136,7 @@ int platform_dma_configure(struct device *dev) int ret = 0; if (dev->of_node) { - ret = of_dma_configure(dev, dev->of_node); + ret = of_dma_configure(dev, dev->of_node, true); } else if (has_acpi_companion(dev)) { attr = acpi_get_dma_attr(to_acpi_device_node(dev->fwnode)); if (attr != DEV_DMA_NOT_SUPPORTED) @@ -1159,7 +1159,6 @@ struct bus_type platform_bus_type = { .uevent = platform_uevent, .dma_configure = platform_dma_configure, .pm = &platform_dev_pm_ops, - .force_dma = true, }; EXPORT_SYMBOL_GPL(platform_bus_type); diff --git a/drivers/bcma/main.c b/drivers/bcma/main.c index e6986c7..fc1f4ac 100644 --- a/drivers/bcma/main.c +++ b/drivers/bcma/main.c @@ -207,7 +207,7 @@ static void bcma_of_fill_device(struct device *parent, core->irq = bcma_of_get_irq(parent, core, 0); - of_dma_configure(&core->dev, node); + of_dma_configure(&core->dev, node, false); } unsigned int bcma_core_irq(struct bcma_device *core, int num) diff --git a/drivers/dma/qcom/hidma_mgmt.c b/drivers/dma/qcom/hidma_mgmt.c index 000c7019..d64edeb 100644 --- a/drivers/dma/qcom/hidma_mgmt.c +++ b/drivers/dma/qcom/hidma_mgmt.c @@ -398,7 +398,7 @@ static int __init hidma_mgmt_of_populate_channels(struct device_node *np) } of_node_get(child); new_pdev->dev.of_node = child; - of_dma_configure(&new_pdev->dev, child); + of_dma_configure(&new_pdev->dev, child, true); /* * It is assumed that calling of_msi_configure is safe on * platforms with or without MSI support. diff --git a/drivers/gpu/host1x/bus.c b/drivers/gpu/host1x/bus.c index a9ec99d..b39c1e9 100644 --- a/drivers/gpu/host1x/bus.c +++ b/drivers/gpu/host1x/bus.c @@ -317,7 +317,7 @@ static int host1x_device_match(struct device *dev, struct device_driver *drv) static int host1x_dma_configure(struct device *dev) { if (dev->of_node) - return of_dma_configure(dev, dev->of_node); + return of_dma_configure(dev, dev->of_node, true); return 0; } @@ -335,7 +335,6 @@ struct bus_type host1x_bus_type = { .match = host1x_device_match, .dma_configure = host1x_dma_configure, .pm = &host1x_device_pm_ops, - .force_dma = true, }; static void __host1x_device_del(struct host1x_device *device) @@ -424,7 +423,7 @@ static int host1x_device_add(struct host1x *host1x, device->dev.bus = &host1x_bus_type; device->dev.parent = host1x->dev; - of_dma_configure(&device->dev, host1x->dev->of_node); + of_dma_configure(&device->dev, host1x->dev->of_node, true); err = host1x_device_parse_dt(device, driver); if (err < 0) { diff --git a/drivers/of/device.c b/drivers/of/device.c index 064c818..33d8551 100644 --- a/drivers/of/device.c +++ b/drivers/of/device.c @@ -76,6 +76,8 @@ int of_device_add(struct platform_device *ofdev) * of_dma_configure - Setup DMA configuration * @dev: Device to apply DMA configuration * @np: Pointer to OF node having DMA configuration + * @force_dma: Whether device is to be set up by of_dma_configure() even if + * DMA capability is not explicitly described by firmware. * * Try to get devices's DMA configuration from DT and update it * accordingly. @@ -84,7 +86,7 @@ int of_device_add(struct platform_device *ofdev) * can use a platform bus notifier and handle BUS_NOTIFY_ADD_DEVICE events * to fix up DMA configuration. */ -int of_dma_configure(struct device *dev, struct device_node *np) +int of_dma_configure(struct device *dev, struct device_node *np, bool force_dma) { u64 dma_addr, paddr, size = 0; int ret; @@ -100,7 +102,7 @@ int of_dma_configure(struct device *dev, struct device_node *np) * DMA configuration regardless of whether "dma-ranges" is * correctly specified or not. */ - if (!dev->bus->force_dma) + if (!force_dma) return ret == -ENODEV ? 0 : ret; dma_addr = offset = 0; diff --git a/drivers/of/of_reserved_mem.c b/drivers/of/of_reserved_mem.c index 9a4f4246..895c83e 100644 --- a/drivers/of/of_reserved_mem.c +++ b/drivers/of/of_reserved_mem.c @@ -353,7 +353,7 @@ int of_reserved_mem_device_init_by_idx(struct device *dev, /* ensure that dma_ops is set for virtual devices * using reserved memory */ - of_dma_configure(dev, np); + of_dma_configure(dev, np, true); dev_info(dev, "assigned reserved memory node %s\n", rmem->name); } else { diff --git a/drivers/pci/pci-driver.c b/drivers/pci/pci-driver.c index ba8c6b4..f8269a7 100644 --- a/drivers/pci/pci-driver.c +++ b/drivers/pci/pci-driver.c @@ -1595,7 +1595,7 @@ static int pci_dma_configure(struct device *dev) if (IS_ENABLED(CONFIG_OF) && bridge->parent && bridge->parent->of_node) { - ret = of_dma_configure(dev, bridge->parent->of_node); + ret = of_dma_configure(dev, bridge->parent->of_node, true); } else if (has_acpi_companion(bridge)) { struct acpi_device *adev = to_acpi_device_node(bridge->fwnode); enum dev_dma_attr attr = acpi_get_dma_attr(adev); @@ -1621,7 +1621,6 @@ struct bus_type pci_bus_type = { .pm = PCI_PM_OPS_PTR, .num_vf = pci_bus_num_vf, .dma_configure = pci_dma_configure, - .force_dma = true, }; EXPORT_SYMBOL(pci_bus_type); diff --git a/include/linux/device.h b/include/linux/device.h index 607c7eb..31dcb29 100644 --- a/include/linux/device.h +++ b/include/linux/device.h @@ -98,8 +98,6 @@ extern int __must_check bus_create_file(struct bus_type *, * @p: The private data of the driver core, only the driver core can * touch this. * @lock_key: Lock class key for use by the lock validator - * @force_dma: Assume devices on this bus should be set up by dma_configure() - * even if DMA capability is not explicitly described by firmware. * * A bus is a channel between the processor and one or more devices. For the * purposes of the device model, all devices are connected via a bus, even if @@ -140,8 +138,6 @@ struct bus_type { struct subsys_private *p; struct lock_class_key lock_key; - - bool force_dma; }; extern int __must_check bus_register(struct bus_type *bus); diff --git a/include/linux/of_device.h b/include/linux/of_device.h index 8da5a1b..165fd30 100644 --- a/include/linux/of_device.h +++ b/include/linux/of_device.h @@ -55,7 +55,9 @@ static inline struct device_node *of_cpu_device_node_get(int cpu) return of_node_get(cpu_dev->of_node); } -int of_dma_configure(struct device *dev, struct device_node *np); +int of_dma_configure(struct device *dev, + struct device_node *np, + bool force_dma); void of_dma_deconfigure(struct device *dev); #else /* CONFIG_OF */ @@ -105,7 +107,9 @@ static inline struct device_node *of_cpu_device_node_get(int cpu) return NULL; } -static inline int of_dma_configure(struct device *dev, struct device_node *np) +static inline int of_dma_configure(struct device *dev, + struct device_node *np, + bool force_dma) { return 0; }