On Sun, Sep 03, 2017 at 07:56:30AM +0300, Artturi Alm wrote:
> On Mon, Jul 03, 2017 at 06:14:06AM +0300, Artturi Alm wrote:
> > Hi,
> > 
> > just the bug fix, so this diff leaves the unused relics around and so,
> > but better than current/nothing, and hopefully small enough to get looked 
> > at.
> > 
> > -Artturi
> > 
> 
> ping?
> a4x bus_space is supposed to be identical to armv7 bus_space w/ offset << 2.
> 
> -Artturi
> 

pong?
just in case the minimal diff was lacking good enough description, i'll
try again, hoping the desire for md correctness will prevail even for an
arch like armv7, atleast for obvious(?) things like this.

for those who don't know what a4x bus_space is for:
/*
 * There are simple bus space functions for IO registers mapped at
 * 32-bit aligned positions.  offset is multiplied by 4.
 */

i guess it should read "These are ..", that was from armv7_a4x_io.S,
another more broken descr. of it can be found from armv7_a4x_space.c:
/*
 * Bus space tag for 8/16-bit devices on 32-bit bus.
 * all registers are located at the address of multiple of 4.
 */

== wrong, as it does support 32-bit reads and writes, but not in multiples.
anyway, it's purpose remains the same, to do offset << 2 before bus op.

So, there's nothing suggesting a4x shouldn't have the barriers added by
diff below, to really make it armv7_bus_space+off*4 i believe it was/is
meant to be, and nothing should use the generic/generic_armv4 bs.

current might work now, will anyone remember this if MULTIPROCESSOR would
ever appear, and have weird problems w/console? idk. i'll just leave this
here:)

-Artturi


