Re: [PATCH 1/2] smc91x: always use 8-bit access if necessary

2016-08-27 Thread kbuild test robot
Hi Arnd, [auto build test WARNING on net-next/master] [also build test WARNING on v4.8-rc3 next-20160825] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to recor

Re: [PATCH 1/2] smc91x: always use 8-bit access if necessary

2016-08-26 Thread kbuild test robot
Hi Arnd, [auto build test WARNING on net-next/master] [also build test WARNING on v4.8-rc3 next-20160825] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to recor

Re: [PATCH 1/2] smc91x: always use 8-bit access if necessary

2016-08-26 Thread Arnd Bergmann
On Thursday, August 25, 2016 11:33:08 PM CEST Russell King - ARM Linux wrote: > > This is insufficient. If you look at how SMC_REG works, you'll notice > that it has side effects which can disrupt other accesses (it reads > the bank register if debugging is enabled.) Ok, got it. Arnd

Re: [PATCH 1/2] smc91x: always use 8-bit access if necessary

2016-08-25 Thread Russell King - ARM Linux
On Thu, Aug 25, 2016 at 04:46:20PM +0200, Arnd Bergmann wrote: > +#define SMC_out16(x, ioaddr, reg) \ > +do { \ > + if (SMC_CAN_USE_8BIT && !SMC_16BIT(lp)) {\

Re: [PATCH 1/2] smc91x: always use 8-bit access if necessary

2016-08-25 Thread kbuild test robot
Hi Arnd, [auto build test WARNING on net-next/master] [also build test WARNING on v4.8-rc3 next-20160825] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to recor

[PATCH 1/2] smc91x: always use 8-bit access if necessary

2016-08-25 Thread Arnd Bergmann
As Russell King found out the hard way, a change I did to fix multiplatform builds with this driver broke the old Assabet/Neponset platform: It turns out that while the driver is runtime configurable in principle, the runtime configuration does not cover the specific case of machines that can not d