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/cannonlake/pmc.c

diff --git a/src/soc/intel/cannonlake/pmc.c b/src/soc/intel/cannonlake/pmc.c
index 817cdd7..d502e06 100644
--- a/src/soc/intel/cannonlake/pmc.c
+++ b/src/soc/intel/cannonlake/pmc.c
@@ -84,7 +84,7 @@
 	res->flags = IORESOURCE_IO | IORESOURCE_ASSIGNED | IORESOURCE_FIXED;
 }
 
-static void pmc_init(struct device *dev)
+static void soc_pmc_enable(struct device *dev)
 {
 	const config_t *config = config_of_soc();
 
@@ -98,7 +98,7 @@
 	config_deep_sx(config->deep_sx_config);
 }
 
-static void soc_acpi_mode_init(struct device *dev)
+static void soc_pmc_init(struct device *dev)
 {
 	/*
 	 * PMC initialization happens earlier for this SoC because FSP-Silicon
@@ -129,8 +129,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	  = pmc_fill_ssdt,
 #endif