Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1470045

Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external MCLKn clocks

From Sylwester Nawrocki <s.nawrocki@samsung.com>
Newsgroups linux.kernel
Subject Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external MCLKn clocks
Date 2016-08-25 12:40 +0200
Message-ID <sa0eK-8eB-17@gated-at.bofh.it> (permalink)
References <s7VCx-8rz-15@gated-at.bofh.it> <s8TIl-4sY-3@gated-at.bofh.it> <s91cS-Nb-35@gated-at.bofh.it> <s9mKl-6om-21@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 08/23/2016 06:28 PM, Charles Keepax wrote:
> Apologies it seems I missed another version of this on the
> mailing list, please do feel free to add me or the
> patches@opensource.wolfsonmicro.com list as a direct CC on
> any patches using our CODECs, I am always more than happy to look
> over things and feel bad when I miss stuff. I will have a look
> at the latest version of the patch.

Actually I should have copied you on the last iteration after
addressing your review comments, sorry about this omission.

> I think we should be able to do something requesting the 32k
> approximately as your existing patch here does, but then
> requesting the main MCLK from the set_pll and set_sysclk
> handlers. Eventually I would like the internal SYSCLK and FLLs
> represented in the clock framework, so I want to have a quick
> think about how that would migrate over. Let me see what I can
> come up with here.

Sounds good, I assume you mean enabling/disabling from the set_pll
and set_syclk handlers. Do you find any issues with that? I think
the CODEC should ensure external MCLK is enabled when setting FLL,
otherwise it will not lock?
Presumably arizona_set_fll() and arizona_set_sysclk() are places
where it should be checked what the root source clock is and enable
it if it's not yet enabled?

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[RFC PATCH] mfd: arizona: Add gating of external MCLKn clocks Sylwester Nawrocki <s.nawrocki@samsung.com> - 2016-08-19 19:20 +0200
  Re: [RFC PATCH] mfd: arizona: Add gating of external MCLKn clocks Charles Keepax <ckeepax@opensource.wolfsonmicro.com> - 2016-08-22 11:30 +0200
    Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external  MCLKn clocks Sylwester Nawrocki <s.nawrocki@samsung.com> - 2016-08-22 19:30 +0200
      Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external  MCLKn clocks Mark Brown <broonie@kernel.org> - 2016-08-22 19:50 +0200
        Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external  MCLKn clocks Sylwester Nawrocki <s.nawrocki@samsung.com> - 2016-08-23 18:50 +0200
      Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external  MCLKn clocks Charles Keepax <ckeepax@opensource.wolfsonmicro.com> - 2016-08-23 18:30 +0200
        Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external  MCLKn clocks Sylwester Nawrocki <s.nawrocki@samsung.com> - 2016-08-25 12:40 +0200
          Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external  MCLKn clocks Charles Keepax <ckeepax@opensource.wolfsonmicro.com> - 2016-08-25 15:40 +0200
            Re: [alsa-devel] [RFC PATCH] mfd: arizona: Add gating of external  MCLKn clocks Sylwester Nawrocki <s.nawrocki@samsung.com> - 2016-08-25 15:50 +0200

csiph-web