commit 38bf496366bfb8c718345869311f6d46a8922486
author Michael Niewöhner <foss@mniewoehner.de> Mon Sep 27 23:55:05 2021 +0200
committer Felix Held <felix-coreboot@felixheld.de> Wed Sep 29 21:19:37 2021 +0000
tree 1938638adc7a24cf8490038c65b7b20e42cbfc9e
parent f67f2b0d4ca4d1d72fa077d69abef916c8c3a38c

soc/intel/{cnl,jsl,tgl,ehl,adl}: rename PMC device init/enable callbacks

The current names of the PMC init/enable callbacks are very confusing.
Rename them.

Change-Id: I69f54f3b4e1ea9a9b4fa5c8dd9c0d454d7cd1283
Signed-off-by: Michael Niewöhner <foss@mniewoehner.de>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57995
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by:  Felix Singer <felixsinger@posteo.net>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>

src/soc/intel/alderlake/pmc.c

diff --git a/src/soc/intel/alderlake/pmc.c b/src/soc/intel/alderlake/pmc.c
index 0e0a568a..6f9b03b 100644
--- a/src/soc/intel/alderlake/pmc.c
+++ b/src/soc/intel/alderlake/pmc.c
@@ -66,7 +66,7 @@
 	write32(pmcbase + DSX_CFG, reg);
 }
 
-static void pmc_init(struct device *dev)
+static void soc_pmc_enable(struct device *dev)
 {
 	const config_t *config = config_of_soc();
 
@@ -140,7 +140,7 @@
 	       dev_path(dev));
 }
 
-static void soc_acpi_mode_init(struct device *dev)
+static void soc_pmc_init(struct device *dev)
 {
 	/*
 	 * pmc_set_acpi_mode() should be delayed until BS_DEV_INIT in order
@@ -165,8 +165,8 @@
 struct device_operations pmc_ops = {
 	.read_resources	  = soc_pmc_read_resources,
 	.set_resources	  = noop_set_resources,
-	.init		  = soc_acpi_mode_init,
-	.enable		  = pmc_init,
+	.init		  = soc_pmc_init,
+	.enable		  = soc_pmc_enable,
 #if CONFIG(HAVE_ACPI_TABLES)
 	.acpi_fill_ssdt	  = soc_pmc_fill_ssdt,
 #endif