[PATCH V4 2/4] clk: samsung: register audio subsystem clocks using common clock framework
Andrew Bresticker
abrestic at chromium.org
Mon Jun 3 19:46:09 EDT 2013
>> +static unsigned long reg_save[][2] = {
>> + {ASS_CLK_SRC, 0},
>> + {ASS_CLK_DIV, 0},
>> + {ASS_CLK_GATE, 0},
>> +};
>> +
>> +/* list of all parent clock list */
>> +static const char *mout_audss_p[] = { "fin_pll", "fout_epll" };
>
> I think this is supposed to be "xxti" which might or might not be
> "fin_pll". In the exynos4 code these are two different clocks that
> are chosen by OM[0]. I'd bet that 99% of the time they are the same,
> but it seems better to use "xxti".
OM[0] also appears to mux the input here between "xxti" and "xusbxti"
as well. So this should probably remain "fin_pll".
> At the moment the 5250 code doesn't expose "xxti". It probably
> should. Andrew (CCed) is looking at this.
Yes, xxti is used directly elsewhere (SPI and UART, from what I can
tell), but not here.
-Andrew
More information about the linux-arm-kernel
mailing list