Received: by 2002:a05:6a10:a0d1:0:0:0:0 with SMTP id j17csp3147813pxa; Tue, 18 Aug 2020 07:48:59 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyW6BDYUbD+sx7OvMQyUh0Wxu8sZNM0jvcdO4xFtunRLoYqtnZLGeMhjv6XWfRZAHWnRv7o X-Received: by 2002:a17:906:bcf3:: with SMTP id op19mr19959150ejb.1.1597762139707; Tue, 18 Aug 2020 07:48:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1597762139; cv=none; d=google.com; s=arc-20160816; b=yRk4uwM1j/P7sdD6/WFcV7891pIz9quEGPnXdP0gP1nLsxs27x5y9JQ29VFb48cVHO nQTgm417v18jPpLDiU1qZPFGXBAcuuSx9lRstaERdmYzZEKf3sWKtvbh6B/WaAAzDcKG LSKk0NdSQD4qCz6yPNViB3aiNnrqOE7obW08nCeiiBAig6RMRNo4ZZ59OgWBBAj9jeJ2 Jp9FWeKlzuCLQR1Jc98T2FEQl//8y//07dtSDAMAS2+IUA3N+o1chbb9mqih7oswJCxC MVAYqqYfNZbZl1g+22+GQJtGwaSIE+BaPNOxIBMv3REKx4msb8vAIsHJjY4RTWl5hmXg 7QYg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=gCtZUEdRfKV/RQHmg4N297y4Uxs7+WArKhyktsEm9lU=; b=Szl/Z+tXqWZjQJNplnTPEn5JikCWBI3C4xiEw/k8+ze4L0UZzCbck/RwK00GZTcVku 9sheC7igBClIRWDgCifGajSeBl1wVuq4sjVYYP0sqfWy7hyb6y9AWGEachv0wK6hh2hy uTVJ+W2bPIK8aB2gMfT3aDu3fBbroCfayQatb2Rdw+kp9SaMyJJ11BAdxO6D6ijKyANN uL4OfWEGFTKfibTd5crypFspDeOlytnU4JQSI5W3RzAfGXURFFk1nKrax68QPt3/bJas lW4r0nCrSmarWao950SB6j/mCVp0kVimkrA9+fi/fNL0pVQ581unfGO8wqdR2vEhIglH V70g== 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 ci15si13291034ejb.107.2020.08.18.07.48.34; Tue, 18 Aug 2020 07:48:59 -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 S1727077AbgHROsC (ORCPT + 99 others); Tue, 18 Aug 2020 10:48:02 -0400 Received: from foss.arm.com ([217.140.110.172]:42710 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726863AbgHROr6 (ORCPT ); Tue, 18 Aug 2020 10:47:58 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id E23541FB; Tue, 18 Aug 2020 07:47:57 -0700 (PDT) Received: from [10.57.40.122] (unknown [10.57.40.122]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 8E81D3F66B; Tue, 18 Aug 2020 07:47:55 -0700 (PDT) Subject: Re: [PATCH 00/16] IOMMU driver for Kirin 960/970 To: Mauro Carvalho Chehab , Greg Kroah-Hartman Cc: devel@driverdev.osuosl.org, devicetree@vger.kernel.org, Joerg Roedel , Manivannan Sadhasivam , Chenfeng , linuxarm@huawei.com, Wei Xu , linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org, Rob Herring , John Stultz , mauro.chehab@huawei.com, Suzhuangluan , linux-arm-kernel@lists.infradead.org References: From: Robin Murphy Message-ID: <5c7918b6-c506-680b-cb0f-9e5f6a7038d9@arm.com> Date: Tue, 18 Aug 2020 15:47:55 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-GB Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2020-08-17 08:49, Mauro Carvalho Chehab wrote: > Add a driver for the Kirin 960/970 iommu. > > As on the past series, this starts from the original 4.9 driver from > the 96boards tree: > > https://github.com/96boards-hikey/linux/tree/hikey970-v4.9 > > The remaining patches add SPDX headers and make it build and run with > the upstream Kernel. > > Chenfeng (1): > iommu: add support for HiSilicon Kirin 960/970 iommu > > Mauro Carvalho Chehab (15): > iommu: hisilicon: remove default iommu_map_sg handler > iommu: hisilicon: map and unmap ops gained new arguments > iommu: hisi_smmu_lpae: rebase it to work with upstream > iommu: hisi_smmu: remove linux/hisi/hisi-iommu.h > iommu: hisilicon: cleanup its code style > iommu: hisi_smmu_lpae: get rid of IOMMU_SEC and IOMMU_DEVICE > iommu: get rid of map/unmap tile functions > iommu: hisi_smmu_lpae: use the right code to get domain-priv data > iommu: hisi_smmu_lpae: convert it to probe_device > iommu: add Hisilicon Kirin970 iommu at the building system > iommu: hisi_smmu_lpae: cleanup printk macros > iommu: hisi_smmu_lpae: make OF compatible more standard Echoing the other comments about none of the driver patches being CC'd to the IOMMU list... Still, I dug the series up on lore and frankly I'm not sure what to make of it - AFAICS the "driver" is just yet another implementation of Arm LPAE pagetable code, with no obvious indication of how those pagetables ever get handed off to IOMMU hardware (and indeed no indication of IOMMU hardware at all). Can you explain how it's supposed to work? And as a pre-emptive strike, we really don't need any more LPAE implementations - that's what the io-pgtable library is all about (which incidentally has been around since 4.0...). I think that should make the issue of preserving authorship largely moot since there's no need to preserve most of the code anyway ;) Robin. > dt: add an spec for the Kirin36x0 SMMU > dt: hi3670-hikey970.dts: load the SMMU driver on Hikey970 > staging: hikey9xx: add an item about the iommu driver > > .../iommu/hisilicon,kirin36x0-smmu.yaml | 55 ++ > .../boot/dts/hisilicon/hi3670-hikey970.dts | 3 + > drivers/staging/hikey9xx/Kconfig | 9 + > drivers/staging/hikey9xx/Makefile | 1 + > drivers/staging/hikey9xx/TODO | 1 + > drivers/staging/hikey9xx/hisi_smmu.h | 196 ++++++ > drivers/staging/hikey9xx/hisi_smmu_lpae.c | 648 ++++++++++++++++++ > 7 files changed, 913 insertions(+) > create mode 100644 Documentation/devicetree/bindings/iommu/hisilicon,kirin36x0-smmu.yaml > create mode 100644 drivers/staging/hikey9xx/hisi_smmu.h > create mode 100644 drivers/staging/hikey9xx/hisi_smmu_lpae.c >