From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from fhigh-a2-smtp.messagingengine.com (fhigh-a2-smtp.messagingengine.com [103.168.172.153]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7CAD820CCE6 for ; Tue, 29 Oct 2024 20:36:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=103.168.172.153 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730234167; cv=none; b=WK3uPgu168PCqO8Oho/ti8+YbS2cbBKzZDWcNQtH3X2+4jouxg8Y/ILAnA/S7pNg1HxKfyHbPPSZiWidtf9b9RydEnE/4SkEY685EyY8n63SwvHJ1bvB146FAGBPo5T5nMdu0n6ajkWcOKm3DvOJiYpywKUfz7w4M0icoQH7dm4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730234167; c=relaxed/simple; bh=poVk1K9IGk+pzKC1czCu9jgTzAIzM1iCikucexGiMtE=; h=MIME-Version:Date:From:To:Cc:Message-Id:In-Reply-To:References: Subject:Content-Type; b=MZOApvpeaANnXswf/41OBYV8LN5b7cIaIxatbT/ydBJBZEGgDLEgnfIQHRWd5C/RGtwTq9GyvNLIIp/mDueIeEOZP0ptAIrxVgM+QOe/makUqtCPmVBdALm3UOWEIYxyu51wuKPjz5rKBtbDnarUoI7NmisCWGaDRBEgbvqO7Ys= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arndb.de; spf=pass smtp.mailfrom=arndb.de; dkim=pass (2048-bit key) header.d=arndb.de header.i=@arndb.de header.b=EkhttdXP; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b=LrNIW0IB; arc=none smtp.client-ip=103.168.172.153 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arndb.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arndb.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=arndb.de header.i=@arndb.de header.b="EkhttdXP"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="LrNIW0IB" Received: from phl-compute-10.internal (phl-compute-10.phl.internal [10.202.2.50]) by mailfhigh.phl.internal (Postfix) with ESMTP id 957AF11401C4; Tue, 29 Oct 2024 16:36:01 -0400 (EDT) Received: from phl-imap-11 ([10.202.2.101]) by phl-compute-10.internal (MEProxy); Tue, 29 Oct 2024 16:36:01 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm2; t=1730234161; x=1730320561; bh=B3ss9pag0NcR14RtAGsaKv3moDarLBu8u2f0bXVnm84=; b= EkhttdXPjfFuPHBZgqjJmtmOB4B34IlLyudV6F3SejXggUVJKijTh2t0Y63hFteu bcXjJC4azzOrnrNJsLW55wIYaygfrE3BBJq/IPpTP8SCwMC4N87c6/KsodCLqNsJ rqNnGnw3iT9ya9OXvToYVc+PI5ejVKX4aLICrW6m0c3bmIDVn2Nq3IyUEZnRxBt7 gNv6zmp4AzPFatuokFeeAGs6x9r20yGSQ9CuuVBpHTf9d6DbxWmRP0hTDCNejRYL pugd8UQRKUMXgWTcvXPo31ETesDRUlgN26Qm73X6dqVud2Hco5UxXcFqOHc91aY/ ULo60cqYT6rxVv+7cMTExw== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm3; t=1730234161; x= 1730320561; bh=B3ss9pag0NcR14RtAGsaKv3moDarLBu8u2f0bXVnm84=; b=L rNIW0IB5yPkOhDJiafrvgYNsrUAMrXQQlWzzm8aJJaQ+XLwzSTe4GRwrsVN0Udd2 w45zjIqS9rqTfzUMyQxDKR/kZ+lCgB1IXvbMV94PFA45oTwH6UEEJRZzhtXjolzG zyAzVg1aZvOYWKAifP02ZGILQdRXL+kHP1PhtY6jTodHPsScE0rAXK/hNMpNDJOj 4t8Oy5uln/rg8ZwY5FfQD5/BFFktkpU7f734XD98G7mzWY6PGYDr4G0sz7ioQuEe CVRf3leLBPWuBIi4qlGm9Oo6t7jOtF3i5HG7nIbvTfdLwLgXyBH1L1g2IULz8lw1 YQ9IdOjb4ItPp9tuNDBMQ== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeeftddrvdekuddgudeffecutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpggftfghnshhusghstghrihgsvgdp uffrtefokffrpgfnqfghnecuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivg hnthhsucdlqddutddtmdenucfjughrpefoggffhffvvefkjghfufgtgfesthhqredtredt jeenucfhrhhomhepfdetrhhnugcuuegvrhhgmhgrnhhnfdcuoegrrhhnugesrghrnhgusg druggvqeenucggtffrrghtthgvrhhnpedvhfdvkeeuudevfffftefgvdevfedvleehvddv geejvdefhedtgeegveehfeeljeenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmh epmhgrihhlfhhrohhmpegrrhhnugesrghrnhgusgdruggvpdhnsggprhgtphhtthhopeei pdhmohguvgepshhmthhpohhuthdprhgtphhtthhopehhuhgrnhhglhgvihekudegseduie efrdgtohhmpdhrtghpthhtoheptggrthgrlhhinhdrmhgrrhhinhgrshesrghrmhdrtgho mhdprhgtphhtthhopeifihhllheskhgvrhhnvghlrdhorhhgpdhrtghpthhtohephhhurg hnghhlvghisehkhihlihhnohhsrdgtnhdprhgtphhtthhopehlihhnuhigqdgrrhhmqdhk vghrnhgvlheslhhishhtshdrihhnfhhrrgguvggrugdrohhrghdprhgtphhtthhopehlih hnuhigqdhkvghrnhgvlhesvhhgvghrrdhkvghrnhgvlhdrohhrgh X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.phl.internal (Postfix, from userid 501) id 770492220071; Tue, 29 Oct 2024 16:35:59 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Date: Tue, 29 Oct 2024 20:35:39 +0000 From: "Arnd Bergmann" To: "Will Deacon" , huanglei Cc: "Catalin Marinas" , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, huanglei Message-Id: <5f93d7a3-082d-455c-b458-a9c2113eb512@app.fastmail.com> In-Reply-To: <20241029132042.GA4416@willie-the-truck> References: <20241025011641.4858-1-huanglei814@163.com> <20241028150544.GA2445@willie-the-truck> <26ed8fbb.1809.192d5ef13e5.Coremail.huanglei814@163.com> <20241029132042.GA4416@willie-the-truck> Subject: Re: [PATCH] arm64/Kconfig: introduce GENERIC_ISA_DMA Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On Tue, Oct 29, 2024, at 13:20, Will Deacon wrote: > On Tue, Oct 29, 2024 at 09:41:08AM +0800, huanglei wrote: >> >> Even if MAX_SMA_CANNELS is not defined, as long as GENETIC-SIA_DM= A is >> enabled=EF=BC=8C request_rma/free_rma function will also be expo= rted, and >> other modules using these two interfaces will not encounter link >> errors. > > Do the drivers do anything useful after they've linked? Can you please > provide some in-tree examples of drivers that can be used on arm64 only > once GENERIC_ISA_DMA is selected? If they just need some stubs for > request_dma() and free_dma(), then it feels to me like that should be > the default implementation, rather than having to select something to > get them. GENERIC_ISA_DMA does nothing by iself, there also needs to be an iplementation of CONFIG_ISA_DMA_API in order to use DMA mode on floppy/parport/soundblaster. I think this should not be enabled by the architecture itself, but by the LPC host driver that provides the ISA style DMA. Most LPC implementations don't have this, and most arm64 machines have no LPC or PC104 at all. Note that PCI-ISA bridges can sometimes provide ISA devices, but ISA DMA won't work behind a PCIe bridge. The patch also needs a better changelog text to explain specifically which arm64 platform has PC style DMA support, and what devices are connected to it.=20 Arnd