Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 17 May 2004 13:36:32 +0200
From:      "Carlos Velasco" <freebsd@newipnet.com>
To:        "M. Warner Losh" <imp@bsdimp.com>
Cc:        freebsd-mobile@freebsd.org
Subject:   Re[2]: Modem + Network in Xircom cards, and maybe others
Message-ID:  <200405171336320804.080CD05E@192.168.128.16>
In-Reply-To: <20040505.155926.128442915.imp@bsdimp.com>
References:  <200405051311330554.1F584985@192.168.128.16> <20040505233435.B15444@gamplex.bde.org> <200405051746300245.2039F9A4@192.168.128.16> <20040505.155926.128442915.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
--=====_108479379229275=_
Content-Type: text/plain; charset="us-ascii"

On 05/05/2004 at 15:59 M. Warner Losh wrote:

>In message: <200405051746300245.2039F9A4@192.168.128.16>
>            "Carlos Velasco" <freebsd@newipnet.com> writes:
>: I'm patching it to activate both functions (usually modem/serial and
>: network).
>: Network is in xe driver, however I'm passing modem to sio.
>: It works, however sio reports interrupt-level buffer overflows when I'm
>: above 2400bps, losing characters, and here is where I'm lost.
>
>I don't think you are getting interrupts like you should.

Warner,

I think my patch is ok, problem is related to sio in normal mode (i think).
Aplying a last patch into sio.c makes my xircom modem work perfectly.

Have you tested other 16 bits cards?

I'm attaching my last complete patch.

Regards,
Carlos Velasco


--=====_108479379229275=_
Content-Type: application/octet-stream; name="xircom_2.patch"
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="xircom_2.patch"

