* [PATCH] iommu/exynos: remove dead code (set_prefbuf)
@ 2013-07-02 16:08 Grant Grundler
2013-08-05 16:39 ` Grant Grundler
2013-08-14 9:30 ` Joerg Roedel
0 siblings, 2 replies; 4+ messages in thread
From: Grant Grundler @ 2013-07-02 16:08 UTC (permalink / raw)
To: Joerg Roedel, Sachin Kamat
Cc: Jiri Kosina, Wei Yongjun, Kukjin Kim, linux-kernel,
Grant Grundler
exynos_sysmmu_set_prefbuf() is not called any where.
Signed-off-by: Grant Grundler <grundler@chromium.org>
---
drivers/iommu/exynos-iommu.c | 44 --------------------------------------------
1 file changed, 44 deletions(-)
diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c
index 3f32d64..0740189 100644
--- a/drivers/iommu/exynos-iommu.c
+++ b/drivers/iommu/exynos-iommu.c
@@ -247,50 +247,6 @@ static void __sysmmu_set_prefbuf(void __iomem *sfrbase, unsigned long base,
__raw_writel(size - 1 + base, sfrbase + REG_PB0_EADDR + idx * 8);
}
-void exynos_sysmmu_set_prefbuf(struct device *dev,
- unsigned long base0, unsigned long size0,
- unsigned long base1, unsigned long size1)
-{
- struct sysmmu_drvdata *data = dev_get_drvdata(dev->archdata.iommu);
- unsigned long flags;
- int i;
-
- BUG_ON((base0 + size0) <= base0);
- BUG_ON((size1 > 0) && ((base1 + size1) <= base1));
-
- read_lock_irqsave(&data->lock, flags);
- if (!is_sysmmu_active(data))
- goto finish;
-
- for (i = 0; i < data->nsfrs; i++) {
- if ((readl(data->sfrbases[i] + REG_MMU_VERSION) >> 28) == 3) {
- if (!sysmmu_block(data->sfrbases[i]))
- continue;
-
- if (size1 == 0) {
- if (size0 <= SZ_128K) {
- base1 = base0;
- size1 = size0;
- } else {
- size1 = size0 -
- ALIGN(size0 / 2, SZ_64K);
- size0 = size0 - size1;
- base1 = base0 + size0;
- }
- }
-
- __sysmmu_set_prefbuf(
- data->sfrbases[i], base0, size0, 0);
- __sysmmu_set_prefbuf(
- data->sfrbases[i], base1, size1, 1);
-
- sysmmu_unblock(data->sfrbases[i]);
- }
- }
-finish:
- read_unlock_irqrestore(&data->lock, flags);
-}
-
static void __set_fault_handler(struct sysmmu_drvdata *data,
sysmmu_fault_handler_t handler)
{
--
1.8.3
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] iommu/exynos: remove dead code (set_prefbuf)
2013-07-02 16:08 [PATCH] iommu/exynos: remove dead code (set_prefbuf) Grant Grundler
@ 2013-08-05 16:39 ` Grant Grundler
2013-08-06 9:47 ` Cho KyongHo
2013-08-14 9:30 ` Joerg Roedel
1 sibling, 1 reply; 4+ messages in thread
From: Grant Grundler @ 2013-08-05 16:39 UTC (permalink / raw)
To: Linux IOMMU, KyongHo Cho
Cc: Joerg Roedel, Sachin Kamat, Jiri Kosina, Wei Yongjun, Kukjin Kim,
LKML
Ping?
Adding linux-iommu ML.
thanks,
grant
On Tue, Jul 2, 2013 at 9:08 AM, Grant Grundler <grundler@chromium.org> wrote:
> exynos_sysmmu_set_prefbuf() is not called any where.
>
> Signed-off-by: Grant Grundler <grundler@chromium.org>
> ---
> drivers/iommu/exynos-iommu.c | 44 --------------------------------------------
> 1 file changed, 44 deletions(-)
>
> diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c
> index 3f32d64..0740189 100644
> --- a/drivers/iommu/exynos-iommu.c
> +++ b/drivers/iommu/exynos-iommu.c
> @@ -247,50 +247,6 @@ static void __sysmmu_set_prefbuf(void __iomem *sfrbase, unsigned long base,
> __raw_writel(size - 1 + base, sfrbase + REG_PB0_EADDR + idx * 8);
> }
>
> -void exynos_sysmmu_set_prefbuf(struct device *dev,
> - unsigned long base0, unsigned long size0,
> - unsigned long base1, unsigned long size1)
> -{
> - struct sysmmu_drvdata *data = dev_get_drvdata(dev->archdata.iommu);
> - unsigned long flags;
> - int i;
> -
> - BUG_ON((base0 + size0) <= base0);
> - BUG_ON((size1 > 0) && ((base1 + size1) <= base1));
> -
> - read_lock_irqsave(&data->lock, flags);
> - if (!is_sysmmu_active(data))
> - goto finish;
> -
> - for (i = 0; i < data->nsfrs; i++) {
> - if ((readl(data->sfrbases[i] + REG_MMU_VERSION) >> 28) == 3) {
> - if (!sysmmu_block(data->sfrbases[i]))
> - continue;
> -
> - if (size1 == 0) {
> - if (size0 <= SZ_128K) {
> - base1 = base0;
> - size1 = size0;
> - } else {
> - size1 = size0 -
> - ALIGN(size0 / 2, SZ_64K);
> - size0 = size0 - size1;
> - base1 = base0 + size0;
> - }
> - }
> -
> - __sysmmu_set_prefbuf(
> - data->sfrbases[i], base0, size0, 0);
> - __sysmmu_set_prefbuf(
> - data->sfrbases[i], base1, size1, 1);
> -
> - sysmmu_unblock(data->sfrbases[i]);
> - }
> - }
> -finish:
> - read_unlock_irqrestore(&data->lock, flags);
> -}
> -
> static void __set_fault_handler(struct sysmmu_drvdata *data,
> sysmmu_fault_handler_t handler)
> {
> --
> 1.8.3
>
^ permalink raw reply [flat|nested] 4+ messages in thread* RE: [PATCH] iommu/exynos: remove dead code (set_prefbuf)
2013-08-05 16:39 ` Grant Grundler
@ 2013-08-06 9:47 ` Cho KyongHo
0 siblings, 0 replies; 4+ messages in thread
From: Cho KyongHo @ 2013-08-06 9:47 UTC (permalink / raw)
To: 'Grant Grundler', 'Linux IOMMU'
Cc: 'Joerg Roedel', 'Sachin Kamat',
'Jiri Kosina', 'Wei Yongjun',
'Kukjin Kim', 'LKML'
> -----Original Message-----
> From: grundler@google.com [mailto:grundler@google.com] On Behalf Of Grant Grundler
> Sent: Tuesday, August 06, 2013 1:40 AM
>
> Ping?
>
> Adding linux-iommu ML.
>
> thanks,
> grant
>
> On Tue, Jul 2, 2013 at 9:08 AM, Grant Grundler <grundler@chromium.org> wrote:
> > exynos_sysmmu_set_prefbuf() is not called any where.
> >
> > Signed-off-by: Grant Grundler <grundler@chromium.org>
Reviewed-by: Cho KyongHo <pullip.cho@samsung.com>
Thank you.
KyongHo.
> > ---
> > drivers/iommu/exynos-iommu.c | 44 --------------------------------------------
> > 1 file changed, 44 deletions(-)
> >
> > diff --git a/drivers/iommu/exynos-iommu.c b/drivers/iommu/exynos-iommu.c
> > index 3f32d64..0740189 100644
> > --- a/drivers/iommu/exynos-iommu.c
> > +++ b/drivers/iommu/exynos-iommu.c
> > @@ -247,50 +247,6 @@ static void __sysmmu_set_prefbuf(void __iomem *sfrbase, unsigned long base,
> > __raw_writel(size - 1 + base, sfrbase + REG_PB0_EADDR + idx * 8);
> > }
> >
> > -void exynos_sysmmu_set_prefbuf(struct device *dev,
> > - unsigned long base0, unsigned long size0,
> > - unsigned long base1, unsigned long size1)
> > -{
> > - struct sysmmu_drvdata *data = dev_get_drvdata(dev->archdata.iommu);
> > - unsigned long flags;
> > - int i;
> > -
> > - BUG_ON((base0 + size0) <= base0);
> > - BUG_ON((size1 > 0) && ((base1 + size1) <= base1));
> > -
> > - read_lock_irqsave(&data->lock, flags);
> > - if (!is_sysmmu_active(data))
> > - goto finish;
> > -
> > - for (i = 0; i < data->nsfrs; i++) {
> > - if ((readl(data->sfrbases[i] + REG_MMU_VERSION) >> 28) == 3) {
> > - if (!sysmmu_block(data->sfrbases[i]))
> > - continue;
> > -
> > - if (size1 == 0) {
> > - if (size0 <= SZ_128K) {
> > - base1 = base0;
> > - size1 = size0;
> > - } else {
> > - size1 = size0 -
> > - ALIGN(size0 / 2, SZ_64K);
> > - size0 = size0 - size1;
> > - base1 = base0 + size0;
> > - }
> > - }
> > -
> > - __sysmmu_set_prefbuf(
> > - data->sfrbases[i], base0, size0, 0);
> > - __sysmmu_set_prefbuf(
> > - data->sfrbases[i], base1, size1, 1);
> > -
> > - sysmmu_unblock(data->sfrbases[i]);
> > - }
> > - }
> > -finish:
> > - read_unlock_irqrestore(&data->lock, flags);
> > -}
> > -
> > static void __set_fault_handler(struct sysmmu_drvdata *data,
> > sysmmu_fault_handler_t handler)
> > {
> > --
> > 1.8.3
> >
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] iommu/exynos: remove dead code (set_prefbuf)
2013-07-02 16:08 [PATCH] iommu/exynos: remove dead code (set_prefbuf) Grant Grundler
2013-08-05 16:39 ` Grant Grundler
@ 2013-08-14 9:30 ` Joerg Roedel
1 sibling, 0 replies; 4+ messages in thread
From: Joerg Roedel @ 2013-08-14 9:30 UTC (permalink / raw)
To: Grant Grundler
Cc: Sachin Kamat, Jiri Kosina, Wei Yongjun, Kukjin Kim, linux-kernel
On Tue, Jul 02, 2013 at 09:08:27AM -0700, Grant Grundler wrote:
> exynos_sysmmu_set_prefbuf() is not called any where.
>
> Signed-off-by: Grant Grundler <grundler@chromium.org>
> ---
> drivers/iommu/exynos-iommu.c | 44 --------------------------------------------
> 1 file changed, 44 deletions(-)
Applied to arm/exynos, thanks.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2013-08-14 9:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-02 16:08 [PATCH] iommu/exynos: remove dead code (set_prefbuf) Grant Grundler
2013-08-05 16:39 ` Grant Grundler
2013-08-06 9:47 ` Cho KyongHo
2013-08-14 9:30 ` Joerg Roedel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).