aboutsummaryrefslogtreecommitdiff
path: root/src/southbridge/intel/i82801dx/early_smbus.c
diff options
context:
space:
mode:
authorAngel Pons <th3fanbus@gmail.com>2020-06-01 23:11:18 +0200
committerAngel Pons <th3fanbus@gmail.com>2020-07-21 18:24:07 +0000
commit16597243753e7fafa42d24389040d885b364c29a (patch)
tree655bf9982a6b21d7aa783a6dd16d4770a1a14f74 /src/southbridge/intel/i82801dx/early_smbus.c
parent7a2cb35262bedec9e6470bf7fbebf9256e855cff (diff)
e7505/i82801dx: Use common code for early SMBus
While it looks different, the early SMBus code for this southbridge is still the same. In addition, this code was not checking the vendor ID before. It is assumed that adding this check does not pose a problem. Change-Id: I95ae4db399ce5592cefca82fa75f349220023b8c Signed-off-by: Angel Pons <th3fanbus@gmail.com> Reviewed-on: https://review.coreboot.org/c/coreboot/+/42006 Reviewed-by: Patrick Rudolph <siro@das-labor.org> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/southbridge/intel/i82801dx/early_smbus.c')
-rw-r--r--src/southbridge/intel/i82801dx/early_smbus.c21
1 files changed, 0 insertions, 21 deletions
diff --git a/src/southbridge/intel/i82801dx/early_smbus.c b/src/southbridge/intel/i82801dx/early_smbus.c
index fc225b0466..76ef1b8a15 100644
--- a/src/southbridge/intel/i82801dx/early_smbus.c
+++ b/src/southbridge/intel/i82801dx/early_smbus.c
@@ -1,7 +1,5 @@
/* SPDX-License-Identifier: GPL-2.0-only */
-#include <device/pci_ops.h>
-#include <device/pci_def.h>
#include <device/smbus_host.h>
#include "i82801dx.h"
@@ -9,22 +7,3 @@ void i82801dx_early_init(void)
{
enable_smbus();
}
-
-uintptr_t smbus_base(void)
-{
- return CONFIG_FIXED_SMBUS_IO_BASE;
-}
-
-int smbus_enable_iobar(uintptr_t base)
-{
- pci_devfn_t dev = PCI_DEV(0x0, 0x1f, 0x3);
-
- /* set smbus iobase */
- pci_write_config32(dev, 0x20, base | 1);
- /* Set smbus enable */
- pci_write_config8(dev, 0x40, 0x01);
- /* Set smbus iospace enable */
- pci_write_config16(dev, 0x4, 0x01);
-
- return 0;
-}