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

2015-02-10 Thread Krzysztof Kozlowski
2015-02-06 21:50 GMT+01:00 Javier Martinez Canillas javier.marti...@collabora.co.uk: Hello Sergei, Thanks a lot for your feedback. On 02/06/2015 08:09 PM, Sergei Shtylyov wrote: Hello. On 02/06/2015 08:37 PM, Javier Martinez Canillas wrote: All the device nodes for the Exynos5420

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

2015-02-10 Thread Sergei Shtylyov
Hello. On 2/10/2015 2:46 PM, Krzysztof Kozlowski wrote: All the device nodes for the Exynos5420 power-domains have a quite generic power-domain name. And this is in conformance to the ePAPR standard. True, I forgot that the ePAPR recommends that the node names should be somewhat

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

2015-02-10 Thread Krzysztof Kozlowski
On wto, 2015-02-10 at 14:00 +0100, Sylwester Nawrocki wrote: On 10/02/15 13:46, Javier Martinez Canillas wrote: 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

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

2015-02-10 Thread Krzysztof Kozlowski
On wto, 2015-02-10 at 14:55 +0300, Sergei Shtylyov wrote: Hello. On 2/10/2015 2:46 PM, Krzysztof Kozlowski wrote: Additionally (on Arndale Octa): $ cat /sys/kernel/debug/pm_genpd/pm_genpd_summary domain status slaves /device

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

2015-02-10 Thread Krzysztof Kozlowski
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

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

2015-02-10 Thread Javier Martinez Canillas
Hello, 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

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

2015-02-10 Thread Sergei Shtylyov
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

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

2015-02-10 Thread Sylwester Nawrocki
On 10/02/15 13:46, Javier Martinez Canillas wrote: 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

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

2015-02-06 Thread Javier Martinez Canillas
Hello Sergei, Thanks a lot for your feedback. On 02/06/2015 08:09 PM, Sergei Shtylyov wrote: Hello. On 02/06/2015 08:37 PM, Javier Martinez Canillas wrote: All the device nodes for the Exynos5420 power-domains have a quite generic power-domain name. And this is in conformance to

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

2015-02-06 Thread Sergei Shtylyov
Hello. On 02/06/2015 08:37 PM, Javier Martinez Canillas wrote: All the device nodes for the Exynos5420 power-domains have a quite generic power-domain name. And this is in conformance to the ePAPR standard. So in case of an error, the Exynos PD driver shows the following (not very

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

2015-02-06 Thread Javier Martinez Canillas
All the device nodes for the Exynos5420 power-domains have a quite generic power-domain name. So in case of an error, the Exynos PD driver shows the following (not very useful) message: Power domain power-domain disable failed Use descriptive names to know on which PD enable or disable failed.