Date: Sat, 01 Nov 1997 08:53:14 -0800 From: David Greenman <dg@root.com> To: nnd@itfs.nsk.su Cc: hackers@FreeBSD.ORG Subject: Re: Cyclades :( Message-ID: <199711011653.IAA03064@implode.root.com> In-Reply-To: Your message of "01 Nov 1997 08:23:22 GMT." <63eotq$37v@news.itfs.nsk.su>
next in thread | previous in thread | raw e-mail | index | archive | help
> Driver's text inspection shows that there is ONLY ONE
>significant difference with 'standard FreeBSD's' cy-driver:
>(this is a 'diff -bu' result for /sys/i386/isa/cy.c version
>1.55 of 1997/09/21)
>
>--- cy.c.ORIG Sat Nov 1 13:33:19 1997
>+++ cy.c Sat Nov 1 13:36:55 1997
>@@ -410,7 +404,7 @@
> #endif
>
> static int cy_chip_offset[] = {
>- 0x0000, 0x0200, 0x0400, 0x0600, 0x0100, 0x0300, 0x0500, 0x0700,
>+ 0x0000, 0x0400, 0x0800, 0x0c00, 0x0200, 0x0600, 0x0a00, 0x0e00
> };
> static int cy_nr_cd1400s[NCY];
> static int cy_total_devices;
>
>
> Can someone with Cyclom-8/16YeP (and problems ;-)
>test this patch ? I myself can not test it because my only ONE
>Cyclom-16YeP now busily serves 12 modems on Linux box.
>
> Partial "correctnes proof" for this patch can be found
>in (working) Linux 'cy' (or 'cz') driver which uses the same
>chip_offset addresses as in "patched" driver, but not as in
>original FreeBSD's 'cy' driver.
If you look at the cy_inb/cy_outb functions in cyreg.h, you'll see that
the offset is adjusted (shifted left by one bit) for the PCI card, making
the appropriate adjustment. The above change (which has the left shift built
in to the numbers) would effectively double this shift. What I'm saying is
that unless I'm really missing something, the patch can't be correct.
-DG
David Greenman
Core-team/Principal Architect, The FreeBSD Project
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?199711011653.IAA03064>