From: Takashi Sakamoto <o-takashi@sakamocchi.jp>
To: Takashi Iwai <tiwai@suse.de>
Cc: clemens@ladisch.de, alsa-devel@alsa-project.org,
linux1394-devel@lists.sourceforge.net,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/3] firewire: assist unit driver to compute packet time stamp
Date: Thu, 7 Apr 2022 09:10:21 +0900 [thread overview]
Message-ID: <Yk4r7VcotHz0iMOU@workstation> (raw)
In-Reply-To: <s5hczhv5wjc.wl-tiwai@suse.de>
On Tue, Apr 05, 2022 at 06:23:35PM +0200, Takashi Iwai wrote:
> On Tue, 05 Apr 2022 09:22:18 +0200,
> Takashi Sakamoto wrote:
> >
> > Hi,
> >
> > Current implementation of Linux FireWire subsystem doesn't allow unit
> > driver to operate content of packet in IR context according to
> > time stamp. Additionally it doesn't allow unit driver to read current value
> > of CYCLE_TIME register in OHCI 1394 controller. It brings disadvantages to
> > drivers in Linux sound subsystem in regards of handling time for sampled
> > data such as PCM frames and MIDI messages.
> >
> > This rerolled patchset is first step to improve the situation.
> >
> > Changes in v3:
> > * Rebase v2 patchset to v5.18-rc1
> > Changes in v2:
> > * Rebase v1 patchset to v5.16 release
> > * https://lore.kernel.org/lkml/20220212022131.199855-1-o-takashi@sakamocchi.jp/
> > V1:
> > * https://lore.kernel.org/lkml/20211202113457.24011-1-o-takashi@sakamocchi.jp/
> >
> > Hector Martin (1):
> > firewire: Add dummy read_csr/write_csr functions
> >
> > Takashi Sakamoto (2):
> > firewire: add kernel API to access CYCLE_TIME register
> > firewire: add kernel API to access packet structure in request
> > structure for AR context
>
> Thanks, applied all three patches now to for-next branch.
Although thanks for your applying them into your tree, I apologize to
trouble you if you overlook that the included changes is just for Linux
FireWire subsystem. It's my fault to send them only to Linux sound
subsystem, but the changes are required to my work in sound drivers...
If you are willing to include patches to Linux FireWire subsystem for
your pull-request to Linus, I can prepare respined patches for it since
I have the list of patches posted to LKML as bug fixes for Linux FireWire
subsystem.
I need any help to solve current situation of Linux FireWire subsystem
that bug fixes and new changes are hardly merged. Of course, IEEE 1394 bus
is already outdated and legacy, but I know that some users still work
with it. If your path is available for it, it's the easiest and the most
convenient way for upstreaming, I think.
Thanks
Takashi Sakamoto
next parent reply other threads:[~2022-04-07 0:12 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20220405072221.226217-1-o-takashi@sakamocchi.jp>
[not found] ` <s5hczhv5wjc.wl-tiwai@suse.de>
2022-04-07 0:10 ` Takashi Sakamoto [this message]
2022-04-07 6:02 ` [PATCH v3 0/3] firewire: assist unit driver to compute packet time stamp Takashi Iwai
2022-04-08 2:15 ` 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=Yk4r7VcotHz0iMOU@workstation \
--to=o-takashi@sakamocchi.jp \
--cc=alsa-devel@alsa-project.org \
--cc=clemens@ladisch.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux1394-devel@lists.sourceforge.net \
--cc=tiwai@suse.de \
/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