RE: [PATCH net-next v2] net: bcmgenet: Use correct I/O accessors

2017-08-30 Thread Florian Fainelli
On August 30, 2017 4:39:52 AM PDT, David Laight wrote: >From: Florian Fainelli >> Sent: 29 August 2017 20:26 >> The GENET driver currently uses __raw_{read,write}l which means >> native I/O endian. This works correctly for an ARM LE kernel >(default) >> but fails miserably on an ARM BE (BE8) kerne

RE: [PATCH net-next v2] net: bcmgenet: Use correct I/O accessors

2017-08-30 Thread David Laight
From: Florian Fainelli > Sent: 29 August 2017 20:26 > The GENET driver currently uses __raw_{read,write}l which means > native I/O endian. This works correctly for an ARM LE kernel (default) > but fails miserably on an ARM BE (BE8) kernel where registers are kept > little endian, so replace uses wi

Re: [PATCH net-next v2] net: bcmgenet: Use correct I/O accessors

2017-08-29 Thread David Miller
From: Florian Fainelli Date: Tue, 29 Aug 2017 12:25:31 -0700 > The GENET driver currently uses __raw_{read,write}l which means > native I/O endian. This works correctly for an ARM LE kernel (default) > but fails miserably on an ARM BE (BE8) kernel where registers are kept > little endian, so repl

[PATCH net-next v2] net: bcmgenet: Use correct I/O accessors

2017-08-29 Thread Florian Fainelli
The GENET driver currently uses __raw_{read,write}l which means native I/O endian. This works correctly for an ARM LE kernel (default) but fails miserably on an ARM BE (BE8) kernel where registers are kept little endian, so replace uses with {read,write}l_relaxed here which is what we want because