sh_eth: make sh_eth_soft_swap() work on ARM
authorSergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Sat, 2 Jun 2018 19:37:42 +0000 (22:37 +0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 4 Jun 2018 19:23:24 +0000 (15:23 -0400)
Browsing  thru the driver disassembly, I noticed that ARM gcc generated
no  code  whatsoever for sh_eth_soft_swap() while building a little-endian
kernel -- apparently __LITTLE_ENDIAN__ was not being #define'd, however
it got implicitly #define'd when building with the SH gcc (I could only
find the explicit #define __LITTLE_ENDIAN that was #include'd when building
a little-endian kernel).  Luckily, the Ether controller  only doing big-
endian DMA is encountered on the early SH771x SoCs only and all ARM SoCs
implement EDMR.DE and thus set 'sh_eth_cpu_data::hw_swap'. But anyway, we
need to fix the #ifdef inside sh_eth_soft_swap() to something that would
work on all architectures...

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/renesas/sh_eth.h

index 5dee19b..d77de76 100644 (file)
@@ -562,7 +562,7 @@ struct sh_eth_private {
 
 static inline void sh_eth_soft_swap(char *src, int len)
 {
-#ifdef __LITTLE_ENDIAN__
+#ifdef __LITTLE_ENDIAN
        u32 *p = (u32 *)src;
        u32 *maxp;
        maxp = p + ((len + sizeof(u32) - 1) / sizeof(u32));