linux-kernel - Re: [PATCH] ASoC: soc-core: add snd_soc_of_parse_sysclk

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20140107162447.GX31886@sirena.org.uk>
Date:	Tue, 7 Jan 2014 16:24:47 +0000
From:	Mark Brown <broonie@...nel.org>
To:	Xiubo Li <Li.Xiubo@...escale.com>
Cc:	lgirdwood@...il.com, perex@...ex.cz, tiwai@...e.de,
	alsa-devel@...a-project.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ASoC: soc-core: add snd_soc_of_parse_sysclk_id() for dt

On Thu, Jan 02, 2014 at 03:34:17PM +0800, Xiubo Li wrote:
> This patch adds snd_soc_of_parse_sysclk_id() and supports below
> style on dt:
> 
> sysclk-id = <XXX>;
> 
> It's an optional property, If this property is absent the defualt
> vaule 0 will be returned.

The trick with this is defining what the clock should be.  Drivers would
have to start defining their clock IDs and then at some point we end up
reinventing the common clock API and bindings.  We really ought to be
moving ASoC's clock handling over towards clk but there's issues with
that given that the clock API is not available at all on some platforms
and the common clock API isn't common.  This is rather a larger bit of
work than I expect you want to do right now though...

What's your use case for this - is it possible to make the driver you're
looking at default to a clock that'll work for most systems?  That'd
sidestep the need to say which clock.  Alternatively is the CODEC only
likely to be used with systems supporting common clk in which case doing
the sysclk configuration via the clk API might be the way forwards?

Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)