sravan/system76-coreboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

northbridge/amd/agesa/family1{4,5,5tn,6kb}: Reduce differences

Browse Source 
Lets cut down on whitespace differences, fix some typos and indents.
Also make use of ARRAY_SIZE() macro instead of a local redefinition.

Fix NULL pointer checks ordering and not to use zero.

Change-Id: I93f344d300c04570d795659d848255cb1832e1d8
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/7528
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Reviewed-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
This commit is contained in:
Edward O'Callaghan
2014-11-20 19:58:09 +11:00
parent 9e999d6a14
commit ae5fd3453a
18 changed files with 91 additions and 85 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/northbridge/amd/agesa/family10/northbridge.h
View File

@@ -17,10 +17,10 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#ifndef NORTHBRIDGE_AMD_AGESA_FAM10H_H #ifndef NORTHBRIDGE_AMD_AGESA_FAM10_H
#define NORTHBRIDGE_AMD_AGESA_FAM10H_H #define NORTHBRIDGE_AMD_AGESA_FAM10_H
static struct device_operations pci_domain_ops; static struct device_operations pci_domain_ops;
static struct device_operations cpu_bus_ops; static struct device_operations cpu_bus_ops;
#endif /* NORTHBRIDGE_AMD_AGESA_FAM10H_H */ #endif /* NORTHBRIDGE_AMD_AGESA_FAM10_H */

6
src/northbridge/amd/agesa/family12/northbridge.h
View File

@@ -17,12 +17,12 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#ifndef NORTHBRIDGE_AMD_AGESA_FAM12H_H #ifndef NORTHBRIDGE_AMD_AGESA_FAM12_H
#define NORTHBRIDGE_AMD_AGESA_FAM12H_H #define NORTHBRIDGE_AMD_AGESA_FAM12_H
static struct device_operations pci_domain_ops; static struct device_operations pci_domain_ops;
static struct device_operations cpu_bus_ops; static struct device_operations cpu_bus_ops;
device_t get_node_pci(u32 nodeid, u32 fn); device_t get_node_pci(u32 nodeid, u32 fn);
#endif /* NORTHBRIDGE_AMD_AGESA_FAM12H_H */ #endif /* NORTHBRIDGE_AMD_AGESA_FAM12_H */

4
src/northbridge/amd/agesa/family14/Kconfig
View File

@@ -32,10 +32,10 @@ config HW_MEM_HOLE_SIZE_AUTO_INC
config MMCONF_BASE_ADDRESS config MMCONF_BASE_ADDRESS
hex hex
default 0xf8000000 default 0xF8000000
config MMCONF_BUS_NUMBER config MMCONF_BUS_NUMBER
int int
default 16 default 16
endif endif # NORTHBRIDGE_AMD_AGESA_FAMILY14

12
src/northbridge/amd/agesa/family14/dimmSpd.c
View File

