[coreboot-gerrit] Change in coreboot[master]: drivers/intel/pmx_mux: Remove redundant declaration

[Nico Huber (Code Review)]

Nico Huber has submitted this change. ( https://review.coreboot.org/c/coreboot/+/41871 )

Change subject: drivers/intel/pmx_mux: Remove redundant declaration
......................................................................

drivers/intel/pmx_mux: Remove redundant declaration

Change-Id: Ie64b267ac01afa9774105e1ab8a7c18021726ff3
Signed-off-by: Kyösti Mälkki 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/41871
Reviewed-by: Paul Menzel 
Reviewed-by: Tim Wawrzynczak 
Reviewed-by: Duncan Laurie 
Tested-by: build bot (Jenkins) 
---
M src/drivers/intel/pmc_mux/chip.h
M src/soc/intel/tigerlake/pmc.c
2 files changed, 3 insertions(+), 2 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Duncan Laurie: Looks good to me, approved
  Paul Menzel: Looks good to me, but someone else must approve
  Tim Wawrzynczak: Looks good to me, approved

diff --git a/src/drivers/intel/pmc_mux/chip.h b/src/drivers/intel/pmc_mux/chip.h
index f73a070..dcca2a3 100644
--- a/src/drivers/intel/pmc_mux/chip.h
+++ b/src/drivers/intel/pmc_mux/chip.h
@@ -3,8 +3,6 @@
 #ifndef __DRIVERS_INTEL_PMC_MUX_H__
 #define __DRIVERS_INTEL_PMC_MUX_H__

-extern struct chip_operations drivers_intel_pmc_mux_ops;
-
 struct drivers_intel_pmc_mux_config {
 };

diff --git a/src/soc/intel/tigerlake/pmc.c b/src/soc/intel/tigerlake/pmc.c
index 8fcbe50..f2f8a06 100644
--- a/src/soc/intel/tigerlake/pmc.c
+++ b/src/soc/intel/tigerlake/pmc.c
@@ -126,6 +126,9 @@
           dev_path(dev));
 }

+/* FIXME: Rewrite loop below without this. */
+extern struct chip_operations drivers_intel_pmc_mux_ops;
+
 /* By default, TGL uses the PMC MUX for all ports, so port_number is unused */
 const struct device *soc_get_pmc_mux_device(int port_number)
 {