On Wednesday 24 October 2012, Gregory CLEMENT wrote:
> > No, I was thinking of making the underlying functions globally visible
> > and have extern declarations in a header file so you can access them
> > directly.
> >
> > Generally speaking, when you run into a problem with common code, your
> >
Dear Gregory CLEMENT,
On Wed, 24 Oct 2012 16:39:13 +0200, Gregory CLEMENT wrote:
> > As others said, the prefix is wrong. Since the file is named coherency,
> > maybe just "coherency_" as the prefix? Not sure, though. Shouldn't the
> > file be named coherency-armada-370-xp.c, as we have done for
On 10/24/2012 02:27 PM, Thomas Petazzoni wrote:
[...]
I will fixed the spelling and complete the comments as suggested
[...]
>> +struct dma_map_ops armada_xp_dma_ops;
>
> static
OK
>
>> +static inline void armada_xp_sync_io_barrier(void)
>> +{
>> +writel(0x1, coherency_cpu_base +
On 10/24/2012 02:24 PM, Arnd Bergmann wrote:
> On Wednesday 24 October 2012, Gregory CLEMENT wrote:
>> On 10/24/2012 01:48 PM, Gregory CLEMENT wrote:
>>> On 10/24/2012 01:36 PM, Arnd Bergmann wrote:
I think it would be cleaner to statically define the operations in a
constant
Hello,
On Wed, 24 Oct 2012 10:04:01 +0200, Gregory CLEMENT wrote:
> Armada 370 and XP come with an unit called coherency fabric. This unit
> allows to use the Armada XP as a nearly coherent architecture. The
the Armada 370/XP
> coherency mechanism uses snoop filters to ensure the coherency
On Wednesday 24 October 2012, Gregory CLEMENT wrote:
> On 10/24/2012 01:48 PM, Gregory CLEMENT wrote:
> > On 10/24/2012 01:36 PM, Arnd Bergmann wrote:
> >>
> >> I think it would be cleaner to statically define the operations in a
> >> constant
> >> structure and point directly to the functions
On 10/24/2012 01:48 PM, Gregory CLEMENT wrote:
> On 10/24/2012 01:36 PM, Arnd Bergmann wrote:
>> On Wednesday 24 October 2012, Gregory CLEMENT wrote:
>>> +void __init armada_370_xp_coherency_iocache_init(void)
>>> +{
>>> + /* When the coherency fabric is available, the Armada XP and
>>> +
On 10/24/2012 10:25 AM, Andrew Lunn wrote:
> On Wed, Oct 24, 2012 at 10:04:01AM +0200, Gregory CLEMENT wrote:
>> Armada 370 and XP come with an unit called coherency fabric. This unit
>> allows to use the Armada XP as a nearly coherent architecture. The
>> coherency mechanism uses snoop filters to
On 10/24/2012 01:36 PM, Arnd Bergmann wrote:
> On Wednesday 24 October 2012, Gregory CLEMENT wrote:
>> +void __init armada_370_xp_coherency_iocache_init(void)
>> +{
>> + /* When the coherency fabric is available, the Armada XP and
>> +* Aramada 370 are close to a coherent
On Wednesday 24 October 2012, Gregory CLEMENT wrote:
> +void __init armada_370_xp_coherency_iocache_init(void)
> +{
> + /* When the coherency fabric is available, the Armada XP and
> +* Aramada 370 are close to a coherent architecture, so we based
> +* our dma ops on the
> -Original Message-
> From: Gregory CLEMENT [mailto:gregory.clem...@free-electrons.com]
> Sent: Wednesday, October 24, 2012 10:04 AM
> To: Jason Cooper; Andrew Lunn; Gregory Clement
> Cc: linux-arm-ker...@lists.infradead.org; Arnd Bergmann; Olof Johansson;
> Russell King; Rob Herring;
-Original Message-
From: Gregory CLEMENT [mailto:gregory.clem...@free-electrons.com]
Sent: Wednesday, October 24, 2012 10:04 AM
To: Jason Cooper; Andrew Lunn; Gregory Clement
Cc: linux-arm-ker...@lists.infradead.org; Arnd Bergmann; Olof Johansson;
Russell King; Rob Herring; Ben
On Wednesday 24 October 2012, Gregory CLEMENT wrote:
+void __init armada_370_xp_coherency_iocache_init(void)
+{
+ /* When the coherency fabric is available, the Armada XP and
+* Aramada 370 are close to a coherent architecture, so we based
+* our dma ops on the coherent
On 10/24/2012 01:36 PM, Arnd Bergmann wrote:
On Wednesday 24 October 2012, Gregory CLEMENT wrote:
+void __init armada_370_xp_coherency_iocache_init(void)
+{
+ /* When the coherency fabric is available, the Armada XP and
+* Aramada 370 are close to a coherent architecture, so we
On 10/24/2012 10:25 AM, Andrew Lunn wrote:
On Wed, Oct 24, 2012 at 10:04:01AM +0200, Gregory CLEMENT wrote:
Armada 370 and XP come with an unit called coherency fabric. This unit
allows to use the Armada XP as a nearly coherent architecture. The
coherency mechanism uses snoop filters to ensure
On 10/24/2012 01:48 PM, Gregory CLEMENT wrote:
On 10/24/2012 01:36 PM, Arnd Bergmann wrote:
On Wednesday 24 October 2012, Gregory CLEMENT wrote:
+void __init armada_370_xp_coherency_iocache_init(void)
+{
+ /* When the coherency fabric is available, the Armada XP and
+* Aramada
On Wednesday 24 October 2012, Gregory CLEMENT wrote:
On 10/24/2012 01:48 PM, Gregory CLEMENT wrote:
On 10/24/2012 01:36 PM, Arnd Bergmann wrote:
I think it would be cleaner to statically define the operations in a
constant
structure and point directly to the functions you need. If
Hello,
On Wed, 24 Oct 2012 10:04:01 +0200, Gregory CLEMENT wrote:
Armada 370 and XP come with an unit called coherency fabric. This unit
allows to use the Armada XP as a nearly coherent architecture. The
the Armada 370/XP
coherency mechanism uses snoop filters to ensure the coherency between
On 10/24/2012 02:24 PM, Arnd Bergmann wrote:
On Wednesday 24 October 2012, Gregory CLEMENT wrote:
On 10/24/2012 01:48 PM, Gregory CLEMENT wrote:
On 10/24/2012 01:36 PM, Arnd Bergmann wrote:
I think it would be cleaner to statically define the operations in a
constant
structure and point
On 10/24/2012 02:27 PM, Thomas Petazzoni wrote:
[...]
I will fixed the spelling and complete the comments as suggested
[...]
+struct dma_map_ops armada_xp_dma_ops;
static
OK
+static inline void armada_xp_sync_io_barrier(void)
+{
+writel(0x1, coherency_cpu_base +
Dear Gregory CLEMENT,
On Wed, 24 Oct 2012 16:39:13 +0200, Gregory CLEMENT wrote:
As others said, the prefix is wrong. Since the file is named coherency,
maybe just coherency_ as the prefix? Not sure, though. Shouldn't the
file be named coherency-armada-370-xp.c, as we have done for the irq
On Wednesday 24 October 2012, Gregory CLEMENT wrote:
No, I was thinking of making the underlying functions globally visible
and have extern declarations in a header file so you can access them
directly.
Generally speaking, when you run into a problem with common code, your
first
22 matches
Mail list logo