Re: [PATCH 0/4] g_NCR5380: PDMA fixes and cleanup

2017-06-23 Thread Finn Thain
On Fri, 23 Jun 2017, Ondrej Zary wrote: > On Friday 23 June 2017 13:01:53 Finn Thain wrote: > > On Fri, 23 Jun 2017, I wrote: > > > Does this patch help? It should be applied on top of this series of 4. > > > > Sorry, I sent the wrong diff. Please try this patch instead. > > Thanks, much better n

Re: [PATCH 0/4] g_NCR5380: PDMA fixes and cleanup

2017-06-23 Thread Ondrej Zary
On Friday 23 June 2017 13:01:53 Finn Thain wrote: > On Fri, 23 Jun 2017, I wrote: > > Does this patch help? It should be applied on top of this series of 4. > > Sorry, I sent the wrong diff. Please try this patch instead. Thanks, much better now: both HDD and CD-ROM seem to work on DTC and non-DTC

Re: [PATCH 0/4] g_NCR5380: PDMA fixes and cleanup

2017-06-23 Thread Finn Thain
On Fri, 23 Jun 2017, I wrote: > > Does this patch help? It should be applied on top of this series of 4. > Sorry, I sent the wrong diff. Please try this patch instead. diff --git a/drivers/scsi/g_NCR5380.c b/drivers/scsi/g_NCR5380.c index e9a942d86865..95ae8edbecbc 100644 --- a/drivers/scsi/g

Re: [PATCH 0/4] g_NCR5380: PDMA fixes and cleanup

2017-06-22 Thread Finn Thain
On Thu, 22 Jun 2017, Ondrej Zary wrote: > Works only with HDD on non-DTC chips. CD-ROM hangs. DTC hangs even with > HDD. The PDMA code really needs to be fixed. > Does this patch help? It should be applied on top of this series of 4. diff --git a/drivers/scsi/g_NCR5380.c b/drivers/scsi/g_NCR5

Re: [PATCH 0/4] g_NCR5380: PDMA fixes and cleanup

2017-06-22 Thread Ondrej Zary
On Thursday 15 June 2017 14:17:56 Finn Thain wrote: > Ondrej, would you please test this patch series? One of your patches > has been modified slightly and the two I wrote are untested. Works only with HDD on non-DTC chips. CD-ROM hangs. DTC hangs even with HDD. The PDMA code really needs to be fi

[PATCH 0/4] g_NCR5380: PDMA fixes and cleanup

2017-06-15 Thread Finn Thain
Ondrej, would you please test this patch series? One of your patches has been modified slightly and the two I wrote are untested. Finn Thain (2): g_NCR5380: Limit sg_tablesize to avoid PDMA read overruns on DTC436 g_NCR5380: Cleanup comments and whitespace Ondrej Zary (2): g_NCR5380: Fix P