From: Michael Schmitz <schmitzmic@gmail.com>
To: Tuomas Vainikka <tuomas.vainikka@aalto.fi>
Cc: Linux/m68k <linux-m68k@vger.kernel.org>,
David Miller <davem@davemloft.net>,
Geert Uytterhoeven <geert@linux-m68k.org>
Subject: Re: [PATCH 0/2] Experimental Amiga Zorro ESP driver
Date: Sat, 17 Aug 2013 13:49:51 +1200 [thread overview]
Message-ID: <11425cb5ec432e2fbb7b675052e8b33d@gmail.com> (raw)
In-Reply-To: <520E76FC.9040803@aalto.fi>
HelloTuomas,
> On 08/16/2013 12:40 AM, Tuomas Vainikka wrote:
>> On 06/06/2013 11:56 PM, Michael Schmitz wrote:
>>> Geert,
>>>
>>> other than 'it compiles', I can't really say much about this one.
>>> I've done what seemed necessary to do the ESP probe setup in the
>>> zorro driver framework (using the NCR7xx driver as a guideline),
>>> and used the old Blizzard 2060 driver as a model for the DMA setup.
>>>
>>> To adapt to another style of ESP integration, the DMA setup function
>>> will need to be changed. The simplest way to cater for multipe boards
>>> in this driver may be to provide a separate DMA setup function for
>>> each, and substitute the correct one for the default esp_ops field in
>>> the driver probe. Please correct me if you see a better or simpler
>>> way.
>> Thank you for rewriting the driver.
>>
>> I copypasted the appropriate DMA code from blz1230.c, and added some
>> code to switch
>> to the appropriate function. I compiled the module, and was able to
>> insert and remove it.
Thanks, I will merge that with my current version of the driver. Thanks
also for testing!
>>
>> The dmesg output is attached. I also attached the modified
>> zorro_esp.c.
>>
>> There is a HDD attached to the SCSI bus, but the scan does not take
>> place.
>>
>> The chip on the controller is a FAS216, but it is identified as a
>> FAS236. Does that matter?
I would not think so - David Miller as author of the ESP core might
know better though. The m68k mac esp driver is the only one I ever
worked on - fifteen years ago.
> I got a little further, but now there seems to be something wrong with
> the IRQ:
>
> [ 297.720000] esp: esp0, regs[80ea8000:80eb0000] irq[2]
> [ 297.730000] esp: esp0 is a FAS236, 40 MHz (ccf=0), SCSI ID 7
> [ 300.750000] scsi0 : esp
> [ 301.020000] scsi 0:0:1:0: Direct-Access SAMSUNG SP1213N
> TL10 PQ: 0 ANSI: 2
> [ 301.030000] scsi target0:0:1: Beginning Domain Validation
> [ 301.060000] scsi target0:0:1: FAST-10 SCSI 10.0 MB/s ST (100 ns,
> offset 15)
> [ 301.090000] scsi target0:0:1: Domain Validation skipping write tests
> [ 301.100000] scsi target0:0:1: Ending Domain Validation
> [ 301.880000] esp: esp0: Reconnect IRQ2 timeout
Are there interrupts logged for IRQ2 at all (cat /proc/interrupts)? It
looks to me as though all DMA transfers fail (the first command to fail
is READ_CAPACITY which would usually be issued right after IDENTIFY
IIRC).
> [ 332.040000] esp: esp0: Aborting command [0f9998a0:12]
> [ 332.040000] esp: esp0: Current command [0f999940:25]
> [ 332.040000] esp: esp0: Queued command [0f9998a0:12]
> [ 332.040000] esp: esp0: Active command [0f999940:25]
> [ 332.040000] esp: esp0: Dumping command log
> ...
>
> I attached a full log containing the dmesg output from both probing
> and removing the module.
Thanks, I will try to make sense of the log...
Cheers,
Michael
>
> -Tuomas
> <zorro_esp.c.gz><zorro_esp_modprobe2.cap.gz>
next prev parent reply other threads:[~2013-08-17 1:50 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-06 20:56 [PATCH 0/2] Experimental Amiga Zorro ESP driver Michael Schmitz
2013-06-06 20:56 ` [PATCH 1/2] m68k/amiga - Zorro ESP SCSI Makefile/Kconfig support Michael Schmitz
2013-06-06 20:56 ` [PATCH 2/2] m68k/amiga - Zorro ESP: convert old driver to ESP core Michael Schmitz
2013-08-15 21:40 ` [PATCH 0/2] Experimental Amiga Zorro ESP driver Tuomas Vainikka
2013-08-16 19:01 ` Tuomas Vainikka
2013-08-17 1:49 ` Michael Schmitz [this message]
2013-08-17 11:33 ` Tuomas Vainikka
2013-08-18 2:05 ` Michael Schmitz
2013-08-18 8:23 ` Geert Uytterhoeven
2013-08-18 8:58 ` Michael Schmitz
2013-08-18 9:10 ` Geert Uytterhoeven
2013-08-19 8:48 ` Michael Schmitz
2013-08-19 11:47 ` Tuomas Vainikka
2013-08-19 12:01 ` Geert Uytterhoeven
2013-08-19 20:46 ` Tuomas Vainikka
2013-08-20 9:36 ` Michael Schmitz
2013-08-20 10:00 ` Tuomas Vainikka
2013-08-22 20:34 ` Michael Schmitz
2013-08-31 10:37 ` Tuomas Vainikka
2013-09-10 21:13 ` Tuomas Vainikka
2013-09-11 10:12 ` Michael Schmitz
2013-09-11 13:08 ` Tuomas Vainikka
2013-09-11 20:14 ` Tuomas Vainikka
2013-09-26 13:44 ` Michael Schmitz
2013-09-26 14:04 ` Tuomas Vainikka
2013-09-27 9:17 ` Michael Schmitz
2013-09-11 14:48 ` Geert Uytterhoeven
2013-09-12 15:36 ` esp_scsi QTAG in FAS216 (was Re: [PATCH 0/2] Experimental Amiga Zorro ESP driver) Tuomas Vainikka
2013-09-26 13:50 ` Michael Schmitz
2014-04-04 20:28 ` esp_scsi QTAG in FAS216 David Miller
2014-04-06 20:33 ` Michael Schmitz
2014-04-07 3:39 ` David Miller
2014-04-10 14:31 ` Kars de Jong
2014-04-11 1:47 ` Michael Schmitz
2014-04-13 14:47 ` Kars de Jong
2014-04-13 22:38 ` Michael Schmitz
2014-04-14 2:14 ` David Miller
2014-04-14 5:05 ` Tuomas Vainikka
2014-04-14 8:51 ` Michael Schmitz
2014-05-25 8:56 ` Geert Uytterhoeven
2014-05-26 1:15 ` Michael Schmitz
2014-04-14 9:01 ` Michael Schmitz
2014-05-04 11:41 ` Tuomas Vainikka
2016-10-28 21:54 ` Finn Thain
2016-10-30 2:33 ` Finn Thain
2016-10-31 8:03 ` Michael Schmitz
2016-10-31 18:54 ` Michael Schmitz
2016-10-31 23:47 ` Finn Thain
2016-11-01 7:09 ` Michael Schmitz
2016-11-01 7:23 ` Finn Thain
2013-08-18 9:14 ` [PATCH 0/2] Experimental Amiga Zorro ESP driver Tuomas Vainikka
2013-08-18 9:42 ` Geert Uytterhoeven
2013-08-18 12:25 ` Tuomas Vainikka
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=11425cb5ec432e2fbb7b675052e8b33d@gmail.com \
--to=schmitzmic@gmail.com \
--cc=davem@davemloft.net \
--cc=geert@linux-m68k.org \
--cc=linux-m68k@vger.kernel.org \
--cc=tuomas.vainikka@aalto.fi \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox