linux-kernel - Re: [PATCH v1 0/8] Redesign the pmc core driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2a3f8cc5-4c6e-7bb9-5a09-1dc20929271e@redhat.com>
Date:   Mon, 21 Nov 2022 10:44:32 +0100
From:   Hans de Goede <hdegoede@...hat.com>
To:     Gayatri Kammela <gayatri.kammela@...ux.intel.com>
Cc:     irenic.rajneesh@...il.com, markgross@...nel.org,
        linux-kernel@...r.kernel.org, platform-driver-x86@...r.kernel.org,
        sukumar.ghorai@...el.com, xi.pardee@...el.com,
        rajvi.jingar@...el.com,
        Rajvi Jingar <rajvi.jingar@...ux.intel.com>,
        David E Box <david.e.box@...ux.intel.com>
Subject: Re: [PATCH v1 0/8] Redesign the pmc core driver

Hi,

On 11/14/22 19:32, Gayatri Kammela wrote:
> This patch series focuses on redesigning the PMC core driver for
> maintainability and readability. Moving PCH specific structures and
> functions to separate c files, simplifies core.c file and makes it
> easier to add new platforms in future. The series includes two
> preparatory patches that lays the ground work for the redesign. The
> patch series also adds legacy support for Meteor Lake.
> 
> This redesign adds following c files to support different PCHs. There
> are no functional changes involved for the already supported platforms.
> 
> spt.c: Sunrise Point PCH supports: Sky Lake, Sky Lake L, Kaby Lake,
> Kaby Lake L
> cnp.c: Cannon Lake Point PCH supports: Cannon Lake L, Comet Lake,
> Comet Lake L
> icl.c: Ice Lake PCH supports: Ice Lake L, Ice Lake NNPI, Jasper Lake
> tgl.c: Tiger Lake PCH supports: Tiger Lake, Tiger Lake L, Alder Lake L,
> Alder Lake N, Rocket Lake, Raptor Lake P, Elkhart Lake
> adl.c: Alder Lake PCH supports: Alder Lake, Raptor Lake, Raptor Lake S
> mtl.c: Meteor Lake PCH supports: Meteor Lake
> 
> Patch 1: platform/x86: intel/pmc: Replace all the reg_map with init
> functions
> Patch 2: platform/x86: intel/pmc: Move variable declarations and
> definitions to header and core.c
> Patch 3: platform/x86: intel/pmc: Relocate Sunrise Point PCH support
> Patch 4: platform/x86: intel/pmc: Relocate Cannon Lake Point PCH
> support
> Patch 5: platform/x86: intel/pmc: Relocate Ice Lake PCH support
> Patch 6: platform/x86: intel/pmc: Relocate Tiger Lake PCH support
> Patch 7: platform/x86: intel/pmc: Relocate Alder Lake PCH support
> Patch 8: platform/x86: intel/pmc: Add Meteor Lake support to pmc core
> driver

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






> Gayatri Kammela (4):
>   platform/x86: intel/pmc: Replace all the reg_map with init functions
>   platform/x86: intel/pmc: Relocate Tiger Lake PCH support
>   platform/x86: intel/pmc: Relocate Alder Lake PCH support
>   platform/x86: intel/pmc/core: Add Meteor Lake support to pmc core
>     driver
> 
> Rajvi Jingar (1):
>   platform/x86: intel/pmc: Relocate Sunrise Point PCH support
> 
> Xi Pardee (3):
>   platform/x86: intel/pmc: Move variable declarations and definitions to
>     header and core.c
>   platform/x86: intel/pmc: Relocate Cannon Lake Point PCH support
>   platform/x86: intel/pmc: Relocate Ice Lake PCH support
> 
>  drivers/platform/x86/intel/pmc/Makefile |   3 +-
>  drivers/platform/x86/intel/pmc/adl.c    | 325 ++++++++
>  drivers/platform/x86/intel/pmc/cnp.c    | 210 +++++
>  drivers/platform/x86/intel/pmc/core.c   | 994 ++----------------------
>  drivers/platform/x86/intel/pmc/core.h   |  89 ++-
>  drivers/platform/x86/intel/pmc/icl.c    |  56 ++
>  drivers/platform/x86/intel/pmc/mtl.c    |  52 ++
>  drivers/platform/x86/intel/pmc/spt.c    | 140 ++++
>  drivers/platform/x86/intel/pmc/tgl.c    | 269 +++++++
>  9 files changed, 1183 insertions(+), 955 deletions(-)
>  create mode 100644 drivers/platform/x86/intel/pmc/adl.c
>  create mode 100644 drivers/platform/x86/intel/pmc/cnp.c
>  create mode 100644 drivers/platform/x86/intel/pmc/icl.c
>  create mode 100644 drivers/platform/x86/intel/pmc/mtl.c
>  create mode 100644 drivers/platform/x86/intel/pmc/spt.c
>  create mode 100644 drivers/platform/x86/intel/pmc/tgl.c
> 
> 
> base-commit: 309e0a6ed6e3fdb4febacc3e91aeb268500b90c6
> 
> Cc: Xi Pardee <xi.pardee@...el.com>
> Cc: Rajvi Jingar <rajvi.jingar@...ux.intel.com>
> Cc: David E Box <david.e.box@...ux.intel.com>