Re: [PATCH 1/1] ARM: dts: Use more descriptive names for Exynos5420 PDs

From: Javier Martinez Canillas
Date: Tue Feb 10 2015 - 07:46:25 EST


On 02/10/2015 01:30 PM, Krzysztof Kozlowski wrote:
> On wto, 2015-02-10 at 15:21 +0300, Sergei Shtylyov wrote:
>> On 2/10/2015 3:17 PM, Krzysztof Kozlowski wrote:
>> >>> Additionally (on Arndale Octa):
>> >>> $ cat /sys/kernel/debug/pm_genpd/pm_genpd_summary
>> >>> domain status slaves
>> >>> /device runtime status
>> >>> ----------------------------------------------------------------------
>> >>> power-domain on
>> >>> /devices/platform/amba/3880000.adma suspended
>> >>> power-domain off
>> >>> power-domain off
>> >>> power-domain off
>> >>> power-domain off
>> >>> power-domain off
>> >>> This really is not helpful. From the power domain debugfs code it is
>> >>> complicated to extract of_node of power domain.

Not very useful indeed.

>> >> You shouldn't need it.
>> >>> It is easier to print
>> >>> the name of power domain. But wait... all names are the same! :) So
>> >>> why do we have the name in the first place?
>> >> I'm not sure why the full platform device names aren't printed -- they
>> >> should all be different.

Yes, but like I said in a previous email the fact that are different doesn't
necessarily mean that they will be helpful for debugging purposes.

>> > This debugfs code iterates over list of generic_pm_domains (gpd_list). I
>> > cannot find function for translating from genpd to its platform device
>> > so only genpd->name can be printed.
>> Then why power domains aren't just named with the platform device names?
> Right, the mach-exynos/pm_domains.c set the name equal to OF node name.
> I'll send a patch extending the name.

IIRC the OF core uses the device node unit address and node name to create
the platform device names so you will have something like 10044000.power-domain.

Same if using the node full_name since it will /power-domain@10044000. In both
cases the DTS should have to be checked to know which power domain really is
unless someone knows by heart the power domains addresses.

But if using generic names for the power domains as suggested by ePAPR is so
important then we should change all the other Exynos DTS files which don't do.

> Best regards,
> Krzysztof

Best regards,
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at