All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nicolin Chen <nicoleotsuka@gmail.com>
To: Mark Brown <broonie@kernel.org>
Cc: Arnaud Ferraris <arnaud.ferraris@collabora.com>,
	Timur Tabi <timur@kernel.org>,
	alsa-devel@alsa-project.org, Xiubo Li <Xiubo.Lee@gmail.com>,
	Liam Girdwood <lgirdwood@gmail.com>,
	linux-kernel@vger.kernel.org, Takashi Iwai <tiwai@suse.com>,
	Rob Herring <robh+dt@kernel.org>,
	kernel@collabora.com, Fabio Estevam <festevam@gmail.com>
Subject: Re: [PATCH 0/4] ASoC: fsl_asrc: allow selecting arbitrary clocks
Date: Wed, 15 Jul 2020 14:03:08 -0700	[thread overview]
Message-ID: <20200715210308.GA14589@Asurada-Nvidia> (raw)
In-Reply-To: <20200715140519.GH5431@sirena.org.uk>

On Wed, Jul 15, 2020 at 03:05:19PM +0100, Mark Brown wrote:
> On Tue, Jul 14, 2020 at 01:50:50PM -0700, Nicolin Chen wrote:
> > On Tue, Jul 14, 2020 at 09:27:53PM +0100, Mark Brown wrote:
> 
> > > The nominal rate might be the same but if they're in different clock
> > > domains then the actual rates might be different (hence the desire for
> > > an ASRC I guess).  I can see the system wanting to choose one clock or
> > > the other on the basis of some system specific property (quality of the
> > > clock sources, tolerances of the devices involved or something) though
> > > it's a rather fun edge case configuration :/ .
> 
> > Thanks for the input. Fox i.MX6, I don't feel it would be that
> > drastically different though. And both SSI1 and SSI2 can simply
> > select the same root clock source to avoid that happen.
> 
> If you've got two radios that both need to sync to some radio derived
> frequency it gets a bit more entertaining.

I'm simply curious what could be a problem. Do you mind educating
me a bit? And ASRC here isn't a radio but a sample rate converter
working as a BE in DPCM setup, using radio-capture for example...

WARNING: multiple messages have this Message-ID (diff)
From: Nicolin Chen <nicoleotsuka@gmail.com>
To: Mark Brown <broonie@kernel.org>
Cc: Arnaud Ferraris <arnaud.ferraris@collabora.com>,
	alsa-devel@alsa-project.org, Timur Tabi <timur@kernel.org>,
	Xiubo Li <Xiubo.Lee@gmail.com>,
	linux-kernel@vger.kernel.org, Takashi Iwai <tiwai@suse.com>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Rob Herring <robh+dt@kernel.org>,
	kernel@collabora.com, Fabio Estevam <festevam@gmail.com>
Subject: Re: [PATCH 0/4] ASoC: fsl_asrc: allow selecting arbitrary clocks
Date: Wed, 15 Jul 2020 14:03:08 -0700	[thread overview]
Message-ID: <20200715210308.GA14589@Asurada-Nvidia> (raw)
In-Reply-To: <20200715140519.GH5431@sirena.org.uk>

On Wed, Jul 15, 2020 at 03:05:19PM +0100, Mark Brown wrote:
> On Tue, Jul 14, 2020 at 01:50:50PM -0700, Nicolin Chen wrote:
> > On Tue, Jul 14, 2020 at 09:27:53PM +0100, Mark Brown wrote:
> 
> > > The nominal rate might be the same but if they're in different clock
> > > domains then the actual rates might be different (hence the desire for
> > > an ASRC I guess).  I can see the system wanting to choose one clock or
> > > the other on the basis of some system specific property (quality of the
> > > clock sources, tolerances of the devices involved or something) though
> > > it's a rather fun edge case configuration :/ .
> 
> > Thanks for the input. Fox i.MX6, I don't feel it would be that
> > drastically different though. And both SSI1 and SSI2 can simply
> > select the same root clock source to avoid that happen.
> 
> If you've got two radios that both need to sync to some radio derived
> frequency it gets a bit more entertaining.

