mbox series

[v3,0/8] Redesign the pmc core driver

Message ID 20221114181207.2062901-1-gayatri.kammela@linux.intel.com (mailing list archive)
Headers show
Series Redesign the pmc core driver | expand

Message

Gayatri Kammela Nov. 14, 2022, 6:11 p.m. UTC
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 

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@intel.com>
Cc: Rajvi Jingar <rajvi.jingar@linux.intel.com>
Cc: David E Box <david.e.box@linux.intel.com>

Comments

Gayatri Kammela Nov. 14, 2022, 6:16 p.m. UTC | #1
On 11/14/2022 10:11 AM, 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
>
> 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@intel.com>
> Cc: Rajvi Jingar <rajvi.jingar@linux.intel.com>
> Cc: David E Box <david.e.box@linux.intel.com>

Hi All,

Sorry, this is v1 and not v3. Please ignore this series. I will resend 
the v1 to avoid confusion.