diff --git a/sys/arch/arm/arm/bus_space_asm_generic.S 
b/sys/arch/arm/arm/bus_space_asm_generic.S
deleted file mode 100644
index 97b71a5cecc..00000000000
--- a/sys/arch/arm/arm/bus_space_asm_generic.S
+++ /dev/null
@@ -1,336 +0,0 @@
-/*     $OpenBSD: bus_space_asm_generic.S,v 1.5 2017/01/06 00:06:02 jsg Exp $   
*/
-/*     $NetBSD: bus_space_asm_generic.S,v 1.3 2003/03/27 19:46:14 mycroft Exp 
$        */
-
-/*
- * Copyright (c) 1997 Causality Limited.
- * Copyright (c) 1997 Mark Brinicombe.
- * All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- * 3. All advertising materials mentioning features or use of this software
- *    must display the following acknowledgement:
- *     This product includes software developed by Mark Brinicombe
- *     for the NetBSD Project.
- * 4. The name of the company nor the name of the author may be used to
- *    endorse or promote products derived from this software without specific
- *    prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR IMPLIED
- * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES OF
- * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
- * IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT,
- * INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
- * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
- * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- */
-
-#include <arm/asm.h>
-#include <arm/cpuconf.h>
-
-/*
- * Generic bus_space functions.
- */
-
-/*
- * read single
- */
-
-ENTRY(generic_bs_r_1)
-       ldrb    r0, [r1, r2]
-       mov     pc, lr
-
-ENTRY(generic_armv4_bs_r_2)
-       ldrh    r0, [r1, r2]
-       mov     pc, lr
-
-ENTRY(generic_bs_r_4)
-       ldr     r0, [r1, r2]
-       mov     pc, lr
-
-/*
- * write single
- */
-
-ENTRY(generic_bs_w_1)
-       strb    r3, [r1, r2]
-       mov     pc, lr
-
-ENTRY(generic_armv4_bs_w_2)
-       strh    r3, [r1, r2]
-       mov     pc, lr
-
-ENTRY(generic_bs_w_4)
-       str     r3, [r1, r2]
-       mov     pc, lr
-
-/*
- * read multiple
- */
-
-ENTRY(generic_bs_rm_1)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldrb    r3, [r0]
-       strb    r3, [r1], #1
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_armv4_bs_rm_2)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldrh    r3, [r0]
-       strh    r3, [r1], #2
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_bs_rm_4)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldr     r3, [r0]
-       str     r3, [r1], #4
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-/*
- * write multiple
- */
-
-ENTRY(generic_bs_wm_1)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldrb    r3, [r1], #1
-       strb    r3, [r0]
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_armv4_bs_wm_2)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldrh    r3, [r1], #2
-       strh    r3, [r0]
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_bs_wm_4)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldr     r3, [r1], #4
-       str     r3, [r0]
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-/*
- * read region
- */
-
-ENTRY(generic_bs_rr_1)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldrb    r3, [r0], #1
-       strb    r3, [r1], #1
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_armv4_bs_rr_2)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldrh    r3, [r0], #2
-       strh    r3, [r1], #2
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_bs_rr_4)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldr     r3, [r0], #4
-       str     r3, [r1], #4
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-/*
- * write region.
- */
-
-ENTRY(generic_bs_wr_1)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldrb    r3, [r1], #1
-       strb    r3, [r0], #1
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_armv4_bs_wr_2)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldrh    r3, [r1], #2
-       strh    r3, [r0], #2
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_bs_wr_4)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     ldr     r3, [r1], #4
-       str     r3, [r0], #4
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-/*
- * set region
- */
-
-ENTRY(generic_bs_sr_1)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     strb    r1, [r0], #1
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_armv4_bs_sr_2)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     strh    r1, [r0], #2
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-ENTRY(generic_bs_sr_4)
-       add     r0, r1, r2
-       mov     r1, r3
-       ldr     r2, [sp, #0]
-       teq     r2, #0
-       moveq   pc, lr
-
-1:     str     r1, [r0], #4
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-/*
- * copy region
- */
-
-ENTRY(generic_armv4_bs_c_2)
-       add     r0, r1, r2
-       ldr     r2, [sp, #0]
-       add     r1, r2, r3
-       ldr     r2, [sp, #4]
-       teq     r2, #0
-       moveq   pc, lr
-
-       cmp     r0, r1
-       blt     2f
-
-1:     ldrh    r3, [r0], #2
-       strh    r3, [r1], #2
-       subs    r2, r2, #1
-       bne     1b
-
-       mov     pc, lr
-
-2:     add     r0, r0, r2, lsl #1
-       add     r1, r1, r2, lsl #1
-       sub     r0, r0, #2
-       sub     r1, r1, #2
-
-3:     ldrh    r3, [r0], #-2
-       strh    r3, [r1], #-2
-       subs    r2, r2, #1
-       bne     3b
-
-       mov     pc, lr
diff --git a/sys/arch/arm/armv7/armv7_a4x_io.S 
b/sys/arch/arm/armv7/armv7_a4x_io.S
index 3fa09faa4c2..5e9db421853 100644
--- a/sys/arch/arm/armv7/armv7_a4x_io.S
+++ b/sys/arch/arm/armv7/armv7_a4x_io.S
@@ -50,14 +50,17 @@
  */
 
 ENTRY(a4x_bs_r_1)
+       dsb     sy
        ldr     r0, [r1, r2, LSL #2]
        mov     pc, lr
 
 ENTRY(a4x_bs_r_2)
+       dsb     sy
        ldr     r0, [r1, r2, LSL #2]
        mov     pc, lr
 
 ENTRY(a4x_bs_r_4)
+       dsb     sy
        ldr     r0, [r1, r2, LSL #2]
        mov     pc, lr
 
@@ -67,14 +70,17 @@ ENTRY(a4x_bs_r_4)
 
 ENTRY(a4x_bs_w_1)
        str     r3, [r1, r2, LSL #2]
+       dsb     sy
        mov     pc, lr
 
 ENTRY(a4x_bs_w_2)
        str     r3, [r1, r2, LSL #2]
+       dsb     sy
        mov     pc, lr
 
 ENTRY(a4x_bs_w_4)
        str     r3, [r1, r2, LSL #2]
+       dsb     sy
        mov     pc, lr
 
 /*
@@ -82,11 +88,11 @@ ENTRY(a4x_bs_w_4)
  */
 ENTRY(a4x_bs_rm_1)
        mov r2, r2, LSL #2
-       b generic_bs_rm_1
+       b armv7_bs_rm_1
        
 ENTRY(a4x_bs_rm_2)
        mov r2, r2, LSL #2
-       b generic_armv4_bs_rm_2
+       b armv7_bs_rm_2
 
 
 
@@ -95,8 +101,8 @@ ENTRY(a4x_bs_rm_2)
  */
 ENTRY(a4x_bs_wm_1)
        mov r2, r2, LSL #2
-       b generic_bs_wm_1
+       b armv7_bs_wm_1
        
 ENTRY(a4x_bs_wm_2)
        mov r2, r2, LSL #2
-       b generic_armv4_bs_wm_2
+       b armv7_bs_wm_2
diff --git a/sys/arch/arm/armv7/armv7_a4x_space.c 
b/sys/arch/arm/armv7/armv7_a4x_space.c
index 845f7704d8d..b5204579326 100644
--- a/sys/arch/arm/armv7/armv7_a4x_space.c
+++ b/sys/arch/arm/armv7/armv7_a4x_space.c
@@ -49,8 +49,6 @@
 /* Prototypes for all the bus_space structure functions */
 bs_protos(armv7);
 bs_protos(a4x);
-bs_protos(generic);
-bs_protos(generic_armv4);
 bs_protos(bs_notimpl);
 
 struct bus_space armv7_a4x_bs_tag = {
diff --git a/sys/arch/arm/armv7/armv7_space.c b/sys/arch/arm/armv7/armv7_space.c
index c7e9b686b8f..b48221803c7 100644
--- a/sys/arch/arm/armv7/armv7_space.c
+++ b/sys/arch/arm/armv7/armv7_space.c
@@ -84,7 +84,6 @@
 
 /* Prototypes for all the bus_space structure functions */
 bs_protos(armv7);
-bs_protos(generic);
 bs_protos(bs_notimpl);
 
 struct bus_space armv7_bs_tag = {
diff --git a/sys/arch/arm/conf/files.arm b/sys/arch/arm/conf/files.arm
index 9dae5a17689..713037c36a6 100644
--- a/sys/arch/arm/conf/files.arm
+++ b/sys/arch/arm/conf/files.arm
@@ -37,7 +37,6 @@ attach        cpu at mainbus
 
 # bus_space(9)
 define bus_space_generic
-file   arch/arm/arm/bus_space_asm_generic.S
 file   arch/arm/arm/bus_space_notimpl.S
 
 file   arch/arm/arm/arm_machdep.c

Reply via email to