From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephan Linz Date: Fri, 12 Oct 2012 00:31:54 +0200 Subject: [U-Boot] [PATCH 2/4] microblaze: Fix byteorder for microblaze In-Reply-To: <1349441933-22840-2-git-send-email-monstr@monstr.eu> References: <1349441933-22840-1-git-send-email-monstr@monstr.eu> <1349441933-22840-2-git-send-email-monstr@monstr.eu> Message-ID: <1349994714.2756.4.camel@keto> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Am Freitag, den 05.10.2012, 14:58 +0200 schrieb Michal Simek: > Just remove ancient code. > > Signed-off-by: Michal Simek > --- > arch/microblaze/include/asm/byteorder.h | 23 ----------------------- > 1 files changed, 0 insertions(+), 23 deletions(-) > > diff --git a/arch/microblaze/include/asm/byteorder.h b/arch/microblaze/include/asm/byteorder.h > index b2757a4..f3a471d 100644 > --- a/arch/microblaze/include/asm/byteorder.h > +++ b/arch/microblaze/include/asm/byteorder.h > @@ -20,29 +20,6 @@ > > #ifdef __GNUC__ > > -/* This is effectively a dupe of the arch-independent byteswap > - code in include/linux/byteorder/swab.h, however we force a cast > - of the result up to 32 bits. This in turn forces the compiler > - to explicitly clear the high 16 bits, which it wasn't doing otherwise. > - > - I think this is a symptom of a bug in mb-gcc. JW 20040303 > -*/ > - > - > -static __inline__ __u16 ___arch__swab16 (__u16 half_word) > -{ > - /* 32 bit temp to cast result, forcing clearing of high word */ > - __u32 temp; > - > - temp = ((half_word & 0x00FFU) << 8) | ((half_word & 0xFF00U) >> 8); > - > - return (__u16) temp; > -} > - > -#define __arch__swab16(x) ___arch__swab16(x) > - > -/* Microblaze has no arch-specific endian conversion insns */ > - Acked-by: Stephan Linz > > #if !defined(__STRICT_ANSI__) || defined(__KERNEL__) > # define __BYTEORDER_HAS_U64__ > # define __SWAB_64_THRU_32__