On Wed, Aug 07, 2013 at 02:40:15PM +0530, Padmavathi Venna wrote: > This patch registers the 16MHz oscillator clock as fixed clk.
> +/* 16.9MHz fixed oscillator clock */
> +static void init_osc_clock(void)
> +{
> + struct device_node *np;
> +
> + np = of_find_compatible_node(NULL, NULL, "osc3_clk16mhz");
> + of_fixed_clk_setup(np);
> +}
> +
> static int smdk_hw_params(struct snd_pcm_substream *substream,
> struct snd_pcm_hw_params *params)
> {
> @@ -173,6 +183,8 @@ static int smdk_audio_probe(struct platform_device *pdev)
> smdk_dai[0].platform_of_node = smdk_dai[0].cpu_of_node;
> }
>
> + init_osc_clock();
> +
This doesn't seem great - it means that this machine driver needs to
know about a specifically named fixed clock which makes it hard to reuse
on other similar boards. For example I'm intending to reuse this on
Arndale. Mike?
signature.asc
Description: Digital signature
