public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Sylwester Nawrocki <s.nawrocki@samsung.com>
To: Javier Martinez Canillas <javier.martinez@collabora.co.uk>,
	Krzysztof Kozlowski <k.kozlowski@samsung.com>
Cc: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>,
	Kukjin Kim <kgene@kernel.org>,
	Andrzej Hajda <a.hajda@samsung.com>,
	linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	Marek Szyprowski <m.szyprowski@samsung.com>
Subject: Re: [PATCH 1/1] ARM: dts: Use more descriptive names for Exynos5420 PDs
Date: Tue, 10 Feb 2015 14:00:55 +0100	[thread overview]
Message-ID: <54DA0107.2020909@samsung.com> (raw)
In-Reply-To: <54D9FD94.9000102@collabora.co.uk>

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 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.

Perhaps we could assign OF aliases to the power domain device nodes in DT
and then in the power domains driver map those aliases to more descriptive
names when creating the power domains?

--
Regards,
Sylwester

  reply	other threads:[~2015-02-10 13:01 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-06 17:37 [PATCH 1/1] ARM: dts: Use more descriptive names for Exynos5420 PDs Javier Martinez Canillas
2015-02-06 19:09 ` Sergei Shtylyov
2015-02-06 20:50   ` Javier Martinez Canillas
2015-02-10 11:46     ` Krzysztof Kozlowski
2015-02-10 11:55       ` Sergei Shtylyov
2015-02-10 12:17         ` Krzysztof Kozlowski
2015-02-10 12:21           ` Sergei Shtylyov
2015-02-10 12:30             ` Krzysztof Kozlowski
2015-02-10 12:46               ` Javier Martinez Canillas
2015-02-10 13:00                 ` Sylwester Nawrocki [this message]
2015-02-10 13:14                   ` Krzysztof Kozlowski
2015-02-10 14:08                     ` Sylwester Nawrocki

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54DA0107.2020909@samsung.com \
    --to=s.nawrocki@samsung.com \
    --cc=a.hajda@samsung.com \
    --cc=javier.martinez@collabora.co.uk \
    --cc=k.kozlowski@samsung.com \
    --cc=kgene@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    --cc=sergei.shtylyov@cogentembedded.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox