Re: [PATCH] alpha: provide ioread64 and iowrite64 implementations

2017-06-24 Thread Logan Gunthorpe
On 24/06/17 10:14 AM, Richard Henderson wrote: > So you would prefer a function containing BUG? Don't be so obtuse. Of course not. > Because I have to have *something* to put into the machvec function > pointer. Well I can't say I understand the point of the machvec code, but if you're saying

Re: [PATCH] alpha: provide ioread64 and iowrite64 implementations

2017-06-24 Thread Richard Henderson
On 06/24/2017 08:32 AM, Logan Gunthorpe wrote: So, IMO, the Jensen inq and outq funtions in your patch, which I did read, should probably just not be provided. So you would prefer a function containing BUG? Because I have to have *something* to put into the machvec function pointer. r~

Re: [PATCH] alpha: provide ioread64 and iowrite64 implementations

2017-06-24 Thread Logan Gunthorpe
On 24/06/17 09:25 AM, Richard Henderson wrote: > On 06/24/2017 08:19 AM, Logan Gunthorpe wrote: >> Hey, >> >> On 24/06/17 09:13 AM, Richard Henderson wrote: >>> All Alpha hosts except for Jensen provide 64-bit I/O operations. >>> >>> Jensen is EISA only, so there ought not be any devices that eve

Re: [PATCH] alpha: provide ioread64 and iowrite64 implementations

2017-06-24 Thread Richard Henderson
On 06/24/2017 08:19 AM, Logan Gunthorpe wrote: Hey, On 24/06/17 09:13 AM, Richard Henderson wrote: All Alpha hosts except for Jensen provide 64-bit I/O operations. Jensen is EISA only, so there ought not be any devices that even attempt such operations. But just in case, use 2 32-bit operatio

Re: [PATCH] alpha: provide ioread64 and iowrite64 implementations

2017-06-24 Thread Logan Gunthorpe
Hey, On 24/06/17 09:13 AM, Richard Henderson wrote: > All Alpha hosts except for Jensen provide 64-bit I/O operations. > > Jensen is EISA only, so there ought not be any devices that even > attempt such operations. But just in case, use 2 32-bit operations. Thanks for this, Richard. However, I

[PATCH] alpha: provide ioread64 and iowrite64 implementations

2017-06-24 Thread Richard Henderson
All Alpha hosts except for Jensen provide 64-bit I/O operations. Jensen is EISA only, so there ought not be any devices that even attempt such operations. But just in case, use 2 32-bit operations. Cc: Logan Gunthorpe Cc: Ivan Kokshaysky Cc: Matt Turner Signed-off-by: Richard Henderson ---