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 >
