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

Groups > linux.kernel > #1381850

Re: [alsa-devel] [PATCH 4/4] ASoC: simple-card: Support for selecting system clocks by ID

Show all headers | View raw

On 02/17/2016 09:52 PM, Peter Ujfalusi wrote:
> On 02/17/2016 02:07 PM, Mark Brown wrote:
>> On Wed, Feb 17, 2016 at 10:13:35AM +0200, Peter Ujfalusi wrote:
>>
>>> With this change we don't need to write custom machine drivers for setup not
>>> using sysclk_id == 0.
>>> I do think this is reasonable change by itself.
>>
>>> However I do think that the current simple-card is flawed regarding to clock
>>> selection and the change Jyri and me are proposing is reasonable.
>>
>> But you define a new ABI to specify it in the process, I'd rather fix
>> the flaws by using the common clock ABI than extend any device stuff.
>> If it didn't define a new ABI I'd probably not worry about it but one of
>> the issues we have with DT is that we do end up making ABIs every time
>> we put something in DT.
> 
> I understand. This should have been supported by simple-card since the
> beginning. When we tried to move all boards to use simple-card, we hit the
> wall by not being able to select and configure other sysclk_id than 0.
> I don't want to create yet another simple card which handles only sysclk_id 1 ;)
> On the other hand this ABI is backwards compatible since if it is missing it
> will default to the configuration we right now have regarding to sysclk_dir
> and sysclk_id.
> 
> I will look at the CCF implementation for McASP first then for aic3x.

The first issue with converting the McASP to use CCF internally for clock
selection, muxing and rate configuration is that the daVinci platform does not
use CCF at all. Given that the davinci-mcasp driver is used by daVinci, we
need to have non CCF way supported in ASoC...

-- 
Péter

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

Thread

Re: [alsa-devel] [PATCH 4/4] ASoC: simple-card: Support for selecting  system clocks by ID Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-04-18 18:00 +0200
  Re: [alsa-devel] [PATCH 4/4] ASoC: simple-card: Support for  selecting system clocks by ID Mark Brown <broonie@kernel.org> - 2016-04-18 18:30 +0200
    Re: [alsa-devel] [PATCH 4/4] ASoC: simple-card: Support for  selecting system clocks by ID Stephen Boyd <sboyd@codeaurora.org> - 2016-04-22 00:30 +0200
      Re: [alsa-devel] [PATCH 4/4] ASoC: simple-card: Support for selecting  system clocks by ID Peter Ujfalusi <peter.ujfalusi@ti.com> - 2016-04-22 14:00 +0200
        Re: [alsa-devel] [PATCH 4/4] ASoC: simple-card: Support for selecting  system clocks by ID Tero Kristo <t-kristo@ti.com> - 2016-04-22 14:10 +0200

csiph-web