I'm simply curious what could be a problem. Do you mind educating
me a bit? And ASRC here isn't a radio but a sample rate converter
working as a BE in DPCM setup, using radio-capture for example...

  parent reply	other threads:[~2020-07-15 21:04 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-02 14:22 [PATCH 0/4] ASoC: fsl_asrc: allow selecting arbitrary clocks Arnaud Ferraris
2020-07-02 14:22 ` Arnaud Ferraris
2020-07-02 14:22 ` Arnaud Ferraris
2020-07-02 14:22 ` [PATCH 1/4] dt-bindings: sound: fsl, asrc: add properties to select in/out clocks Arnaud Ferraris
2020-07-02 14:22   ` [PATCH 1/4] dt-bindings: sound: fsl,asrc: " Arnaud Ferraris
2020-07-02 14:22   ` [PATCH 1/4] dt-bindings: sound: fsl, asrc: " Arnaud Ferraris
2020-07-02 14:22 ` [PATCH 2/4] ASoC: fsl_asrc: allow using arbitrary input and output clocks Arnaud Ferraris
2020-07-02 14:22   ` Arnaud Ferraris
2020-07-02 14:22   ` Arnaud Ferraris
2020-07-02 17:42   ` kernel test robot
2020-07-02 21:33   ` kernel test robot
2020-07-02 14:22 ` [PATCH 3/4] ASoC: fsl_asrc: always use ratio for conversion Arnaud Ferraris
2020-07-02 14:22   ` Arnaud Ferraris
2020-07-02 14:22   ` Arnaud Ferraris
2020-07-02 14:22 ` [PATCH 4/4] ASoC: fsl_asrc: swap input and output clocks in capture mode Arnaud Ferraris
2020-07-02 14:22   ` Arnaud Ferraris
2020-07-02 14:22   ` Arnaud Ferraris
2020-07-02 19:34   ` kernel test robot
2020-07-02 18:42 ` [PATCH 0/4] ASoC: fsl_asrc: allow selecting arbitrary clocks Nicolin Chen
2020-07-02 18:42   ` Nicolin Chen
2020-07-02 18:42   ` Nicolin Chen
2020-07-03  9:38   ` Arnaud Ferraris
2020-07-03  9:38     ` Arnaud Ferraris
2020-07-14 16:20     ` Arnaud Ferraris
2020-07-14 16:20       ` Arnaud Ferraris
2020-07-14 20:15       ` Nicolin Chen
2020-07-14 20:15         ` Nicolin Chen
2020-07-14 20:27         ` Mark Brown
2020-07-14 20:27           ` Mark Brown
2020-07-14 20:50           ` Nicolin Chen
2020-07-14 20:50             ` Nicolin Chen
2020-07-15 14:05             ` Mark Brown
2020-07-15 14:05               ` Mark Brown
2020-07-15 16:18               ` Arnaud Ferraris
2020-07-15 16:18                 ` Arnaud Ferraris
2020-07-15 16:22                 ` Mark Brown
2020-07-15 16:22                   ` Mark Brown
2020-07-15 16:32                   ` Arnaud Ferraris
2020-07-15 16:32                     ` Arnaud Ferraris
2020-07-15 20:46                 ` Nicolin Chen
2020-07-15 20:46                   ` Nicolin Chen
2020-07-16  9:54                   ` Arnaud Ferraris
2020-07-16  9:54                     ` Arnaud Ferraris
2020-07-15 21:03               ` Nicolin Chen [this message]
2020-07-15 21:03                 ` Nicolin Chen
2020-07-16 12:18                 ` Mark Brown
2020-07-16 12:18                   ` Mark Brown
2020-07-16 14:26                   ` Arnaud Ferraris
2020-07-16 14:26                     ` Arnaud Ferraris
2020-07-17 11:16   ` Arnaud Ferraris
2020-07-17 11:16     ` Arnaud Ferraris
2020-07-17 11:16     ` Arnaud Ferraris
2020-07-23  5:46     ` Nicolin Chen
2020-07-23  5:46       ` Nicolin Chen
2020-07-23  5:46       ` Nicolin Chen

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=20200715210308.GA14589@Asurada-Nvidia \
    --to=nicoleotsuka@gmail.com \
    --cc=Xiubo.Lee@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=arnaud.ferraris@collabora.com \
    --cc=broonie@kernel.org \
    --cc=festevam@gmail.com \
    --cc=kernel@collabora.com \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robh+dt@kernel.org \
    --cc=timur@kernel.org \
    --cc=tiwai@suse.com \
    /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.