Re: [PATCH 4/9] iommu: mtk_iommu: make it explicitly non-modular

2018-11-26 Thread Honghui Zhang
nchanged with this commit. > > We also delete the MODULE_LICENSE tag etc. since all that information > was (or is now) contained at the top of the file in the comments. > > Cc: Joerg Roedel > Cc: Matthias Brugger > Cc: Honghui Zhang > Cc: iommu@lists.linux-foundation.org &g

Re: [PATCH v2 0/2] PCI: mediatek: Fixups for the IRQ handle routine and MT7622's class code

2017-12-20 Thread Honghui Zhang
On Thu, 2017-12-21 at 10:08 +0800, honghui.zh...@mediatek.com wrote: > From: Honghui Zhang <honghui.zh...@mediatek.com> > > Two fixups for mediatek's host bridge: > The first patch fixup the IRQ handle routine to avoid IRQ reentry which > may exist for both MT2712 and MT762

Re: [PATCH v2] memory: mtk-smi: Use of_device_get_match_data helper

2017-08-01 Thread Honghui Zhang
On Wed, 2017-07-26 at 20:37 +0800, honghui.zh...@mediatek.com wrote: > From: Honghui Zhang <honghui.zh...@mediatek.com> > > Replace custom code with generic helper to retrieve driver data. Hi, Joerg, would you please take this one, because my last patch[1]([patch v2 0/3] Add larb

Re: [PATCH 1/3] memory: mtk-smi: add larbid init routine

2017-07-28 Thread Honghui Zhang
On Fri, 2017-07-28 at 10:59 +0100, Robin Murphy wrote: > On 28/07/17 02:15, Honghui Zhang wrote: > > On Thu, 2017-07-27 at 17:42 +0200, Matthias Brugger wrote: > >> > >> On 07/27/2017 04:01 AM, honghui.zh...@mediatek.com wrote: > >>> From: Ho

Re: [PATCH 1/3] memory: mtk-smi: add larbid init routine

2017-07-27 Thread Honghui Zhang
On Thu, 2017-07-27 at 17:42 +0200, Matthias Brugger wrote: > > On 07/27/2017 04:01 AM, honghui.zh...@mediatek.com wrote: > > From: Honghui Zhang <honghui.zh...@mediatek.com> > > > > In the commit 3c8f4ad85c4b ("memory/mediatek: add support for mt270

Re: [PATCH 0/3] Add larbid init routine for mediatek's gen1 smi larb driver

2017-07-27 Thread Honghui Zhang
On Thu, 2017-07-27 at 14:58 +0200, Joerg Roedel wrote: > On Thu, Jul 27, 2017 at 10:01:09AM +0800, honghui.zh...@mediatek.com wrote: > > From: Honghui Zhang <honghui.zh...@mediatek.com> > > > > Mediatek's gen1 smi need the hardware larbid to identify the offset for >

Re: [PATCH v4 5/5] ARM: dts: mt2701: add iommu/smi dtsi node for mt2701

2017-01-15 Thread Honghui Zhang
On Fri, 2017-01-13 at 15:54 +0100, Matthias Brugger wrote: > > On 04/07/16 10:00, Matthias Brugger wrote: > > > > > > On 04/07/16 03:32, Honghui Zhang wrote: > >> On Sun, 2016-07-03 at 21:12 +0200, Matthias Brugger wrote: > >>> > &g

Re: [PATCH] iommu: mtk: add common-clk dependency

2016-11-17 Thread Honghui Zhang
On Thu, 2016-11-17 at 15:35 -0800, Stephen Boyd wrote: > On 11/17, Honghui Zhang wrote: > > On Wed, 2016-11-16 at 11:38 -0800, Stephen Boyd wrote: > > > On 11/16, Arnd Bergmann wrote: > > > > After the MT2701 clock driver was added, we get a harmless warnin

Re: [PATCH] iommu: mtk: add common-clk dependency

2016-11-16 Thread Honghui Zhang
On Wed, 2016-11-16 at 11:38 -0800, Stephen Boyd wrote: > On 11/16, Arnd Bergmann wrote: > > After the MT2701 clock driver was added, we get a harmless warning for > > the iommu driver that selects it, when compile-testing without > > COMMON_CLK. > > > > warning: (MTK_IOMMU_V1) selects

Re: [PATCH 2/2] iommu/mediatek: Convert M4Uv1 to iommu_fwspec

2016-10-19 Thread Honghui Zhang
e can't rely on the of_xlate() mechanism to initialise things until the > 32-bit ARM DMA code learns about groups and default domains, it still > results in a reasonable simplification overall. > > CC: Honghui Zhang <honghui.zh...@mediatek.com> > Signed-off-by: Robin Murphy <robin.m

Re: [PATCH v4 5/5] ARM: dts: mt2701: add iommu/smi dtsi node for mt2701

2016-07-03 Thread Honghui Zhang
On Sun, 2016-07-03 at 21:12 +0200, Matthias Brugger wrote: > > On 07/03/2016 08:24 AM, Matthias Brugger wrote: > > > > > > On 06/08/2016 11:51 AM, honghui.zh...@mediatek.com wrote: > >> From: Honghui Zhang <honghui.zh...@mediatek.com> > >> &g

Re: [PATCH v4 5/5] ARM: dts: mt2701: add iommu/smi dtsi node for mt2701

2016-06-22 Thread Honghui Zhang
On Wed, 2016-06-22 at 13:45 +0800, Eddie Huang wrote: > On Tue, 2016-06-21 at 17:57 +0800, Joerg Roedel wrote: > > On Wed, Jun 08, 2016 at 05:51:01PM +0800, honghui.zh...@mediatek.com wrote: > > > From: Honghui Zhang <honghui.zh...@mediatek.com> > > > > &g

Re: [PATCH v4 5/5] ARM: dts: mt2701: add iommu/smi dtsi node for mt2701

2016-06-21 Thread Honghui Zhang
On Tue, 2016-06-21 at 11:41 +0200, Joerg Roedel wrote: > On Wed, Jun 08, 2016 at 05:51:01PM +0800, honghui.zh...@mediatek.com wrote: > > From: Honghui Zhang <honghui.zh...@mediatek.com> > > > > Add the dtsi node of iommu and smi for mt2701. > > > > S

Re: [PATCH v3 0/5] MT2701 iommu support

2016-06-05 Thread Honghui Zhang
On Fri, 2016-05-27 at 16:56 +0800, honghui.zh...@mediatek.com wrote: > From: Honghui Zhang <honghui.zh...@mediatek.com> > Is there will be some comments for this version patchset? Any comments is welcome. Thanks. > Mediatek's m4u(Multimedia Memory Management Unit) and SMI(Sma

Re: [PATCH v2 4/5] iommu/mediatek: add support for mtk iommu generation one HW

2016-05-25 Thread Honghui Zhang
On Tue, 2016-05-24 at 16:36 +0100, Robin Murphy wrote: > On 24/05/16 10:57, Honghui Zhang wrote: > [...] > >>> @@ -48,6 +48,9 @@ struct mtk_iommu_domain { > >>> struct io_pgtable_ops *iop; > >>> > >>>

Re: [PATCH v2 4/5] iommu/mediatek: add support for mtk iommu generation one HW

2016-05-24 Thread Honghui Zhang
Hi, Robin, Thanks very much for your comments. On Mon, 2016-05-23 at 20:31 +0100, Robin Murphy wrote: > On 19/05/16 12:49, honghui.zh...@mediatek.com wrote: > > From: Honghui Zhang <honghui.zh...@mediatek.com> > > > > Mediatek SoC's M4U has two generations of HW

Re: [PATCH 4/5] iommu/mediatek: add support for mtk iommu generation one HW

2016-05-12 Thread Honghui Zhang
Hi, Robin, Thanks very much for your comment, and sorry for the last reply format. On Tue, 2016-05-10 at 11:28 +0100, Robin Murphy wrote: > On 09/05/16 09:00, honghui.zh...@mediatek.com wrote: > [...] > > +static void *mtk_iommu_alloc_pgt(struct device *dev, size_t size, gfp_t > > gfp) > > +{ >