lkml.org 
[lkml]   [2017]   [Apr]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 3/3] bus: fsl-mc: dpio: change CENA regs to be cacheable
On Thu, Apr 20, 2017 at 03:34:18PM -0400, Haiying Wang wrote:
> plus non-shareable to meet the performance requirement.
> QMan's CENA region contains registers and structures that
> are 64byte in size and are inteneded to be accessed using a
> single 64 byte bus transaction, therefore this portal
> memory should be configured as cache-enabled. Also because
> the write allocate stash transcations of QBMan should be
> issued as cachable and non-coherent(non-sharable), we
> need to configure this region to be non-shareable.
>
> Signed-off-by: Haiying Wang <Haiying.Wang@nxp.com>
> ---
> drivers/staging/fsl-mc/bus/dpio/dpio-driver.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/staging/fsl-mc/bus/dpio/dpio-driver.c b/drivers/staging/fsl-mc/bus/dpio/dpio-driver.c
> index e36da20..97f909c 100644
> --- a/drivers/staging/fsl-mc/bus/dpio/dpio-driver.c
> +++ b/drivers/staging/fsl-mc/bus/dpio/dpio-driver.c
> @@ -168,10 +168,10 @@ static int dpaa2_dpio_probe(struct fsl_mc_device *dpio_dev)
> desc.cpu = next_cpu;
>
> /*
> - * Set the CENA regs to be the cache inhibited area of the portal to
> - * avoid coherency issues if a user migrates to another core.
> + * Set the CENA regs to be the cache enalbed area of the portal to
> + * archieve the best performance.

Is migrating to another core no longer a concern?

As with my prior comments regarding the Non-Shareable mapping, I do not
thing this makes sense.

Thanks,
Mark.

> */
> - desc.regs_cena = ioremap_wc(dpio_dev->regions[1].start,
> + desc.regs_cena = ioremap_cache_ns(dpio_dev->regions[1].start,
> resource_size(&dpio_dev->regions[1]));
> desc.regs_cinh = ioremap(dpio_dev->regions[1].start,
> resource_size(&dpio_dev->regions[1]));
> --
> 2.7.4
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

\
 
 \ /
  Last update: 2017-04-21 11:29    [W:0.405 / U:0.000 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site