On Sun, Jan 12, 2014 at 08:34:10PM +0100, Tomasz Figa wrote:

On 12.01.2014 20:29, Mark Brown wrote:

quoted

On Sat, Jan 11, 2014 at 08:42:48PM +0100, Tomasz Figa wrote:

quoted

quoted

This patch adds support for registering power domains of S3C64xx SoCs
and binding devices to them using device tree.

quoted

quoted

+#ifdef CONFIG_OF
+static struct of_device_id s3c64xx_pd_matches[] = {
+	{ .compatible = "samsung,s3c6400-clock", },
+	{ .compatible = "samsung,s3c6410-clock", },
+	{ },
 };

quoted

Looks like a rebasing issue?  Otherwise this looks nice.

Hmm, what's the problem here? Maybe I'm just blind, but I fail to
see anything wrong (except that the array should be const...).

Thanks for reviewing this.

I wasn't expecting to see references to clocks come up in power domain
code.