On Sat, Jul 19, 2014 at 1:31 AM, Arnd Bergmann [off-list ref] wrote:

On Saturday 19 July 2014 01:11:53 Chanwoo Choi wrote:

quoted

  Exynos3250/Exynos4/Exynos5 has 'adc' clock as following:
  - 'adc' clock: bus clock for ADC

  Exynos3250 has additional 'sclk_adc' clock as following:
  - 'sclk_adc' clock: special clock for ADC which provide clock to internal ADC

The bus clock is often named "apb_pclk", but it's too late to change
that now, and it's not clear if the device is actually on APB.

The problem with "sclk_adc" is that it's very close to the name used
in the clock provider, with is CLK_SCLK_TSADC. Please try to avoid
the ambiguity here and make it clear that they are not referring
to the same thing. "internal" or "special" might be good if you want
to avoid "sclk".

OK, thanks for your feedback.
I'll try to find proper name.

Chanwoo Choi