dany/u-boot-megous
1
0
Fork 0
mirror of https://xff.cz/git/u-boot/ synced 2025-09-26 21:11:18 +02:00
Code Issues Packages Projects Releases Wiki Activity

m68k: fix multiple memory accesses on swap operations

Browse Source 
On a
	u32 val = __sw32(*addr);

multiple memory accesses are not welcome, since "addr" may
be an IO peripheral register address.
This patch changes __sw16/32 to perform a single memory
access for the source value.

Signed-off-by: Angelo Dureghello <angelo@sysam.it>
This commit is contained in:
Angelo Dureghello
2018-08-04 23:02:56 +02:00
parent a1ed3a83be
commit faae49543a
1 changed files with 20 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

34
arch/m68k/include/asm/byteorder.h
View File

@@ -10,21 +10,28 @@
#include <asm/types.h> #include <asm/types.h>
#ifdef __GNUC__ #ifdef __GNUC__
#define __sw16(x) \
((__u16)( \ static inline __u32 __sw32(__u32 x)
(((__u16)(x) & (__u16)0x00ffU) << 8) | \ {
(((__u16)(x) & (__u16)0xff00U) >> 8) )) __u32 v = x;
#define __sw32(x) \
((__u32)( \ return v << 24 |
(((__u32)(x)) << 24) | \ (v & (__u32)0x0000ff00UL) << 8 |
(((__u32)(x) & (__u32)0x0000ff00UL) << 8) | \ (v & (__u32)0x00ff0000UL) >> 8 |
(((__u32)(x) & (__u32)0x00ff0000UL) >> 8) | \ v >> 24;
(((__u32)(x)) >> 24) )) }
static inline __u16 __sw16(__u16 x)
{
__u16 v = x;
return (v & (__u16)0x00ffU) << 8 |
(v & (__u16)0xff00U) >> 8;
}
static __inline__ unsigned ld_le16(const volatile unsigned short *addr) static __inline__ unsigned ld_le16(const volatile unsigned short *addr)
{ {
unsigned result = *addr; return __sw16(*addr);
return __sw16(result);
} }
static __inline__ void st_le16(volatile unsigned short *addr, static __inline__ void st_le16(volatile unsigned short *addr,
@@ -35,8 +42,7 @@ static __inline__ void st_le16(volatile unsigned short *addr,
static __inline__ unsigned ld_le32(const volatile unsigned *addr) static __inline__ unsigned ld_le32(const volatile unsigned *addr)
{ {
unsigned result = *addr; return __sw32(*addr);
return __sw32(result);
} }
static __inline__ void st_le32(volatile unsigned *addr, const unsigned val) static __inline__ void st_le32(volatile unsigned *addr, const unsigned val)