From: Takashi Iwai <tiwai@suse.de>
To: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Cc: alsa-devel@alsa-project.org, clemens@ladisch.de,
ffado-devel@lists.sf.net
Subject: Re: [PATCH 0/2] ALSA: fireworks/firewire-lib: add support for recent firmware quirk
Date: Tue, 11 Aug 2015 07:30:43 +0200 [thread overview]
Message-ID: <s5hbnee2xz0.wl-tiwai@suse.de> (raw)
In-Reply-To: <55C92933.50207@sakamocchi.jp>
On Tue, 11 Aug 2015 00:44:03 +0200,
Takashi Sakamoto wrote:
>
> Hi,
>
> On Aug 5 2015 09:21, Takashi Sakamoto wrote:
> > Some users have reported current ALSA Fireworks driver stops packet
> > streaming due to detecting discontinuity.
> >
> > [alsa-devel] ALSA snd-fireworks Echo AudioFire 2
> > http://mailman.alsa-project.org/pipermail/alsa-devel/2014-August/080379.html
> >
> > [alsa-devel] AudioFire4 quirk
> > http://mailman.alsa-project.org/pipermail/alsa-devel/2015-July/095697.html
> >
> > Linux 3.16 or later have this issue.
> >
> > As long as I investigate, this issue depends on firmware version installed
> > in these devices. While, it's not clear that which version of firmwares for
> > which model has this quirk. At least, firmware version 5.7.0, 5.7.3 and
> > 5.8.0 for AudioFire2/AudioFire4/AudioFirePre8 have this quirk.
> >
> > This patchset applies a solution for this issue, with my assumption that
> > 'the same firmware binary has the same quirk'. These models, AudioFire8
> > (since Jul 2009) and Gibson Robot Interface Pack (RIP) series uses the
> > same firmware binary in driver package.
> >
> > If possible, the second patch should be applied to all of stable kernels.
> >
> > This patchset updates my previous RFC:
> > [alsa-devel] [RFC][PATCH 0/2] ALSA: fireworks/firewire-lib: add support for recent firmware quirk
> > http://mailman.alsa-project.org/pipermail/alsa-devel/2015-August/095849.html
> >
> > Takashi Sakamoto (2):
> > Revert "ALSA: fireworks: add support for AudioFire2 quirk"
> > ALSA: fireworks/firewire-lib: add support for recent firmware quirk
> >
> > sound/firewire/amdtp.c | 5 +++--
> > sound/firewire/amdtp.h | 2 ++
> > sound/firewire/fireworks/fireworks.c | 10 ++++++++--
> > sound/firewire/fireworks/fireworks.h | 2 +-
> > sound/firewire/fireworks/fireworks_stream.c | 12 +++++++++---
> > 5 files changed, 23 insertions(+), 8 deletions(-)
>
> Thanks for applying them. (And sorry to be late for reply because I got
> a short summer vacation.)
>
> Currently I'm preparing for the next patchset (Digi00x and TASCAM
> drivers). As a quick glance, for-next branch in sound.git doesn't still
> includes these two patches. In this case, which branch should my work
> reabases to?
Use for-linus branch for now.
Takashi
next prev parent reply other threads:[~2015-08-11 5:30 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-05 0:21 [PATCH 0/2] ALSA: fireworks/firewire-lib: add support for recent firmware quirk Takashi Sakamoto
2015-08-05 0:21 ` [PATCH 1/2] Revert "ALSA: fireworks: add support for AudioFire2 quirk" Takashi Sakamoto
2015-08-05 5:54 ` Takashi Iwai
2015-08-05 0:21 ` [PATCH 2/2] ALSA: fireworks/firewire-lib: add support for recent firmware quirk Takashi Sakamoto
2015-08-05 5:55 ` Takashi Iwai
2015-08-10 22:44 ` [PATCH 0/2] " Takashi Sakamoto
2015-08-11 5:30 ` Takashi Iwai [this message]
2015-08-11 11:22 ` Takashi Sakamoto
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=s5hbnee2xz0.wl-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@alsa-project.org \
--cc=clemens@ladisch.de \
--cc=ffado-devel@lists.sf.net \
--cc=o-takashi@sakamocchi.jp \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox