PC Card", "Version 01.02", ""
>manfid: 0x0156, 0x0002
>function: 6 (network)
>
> Signed-off-by: Marcin Juszkiewicz <[EMAIL PROTECTED]>
Acked-by: Pavel Roskin <[EMAIL PROTECTED]>
> diff --git a/drivers/net/wireless/hostap/hostap_cs.c
&
Quoting Marcin Juszkiewicz <[EMAIL PROTECTED]>:
+ PCMCIA_DEVICE_PROD_ID1233(
+ "The Linksys Group, Inc.", "Wireless Network CF
Card", "ISL37300P",
+ 0xa5f472c2, 0x9c05598d, 0xc9049a39),
Acked-by: Pavel Roskin <[EMAI
a5f472c2, 0x9c05598d, 0xc9049a39, 0x57a66194),
I would prefer that revisions are not used for card identification in
presence of the chip name. It's not like RevB would need another driver
as long as "ISL37300P" is still a part of the PCMCIA ID.
PCMCIA_DEVICE_PROD_ID123 should be eno
at least the compatible kernel API) should go away soon.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
. It's just something that would help
a lot.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
though I haven't seen a problem like yours, I strongly suggest that
you try the above repository and post your results to the bcm43xx-dev
list. Even if the results are more positive :)
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
->name,
> newname);
I think KERN_INFO is the right level, but maybe KERN_DEBUG would be more
readily accepted.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
gt; rather than in the top level of /sys/devices.
I've just checked, pcmcia/ciscode.h is not needed. The rest is needed.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info
couple of wrinkles in the standalone build
system are irrelevant to the kernel interation.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
C is defined twice in Makefile. That must be wrong.
I'm going to try everything on a more traditional build in the tree and
without d80211. And then I'll try to break some assumptions.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
fficial ipw3945 drivers as well.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
ou may want to send further replies to
[EMAIL PROTECTED] where it would be seen by those
actually using the driver.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
f
good programming style.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
needs to be reconfigured.
Without this patch, bcm43xx_d80211 needs the channel to be set again
after the interface goes down and up. Similar problems are reported for
rt2x00 drivers.
Failure in ieee80211_hw_config() leads to the interface staying down.
Signed-off-by: Pavel Roskin <[EMAIL PROTEC
iples.
If the hardware drivers are supposed to do it, here's my patch. It is
working fine for me and ready to be applied. The changelog is in the
subject.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/d80211/bcm43xx/bcm43xx_main.c |3 +++
1 files changed,
On Tue, 2007-01-23 at 10:21 +0100, Michael Buesch wrote:
> On Tuesday 23 January 2007 07:14, Pavel Roskin wrote:
> > I have tried the patch, and it doesn't fix the problem. It's a separate
> > problem. It happens when bcm43xx_interrupt_handler() is called on a
> &g
but ARP and ping packets from AP to
> > STA are lost. The frames are even acknowledged, but not seen on the
> > station side. It takes from one to ten minutes util ping suddenly
> > starts working.
>
> Hm, is this 4318? It is known to loose lots of packets.
No, it
STA are lost. The frames are even acknowledged, but not seen on the
station side. It takes from one to ten minutes util ping suddenly
starts working.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Set phy->lo_control to NULL whenever it's freed. Failure to do so leads
to zeroing a block of memory that uses to hold *phy->lo_control, which
caused random crashes down the road.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/d80211/bcm43xx/bcm4
onfirm my suspicion that the actual corruption happened before
this point. We are just hitting it when trying to allocate more memory.
Help with debugging this problem will be appreciated. I've never hunted
down such problems, especially in kernel space.
--
Regards,
Pavel Roskin
-
To u
ible. It
appears that most loss is on the receiving side.
I'll try to debug the problem when I have time. At least I'll try to
find out if it's specific to the architecture or to another kernel
option.
Anyway, it's exciting that I could send first packets today!
--
Regards,
On Tue, 2007-01-16 at 23:07 +0100, Michael Buesch wrote:
> On Tuesday 16 January 2007 22:50, Pavel Roskin wrote:
> > On Tue, 2007-01-16 at 20:23 +0100, Michael Buesch wrote:
> >
> > > A patch for that is already upstream.
> >
> > I don't see it. It'
l as on x86_64. The dump was for
x86_64, as evidenced by the register size. Maybe you have less
debugging options enabled?
> Patch was
> [PATCH] bcm43xx-d80211: Fix DMA TX skb doublefree
Even with this hint, I cannot spot the bug immediately, so it would be
great if you sync the public r
uot;bcm43xx-d80211: Various cleanups all over the code"
SMP is disabled this time, just to make things simpler.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at
Quoting Michael Buesch <[EMAIL PROTECTED]>:
> Ok, nice.
> Can you send me a complete dmesg log after you brought up the card
> and tried to assoc/send some packets?
Attached.
--
Regards,
Pavel Roskin
ACPI: PCI Interrupt :0c:00.0[A] -> GSI 17 (level, low) -> IRQ 17
PCI:
7;t work over and
> over again.
> So, well. What to do? :)
I'll check in with a HostAP based AP and a separate sniffer, so I can see what's
on the air and what the AP sees.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev&
My immediate suspect is that we
> have
> > too many different fields called "dev". All it takes is one cast to hide a
> > horrible mistake. Although I think it would have affected you as well
> (unless
>
> We don't cast devs.
Yet I was pretty close :)
--
for me to think that your changes would fix
accociation to the AP?
I have never had any success with any d80211 based driver, so I have a strong
suspicion that I'm doing something wrong. I'll try to report a clear-cut case
with a less problematic driver in a separate thread.
--
Regard
tried wpa_supplicant, but it won't associate
either. The kernel messages and the wpa_supplicant output are attached. I
could not compile wpa_supplicant with MLME (too much code to fix), but I don't
think it should be required.
I tried the current version from your branch without SM
x-2.6/drivers/net/wireless/d80211/bcm43xx/bcm43xx_phy.c:1650
bcm43xx_d80211: Chip initialized
bcm43xx_d80211: ERROR: This card does not support PIO operation mode.
Please use DMA mode (module parameter pio=0).
bcm43xx_d80211: Radio turned off
--
Regards,
Pavel Roskin
-
To unsubscribe from this list
On Thu, 2006-12-21 at 21:53 -0500, Pavel Roskin wrote:
> On Sun, 2006-12-10 at 17:21 +0100, Johannes Berg wrote:
> > John, please apply this to wireless-dev if Pavel says it works (I think
> > it should).
>
> Yes, it's working for me. Please apply the patch.
Actually
On Sun, 2006-12-10 at 17:21 +0100, Johannes Berg wrote:
> John, please apply this to wireless-dev if Pavel says it works (I think
> it should).
Yes, it's working for me. Please apply the patch.
Sorry for answering so late.
--
Regards,
Pavel Roskin
-
To unsubscribe from this lis
they
won't be able to debug their drivers effectifely; they'll have to fix
cfg80211 first.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
ider and longer discussion (at least give driver
maintainers and the top kernel folks a chance to look at what you are
suggesting).
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
n be reshuffled and eliminated by
one person in a few days.
Making things work properly takes years. That's what MadWifi has been
working on for a long time, using contributions and bug reports from
scores of users and developers.
Rejecting MadWifi because it's HAL based is like thro
hostap_plx: fix two related off-by-one errors in CIS parser
From: Pavel Roskin <[EMAIL PROTECTED]>
The length of the manfid CIS should be at least 4, and it's normally 4.
It's incorrect to require it to be at least 5. This breaks support for
most (if not all) cards.
The right
Hello, Jouni!
On Fri, 2006-10-20 at 18:19 -0700, Jouni Malinen wrote:
> On Fri, Oct 20, 2006 at 06:20:15PM -0400, Pavel Roskin wrote:
>
> > The record length for numerical manufacturer ID should be at least 4
> > bytes (two 16-bit words). The code required 5 bytes, which w
feels more comfortable with an additional check, I'll add it.
I don't have Coverity to check, and I think the results of Coverity were
misinterpreted. It doesn't know anything about CIS structure. Anyway,
let me just check Jouni's suggestion on a real PLX card.
--
Regards,
Pave
hostap_plx: fix CIS verification
The record length for numerical manufacturer ID should be at least 4
bytes (two 16-bit words). The code required 5 bytes, which would break
for most (if not all) cards. Reported by [EMAIL PROTECTED]
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
This
that could harm the chances of d80211 to make it to the mainline
> > kernel.
>
> Christoph et. all will insist that everything be EXPORT_SYMBOL_GPL.
That shouldn't be an issue at all. HAL does not and can not call any
Linux functions directly. It can only communicate
rs, and they will ensure that
useful features of d80211 are not lost.
Unless I'm missing something obvious, I don't see absolutely anything
that could harm the chances of d80211 to make it to the mainline
kernel.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line &
compatibility.
I'm trying to maintain a Subversion repository with the driver modified
to be compatible with a few latest kernels. But it looks like it's an
uphill battle that I'm not going to win.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsub
From: Pavel Roskin <[EMAIL PROTECTED]>
Intersil firmware 1.7.4 (and possibly others) loses the antenna
selection settings when the port is reset.
Signed-off-by: David Acker <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/hostap/
From: Pavel Roskin <[EMAIL PROTECTED]>
SPARC architecture has been fixed, so it's no longer needed.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco.h |8 ++--
1 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/net/w
From: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/orinoco.c b/drivers/net/wireless/orinoco.c
index 317ace7..1174ff5 100644
--- a/drivers/net/wireless/orinoco.c
+++ b/d
From: Pavel Roskin <[EMAIL PROTECTED]>
This bug was introduced during the PCMCIA API conversion and broke
spectrum_cs completely.
Tracked down by Fredrik Tolf <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
This patch should go to 2.6.18 and 2.6.17.y.
/
> >
> > Didn't that get fixed up for SPARC a year or so back?
>
> That's right, this problem no longer exists.
Then, I guess, "static inline" would be just fine, right?
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netd
errors.
>
> Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
Thanks for the patch! I'm removing the last sentence in the comment as
irrelevant and fixing the comment. The patch is being sent to netdev by
StGIT, cc: author.
--
Regards,
Pavel Roskin
-
To unsubscribe from this
From: Pavel Roskin <[EMAIL PROTECTED]>
This is exactly the case when they are needed. This also fixed a
warning with -Wmissing-prototypes
Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco.h |
From: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/3c59x.c |2 +-
drivers/net/8139cp.c |2 +-
drivers/net/8139too.c |2 +-
drivers/net/acenic.c |2 +-
drivers/net/amd8
On Tue, 2006-07-25 at 14:26 -0700, Stephen Hemminger wrote:
> On Tue, 25 Jul 2006 17:08:11 -0400
> Pavel Roskin <[EMAIL PROTECTED]> wrote:
> > Considering the drivers that are already in the kernel, you may prefer
> > to have a more high-level function that would clone t
k netdev_get_pdev()
would be mostly used for such cloning if implemented.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
t_device. It
should be done before class_dev is removed or in the same time.
Should I send a patch?
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
CL-11", "ISL37300P",
0xa21501a, 0x59868926, 0xc9049a39),
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
;WL PCCL-11", "ISL37300P", "RevA"
manfid 0xc00f, 0x
bind "hostap_cs"
(from http://www.handhelds.org/hypermail/oe-commits/41/4121.html)
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
ficient
details. I haven't seen any detailed reports of this problem.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
rily for
systems that cannot use udev. But it's far from the top of my TODO
list. And I'm not sure it would actually help users of desktop
distributions.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
lkml/2006/5/30/433
Thanks for heads up. Actually, this e-mail was copied to the orinoco
mailing list, so I'm aware of it.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo
Signed-off-by: Marcin Juszkiewicz <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
> drivers/net/wireless/hostap/hostap_cs.c |6 ++
> 1 file changed, 6 insertions(+)
>
> Index: linux/d
.org/forums/index.php?showtopic=14544
> + PCMCIA_DEVICE_PROD_ID123(
> + "Pretec", "CompactWLAN Card 802.11b", "2.5",
> + 0x1cadd3e5, 0xe697636c, 0x7a5bfcf1),
That's a good one.
> + PCMCIA_DEVICE_PROD_ID1234(
> + "U.S. Robotics", "IEEE 802.11b PC-CARD", "Version 01.02", ""
> + 0xc7b8df9d, 0x1700d087, 0x4b74baa0, 0x),
Another useful entry, but please don't match the empty string.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, 2006-05-12 at 15:21 +0200, Marcin Juszkiewicz wrote:
> All patches require 24_hostap_cs_id.diff from Pavel Roskin.
This patch was never submitted. Please ignore the series.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
ot with patches that remove IDs for compatible cards
that were carefully collected over the years.
Any patches for Orinoco drivers that don't come from kernel subsystem
maintainers should be sent to the driver mailing list at
[EMAIL PROTECTED]
--
Regards,
Pavel Roskin
-
To unsubscribe from
footprint of hostap).
Once again, I really appreciate your vigilance, Jean :-)
I strongly believe that every driver should list all devices it can
support. In case of multiple drivers supporting one device, the choice
should be done in userspace.
--
Regards,
Pavel Roskin
-
To unsubscribe fr
m). That's what I want to see fixed.
One tricky case would be when the driver sets the max signal e.g. to 30
and reports 35 (i.e. a positive value within the "reasonable" dBm
range). I would probably prefer to show it as 30/30 rather than 35dBm,
but the driver is nuts any
n integer) and pushes it into
unsigned char in the hope that the client side can understand it using
the common sense. Interpreting 0dBm as -256dBm goes against the common
sense. This has to be specifically worked around in the drivers.
Of course, it would be nice to document it somewhere, and to
> don't want to cutoff the bottom of the range (is -128dBm enough ?).
> I tried to use 'signed' in the struct a long while ago, and
> for some reason it broke left and right, I don't remember the
> details. So, whatever we do, it would not be straightforward.
I sugg
would put the cutoff and 127dBm. 500 gigawatts should be enough
for everyone :-)
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
h because it's
> clear you don't know what you are doing.
> Some of those cards are Orinoco or Symbol cards.
I agree. Please don't apply this patch.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of
From: Pavel Roskin <[EMAIL PROTECTED]>
Hardware resources should not be made available to other devices while
the network device is still registered. Also remove the related debug
statements.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_
From: Pavel Roskin <[EMAIL PROTECTED]>
Report only the first I/O window and IRQ, and also add the driver name.
The second I/O window, Vpp and configuration index are not interesting
to most users. They can be found by PCMCIA debug tools if needed.
Signed-off-by: Pavel Roskin <[EMAIL
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_pci.h |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/orinoco_pci.h
b/drivers/net/wireless/orinoco_pci.h
i
From: Pavel Roskin <[EMAIL PROTECTED]>
The resource data in the network device is intended for ISA and other
older busses, but not for PCI. Don't put PCI data there. Don't (ab)use
the network device for keeping the IRQ number.
Retire orinoco_pci_setup_netdev(), and pr
From: Pavel Roskin <[EMAIL PROTECTED]>
Don't use flags in the spinlocks - the PCMCIA resume functions may not
be called under lock. Don't ignore any errors.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_cs.c |7 +++
Hello!
I'm about to send some more patches for Orinoco drivers. Unlike the
previous set, the new patches are mostly cosmetic changes based on the
feedback I received from the already submitted patches. Please apply to
the trees that have the previous patches.
--
Regards,
Pavel Roskin
From: Pavel Roskin <[EMAIL PROTECTED]>
This is the right thing to do and it prevents kernel BUG on unload.
Some PCMCIA network drivers use link->dev_node as a flag indicating that
the network device has been successfully registered. Recent code
changes cause this flag to be 0 aft
From: Pavel Roskin <[EMAIL PROTECTED]>
The original code was doing arithmetics on a little-endian value.
Reported by Stelios Koroneos <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/hostap/hostap_ioctl.c |4 ++--
1 files cha
t netdev->irq? Is it going to be obsolete too? Then I can easily
remove orinoco_pci_setup_netdev() with very minimal adjustments.
--
Regards,
Pavel Roskin
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
rinoco_up(dev);
>
> I wonder if it would be enough to issue hermes_set_irqmask() later
> in __orinoco_up() to release this irq disabled section.
Maybe, but I prefer not to touch this code, since it's about to undergo
a much more radical rework.
--
Regards,
Pavel Roskin
-
To unsubs
nge;
> > - dev->mem_end = pci_iorange + pci_iolen - 1;
> > + dev->mem_start = pci_resource_start(pdev, 0);
> > + dev->mem_end = dev->mem_start + pci_resource_len(pdev, 0) - 1;
>
> Is there a reason why dev->mem_{start/end} should not be removed ?
Is
Hello!
On Fri, 2006-04-07 at 23:24 +0200, Francois Romieu wrote:
> Pavel Roskin <[EMAIL PROTECTED]> :
> [...]
> > diff --git a/drivers/net/wireless/orinoco_pci.c
> > b/drivers/net/wireless/orinoco_pci.c
> > index 5362c21..e57e92b 100644
> > --- a/drivers/
ion of the problem can be found at
https://bugzilla.novell.com/show_bug.cgi?id=154773
The same problem with different card is reported at
http://sourceforge.net/mailarchive/message.php?msg_id=14597046
Signed-off-by: Jiri Benc <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/spectrum_cs.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/wireless/spectrum_cs.c
b/drivers/net/wireless/spectrum_cs.c
i
From: Pavel Roskin <[EMAIL PROTECTED]>
When processing Tx exception, only read data until addr1. Rename
hermes_tx_descriptor_802_11 to hermes_txexc_data since it's only used to
Tx exceptions. Reuse existing hermes_tx_descriptor structure. Remove
fields after addr1 - they are not re
From: Pavel Roskin <[EMAIL PROTECTED]>
Use skb_pull() to strip the addresses from the original packet. Don't
strip protocol bytes.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco.c | 44 ++--
drivers/net/w
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco.h |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/net/wireless/orinoco.h b/drivers/net/wireless/orinoco.h
index ca01e45..16db3e1
From: Pavel Roskin <[EMAIL PROTECTED]>
The result of orinoco_xmit() can be OK, dropped packet and busy
transmitter. Rename labels accordingly. Increment stats->tx_errors in
one place. Increment stats->tx_dropped - nobody is doing it for us.
Signed-off-by: Pavel Roskin <[
From: Pavel Roskin <[EMAIL PROTECTED]>
Copy PCI suspend/resume functions from orinoco_pci.c.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_nortel.c | 87 +++--
drivers/net/wireless/orinoco_plx
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_pci.c | 11 +++
1 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/drivers/net/wireless/orinoco_pci.c
b/drivers/net/wireless/orinoco
From: Pavel Roskin <[EMAIL PROTECTED]>
The new function can write an odd number of bytes, thus making padding
unnecessary.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/hermes.c | 17 -
drivers/net/wireless/hermes.h |7
From: Pavel Roskin <[EMAIL PROTECTED]>
hermes_bap_pwrite() supports odd-sized packets now. There is no
minimal packet size for 802.11. Also, hermes_bap_pwrite() supports
odd-sized packets now. This removes all reasons to pad the Tx data.
Signed-off-by: Pavel Roskin <[EMAIL
From: Pavel Roskin <[EMAIL PROTECTED]>
Make all Orinoco PCI drivers (orinoco_pci, orinoco_plx, orinoco_tmd and
orinoco_nortel) as similar as possible. Use the best implementation of
error handling, the best error messages, the best comments.
Put common code to orinoco_pci.h. For now
From: Pavel Roskin <[EMAIL PROTECTED]>
This is needed to identify the card before possible allocation problems,
so that the user at least can report the firmware version that fails.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orin
From: Pavel Roskin <[EMAIL PROTECTED]>
Don't ever return -errno from orinoco_xmit() - the network layer doesn't
expect it.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco.c | 12 ++--
1 files changed, 6 insertions(+), 6 dele
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_nortel.c |7 +--
drivers/net/wireless/orinoco_pci.c| 69 +
drivers/net/wireless/orinoco_plx.c| 42 +++--
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_pci.c | 15 ++-
1 files changed, 6 insertions(+), 9 deletions(-)
diff --git a/drivers/net/wireless/orinoco_pci.c
b/drivers/net/wireless/orinoco
From: Pavel Roskin <[EMAIL PROTECTED]>
The PCMCIA drivers would never be loaded if the CIS were wrong.
No other PCMCIA drivers validate CIS.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_cs.c |3 ---
drivers/net/wireless/spectrum_cs.c |3
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco.c | 83
drivers/net/wireless/orinoco.h |3 -
2 files changed, 0 insertions(+), 86 deletions(-)
diff --git a
From: Pavel Roskin <[EMAIL PROTECTED]>
Symbol firmware F3.91-71 has an additional word in the commsquality RID.
Extend the receiving buffer by one word to accomodate it.
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco.c |2 +-
1 files changed,
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/hermes.c | 19 ---
drivers/net/wireless/hermes.h | 34 --
2 files changed, 0 insertions(+), 53 deletions(-)
diff --g
From: Pavel Roskin <[EMAIL PROTECTED]>
Signed-off-by: Pavel Roskin <[EMAIL PROTECTED]>
---
drivers/net/wireless/orinoco_cs.c |6 --
drivers/net/wireless/spectrum_cs.c |6 --
2 files changed, 0 insertions(+), 12 deletions(-)
diff --git a/drivers/net/wireless/ori
1 - 100 of 122 matches
Mail list logo