Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
Hi Joerg, Am Dienstag, den 16.02.2016, 17:26 +0100 schrieb Joerg Roedel: > On Tue, Feb 16, 2016 at 04:20:00PM +, Will Deacon wrote: > > I'm more than happy to bake a branch containing all the page table stuff, > > but Yong's stuff depends on it so I'll work with whatever is easiest for > > you. > > Okay, to the best is if you would collect all the page table patches and > send me a pull-request. I pull that into my tree then and put Yong's > stuff on-top. could you provide a branch that I may then pull in, stack the mediatek-drm driver patches (that depend on the SMI driver) on top and have them merged via drm-next? regards Philipp ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
On Tue, 2016-02-16 at 15:11 +0100, Joerg Roedel wrote: > On Sun, Jan 31, 2016 at 12:00:41PM +0100, Matthias Brugger wrote: > > > > > > On 26/01/16 05:12, Yong Wu wrote: > > >This patch add SMI(Smart Multimedia Interface) driver. This driver > > >is responsible to enable/disable iommu and control the power domain > > >and clocks of each local arbiter. > > > > > >Signed-off-by: Yong Wu> > >Tested-by: Philipp Zabel > > >--- > > > > Signed-off-by: Matthias Brugger > > > > Joerg would you mind to take this through your branch? > > Hmm, I somehow missed the patch-set, at least I can't find the v8 > patches in my inbox. > > Yong, can you please re-send with all Acks and other tags added? Hi Joerg, I have re-sent v9[1] which has added all the tags yesterday. Could you find them in your inbox? If not again, please tell me. Thanks. [1]:http://lists.linuxfoundation.org/pipermail/iommu/2016-February/015713.html > > > Thanks, > > Joerg > ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
On Tue, Feb 16, 2016 at 05:26:41PM +0100, Joerg Roedel wrote: > On Tue, Feb 16, 2016 at 04:20:00PM +, Will Deacon wrote: > > I'm more than happy to bake a branch containing all the page table stuff, > > but Yong's stuff depends on it so I'll work with whatever is easiest for > > you. > > Okay, to the best is if you would collect all the page table patches and > send me a pull-request. I pull that into my tree then and put Yong's > stuff on-top. Okey doke, I'll put something together this week. Cheers, Will ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
On Tue, Feb 16, 2016 at 04:20:00PM +, Will Deacon wrote: > I'm more than happy to bake a branch containing all the page table stuff, > but Yong's stuff depends on it so I'll work with whatever is easiest for > you. Okay, to the best is if you would collect all the page table patches and send me a pull-request. I pull that into my tree then and put Yong's stuff on-top. Joerg ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
On Tue, Feb 16, 2016 at 05:10:09PM +0100, Joerg Roedel wrote: > On Tue, Feb 16, 2016 at 03:32:11PM +, Robin Murphy wrote: > > On 16/02/16 14:11, Joerg Roedel wrote: > > >On Sun, Jan 31, 2016 at 12:00:41PM +0100, Matthias Brugger wrote: > > >>On 26/01/16 05:12, Yong Wu wrote: > > >>>This patch add SMI(Smart Multimedia Interface) driver. This driver > > >>>is responsible to enable/disable iommu and control the power domain > > >>>and clocks of each local arbiter. > > >>> > > >>>Signed-off-by: Yong Wu> > >>>Tested-by: Philipp Zabel > > >>>--- > > >> > > >>Signed-off-by: Matthias Brugger > > >> > > >>Joerg would you mind to take this through your branch? > > > > > >Hmm, I somehow missed the patch-set, at least I can't find the v8 > > >patches in my inbox. > > > > > >Yong, can you please re-send with all Acks and other tags added? > > > > Just to note patch 4 of this series also depends on the v3 > > io-pgtable short-descriptor series posted separately[1] - is that on > > your radar already or would you like that resent as well? > > No, that was not on my radar, I somehow expected Will to handle this > series :) I'm more than happy to bake a branch containing all the page table stuff, but Yong's stuff depends on it so I'll work with whatever is easiest for you. Will ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
On Tue, Feb 16, 2016 at 03:32:11PM +, Robin Murphy wrote: > Hi Joerg, > > On 16/02/16 14:11, Joerg Roedel wrote: > >On Sun, Jan 31, 2016 at 12:00:41PM +0100, Matthias Brugger wrote: > >> > >> > >>On 26/01/16 05:12, Yong Wu wrote: > >>>This patch add SMI(Smart Multimedia Interface) driver. This driver > >>>is responsible to enable/disable iommu and control the power domain > >>>and clocks of each local arbiter. > >>> > >>>Signed-off-by: Yong Wu> >>>Tested-by: Philipp Zabel > >>>--- > >> > >>Signed-off-by: Matthias Brugger > >> > >>Joerg would you mind to take this through your branch? > > > >Hmm, I somehow missed the patch-set, at least I can't find the v8 > >patches in my inbox. > > > >Yong, can you please re-send with all Acks and other tags added? > > Just to note patch 4 of this series also depends on the v3 > io-pgtable short-descriptor series posted separately[1] - is that on > your radar already or would you like that resent as well? No, that was not on my radar, I somehow expected Will to handle this series :) So please resend that as well. Thanks, Joerg ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
Hi Joerg, On 16/02/16 14:11, Joerg Roedel wrote: On Sun, Jan 31, 2016 at 12:00:41PM +0100, Matthias Brugger wrote: On 26/01/16 05:12, Yong Wu wrote: This patch add SMI(Smart Multimedia Interface) driver. This driver is responsible to enable/disable iommu and control the power domain and clocks of each local arbiter. Signed-off-by: Yong WuTested-by: Philipp Zabel --- Signed-off-by: Matthias Brugger Joerg would you mind to take this through your branch? Hmm, I somehow missed the patch-set, at least I can't find the v8 patches in my inbox. Yong, can you please re-send with all Acks and other tags added? Just to note patch 4 of this series also depends on the v3 io-pgtable short-descriptor series posted separately[1] - is that on your radar already or would you like that resent as well? Robin. [1]:http://thread.gmane.org/gmane.linux.kernel.iommu/12007 Thanks, Joerg ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
On Sun, Jan 31, 2016 at 12:00:41PM +0100, Matthias Brugger wrote: > > > On 26/01/16 05:12, Yong Wu wrote: > >This patch add SMI(Smart Multimedia Interface) driver. This driver > >is responsible to enable/disable iommu and control the power domain > >and clocks of each local arbiter. > > > >Signed-off-by: Yong Wu> >Tested-by: Philipp Zabel > >--- > > Signed-off-by: Matthias Brugger > > Joerg would you mind to take this through your branch? Could you put it on a separate branch that I may also merge into the DRM driver pull request? regards Philipp ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
On 26/01/16 05:12, Yong Wu wrote: This patch add SMI(Smart Multimedia Interface) driver. This driver is responsible to enable/disable iommu and control the power domain and clocks of each local arbiter. Signed-off-by: Yong WuTested-by: Philipp Zabel --- Signed-off-by: Matthias Brugger Joerg would you mind to take this through your branch? Thanks, Matthias ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH v8 3/5] memory: mediatek: Add SMI driver
On Tue, Jan 26, 2016 at 12:12 PM, Yong Wuwrote: > This patch add SMI(Smart Multimedia Interface) driver. This driver > is responsible to enable/disable iommu and control the power domain > and clocks of each local arbiter. > > Signed-off-by: Yong Wu > Tested-by: Philipp Zabel Reviewed-by: Daniel Kurtz Tested-by: Daniel Kurtz Thanks! ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
[PATCH v8 3/5] memory: mediatek: Add SMI driver
This patch add SMI(Smart Multimedia Interface) driver. This driver is responsible to enable/disable iommu and control the power domain and clocks of each local arbiter. Signed-off-by: Yong WuTested-by: Philipp Zabel --- drivers/memory/Kconfig | 8 ++ drivers/memory/Makefile| 1 + drivers/memory/mtk-smi.c | 272 + include/soc/mediatek/smi.h | 58 ++ 4 files changed, 339 insertions(+) create mode 100644 drivers/memory/mtk-smi.c create mode 100644 include/soc/mediatek/smi.h diff --git a/drivers/memory/Kconfig b/drivers/memory/Kconfig index 6f31546..51d5cd2 100644 --- a/drivers/memory/Kconfig +++ b/drivers/memory/Kconfig @@ -114,6 +114,14 @@ config JZ4780_NEMC the Ingenic JZ4780. This controller is used to handle external memory devices such as NAND and SRAM. +config MTK_SMI + bool + depends on ARCH_MEDIATEK || COMPILE_TEST + help + This driver is for the Memory Controller module in MediaTek SoCs, + mainly help enable/disable iommu and control the power domain and + clocks for each local arbiter. + source "drivers/memory/tegra/Kconfig" endif diff --git a/drivers/memory/Makefile b/drivers/memory/Makefile index 1c46af5..890bdf4 100644 --- a/drivers/memory/Makefile +++ b/drivers/memory/Makefile @@ -15,5 +15,6 @@ obj-$(CONFIG_FSL_IFC) += fsl_ifc.o obj-$(CONFIG_MVEBU_DEVBUS) += mvebu-devbus.o obj-$(CONFIG_TEGRA20_MC) += tegra20-mc.o obj-$(CONFIG_JZ4780_NEMC) += jz4780-nemc.o +obj-$(CONFIG_MTK_SMI) += mtk-smi.o obj-$(CONFIG_TEGRA_MC) += tegra/ diff --git a/drivers/memory/mtk-smi.c b/drivers/memory/mtk-smi.c new file mode 100644 index 000..702f0d2 --- /dev/null +++ b/drivers/memory/mtk-smi.c @@ -0,0 +1,272 @@ +/* + * Copyright (c) 2015-2016 MediaTek Inc. + * Author: Yong Wu + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + */ +#include +#include +#include +#include +#include +#include +#include +#include +#include + +#define SMI_LARB_MMU_EN0xf00 + +struct mtk_smi { + struct device *dev; + struct clk *clk_apb, *clk_smi; +}; + +struct mtk_smi_larb { /* larb: local arbiter */ + struct mtk_smi smi; + void __iomem*base; + struct device *smi_common_dev; + u32 *mmu; +}; + +static int mtk_smi_enable(const struct mtk_smi *smi) +{ + int ret; + + ret = pm_runtime_get_sync(smi->dev); + if (ret < 0) + return ret; + + ret = clk_prepare_enable(smi->clk_apb); + if (ret) + goto err_put_pm; + + ret = clk_prepare_enable(smi->clk_smi); + if (ret) + goto err_disable_apb; + + return 0; + +err_disable_apb: + clk_disable_unprepare(smi->clk_apb); +err_put_pm: + pm_runtime_put_sync(smi->dev); + return ret; +} + +static void mtk_smi_disable(const struct mtk_smi *smi) +{ + clk_disable_unprepare(smi->clk_smi); + clk_disable_unprepare(smi->clk_apb); + pm_runtime_put_sync(smi->dev); +} + +int mtk_smi_larb_get(struct device *larbdev) +{ + struct mtk_smi_larb *larb = dev_get_drvdata(larbdev); + struct mtk_smi *common = dev_get_drvdata(larb->smi_common_dev); + int ret; + + /* Enable the smi-common's power and clocks */ + ret = mtk_smi_enable(common); + if (ret) + return ret; + + /* Enable the larb's power and clocks */ + ret = mtk_smi_enable(>smi); + if (ret) { + mtk_smi_disable(common); + return ret; + } + + /* Configure the iommu info for this larb */ + writel(*larb->mmu, larb->base + SMI_LARB_MMU_EN); + + return 0; +} + +void mtk_smi_larb_put(struct device *larbdev) +{ + struct mtk_smi_larb *larb = dev_get_drvdata(larbdev); + struct mtk_smi *common = dev_get_drvdata(larb->smi_common_dev); + + /* +* Don't de-configure the iommu info for this larb since there may be +* several modules in this larb. +* The iommu info will be reset after power off. +*/ + + mtk_smi_disable(>smi); + mtk_smi_disable(common); +} + +static int +mtk_smi_larb_bind(struct device *dev, struct device *master, void *data) +{ + struct mtk_smi_larb *larb = dev_get_drvdata(dev); + struct mtk_smi_iommu *smi_iommu = data; + unsigned int i; + + for (i = 0; i < smi_iommu->larb_nr; i++) { + if (dev ==