a996320146
Without marking the asm __dcc_getstatus() volatile my compiler
decides it can cache the value of __ret in a register and then
check the value of it continually in hvc_dcc_put_chars() (I had
to replace get_wait/put_wait with 1 and fixup the branch
otherwise my disassembler barfed on __dcc_(get|put)char).
00000000 <hvc_dcc_put_chars>:
0: ee103e11 mrc 14, 0, r3, cr0, cr1, {0}
4: e3a0c000 mov ip, #0 ; 0x0
8: e2033202 and r3, r3, #536870912 ; 0x20000000
c: ea000006 b 2c <hvc_dcc_put_chars+0x2c>
10: e3530000 cmp r3, #0 ; 0x0
14: 1afffffd bne 10 <hvc_dcc_put_chars+0x10>
18: e7d1000c ldrb r0, [r1, ip]
1c: ee10fe11 mrc 14, 0, pc, cr0, cr1, {0}
20: 2afffffd bcs 1c <hvc_dcc_put_chars+0x1c>
24: ee000e15 mcr 14, 0, r0, cr0, cr5, {0}
28: e28cc001 add ip, ip, #1 ; 0x1
2c: e15c0002 cmp ip, r2
30: bafffff6 blt 10 <hvc_dcc_put_chars+0x10>
34: e1a00002 mov r0, r2
38: e12fff1e bx lr
As you can see, the value of the mrc is checked against
DCC_STATUS_TX (bit 29) and then stored in r3 for later use.
Marking the asm volatile produces the following:
00000000 <hvc_dcc_put_chars>:
0: e3a03000 mov r3, #0 ; 0x0
4: ea000007 b 28 <hvc_dcc_put_chars+0x28>
8: ee100e11 mrc 14, 0, r0, cr0, cr1, {0}
c: e3100202 tst r0, #536870912 ; 0x20000000
10: 1afffffc bne 8 <hvc_dcc_put_chars+0x8>
14: e7d10003 ldrb r0, [r1, r3]
18: ee10fe11 mrc 14, 0, pc, cr0, cr1, {0}
1c: 2afffffd bcs 18 <hvc_dcc_put_chars+0x18>
20: ee000e15 mcr 14, 0, r0, cr0, cr5, {0}
24: e2833001 add r3, r3, #1 ; 0x1
28: e1530002 cmp r3, r2
2c: bafffff5 blt 8 <hvc_dcc_put_chars+0x8>
30: e1a00002 mov r0, r2
34: e12fff1e bx lr
which looks better and actually works. Mark all the inline
assembly in this file as volatile since we don't want the
compiler to optimize away these statements or move them around
in any way.
Acked-by: Tony Lindgren <tony@atomide.com>
Cc: Arnd Bergmann <arnd@arndb.de>
Acked-by: Nicolas Pitre <nicolas.pitre@linaro.org>
Cc: Daniel Walker <dwalker@codeaurora.org>
Signed-off-by: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
133 lines
2.9 KiB
C
133 lines
2.9 KiB
C
/* Copyright (c) 2010, Code Aurora Forum. All rights reserved.
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License version 2 and
|
|
* only 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.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program; if not, write to the Free Software
|
|
* Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA
|
|
* 02110-1301, USA.
|
|
*/
|
|
|
|
#include <linux/console.h>
|
|
#include <linux/delay.h>
|
|
#include <linux/err.h>
|
|
#include <linux/init.h>
|
|
#include <linux/moduleparam.h>
|
|
#include <linux/types.h>
|
|
|
|
#include <asm/processor.h>
|
|
|
|
#include "hvc_console.h"
|
|
|
|
/* DCC Status Bits */
|
|
#define DCC_STATUS_RX (1 << 30)
|
|
#define DCC_STATUS_TX (1 << 29)
|
|
|
|
static inline u32 __dcc_getstatus(void)
|
|
{
|
|
u32 __ret;
|
|
asm volatile("mrc p14, 0, %0, c0, c1, 0 @ read comms ctrl reg"
|
|
: "=r" (__ret) : : "cc");
|
|
|
|
return __ret;
|
|
}
|
|
|
|
|
|
#if defined(CONFIG_CPU_V7)
|
|
static inline char __dcc_getchar(void)
|
|
{
|
|
char __c;
|
|
|
|
asm volatile("get_wait: mrc p14, 0, pc, c0, c1, 0 \n\
|
|
bne get_wait \n\
|
|
mrc p14, 0, %0, c0, c5, 0 @ read comms data reg"
|
|
: "=r" (__c) : : "cc");
|
|
|
|
return __c;
|
|
}
|
|
#else
|
|
static inline char __dcc_getchar(void)
|
|
{
|
|
char __c;
|
|
|
|
asm volatile("mrc p14, 0, %0, c0, c5, 0 @ read comms data reg"
|
|
: "=r" (__c));
|
|
|
|
return __c;
|
|
}
|
|
#endif
|
|
|
|
#if defined(CONFIG_CPU_V7)
|
|
static inline void __dcc_putchar(char c)
|
|
{
|
|
asm volatile("put_wait: mrc p14, 0, pc, c0, c1, 0 \n\
|
|
bcs put_wait \n\
|
|
mcr p14, 0, %0, c0, c5, 0 "
|
|
: : "r" (c) : "cc");
|
|
}
|
|
#else
|
|
static inline void __dcc_putchar(char c)
|
|
{
|
|
asm volatile("mcr p14, 0, %0, c0, c5, 0 @ write a char"
|
|
: /* no output register */
|
|
: "r" (c));
|
|
}
|
|
#endif
|
|
|
|
static int hvc_dcc_put_chars(uint32_t vt, const char *buf, int count)
|
|
{
|
|
int i;
|
|
|
|
for (i = 0; i < count; i++) {
|
|
while (__dcc_getstatus() & DCC_STATUS_TX)
|
|
cpu_relax();
|
|
|
|
__dcc_putchar((char)(buf[i] & 0xFF));
|
|
}
|
|
|
|
return count;
|
|
}
|
|
|
|
static int hvc_dcc_get_chars(uint32_t vt, char *buf, int count)
|
|
{
|
|
int i;
|
|
|
|
for (i = 0; i < count; ++i) {
|
|
int c = -1;
|
|
|
|
if (__dcc_getstatus() & DCC_STATUS_RX)
|
|
c = __dcc_getchar();
|
|
if (c < 0)
|
|
break;
|
|
buf[i] = c;
|
|
}
|
|
|
|
return i;
|
|
}
|
|
|
|
static const struct hv_ops hvc_dcc_get_put_ops = {
|
|
.get_chars = hvc_dcc_get_chars,
|
|
.put_chars = hvc_dcc_put_chars,
|
|
};
|
|
|
|
static int __init hvc_dcc_console_init(void)
|
|
{
|
|
hvc_instantiate(0, 0, &hvc_dcc_get_put_ops);
|
|
return 0;
|
|
}
|
|
console_initcall(hvc_dcc_console_init);
|
|
|
|
static int __init hvc_dcc_init(void)
|
|
{
|
|
hvc_alloc(0, 0, &hvc_dcc_get_put_ops, 128);
|
|
return 0;
|
|
}
|
|
device_initcall(hvc_dcc_init);
|