Hi Geert,

On 05/29/2017 11:12 AM, Geert Uytterhoeven wrote:

Hi Thara,

On Fri, May 26, 2017 at 1:51 AM, Thara Gopinath
[off-list ref] wrote:

quoted

This patch extends the existing generic power domain debugfs.
Changes involve the following
- Remove the current flat format of displaying generic power domain
  summary.
- Introduce a unique debugfs entry for each generic power domain with the
  following attributes
        - current_state - Displays current state of the domain.
        - devices - Displays the devices associated with this domain.
        - slaves - Displays the sub power domains.
        - on_time - Displays the time the domain was in on state in ms.
        - idle_time - Displays the time the domain was in any of the idle
                        states in ms.
        - idle_states - Displays the various idle states and the time
                        spent in each idle state in ms.

Signed-off-by: Thara Gopinath <redacted>

While I don't doubt the merits of having all this additional information in
separate debugfs files, I think it would be a good idea to keep the existing
summary.

Since you have a need for the summary, I will keep it intact and post a
new version of the patches.

Cfr. the clock subsystem, which provides both a summary, and clock-specific
information in individual files.

BTW, if people disagree, I'll have to write a script to extract all information
and combine it into a single file ;-) I tend to compare these summaries across
boots...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

-- 
Regards
Thara