Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755542AbbBJMqW (ORCPT ); Tue, 10 Feb 2015 07:46:22 -0500 Received: from bhuna.collabora.co.uk ([93.93.135.160]:42640 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752354AbbBJMqU (ORCPT ); Tue, 10 Feb 2015 07:46:20 -0500 Message-ID: <54D9FD94.9000102@collabora.co.uk> Date: Tue, 10 Feb 2015 13:46:12 +0100 From: Javier Martinez Canillas User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Icedove/31.2.0 MIME-Version: 1.0 To: Krzysztof Kozlowski , Sergei Shtylyov CC: Kukjin Kim , Andrzej Hajda , linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Marek Szyprowski Subject: Re: [PATCH 1/1] ARM: dts: Use more descriptive names for Exynos5420 PDs References: <1423244258-24314-1-git-send-email-javier.martinez@collabora.co.uk> <54D51170.10903@cogentembedded.com> <54D52905.6040100@collabora.co.uk> <54D9F1A4.1000908@cogentembedded.com> <1423570670.6241.2.camel@AMDC1943> <54D9F7B1.3030106@cogentembedded.com> <1423571438.6241.4.camel@AMDC1943> In-Reply-To: <1423571438.6241.4.camel@AMDC1943> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2684 Lines: 71 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 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, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/