On 24.06.2014 13:28, Pankaj Dubey wrote:

On Tuesday, June 17 2014, Tomasz Figa wrote:

quoted

On 10.05.2014 08:56, Pankaj Dubey wrote:

[snip]

quoted

quoted

+
+	ret = platform_driver_register(&exynos_pmu_driver);
+	if (ret < 0)
+		goto out;
+
+	exynos_pmu_pdev = platform_device_register_simple("exynos-pmu", -1,
+			NULL, 0);

Hmm, I don't see the point of making this a platform driver only to

register respective

quoted

platform device few lines below. If you take into account the patch for

syscon I

quoted

posted as a reply to patch 06/11, you will be able to make this a normal

platform

quoted

driver that binds to DT node directly and then register a syscon in probe

function

quoted

above.

Well I updated PMU to be a syscon provider using your patch. It worked well.
I have following two points to mention here:

1: For making PMU a normal platform driver, other than making PMU a syscon
provider we need to change PMU's DT binding. Basically we need to remove
"syscon"
as second compatibility string. I hope this should not be an issue.

I don't see the reason for this. Could you elaborate on this?

I know that the way Linux currently handles multiple compatible strings
is broken, but despite this, if you just make sure your platform driver
registers before syscon platform driver, it should be fine.

2: I can see for your syscon patch Arnd has some comments, hopefully we can
address
his concern.

Yes. I will get to it after sorting out few other things from my queue.
Or you can do it if you want - the purpose of that RFC patch was just to
show the idea I had in my mind.

Best regards,
Tomasz