Re: [PATCH 2/2] iommu: add Unisoc iommu basic driver

2021-02-02 Thread Chunyan Zhang
On Wed, 3 Feb 2021 at 02:02, Robin Murphy wrote: > > On 2021-02-02 14:41, Joerg Roedel wrote: > > On Tue, Feb 02, 2021 at 02:34:34PM +, Robin Murphy wrote: > >> Nope, I believe if Arm Ltd. had any involvement in this I'd know about it > >> :) > > > > Okay, got confused by thinking of ARM as

Re: [PATCH 2/2] iommu: add Unisoc iommu basic driver

2021-02-02 Thread Chunyan Zhang
On Tue, 2 Feb 2021 at 22:14, Joerg Roedel wrote: > > On Tue, Feb 02, 2021 at 06:42:57PM +0800, Chunyan Zhang wrote: > > +static phys_addr_t sprd_iommu_iova_to_phys(struct iommu_domain *domain, > > +dma_addr_t iova) > > +{ > > + struct sprd_iommu_domain

Re: [PATCH 2/2] iommu: add Unisoc iommu basic driver

2021-02-02 Thread Joerg Roedel
On Tue, Feb 02, 2021 at 06:42:57PM +0800, Chunyan Zhang wrote: > From: Chunyan Zhang > > This iommu module can be used by Unisoc's multimedia devices, such as > display, Image codec(jpeg) and a few signal processors, including > VSP(video), GSP(graphic), ISP(image), and CPP(camera pixel

Re: [PATCH 2/2] iommu: add Unisoc iommu basic driver

2021-02-02 Thread Robin Murphy
On 2021-02-02 14:41, Joerg Roedel wrote: On Tue, Feb 02, 2021 at 02:34:34PM +, Robin Murphy wrote: Nope, I believe if Arm Ltd. had any involvement in this I'd know about it :) Okay, got confused by thinking of ARM as the CPU architecture, not the company :) But given the intel/ and amd/

Re: [PATCH 2/2] iommu: add Unisoc iommu basic driver

2021-02-02 Thread Joerg Roedel
On Tue, Feb 02, 2021 at 06:42:57PM +0800, Chunyan Zhang wrote: > +static phys_addr_t sprd_iommu_iova_to_phys(struct iommu_domain *domain, > +dma_addr_t iova) > +{ > + struct sprd_iommu_domain *dom = to_sprd_domain(domain); > + unsigned long flags; >

Re: [PATCH 2/2] iommu: add Unisoc iommu basic driver

2021-02-02 Thread Robin Murphy
On 2021-02-02 14:01, Joerg Roedel wrote: On Tue, Feb 02, 2021 at 06:42:57PM +0800, Chunyan Zhang wrote: From: Chunyan Zhang This iommu module can be used by Unisoc's multimedia devices, such as display, Image codec(jpeg) and a few signal processors, including VSP(video), GSP(graphic),

Re: [PATCH 2/2] iommu: add Unisoc iommu basic driver

2021-02-02 Thread Joerg Roedel
On Tue, Feb 02, 2021 at 02:34:34PM +, Robin Murphy wrote: > Nope, I believe if Arm Ltd. had any involvement in this I'd know about it :) Okay, got confused by thinking of ARM as the CPU architecture, not the company :) But given the intel/ and amd/ subdirectories refer to company names as

[PATCH 2/2] iommu: add Unisoc iommu basic driver

2021-02-02 Thread Chunyan Zhang
From: Chunyan Zhang This iommu module can be used by Unisoc's multimedia devices, such as display, Image codec(jpeg) and a few signal processors, including VSP(video), GSP(graphic), ISP(image), and CPP(camera pixel processor), etc. Signed-off-by: Chunyan Zhang --- drivers/iommu/Kconfig |

[RFC PATCH 2/2] iommu: add Unisoc iommu basic driver

2020-12-23 Thread Chunyan Zhang
From: Chunyan Zhang This patch only adds display iommu support, the driver was tested with sprd dpu. The iommu support for others would be added once finished tests with those devices, such as Image codec(jpeg) processor, a few signal processors, including VSP(video), GSP(graphic), ISP(image),