commit 3f7bb7dc4457f0513fcdbc6e8760d94ea063367a
author Angel Pons <th3fanbus@gmail.com> Wed Jan 27 13:03:20 2021 +0100
committer Patrick Georgi <pgeorgi@google.com> Mon Mar 01 08:30:51 2021 +0000
tree d558d203953adb3ef186e0f1de007f8f1baa4db7
parent d882bd478d003b36aca7bd43890500dcfa6827ce

sb/intel/bd82x6x: Turn ME PCI register structs into unions

This allows dropping the `pci_read_dword_ptr` and `pci_write_dword_ptr`
wrappers.

Change-Id: I7a6916e535fbba9f05451d5302261418f950be83
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49993
Reviewed-by: Nico Huber <nico.h@gmx.de>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>

src/southbridge/intel/bd82x6x/me_smm.c

diff --git a/src/southbridge/intel/bd82x6x/me_smm.c b/src/southbridge/intel/bd82x6x/me_smm.c
index 11f55c2..e1eb900 100644
--- a/src/southbridge/intel/bd82x6x/me_smm.c
+++ b/src/southbridge/intel/bd82x6x/me_smm.c
@@ -64,8 +64,7 @@
 
 void intel_me_finalize_smm(void)
 {
-	struct me_hfs hfs;
-	u32 reg32;
+	union me_hfs hfs;
 
 	update_mei_base_address();
 
@@ -74,8 +73,7 @@
 		return;
 
 	/* Make sure ME is in a mode that expects EOP */
-	reg32 = pci_read_config32(PCH_ME_DEV, PCI_ME_HFS);
-	memcpy(&hfs, &reg32, sizeof(u32));
+	hfs.raw = pci_read_config32(PCH_ME_DEV, PCI_ME_HFS);
 
 	/* Abort and leave device alone if not normal mode */
 	if (hfs.fpt_bad ||