Re: [PATCH] ipr: qc_fill_rtf() method should not store alternate status register

2013-06-02 Thread Tejun Heo
On Sat, Jun 01, 2013 at 02:37:00AM +0400, Sergei Shtylyov wrote: > Hello. > > On 05/31/2013 04:46 PM, Brian King wrote: > > > > >>The 'ctl' field of the 'struct ata_taskfile' is not really dual purpose, > >>i.e. > >>it is not intended for storing the alternate status register (which is > >>ma

Re: [PATCH] ipr: qc_fill_rtf() method should not store alternate status register

2013-05-31 Thread Sergei Shtylyov
Hello. On 05/31/2013 04:46 PM, Brian King wrote: The 'ctl' field of the 'struct ata_taskfile' is not really dual purpose, i.e. it is not intended for storing the alternate status register (which is mapped at the same address in the legacy IDE controllers) in the qc_fill_rtf() method. No oth

Re: [PATCH] ipr: qc_fill_rtf() method should not store alternate status register

2013-05-31 Thread Brian King
On 05/21/2013 02:24 PM, Sergei Shtylyov wrote: > The 'ctl' field of the 'struct ata_taskfile' is not really dual purpose, i.e. > it is not intended for storing the alternate status register (which is mapped > at the same address in the legacy IDE controllers) in the qc_fill_rtf() > method. > No

[PATCH] ipr: qc_fill_rtf() method should not store alternate status register

2013-05-21 Thread Sergei Shtylyov
The 'ctl' field of the 'struct ata_taskfile' is not really dual purpose, i.e. it is not intended for storing the alternate status register (which is mapped at the same address in the legacy IDE controllers) in the qc_fill_rtf() method. No other 'libata' driver except 'drivers/scsi/ipr.c' stores t