@@ -34,19 +34,21 @@
//#pragma optimize ("", off) //#pragma optimize ("", off)
/** /**
* Gets the SMBUS address for an SPD from the array in devicetree.cb * Gets the SMBus address for an SPD from the array in devicetree.cb
* then read the SPD into the supplied buffer. * then read the SPD into the supplied buffer.
*/ */
AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PARAMS *info) AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PARAMS *info)
{ {
UINT8 spdAddress; UINT8 spdAddress;
ROMSTAGE_CONST struct device *dev = dev_find_slot(0, PCI_DEVFN(0x18, 2));
ROMSTAGE_CONST struct northbridge_amd_agesa_family14_config *config = NULL;
if ((dev == 0) || (dev->chip_info == 0)) ROMSTAGE_CONST struct device *dev = dev_find_slot(0, PCI_DEVFN(0x18, 2));
if (dev == NULL)
return AGESA_ERROR;
ROMSTAGE_CONST struct northbridge_amd_agesa_family14_config *config = dev->chip_info;
if (config == NULL)
return AGESA_ERROR; return AGESA_ERROR;
config = dev->chip_info;
if (info->SocketId >= ARRAY_SIZE(config->spdAddrLookup)) if (info->SocketId >= ARRAY_SIZE(config->spdAddrLookup))
return AGESA_ERROR; return AGESA_ERROR;
if (info->MemChannelId >= ARRAY_SIZE(config->spdAddrLookup[0])) if (info->MemChannelId >= ARRAY_SIZE(config->spdAddrLookup[0]))

6
src/northbridge/amd/agesa/family14/northbridge.h
View File

@@ -17,12 +17,12 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#ifndef NORTHBRIDGE_AMD_AGESA_FAM14H_H #ifndef NORTHBRIDGE_AMD_AGESA_FAM14_H
#define NORTHBRIDGE_AMD_AGESA_FAM14H_H #define NORTHBRIDGE_AMD_AGESA_FAM14_H
static struct device_operations pci_domain_ops; static struct device_operations pci_domain_ops;
static struct device_operations cpu_bus_ops; static struct device_operations cpu_bus_ops;
device_t get_node_pci(u32 nodeid, u32 fn); device_t get_node_pci(u32 nodeid, u32 fn);
#endif /* NORTHBRIDGE_AMD_AGESA_FAM14H_H */ #endif /* NORTHBRIDGE_AMD_AGESA_FAM14_H */

6
src/northbridge/amd/agesa/family15/Kconfig
View File

@@ -31,13 +31,17 @@ config HT3_SUPPORT
config HW_MEM_HOLE_SIZEK config HW_MEM_HOLE_SIZEK
hex hex
default 0x100000 default 0x100000
config HW_MEM_HOLE_SIZE_AUTO_INC config HW_MEM_HOLE_SIZE_AUTO_INC
bool bool
default n default n
config MMCONF_BASE_ADDRESS config MMCONF_BASE_ADDRESS
hex hex
default 0xF8000000 default 0xF8000000
config MMCONF_BUS_NUMBER config MMCONF_BUS_NUMBER
int int
default 64 default 64
endif #NORTHBRIDGE_AMD_AGESA_FAMILY15
endif # NORTHBRIDGE_AMD_AGESA_FAMILY15

11
src/northbridge/amd/agesa/family15/dimmSpd.c
View File

@@ -16,6 +16,7 @@
* along with this program; if not, write to the Free Software * along with this program; if not, write to the Free Software
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#include <device/pci_def.h> #include <device/pci_def.h>
#include <device/device.h> #include <device/device.h>
#include <stdlib.h> #include <stdlib.h>
@@ -39,13 +40,15 @@
AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PARAMS *info) AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PARAMS *info)
{ {
UINT8 spdAddress; UINT8 spdAddress;
ROMSTAGE_CONST struct device *dev = dev_find_slot(0, PCI_DEVFN(0x18, 2));
ROMSTAGE_CONST struct northbridge_amd_agesa_family15_config *config = NULL;
if ((dev == 0) || (dev->chip_info == 0)) ROMSTAGE_CONST struct device *dev = dev_find_slot(0, PCI_DEVFN(0x18, 2));
if (dev == NULL)
return AGESA_ERROR;
ROMSTAGE_CONST struct northbridge_amd_agesa_family15_config *config = dev->chip_info;
if (config == NULL)
return AGESA_ERROR; return AGESA_ERROR;
config = dev->chip_info;
if (info->SocketId >= ARRAY_SIZE(config->spdAddrLookup)) if (info->SocketId >= ARRAY_SIZE(config->spdAddrLookup))
return AGESA_ERROR; return AGESA_ERROR;
if (info->MemChannelId >= ARRAY_SIZE(config->spdAddrLookup[0])) if (info->MemChannelId >= ARRAY_SIZE(config->spdAddrLookup[0]))

23
src/northbridge/amd/agesa/family15/northbridge.c
View File

@@ -63,7 +63,6 @@ static device_t __f2_dev[MAX_NODE_NUMS];
static device_t __f4_dev[MAX_NODE_NUMS]; static device_t __f4_dev[MAX_NODE_NUMS];
static unsigned fx_devs = 0; static unsigned fx_devs = 0;
static dram_base_mask_t get_dram_base_mask(u32 nodeid) static dram_base_mask_t get_dram_base_mask(u32 nodeid)
{ {
device_t dev; device_t dev;
@@ -202,8 +201,8 @@ static void set_vga_enable_reg(u32 nodeid, u32 linkn)
/** /**
* @return * @return
* @retval 2 resoure not exist, usable * @retval 2 resoure does not exist, usable
* @retval 0 resource exist, not usable * @retval 0 resource exists, not usable
* @retval 1 resource exist, resource has been allocated before * @retval 1 resource exist, resource has been allocated before
*/ */
static int reg_useable(unsigned reg, device_t goal_dev, unsigned goal_nodeid, static int reg_useable(unsigned reg, device_t goal_dev, unsigned goal_nodeid,
@@ -287,7 +286,6 @@ static struct resource *amdfam15_find_mempair(device_t dev, u32 nodeid, u32 link
return resource; return resource;
} }
static void amdfam15_link_read_bases(device_t dev, u32 nodeid, u32 link) static void amdfam15_link_read_bases(device_t dev, u32 nodeid, u32 link)
{ {
struct resource *resource; struct resource *resource;
@@ -317,7 +315,6 @@ static void amdfam15_link_read_bases(device_t dev, u32 nodeid, u32 link)
resource->flags |= IORESOURCE_BRIDGE; resource->flags |= IORESOURCE_BRIDGE;
} }
/* Initialize the memory constraints on the current bus */ /* Initialize the memory constraints on the current bus */
resource = amdfam15_find_mempair(dev, nodeid, link); resource = amdfam15_find_mempair(dev, nodeid, link);
if (resource) { if (resource) {
@@ -331,7 +328,6 @@ static void amdfam15_link_read_bases(device_t dev, u32 nodeid, u32 link)
} }
static void nb_read_resources(device_t dev) static void nb_read_resources(device_t dev)
{ {
u32 nodeid; u32 nodeid;
@@ -356,17 +352,14 @@ static void nb_read_resources(device_t dev)
resource->flags = IORESOURCE_MEM | IORESOURCE_RESERVE | resource->flags = IORESOURCE_MEM | IORESOURCE_RESERVE |
IORESOURCE_FIXED | IORESOURCE_STORED | IORESOURCE_ASSIGNED; IORESOURCE_FIXED | IORESOURCE_STORED | IORESOURCE_ASSIGNED;
#endif #endif
} }
static void set_resource(device_t dev, struct resource *resource, u32 nodeid) static void set_resource(device_t dev, struct resource *resource, u32 nodeid)
{ {
resource_t rbase, rend; resource_t rbase, rend;
unsigned reg, link_num; unsigned reg, link_num;
char buf[50]; char buf[50];
/* Make certain the resource has actually been set */ /* Make certain the resource has actually been set */
if (!(resource->flags & IORESOURCE_ASSIGNED)) { if (!(resource->flags & IORESOURCE_ASSIGNED)) {
return; return;
@@ -399,7 +392,7 @@ static void set_resource(device_t dev, struct resource *resource, u32 nodeid)
set_io_addr_reg(dev, nodeid, link_num, reg, rbase>>8, rend>>8); set_io_addr_reg(dev, nodeid, link_num, reg, rbase>>8, rend>>8);
} }
else if (resource->flags & IORESOURCE_MEM) { else if (resource->flags & IORESOURCE_MEM) {
set_mmio_addr_reg(nodeid, link_num, reg, (resource->index >>24), rbase>>8, rend>>8, node_nums) ;// [39:8] set_mmio_addr_reg(nodeid, link_num, reg, (resource->index >>24), rbase>>8, rend>>8, node_nums);// [39:8]
} }
resource->flags |= IORESOURCE_STORED; resource->flags |= IORESOURCE_STORED;
snprintf(buf, sizeof (buf), " <node %x link %x>", snprintf(buf, sizeof (buf), " <node %x link %x>",
@@ -416,7 +409,6 @@ static void create_vga_resource(device_t dev, unsigned nodeid)
{ {
struct bus *link; struct bus *link;
/* find out which link the VGA card is connected, /* find out which link the VGA card is connected,
* we only deal with the 'first' vga card */ * we only deal with the 'first' vga card */
for (link = dev->link_list; link; link = link->next) { for (link = dev->link_list; link; link = link->next) {
@@ -442,7 +434,6 @@ static void create_vga_resource(device_t dev, unsigned nodeid)
set_vga_enable_reg(nodeid, sblink); set_vga_enable_reg(nodeid, sblink);
} }
static void nb_set_resources(device_t dev) static void nb_set_resources(device_t dev)
{ {
unsigned nodeid; unsigned nodeid;
@@ -526,7 +517,6 @@ static void domain_read_resources(device_t dev)
{ {
unsigned reg; unsigned reg;
/* Find the already assigned resource pairs */ /* Find the already assigned resource pairs */
get_fx_devs(); get_fx_devs();
for (reg = 0x80; reg <= 0xd8; reg+= 0x08) { for (reg = 0x80; reg <= 0xd8; reg+= 0x08) {
@@ -560,7 +550,6 @@ static void domain_read_resources(device_t dev)
#if !CONFIG_PCI_64BIT_PREF_MEM #if !CONFIG_PCI_64BIT_PREF_MEM
pci_domain_read_resources(dev); pci_domain_read_resources(dev);
#else #else
struct bus *link; struct bus *link;
struct resource *resource; struct resource *resource;
@@ -596,7 +585,6 @@ static void domain_enable_resources(device_t dev)
printk(BIOS_DEBUG, " Fam15 - leaving %s.\n", __func__); printk(BIOS_DEBUG, " Fam15 - leaving %s.\n", __func__);
} }
#if CONFIG_HW_MEM_HOLE_SIZEK != 0 #if CONFIG_HW_MEM_HOLE_SIZEK != 0
struct hw_mem_hole_info { struct hw_mem_hole_info {
unsigned hole_startk; unsigned hole_startk;
@@ -677,6 +665,7 @@ static void setup_uma_memory(void)
#endif #endif
} }
static void domain_set_resources(device_t dev) static void domain_set_resources(device_t dev)
{ {
#if CONFIG_PCI_64BIT_PREF_MEM #if CONFIG_PCI_64BIT_PREF_MEM
@@ -788,7 +777,6 @@ static void domain_set_resources(device_t dev)
sizek = limitk - basek; sizek = limitk - basek;
/* see if we need a hole from 0xa0000 to 0xbffff */ /* see if we need a hole from 0xa0000 to 0xbffff */
if ((basek < ((8*64)+(8*16))) && (sizek > ((8*64)+(16*16)))) { if ((basek < ((8*64)+(8*16))) && (sizek > ((8*64)+(16*16)))) {
ram_resource(dev, (idx | i), basek, ((8*64)+(8*16)) - basek); ram_resource(dev, (idx | i), basek, ((8*64)+(8*16)) - basek);
@@ -845,7 +833,6 @@ static void domain_set_resources(device_t dev)
} }
} }
static struct device_operations pci_domain_ops = { static struct device_operations pci_domain_ops = {
.read_resources = domain_read_resources, .read_resources = domain_read_resources,
.set_resources = domain_set_resources, .set_resources = domain_set_resources,
@@ -855,7 +842,6 @@ static struct device_operations pci_domain_ops = {
.ops_pci_bus = pci_bus_default_ops, .ops_pci_bus = pci_bus_default_ops,
}; };
static void sysconf_init(device_t dev) // first node static void sysconf_init(device_t dev) // first node
{ {
sblink = (pci_read_config32(dev, 0x64)>>8) & 7; // don't forget sublink1 sblink = (pci_read_config32(dev, 0x64)>>8) & 7; // don't forget sublink1
@@ -1096,7 +1082,6 @@ static struct device_operations cpu_bus_ops = {
.scan_bus = cpu_bus_scan, .scan_bus = cpu_bus_scan,
}; };
static void root_complex_enable_dev(struct device *dev) static void root_complex_enable_dev(struct device *dev)
{ {
static int done = 0; static int done = 0;

2
src/northbridge/amd/agesa/family15tn/Kconfig
View File

@@ -38,4 +38,4 @@ config MMCONF_BUS_NUMBER
int int
default 64 default 64
endif endif # NORTHBRIDGE_AMD_AGESA_FAMILY15_TN

4
src/northbridge/amd/agesa/family15tn/chip.h
View File

@@ -17,8 +17,8 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#ifndef _AGESA_FAM15TN_CHIP_H_ #ifndef _NB_AGESA_CHIP_H_
#define _AGESA_FAM15TN_CHIP_H_ #define _NB_AGESA_CHIP_H_
struct northbridge_amd_agesa_family15tn_config struct northbridge_amd_agesa_family15tn_config
{ {

26
src/northbridge/amd/agesa/family15tn/dimmSpd.c
View File

@@ -19,6 +19,7 @@
#include <device/pci_def.h> #include <device/pci_def.h>
#include <device/device.h> #include <device/device.h>
#include <stdlib.h>
/* warning: Porting.h includes an open #pragma pack(1) */ /* warning: Porting.h includes an open #pragma pack(1) */
#include "Porting.h" #include "Porting.h"
@@ -26,28 +27,33 @@
#include "amdlib.h" #include "amdlib.h"
#include "chip.h" #include "chip.h"
#include "northbridge/amd/agesa/dimmSpd.h" #include <northbridge/amd/agesa/dimmSpd.h>
#define DIMENSION(array)(sizeof (array)/ sizeof (array [0]))
/**
* Gets the SMBus address for an SPD from the array in devicetree.cb
* then read the SPD into the supplied buffer.
*/
AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PARAMS *info) AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PARAMS *info)
{ {
int spdAddress; UINT8 spdAddress;
ROMSTAGE_CONST struct device *dev = dev_find_slot(0, PCI_DEVFN(0x18, 2)); ROMSTAGE_CONST struct device *dev = dev_find_slot(0, PCI_DEVFN(0x18, 2));
if (dev == NULL)
return AGESA_ERROR;
ROMSTAGE_CONST struct northbridge_amd_agesa_family15tn_config *config = dev->chip_info; ROMSTAGE_CONST struct northbridge_amd_agesa_family15tn_config *config = dev->chip_info;
if (config == NULL)
if ((dev == 0) || (config == 0))
return AGESA_ERROR; return AGESA_ERROR;
if (info->SocketId >= DIMENSION(config->spdAddrLookup )) if (info->SocketId >= ARRAY_SIZE(config->spdAddrLookup))
return AGESA_ERROR; return AGESA_ERROR;
if (info->MemChannelId >= DIMENSION(config->spdAddrLookup[0] )) if (info->MemChannelId >= ARRAY_SIZE(config->spdAddrLookup[0]))
return AGESA_ERROR; return AGESA_ERROR;
if (info->DimmId >= DIMENSION(config->spdAddrLookup[0][0])) if (info->DimmId >= ARRAY_SIZE(config->spdAddrLookup[0][0]))
return AGESA_ERROR; return AGESA_ERROR;
spdAddress = config->spdAddrLookup spdAddress = config->spdAddrLookup
[info->SocketId] [info->MemChannelId] [info->DimmId]; [info->SocketId][info->MemChannelId][info->DimmId];
if (spdAddress == 0) if (spdAddress == 0)
return AGESA_ERROR; return AGESA_ERROR;

20
src/northbridge/amd/agesa/family15tn/northbridge.c
View File

@@ -201,8 +201,8 @@ static void set_vga_enable_reg(u32 nodeid, u32 linkn)
/** /**
* @return * @return
* @retval 2 resoure not exist, usable * @retval 2 resoure does not exist, usable
* @retval 0 resource exist, not usable * @retval 0 resource exists, not usable
* @retval 1 resource exist, resource has been allocated before * @retval 1 resource exist, resource has been allocated before
*/ */
static int reg_useable(unsigned reg, device_t goal_dev, unsigned goal_nodeid, static int reg_useable(unsigned reg, device_t goal_dev, unsigned goal_nodeid,
@@ -392,7 +392,7 @@ static void set_resource(device_t dev, struct resource *resource, u32 nodeid)
set_io_addr_reg(dev, nodeid, link_num, reg, rbase>>8, rend>>8); set_io_addr_reg(dev, nodeid, link_num, reg, rbase>>8, rend>>8);
} }
else if (resource->flags & IORESOURCE_MEM) { else if (resource->flags & IORESOURCE_MEM) {
set_mmio_addr_reg(nodeid, link_num, reg, (resource->index >>24), rbase>>8, rend>>8, node_nums) ;// [39:8] set_mmio_addr_reg(nodeid, link_num, reg, (resource->index >>24), rbase>>8, rend>>8, node_nums);// [39:8]
} }
resource->flags |= IORESOURCE_STORED; resource->flags |= IORESOURCE_STORED;
snprintf(buf, sizeof (buf), " <node %x link %x>", snprintf(buf, sizeof (buf), " <node %x link %x>",
@@ -749,7 +749,7 @@ static void domain_set_resources(device_t dev)
if (!(d.mask & 1)) continue; if (!(d.mask & 1)) continue;
basek = ((resource_t)(d.base & 0x1fffff00)) << 9; // could overflow, we may lost 6 bit here basek = ((resource_t)(d.base & 0x1fffff00)) << 9; // could overflow, we may lost 6 bit here
limitk = ((resource_t)(((d.mask & ~1) + 0x000FF) & 0x1fffff00)) << 9 ; limitk = ((resource_t)(((d.mask & ~1) + 0x000FF) & 0x1fffff00)) << 9;
sizek = limitk - basek; sizek = limitk - basek;
@@ -1078,16 +1078,16 @@ static void root_complex_enable_dev(struct device *dev)
} }
struct chip_operations northbridge_amd_agesa_family15tn_root_complex_ops = { struct chip_operations northbridge_amd_agesa_family15tn_root_complex_ops = {
CHIP_NAME("AMD FAM15 Root Complex") CHIP_NAME("AMD FAM15tn Root Complex")
.enable_dev = root_complex_enable_dev, .enable_dev = root_complex_enable_dev,
}; };
/******************************************************************** /*********************************************************************
* Change the vendor / device IDs to match the generic VBIOS header. * Change the vendor / device IDs to match the generic VBIOS header. *
********************************************************************/ *********************************************************************/
u32 map_oprom_vendev(u32 vendev) u32 map_oprom_vendev(u32 vendev)
{ {
u32 new_vendev=vendev; u32 new_vendev = vendev;
switch(vendev) { switch(vendev) {
case 0x10029900: /* AMD Radeon HD 7660G (Trinity) */ case 0x10029900: /* AMD Radeon HD 7660G (Trinity) */
@@ -1124,7 +1124,7 @@ u32 map_oprom_vendev(u32 vendev)
case 0x100299A0: /* AMD Radeon HD 7520G (Trinity) */ case 0x100299A0: /* AMD Radeon HD 7520G (Trinity) */
case 0x100299A2: /* AMD Radeon HD 7420G (Trinity) */ case 0x100299A2: /* AMD Radeon HD 7420G (Trinity) */
case 0x100299A4: /* AMD Radeon HD 7400G (Trinity) */ case 0x100299A4: /* AMD Radeon HD 7400G (Trinity) */
new_vendev=0x10029901; new_vendev = 0x10029901;
break; break;
} }

6
src/northbridge/amd/agesa/family15tn/northbridge.h
View File

@@ -17,10 +17,10 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#ifndef NORTHBRIDGE_AMD_AGESA_FAM15H_H #ifndef NORTHBRIDGE_AMD_AGESA_FAM15_H
#define NORTHBRIDGE_AMD_AGESA_FAM15H_H #define NORTHBRIDGE_AMD_AGESA_FAM15_H
static struct device_operations pci_domain_ops; static struct device_operations pci_domain_ops;
static struct device_operations cpu_bus_ops; static struct device_operations cpu_bus_ops;
#endif /* NORTHBRIDGE_AMD_AGESA_FAM15H_H */ #endif /* NORTHBRIDGE_AMD_AGESA_FAM15_H */

2
src/northbridge/amd/agesa/family16kb/Kconfig
View File

@@ -46,4 +46,4 @@ config VGA_BIOS_ID
The default VGA BIOS PCI vendor/device ID should be set to the The default VGA BIOS PCI vendor/device ID should be set to the
result of the map_oprom_vendev() function in northbridge.c. result of the map_oprom_vendev() function in northbridge.c.
endif endif # NORTHBRIDGE_AMD_AGESA_FAMILY16_KB

4
src/northbridge/amd/agesa/family16kb/chip.h
View File

@@ -17,8 +17,8 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#ifndef _AGESA_FAM16KB_CHIP_H_ #ifndef _NB_AGESA_CHIP_H_
#define _AGESA_FAM16KB_CHIP_H_ #define _NB_AGESA_CHIP_H_
struct northbridge_amd_agesa_family16kb_config struct northbridge_amd_agesa_family16kb_config
{ {

26
src/northbridge/amd/agesa/family16kb/dimmSpd.c
View File

@@ -19,6 +19,7 @@
#include <device/pci_def.h> #include <device/pci_def.h>
#include <device/device.h> #include <device/device.h>
#include <stdlib.h>
/* warning: Porting.h includes an open #pragma pack(1) */ /* warning: Porting.h includes an open #pragma pack(1) */
#include "Porting.h" #include "Porting.h"
@@ -26,28 +27,33 @@
#include "amdlib.h" #include "amdlib.h"
#include "chip.h" #include "chip.h"
#include "northbridge/amd/agesa/dimmSpd.h" #include <northbridge/amd/agesa/dimmSpd.h>
#define DIMENSION(array)(sizeof (array)/ sizeof (array [0]))
/**
* Gets the SMBus address for an SPD from the array in devicetree.cb
* then read the SPD into the supplied buffer.
*/
AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PARAMS *info) AGESA_STATUS AmdMemoryReadSPD (UINT32 unused1, UINT32 unused2, AGESA_READ_SPD_PARAMS *info)
{ {
int spdAddress; UINT8 spdAddress;
ROMSTAGE_CONST struct device *dev = dev_find_slot(0, PCI_DEVFN(0x18, 2)); ROMSTAGE_CONST struct device *dev = dev_find_slot(0, PCI_DEVFN(0x18, 2));
if (dev == NULL)
return AGESA_ERROR;
ROMSTAGE_CONST struct northbridge_amd_agesa_family16kb_config *config = dev->chip_info; ROMSTAGE_CONST struct northbridge_amd_agesa_family16kb_config *config = dev->chip_info;
if (config == NULL)
if ((dev == 0) || (config == 0))
return AGESA_ERROR; return AGESA_ERROR;
if (info->SocketId >= DIMENSION(config->spdAddrLookup )) if (info->SocketId >= ARRAY_SIZE(config->spdAddrLookup))
return AGESA_ERROR; return AGESA_ERROR;
if (info->MemChannelId >= DIMENSION(config->spdAddrLookup[0] )) if (info->MemChannelId >= ARRAY_SIZE(config->spdAddrLookup[0]))
return AGESA_ERROR; return AGESA_ERROR;
if (info->DimmId >= DIMENSION(config->spdAddrLookup[0][0])) if (info->DimmId >= ARRAY_SIZE(config->spdAddrLookup[0][0]))
return AGESA_ERROR; return AGESA_ERROR;
spdAddress = config->spdAddrLookup spdAddress = config->spdAddrLookup
[info->SocketId] [info->MemChannelId] [info->DimmId]; [info->SocketId][info->MemChannelId][info->DimmId];
if (spdAddress == 0) if (spdAddress == 0)
return AGESA_ERROR; return AGESA_ERROR;

4
src/northbridge/amd/agesa/family16kb/northbridge.c
View File

@@ -379,7 +379,7 @@ static void set_resource(device_t dev, struct resource *resource, u32 nodeid)
set_io_addr_reg(dev, nodeid, link_num, reg, rbase>>8, rend>>8); set_io_addr_reg(dev, nodeid, link_num, reg, rbase>>8, rend>>8);
} }
else if (resource->flags & IORESOURCE_MEM) { else if (resource->flags & IORESOURCE_MEM) {
set_mmio_addr_reg(nodeid, link_num, reg, (resource->index >>24), rbase>>8, rend>>8, node_nums) ;// [39:8] set_mmio_addr_reg(nodeid, link_num, reg, (resource->index >>24), rbase>>8, rend>>8, node_nums);// [39:8]
} }
resource->flags |= IORESOURCE_STORED; resource->flags |= IORESOURCE_STORED;
snprintf(buf, sizeof (buf), " <node %x link %x>", snprintf(buf, sizeof (buf), " <node %x link %x>",
@@ -757,7 +757,7 @@ static void domain_set_resources(device_t dev)
if (!(d.mask & 1)) continue; if (!(d.mask & 1)) continue;
basek = ((resource_t)(d.base & 0x1fffff00)) << 9; // could overflow, we may lost 6 bit here basek = ((resource_t)(d.base & 0x1fffff00)) << 9; // could overflow, we may lost 6 bit here
limitk = ((resource_t)(((d.mask & ~1) + 0x000FF) & 0x1fffff00)) << 9 ; limitk = ((resource_t)(((d.mask & ~1) + 0x000FF) & 0x1fffff00)) << 9;
sizek = limitk - basek; sizek = limitk - basek;

6
src/northbridge/amd/agesa/family16kb/northbridge.h
View File

@@ -17,10 +17,10 @@
* Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
*/ */
#ifndef NORTHBRIDGE_AMD_AGESA_FAM16H_H #ifndef NORTHBRIDGE_AMD_AGESA_FAM16_H
#define NORTHBRIDGE_AMD_AGESA_FAM16H_H #define NORTHBRIDGE_AMD_AGESA_FAM16_H
static struct device_operations pci_domain_ops; static struct device_operations pci_domain_ops;
static struct device_operations cpu_bus_ops; static struct device_operations cpu_bus_ops;
#endif /* NORTHBRIDGE_AMD_AGESA_FAM16H_H */ #endif /* NORTHBRIDGE_AMD_AGESA_FAM16_H */