| Message ID | 20210417031252.3020837-1-david.e.box@linux.intel.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | intel_pmc_core: Add sub-state requirements and mode | expand |
Hi, On 4/17/21 5:12 AM, David E. Box wrote: > - Patch 1 and 2 remove the use of the global struct pmc_dev > - Patches 3-7 add support for reading low power mode sub-state > requirements, latching sub-state status on different low power mode > events, and displaying the sub-state residency in microseconds > - Patch 8 adds missing LTR IPs for TGL > - Patch 9 adds support for ADL-P which is based on TGL > > Applied on top of latest hans-review/review-hans Thank you, the series looks good to me know, except for one very minor issue in patch 5, which as I already mentioned in a reply to patch 5, I can fixup while merging this. Once I have you ack for the prososed changes to patch 5 I'll merge this into me review-hans branch. Depending on if Linus does a rc8 or not I'll then push this to for-next for 5.13, or this will have to wait for 5.14 : Linus does a rc8 -> I'll promote this from review-hans to for-next in time for 5.13 Linus releases 5.12 final -> I'll rebase my review-hans on top of 5.13-rc1 once released and then push this to for-next Regards, Hans > > Patches that changed in V2: > Patch 3: Variable name change > Patch 5: Do proper cleanup after fail > Patch 7: Debugfs write function fixes > > David E. Box (4): > platform/x86: intel_pmc_core: Don't use global pmcdev in quirks > platform/x86: intel_pmc_core: Remove global struct pmc_dev > platform/x86: intel_pmc_core: Add option to set/clear LPM mode > platform/x86: intel_pmc_core: Add support for Alder Lake PCH-P > > Gayatri Kammela (5): > platform/x86: intel_pmc_core: Handle sub-states generically > platform/x86: intel_pmc_core: Show LPM residency in microseconds > platform/x86: intel_pmc_core: Get LPM requirements for Tiger Lake > platform/x86: intel_pmc_core: Add requirements file to debugfs > platform/x86: intel_pmc_core: Add LTR registers for Tiger Lake > > drivers/platform/x86/intel_pmc_core.c | 384 +++++++++++++++++++++++--- > drivers/platform/x86/intel_pmc_core.h | 47 +++- > 2 files changed, 395 insertions(+), 36 deletions(-) > > > base-commit: 823b31517ad3196324322804ee365d5fcff704d6 >
Hi, On 4/17/21 5:12 AM, David E. Box wrote: > - Patch 1 and 2 remove the use of the global struct pmc_dev > - Patches 3-7 add support for reading low power mode sub-state > requirements, latching sub-state status on different low power mode > events, and displaying the sub-state residency in microseconds > - Patch 8 adds missing LTR IPs for TGL > - Patch 9 adds support for ADL-P which is based on TGL > > Applied on top of latest hans-review/review-hans Thank you for your patch-series, I've applied the series to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans Note it will show up in my review-hans branch once I've pushed my local branch there, which might take a while. Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > Patches that changed in V2: > Patch 3: Variable name change > Patch 5: Do proper cleanup after fail > Patch 7: Debugfs write function fixes > > David E. Box (4): > platform/x86: intel_pmc_core: Don't use global pmcdev in quirks > platform/x86: intel_pmc_core: Remove global struct pmc_dev > platform/x86: intel_pmc_core: Add option to set/clear LPM mode > platform/x86: intel_pmc_core: Add support for Alder Lake PCH-P > > Gayatri Kammela (5): > platform/x86: intel_pmc_core: Handle sub-states generically > platform/x86: intel_pmc_core: Show LPM residency in microseconds > platform/x86: intel_pmc_core: Get LPM requirements for Tiger Lake > platform/x86: intel_pmc_core: Add requirements file to debugfs > platform/x86: intel_pmc_core: Add LTR registers for Tiger Lake > > drivers/platform/x86/intel_pmc_core.c | 384 +++++++++++++++++++++++--- > drivers/platform/x86/intel_pmc_core.h | 47 +++- > 2 files changed, 395 insertions(+), 36 deletions(-) > > > base-commit: 823b31517ad3196324322804ee365d5fcff704d6 >