Re: [PATCH 2.6.13-rc6] remove dead reset function from cpqfcTS driver

[Rolf Eike Beer]

Martin K. Petersen wrote:
>>>>>> "Rolf" == Rolf Eike Beer <eike-kernel@xxxxxxxxx> writes:
>
>Hey Rolf!

I should go and use "R. Eike Beer" ;)

>Rolf> There was a request on lkml last week for a working version of
>Rolf> this driver. For the moment I try to clean this up a bit before
>Rolf> doing some real work. I found 4 major things that should be
>Rolf> done, for half of them I have patches in a proof-of-concept
>Rolf> state.
>
>As Christoph said I'm working on a driver for the TachLite TL/TS/XL2
>chips.
>
>Initially I just wanted to add support for the integrated PHY on XL2
>so we could support those cards on PA-RISC.  But when I started
>looking at the driver I came to the conclusion that it was just too
>ugly to live.  Architecturally, the overall design of cpqfc just
>doesn't fit in well with Linux.  So I'm rewriting it from scratch -
>but that obviously takes a while.

>I think it's cool that you want to hack on cpqfcTS.  But be aware that
>it's not just a matter of running lindent and making it compile in
>2.6.late.  And without hardware it's going to be hard.  Fibre channel
>is very finicky.

For the moment I'm trying to fix the most buggy parts. The Lindent run is 
scheduled to be done last, it's too big and adds nothing if it would go into 
kernel for now. It won't even compile now, my tests are only compile tests 
done with the two #error commented out. That should prevent most users from 
using it without all patches applied. The rest either knows what he's doing 
and/or would get crashes anyway ;)

Next to come will be splitting up the ISR and then fixing the #errors. After 
this is might even work. :)

>If you manage to get your hands on hardware (cards - avoid Tachyon
>5000 series. TachLite 5100, 5166 or 5200 is what you want, disk array,
>hub/switch, GBICs, etc.) I wouldn't mind some help...

Give me some code and we can see...

Bolke, what kind of adapter do you have?

Eike

Attachment: pgp00000.pgp
Description: PGP signature