RE: RocketPort Linux driver errors on module reload

2007-10-18 Thread Nick Thompson
>-Original Message-
>From: Jiri Slaby [mailto:[EMAIL PROTECTED]
>Sent: Thursday, October 18, 2007 5:08 PM
>To: Nick Thompson
>Cc: Jiri Slaby; linux-kernel@vger.kernel.org; Ferenc Wagner
>Subject: Re: RocketPort Linux driver errors on module reload
>
>
>>On 10/17/2007 09:48 PM, Nick Thompson wrote:
>>  Thanks a lot for the feedback, it's very much appreciated. Like I said, 
>> there is an  >>active project here at Comtrol to get 3.08 beta up to 
>> Linux kernel coding standards, once   >>that gets somewhere near done, a 
>> patch will be forthcoming.
>
>And the last question, when do you expect/estimate this would happen?
>
>thanks,
>-- 
>Jiri Slaby ([EMAIL PROTECTED])
>Faculty of Informatics, Masaryk University

Jiri,
 To be honest, it would be hard to estimate at this point. I've discussed it 
with one of our software engineers, but we do not know how long it will be at 
this point, as there is a lot of work to do to get it up to kernel coding 
standards. I will definitely keep the list up to date.

Regards,

Nick

Nick Thompson
Technical Support Specialist [EMAIL PROTECTED]
Comtrol Corporation ftp://ftp.comtrol.com
6655 Wedgwood Rd. Ste 120 http://www.comtrol.com
Maple Grove, MN. 55311 Sales: (800)GO-ROCKET
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: RocketPort Linux driver errors on module reload

2007-10-17 Thread Nick Thompson
>On 10/15/2007 02:57 PM, Ferenc Wagner wrote:
>> You are the man, Jiri!  That indeed fixes the problem.
>> Thank you very much.  What a turnaround...
>
>You're welcome, that's why we're here.
>
>> Shouldn't Documentation/rocket.txt also be changed?  I'm not sure
>> whether that support email is still valid.
>
>And so am I. I wouldn't change it, maybe they will contact you some later ;).
>

Darn. I don't read the mailing list for one day and there is a Comtrol related 
post. ;)
Anyhow, as far as the support address, Comtrol and Comtrol UK are now separate 
companies. I work for Comtrol. The address you will want to send Linux related 
Comtrol issues to is [EMAIL PROTECTED] You can also email me directly if you 
wish, as I handle Linux issues here in tech support at Comtrol. As far as 
changing the address in the rocket.txt file, it would be a good idea to change 
it to [EMAIL PROTECTED]

>Anyway the driver faces another problem with pci refcounting (it doesn't
>increment the counter) and the device might be removed from it any time the pci
>bus code decides. Are you willing to test to-pci-probing patches (i.e. patches
>which converts the driver to the model introduced in linux 2.4)? If not, I'll
>only increment the counter on modprobe and decrement it on rmmod, since it 
>would
>be a safe (in the meaning of not changing that much code) way of fixing the 
>problem.
>
>thanks,
>-- 
>Jiri Slaby ([EMAIL PROTECTED])
>Faculty of Informatics, Masaryk University

I've been wanting to get the driver that is included in the kernel up to date, 
and there is an active project here at comtrol to get the current code (driver 
version 3.08 beta) up to Linux kernel programming standards. The driver version 
that is included in the kernel (and has been patched a bit I'm sure since it's 
been in there a while) is version 2.09. The latest version (3.08 beta) can be 
obtained from: 

ftp://ftp.comtrol.com/beta/rport/drivers/u_pci/linux/rocketport-linux-3.08.tar.gz

If anyone has any helpful pointers (no pun intended) or comments on the code, 
I'd love to listen. 

Regards,

Nick

Nick Thompson
Technical Support Specialist [EMAIL PROTECTED]
Comtrol Corporation ftp://ftp.comtrol.com
6655 Wedgwood Rd. Ste 120 http://www.comtrol.com
Maple Grove, MN. 55311 Sales: (800)GO-ROCKET
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


RE: RocketPort Linux driver errors on module reload

2007-10-17 Thread Nick Thompson
>-Original Message-
>From: Jiri Slaby [mailto:[EMAIL PROTECTED]
>Sent: Wednesday, October 17, 2007 2:35 PM
>To: Nick Thompson
>Cc: linux-kernel@vger.kernel.org
>Subject: Re: RocketPort Linux driver errors on module reload
>
>
>On 10/17/2007 07:06 PM, Nick Thompson wrote:
>> I've been wanting to get the driver that is included in the kernel up to 
>> date, and there >>is an active project here at comtrol to get the current 
>> code (driver version 3.08 beta) up >>to Linux kernel programming standards. 
>> The driver version that is included in the kernel >>(and has been 
>> patched a bit I'm sure since it's been in there a while)is version 2.09. The 
>> >>latest version (3.08 beta) can be obtained from: 
>
>the easiest way is to get diff -u 2.09 3.08 and port it by hand, at least I
>ususally do it this way.
>
>> ftp://ftp.comtrol.com/beta/rport/drivers/u_pci/linux/rocketport-linux-3.08.tar.gz
>> 
>> If anyone has any helpful pointers (no pun intended) or comments on the 
>> code, I'd love to >>listen. 
>
>Convert to pci probing and in-kernel firmware loading :). The code is big mess
>due to ifdefs and much unneeded material, it's likely impossible to say this is
>wrong and this not, if you won't post a patch, where people will see the 
>changes
>you want to make. Then you will have feedback :).
>
>regards,
>-- 
>Jiri Slaby ([EMAIL PROTECTED])
>Faculty of Informatics, Masaryk University

Jiri,
 Thanks a lot for the feedback, it's very much appreciated. Like I said, there 
is an active project here at Comtrol to get 3.08 beta up to Linux kernel coding 
standards, once that gets somewhere near done, a patch will be forthcoming.

Thanks for taking the time to write.

Regards,

Nick

Nick Thompson
Technical Support Specialist [EMAIL PROTECTED]
Comtrol Corporation ftp://ftp.comtrol.com
6655 Wedgwood Rd. Ste 120 http://www.comtrol.com
Maple Grove, MN. 55311 Sales: (800)GO-ROCKET
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/