ZGlmZiAtcnUgc3lzL2Rldi9wY2NhcmQvcGNjYXJkLmMgc3lzbmV3L2Rldi9w
Y2NhcmQvcGNjYXJkLmMKLS0tIHN5cy9kZXYvcGNjYXJkL3BjY2FyZC5jCVdl
ZCBNYXIgMTcgMTc6NTA6MzggMjAwNAorKysgc3lzbmV3L2Rldi9wY2NhcmQv
cGNjYXJkLmMJU2F0IE1heSAgMSAwOTo0Nzo1NyAyMDA0CkBAIC05NTUsOCAr
OTU1LDggQEAKIAlzdHJ1Y3QgcGNjYXJkX3NvZnRjICpzYyA9IFBDQ0FSRF9T
T0ZUQyhidXMpOwogCiAJZGV2aWNlX3ByaW50ZihidXMsICI8dW5rbm93biBj
YXJkPiIpOwotCXByaW50ZigiIChtYW51ZmFjdHVyZXI9MHglMDR4LCBwcm9k
dWN0PTB4JTA0eCkgYXQgZnVuY3Rpb24gJWRcbiIsCi0JICBzYy0+Y2FyZC5t
YW51ZmFjdHVyZXIsIHNjLT5jYXJkLnByb2R1Y3QsIGZ1bmMtPm51bWJlcik7
CisJcHJpbnRmKCIgKG1hbnVmYWN0dXJlcj0weCUwNHgsIHByb2R1Y3Q9MHgl
MDR4LCBwcm9kZXh0PTB4JTAyeCkgYXQgZnVuY3Rpb24gJWRcbiIsCisJICBz
Yy0+Y2FyZC5tYW51ZmFjdHVyZXIsIHNjLT5jYXJkLnByb2R1Y3QsIHNjLT5j
YXJkLnByb2RleHQsIGZ1bmMtPm51bWJlcik7CiAJZGV2aWNlX3ByaW50Zihi
dXMsICIgICBDSVMgaW5mbzogJXMsICVzLCAlc1xuIiwgc2MtPmNhcmQuY2lz
MV9pbmZvWzBdLAogCSAgc2MtPmNhcmQuY2lzMV9pbmZvWzFdLCBzYy0+Y2Fy
ZC5jaXMxX2luZm9bMl0pOwogCXJldHVybjsKQEAgLTEwNzUsNiArMTA3NSw3
IEBACiAJaW50IHBhc3N0aHJvdWdoID0gKGRldmljZV9nZXRfcGFyZW50KGNo
aWxkKSAhPSBkZXYpOwogCWludCBpc2RlZmF1bHQgPSAoc3RhcnQgPT0gMCAm
JiBlbmQgPT0gfjBVTCAmJiBjb3VudCA9PSAxKTsKIAlzdHJ1Y3QgcmVzb3Vy
Y2UgKnIgPSBOVUxMOworICAgICAgIHVfaW50IGFsaWduOwogCiAJLyogWFhY
IEknbSBubyBsb25nZXIgc3VyZSB0aGlzIGlzIHJpZ2h0ICovCiAJaWYgKHBh
c3N0aHJvdWdoKSB7CkBAIC0xMDkwLDggKzEwOTEsMTUgQEAKIAlpZiAocmxl
ID09IE5VTEwgfHwgcmxlLT5yZXMgPT0gTlVMTCkgewogCQkvKiBEbyB3ZSB3
YW50IHRoaXMgZGV2aWNlIHRvIG93biBpdD8gKi8KIAkJLyogWFhYIEkgdGhp
bmsgc28sIGJ1dCB0aGF0IG1pZ2h0IGJlIGxhbWUgWFhYICovCisKKyAgICAg
ICAgICAgICAgIC8qIGZvcmNlIDY0ayBwYWdlIGFsaWduICovCisgICAgICAg
ICAgICAgICBpZiAodHlwZSA9PSBTWVNfUkVTX01FTU9SWSkKKyAgICAgICAg
ICAgICAgICAgICBhbGlnbiA9IChmbGFncyAmIH5SRl9BTElHTk1FTlRfTUFT
SykgfAorICAgICAgICAgICAgICAgICAgICAgICBybWFuX21ha2VfYWxpZ25t
ZW50X2ZsYWdzKDY0KjEwMjQpOworICAgICAgICAgICAgICAgZWxzZQorICAg
ICAgICAgICAgICAgICAgIGFsaWduID0gZmxhZ3M7CiAJCXIgPSBidXNfYWxs
b2NfcmVzb3VyY2UoZGV2LCB0eXBlLCByaWQsIHN0YXJ0LCBlbmQsCi0JCSAg
Y291bnQsIGZsYWdzIC8qIFhYWCBhbGlnbWVudD8gKi8pOworICAgICAgICAg
ICAgICAgICBjb3VudCwgYWxpZ24pOwogCQlpZiAociA9PSBOVUxMKQogCQkg
ICAgZ290byBiYWQ7CiAJCXJlc291cmNlX2xpc3RfYWRkKCZkaW5mby0+cmVz
b3VyY2VzLCB0eXBlLCAqcmlkLApkaWZmIC1ydSBzeXMvZGV2L3BjY2FyZC9w
Y2NhcmRfY2lzLmMgc3lzbmV3L2Rldi9wY2NhcmQvcGNjYXJkX2Npcy5jCi0t
LSBzeXMvZGV2L3BjY2FyZC9wY2NhcmRfY2lzLmMJTW9uIEFwciAxMiAyMDo1
NjozNCAyMDA0CisrKyBzeXNuZXcvZGV2L3BjY2FyZC9wY2NhcmRfY2lzLmMJ
U2F0IE1heSAgMSAwOTo0Nzo1NyAyMDA0CkBAIC05Niw2ICs5Niw4IEBACiAK
IAlzdGF0ZS5wZiA9IE5VTEw7CiAKKwlzdGF0ZS5jYXJkLT5tZmMgPSAwOwor
CiAJdHNsZWVwKCZzdGF0ZSwgMCwgInBjY2FyZCIsIGh6KTsKIAlpZiAocGNj
YXJkX3NjYW5fY2lzKHNjLT5kZXYsIHBjY2FyZF9wYXJzZV9jaXNfdHVwbGUs
CiAJICAgICZzdGF0ZSkgPT0gLTEpCkBAIC02NjMsNiArNjY1LDcgQEAKIAkJ
ICogdXAuCiAJCSAqLwogCQlzdGF0ZS0+Z290bWZjID0gMTsKKwkJc3RhdGUt
PmNhcmQtPm1mYyA9IDE7CiAJCWJyZWFrOwogI2lmZGVmIFBDQ0FSRENJU0RF
QlVHCiAJY2FzZSBDSVNUUExfREVWSUNFOgpAQCAtODAzLDYgKzgwNiw0MiBA
QAogCiAJCQlTVEFJTFFfSU5TRVJUX1RBSUwoJnN0YXRlLT5jYXJkLT5wZl9o
ZWFkLCBzdGF0ZS0+cGYsCiAJCQkgICAgcGZfbGlzdCk7CisJCX0gZWxzZSBp
ZiAoc3RhdGUtPnBmLT5mdW5jdGlvbiAhPSBQQ0NBUkRfRlVOQ1RJT05fVU5T
UEVDKSB7CisJCQkvKiBXZSBoYXZlIGEgbm9uLU1GQyBjb21wbGlhbnQgY2Fy
ZCwgaXQgcHV0cyBtb3JlCisJCQkgKiB0aGFuIDEgRlVOQ0lEIGluIHRoZSBD
SVMgd2l0aG91dCBMT05HTElOSy4KKwkJCSAqIGEpIFdlIHB1dCB0aGUgZnVu
Y3Rpb25zIGluIHRoZSBsaXN0IGFsYSBNRkMuCisJCQkgKiBiKSBDb3B5IGxh
c3QgQ0ZHIGVudHJ5IGZvciBwcmV2aW91cyBmdW5jdGlvbiwKKwkJCSAqICAg
IG5vdCBzdXJlIGlmIHRoaXMgaXMgcmlnaHQsIGJ1dCB1c3VhbGx5IHdvcmtz
LgorCQkJICovCisJCQlzdHJ1Y3QgcGNjYXJkX2NvbmZpZ19lbnRyeSAqY2Zl
LCAqcWNmZTsKKwkJCXVpbnQzMl90CWNjcl9iYXNlID0gc3RhdGUtPnBmLT5j
Y3JfYmFzZTsKKwkJCXVpbnQzMl90CWNjcl9tYXNrID0gc3RhdGUtPnBmLT5j
Y3JfbWFzazsKKworCQkJY2ZlID0gTlVMTDsKKwkJCVNUQUlMUV9GT1JFQUNI
KHFjZmUsICZzdGF0ZS0+cGYtPmNmZV9oZWFkLCBjZmVfbGlzdCkgeworCQkJ
CWlmIChxY2ZlLT5udW1iZXIgPT0gc3RhdGUtPnBmLT5sYXN0X2NvbmZpZ19p
bmRleCkgeworCQkJCWNmZSA9IChzdHJ1Y3QgcGNjYXJkX2NvbmZpZ19lbnRy
eSAqKQorCQkJCSAgICBtYWxsb2Moc2l6ZW9mKCpjZmUpLCBNX0RFVkJVRiwg
TV9OT1dBSVQpOworCQkJCSpjZmUgPSAqcWNmZTsKKwkJCQlicmVhazsKKwkJ
CQl9CisJCQl9CisKKwkJCXN0YXRlLT5wZiA9IG1hbGxvYyhzaXplb2YoKnN0
YXRlLT5wZiksIE1fREVWQlVGLAorCQkJICAgIE1fTk9XQUlUIHwgTV9aRVJP
KTsKKwkJCXN0YXRlLT5wZi0+bnVtYmVyID0gc3RhdGUtPmNvdW50Kys7CisJ
CQlzdGF0ZS0+cGYtPmxhc3RfY29uZmlnX2luZGV4ID0gY2ZlLT5udW1iZXI7
CisJCQlzdGF0ZS0+cGYtPmNjcl9iYXNlID0gY2NyX2Jhc2U7CisJCQlzdGF0
ZS0+cGYtPmNjcl9tYXNrID0gY2NyX21hc2s7CisKKwkJCVNUQUlMUV9JTklU
KCZzdGF0ZS0+cGYtPmNmZV9oZWFkKTsKKwkJCWlmIChjZmUpIAorCQkJCVNU
QUlMUV9JTlNFUlRfVEFJTCgmc3RhdGUtPnBmLT5jZmVfaGVhZCwKKwkJCQkg
ICAgY2ZlLCBjZmVfbGlzdCk7CisKKwkJCVNUQUlMUV9JTlNFUlRfVEFJTCgm
c3RhdGUtPmNhcmQtPnBmX2hlYWQsIHN0YXRlLT5wZiwKKwkJCSAgICBwZl9s
aXN0KTsKKwkJCiAJCX0KIAkJc3RhdGUtPnBmLT5mdW5jdGlvbiA9IHBjY2Fy
ZF90dXBsZV9yZWFkXzEodHVwbGUsIDApOwogCmRpZmYgLXJ1IHN5cy9kZXYv
cGNjYXJkL3BjY2FyZHZhci5oIHN5c25ldy9kZXYvcGNjYXJkL3BjY2FyZHZh
ci5oCi0tLSBzeXMvZGV2L3BjY2FyZC9wY2NhcmR2YXIuaAlTdW4gTm92ICAy
IDIwOjE4OjE5IDIwMDMKKysrIHN5c25ldy9kZXYvcGNjYXJkL3BjY2FyZHZh
ci5oCVNhdCBNYXkgIDEgMDk6NDc6NTcgMjAwNApAQCAtMTc5LDYgKzE3OSw3
IEBACiAJaW50MzJfdAkJcHJvZHVjdDsKICNkZWZpbmUJUENNQ0lBX1BST0RV
Q1RfSU5WQUxJRAkJLTEKIAlpbnQxNl90CQlwcm9kZXh0OworCWludAkJbWZj
OwogCXVpbnQxNl90CWVycm9yOwogI2RlZmluZQlQQ01DSUFfQ0lTX0lOVkFM
SUQJCXsgTlVMTCwgTlVMTCwgTlVMTCwgTlVMTCB9CiAJU1RBSUxRX0hFQUQo
LCBwY2NhcmRfZnVuY3Rpb24pIHBmX2hlYWQ7CkBAIC0yODQsOCArMjg1LDkg
QEAKICNkZWZpbmUJUENDQVJEX1NQQUNFX0lPCQkyCiAKICNkZWZpbmUJcGNj
YXJkX21mYyhzYykJCQkJCQkJXAotCQkoU1RBSUxRX0ZJUlNUKCYoc2MpLT5j
YXJkLnBmX2hlYWQpICYmCQkJXAotCQkgU1RBSUxRX05FWFQoU1RBSUxRX0ZJ
UlNUKCYoc2MpLT5jYXJkLnBmX2hlYWQpLHBmX2xpc3QpKQorCQkoc2MtPmNh
cmQubWZjKQorLyoJCShTVEFJTFFfRklSU1QoJihzYyktPmNhcmQucGZfaGVh
ZCkgJiYJCQlcCisJCSBTVEFJTFFfTkVYVChTVEFJTFFfRklSU1QoJihzYykt
PmNhcmQucGZfaGVhZCkscGZfbGlzdCkpICovCiAKICNkZWZpbmUJcGNjYXJk
X2lvX2FsbG9jKHBmLCBzdGFydCwgc2l6ZSwgYWxpZ24sIHBjaW9wKQkJCVwK
IAkocGNjYXJkX2NoaXBfaW9fYWxsb2MoKHBmKS0+c2MtPnBjdCwgcGYtPnNj
LT5wY2gsIChzdGFydCksCVwKZGlmZiAtcnUgc3lzL2Rldi9zaW8vc2lvLmMg
c3lzbmV3L2Rldi9zaW8vc2lvLmMKLS0tIHN5cy9kZXYvc2lvL3Npby5jCU1v
biBNYXkgIDMgMjI6MzU6MjggMjAwNAorKysgc3lzbmV3L2Rldi9zaW8vc2lv
LmMJV2VkIE1heSAgNSAxMzowNDoxNCAyMDA0CkBAIC0yMzc0LDcgKzIzNzQs
NyBAQAogCSAqIChhYm91dCAzIHRpY2tzIGlmIGlucHV0IGZsb3cgY29udHJv
bCBpcyBub3QgdXNlZCBvciBub3QgaG9ub3VyZWQsCiAJICogYnV0IGEgYml0
IGxlc3MgZm9yIENTNS1DUzcgbW9kZXMpLgogCSAqLwotCWNwNHRpY2tzID0g
c3BlZWQgLyAxMCAvIGh6ICogNDsKKwljcDR0aWNrcyA9IHNwZWVkIC8gMTAg
LyBoeiAqIDQwOwogCWZvciAoaWJ1ZnNpemUgPSAxMjg7IGlidWZzaXplIDwg
Y3A0dGlja3M7KQogCQlpYnVmc2l6ZSA8PD0gMTsKIAlpZiAoaWJ1ZnNpemUg
PT0gY29tLT5pYnVmc2l6ZSkgewpkaWZmIC1ydSBzeXMvZGV2L3hlL2lmX3hl
X3BjY2FyZC5jIHN5c25ldy9kZXYveGUvaWZfeGVfcGNjYXJkLmMKLS0tIHN5
cy9kZXYveGUvaWZfeGVfcGNjYXJkLmMJU3VuIEFwciAxMSAxNjozNDoyOSAy
MDA0CisrKyBzeXNuZXcvZGV2L3hlL2lmX3hlX3BjY2FyZC5jCVNhdCBNYXkg
IDEgMDk6NDg6MjQgMjAwNApAQCAtNDAyLDYgKzQwMiw3IEBACiAJY29uc3Qg
c3RydWN0IHhlX3BjY2FyZF9wcm9kdWN0KiB4cHA7CiAJdV9pbnQxNl90IHBy
b2RleHQ7CiAKKwogCURFVlBSSU5URigyLCAoZGV2LCAicGNjYXJkX3Byb2R1
Y3RfbWF0Y2hcbiIpKTsKIAogCXhwcCA9IChjb25zdCBzdHJ1Y3QgeGVfcGNj
YXJkX3Byb2R1Y3QqKWVudDsKQEAgLTQwOSw2ICs0MTAsOCBAQAogCiAJaWYg
KHhwcC0+cHJvZGV4dCAhPSBwcm9kZXh0KQogCQl2cGZtYXRjaCA9IDA7CisJ
ZWxzZQorCQl2cGZtYXRjaCsrOwogCiAJcmV0dXJuICh2cGZtYXRjaCk7CiB9
CkBAIC00MTYsOCArNDE5LDE5IEBACiBzdGF0aWMgaW50CiB4ZV9wY2NhcmRf
bWF0Y2goZGV2aWNlX3QgZGV2KQogeworCWludAkJZXJyb3IgPSAwOworCXVf
aW50MzJfdAlmY24gPSBQQ0NBUkRfRlVOQ1RJT05fVU5TUEVDOwogCWNvbnN0
IHN0cnVjdCBwY2NhcmRfcHJvZHVjdCAqcHA7CiAKKwllcnJvciA9IHBjY2Fy
ZF9nZXRfZnVuY3Rpb24oZGV2LCAmZmNuKTsKKwlpZiAoZXJyb3IgIT0gMCkK
KwkJcmV0dXJuIChlcnJvcik7CisJLyoKKwkgKiBJZiBub3QgYSBuZXR3b3Jr
IGNhcmQsIHdlIGFyZSBub3QgdGhlIHJpZ2h0IGRyaXZlci4KKwkgKi8KKwlp
ZiAoZmNuICE9IFBDQ0FSRF9GVU5DVElPTl9ORVRXT1JLKQorCQlyZXR1cm4g
KEVOWElPKTsKKwogCURFVlBSSU5URigyLCAoZGV2LCAicGNjYXJkX21hdGNo
XG4iKSk7CiAKIAlwcCA9IChjb25zdCBzdHJ1Y3QgcGNjYXJkX3Byb2R1Y3Qq
KXhlX3BjY2FyZF9wcm9kdWN0czsKQEAgLTQyNSw3ICs0MzksNiBAQAogCWlm
ICgocHAgPSBwY2NhcmRfcHJvZHVjdF9sb29rdXAoZGV2LCBwcCwKIAkgICAg
IHNpemVvZih4ZV9wY2NhcmRfcHJvZHVjdHNbMF0pLCB4ZV9wY2NhcmRfcHJv
ZHVjdF9tYXRjaCkpICE9IE5VTEwpCiAJCXJldHVybiAoMCk7Ci0KIAlyZXR1
cm4gKEVJTyk7CiB9CiAK

--=====_108479379229275=_--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200405171336320804.080CD05E>