All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael Büsch" <m@bues.ch>
To: b43-dev@lists.infradead.org
Subject: [PATCH] b43: use rx desc underrun interrupt
Date: Sat, 20 Apr 2013 22:38:48 +0200	[thread overview]
Message-ID: <20130420223848.7ce0ddec@milhouse> (raw)
In-Reply-To: <alpine.DEB.2.02.1304202154540.19161@kelly.ryd.net>

On Sat, 20 Apr 2013 22:16:26 +0200 (CEST)
Thommy Jakobsson <thommyj@gmail.com> wrote:

> 
> 
> On Sat, 20 Apr 2013, Michael B?sch wrote:
> 
> > On Sat, 20 Apr 2013 21:47:10 +0200 (CEST)
> > Thommy Jakobsson <thommyj@gmail.com> wrote:
> > 
> > > > And I think you need to set ring->current_slot to the value of ring->ops->get_current_slot().
> > > > Not 100% sure, though, since it is years since I worked on that code.
> > > > 
> > > Is that really needed? They should already be the same since the device 
> > > has thrown the underflow interrupt
> > 
> > Hm not sure. But should be easy to verify.
> I did check it in the beginning when I was testing the patch, just printed 
> the indexes in the kernel log. But of course if there is a race condition
> it could be hard to catch.

Ok. I don't think there should be a race.
Thanks for checking.

-- 
Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/b43-dev/attachments/20130420/cefb6f3a/attachment-0001.sig>

  reply	other threads:[~2013-04-20 20:38 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-15  8:02 [PATCH] b43: use rx desc underrun interrupt Thommy Jakobsson
2013-04-19 14:32 ` Jonas Gorski
2013-04-19 15:55   ` Michael Büsch
2013-04-19 15:59     ` Jonas Gorski
2013-04-19 18:17   ` Larry Finger
2013-04-19 18:57     ` Michael Büsch
2013-04-19 20:37       ` Thommy Jakobsson
2013-04-19 20:46         ` Michael Büsch
2013-04-19 21:05           ` Larry Finger
2013-04-19 20:30   ` Thommy Jakobsson
2013-04-20 14:14   ` Thommy
2013-04-20 18:35     ` Larry Finger
2013-04-20 22:47       ` Thommy Jakobsson
2013-04-20 19:12     ` Michael Büsch
2013-04-20 19:47       ` Thommy Jakobsson
2013-04-20 19:50         ` Michael Büsch
2013-04-20 20:16           ` Thommy Jakobsson
2013-04-20 20:38             ` Michael Büsch [this message]
2013-04-20 20:56             ` Piotras
     [not found]             ` <CAFzhf4rSUO9mn_RKw4Vehw2LKggFdC_W0v-K_KKCZJez1AwynQ@mail.gmail.com>
2013-04-20 21:01               ` Thommy Jakobsson
2013-04-20 21:10                 ` Thommy Jakobsson
2013-04-20 21:23                   ` Thommy Jakobsson
2013-04-21  6:38                     ` Michael Büsch
2013-04-21  8:22                       ` Thommy Jakobsson
2013-04-21  8:44                         ` Michael Büsch
2013-04-21  9:01                           ` Thommy Jakobsson
2013-04-21  9:21                             ` Michael Büsch
2013-04-21 12:12                               ` Thommy Jakobsson
2013-04-21 14:46                                 ` Piotras
2013-04-21 14:59                                   ` Michael Büsch
2013-04-21 15:24                                     ` Piotras
2013-04-21 16:35                                       ` Michael Büsch
2013-04-21 15:11                                   ` Thommy Jakobsson
2013-04-21 16:31                                     ` Michael Büsch
2013-04-21 18:07                           ` Thommy Jakobsson
2013-04-21 18:26                             ` Michael Büsch
2013-04-21 18:44                               ` Thommy Jakobsson
     [not found]                               ` <CAOTQ16LCWpAOndQsHBxN1-X_8SLpcU8V=mM41gvcNRocA=GhsA@mail.gmail.com>
2013-04-21 18:44                                 ` Michael Büsch
2013-04-21 20:13                                   ` Thommy Jakobsson
2013-04-21 21:53                                     ` Michael Büsch
2013-04-22  8:11                                       ` Thommy Jakobsson
2013-04-22 10:00                                         ` Michael Büsch
2013-04-23  6:22                                           ` Michael Büsch
2013-04-21 14:27                       ` Piotras
2013-04-21 14:51                         ` Michael Büsch

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130420223848.7ce0ddec@milhouse \
    --to=m@bues.ch \
    --cc=b43-dev@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.