RE: [PATCH v1 0/7] platform/x86: intel_pmc_core: Add debugfs entries

[Kammela, Gayatri]

> -----Original Message-----
> From: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
> Sent: Monday, January 27, 2020 2:49 AM
> To: Kammela, Gayatri <gayatri.kammela@xxxxxxxxx>
> Cc: Platform Driver <platform-driver-x86@xxxxxxxxxxxxxxx>; Linux Kernel
> Mailing List <linux-kernel@xxxxxxxxxxxxxxx>; Somayaji, Vishwanath
> <vishwanath.somayaji@xxxxxxxxx>; Darren Hart <dvhart@xxxxxxxxxxxxx>;
> Westerberg, Mika <mika.westerberg@xxxxxxxxx>; Peter Zijlstra (Intel)
> <peterz@xxxxxxxxxxxxx>; Prestopine, Charles D
> <charles.d.prestopine@xxxxxxxxx>; Pandruvada, Srinivas
> <srinivas.pandruvada@xxxxxxxxx>; Andy Shevchenko
> <andriy.shevchenko@xxxxxxxxxxxxxxx>; Box, David E
> <david.e.box@xxxxxxxxx>
> Subject: Re: [PATCH v1 0/7] platform/x86: intel_pmc_core: Add debugfs
> entries
> 
> On Fri, Jan 24, 2020 at 10:59 PM Gayatri Kammela
> <gayatri.kammela@xxxxxxxxx> wrote:
> >
> > Hi,
> >
> > Tiger Lake platform supports 8 sub-states/low power modes of S0ix. Add
> > debugfs entries to pmc_core driver to access low power mode registers
> > and residencies.
> >
> > Patch 1: Add debugfs entry to access sub-state residencies Patch 2:
> > Add debugfs entry for low power mode status registers Patch 3:
> > Refactor the driver by removing redundant code Patch 4: Remove slp_s0
> > attributes from tgl_reg_map Patch 5: Add an additional parameter to
> > pmc_core_lpm_display_map() Patch 6: Dump low power status registers on
> > an S0ix.y failure Patch 7: Add debugfs support to access live status
> > registers
> 
> It doesn't apply to our for-next branch.
Sorry to hear that. I've rebased the series on mainline 5.5-rc7. Should I rebase it on for-next branch?
> 
> >
> > Gayatri Kammela (7):
> >   platform/x86: intel_pmc_core: Add debugfs entry to access sub-state
> >     residencies
> >   platform/x86: intel_pmc_core: Add debugfs entry for low power mode
> >     status registers
> >   platform/x86: intel_pmc_core: Refactor the driver by removing
> >     redundant code
> >   platform/x86: intel_pmc_core: Remove slp_s0 attributes from
> >     tgl_reg_map
> >   platform/x86: intel_pmc_core: Add an additional parameter to
> >     pmc_core_lpm_display()
> >   platform/x86: intel_pmc_core: Dump low power status registers on an
> >     S0ix.y failure
> >   platform/x86: intel_pmc_core: Add debugfs support to access live
> >     status registers
> >
> >  drivers/platform/x86/intel_pmc_core.c | 301
> > +++++++++++++++++++++++---  drivers/platform/x86/intel_pmc_core.h |
> > 28 +++
> >  2 files changed, 302 insertions(+), 27 deletions(-)
> >
> > Cc: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxx>
> > Cc: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > Cc: David Box <david.e.box@xxxxxxxxx>
> > --
> > 2.17.1
> >
> 
> 
> --
> With Best Regards,
> Andy Shevchenko