[coreboot] [commit] r5837 - in trunk/src/southbridge/amd: rs690 rs780 sb600 sb700

repository service svn at coreboot.org
Sat Sep 25 01:37:26 CEST 2010


Author: uwe
Date: Sat Sep 25 01:37:25 2010
New Revision: 5837
URL: https://tracker.coreboot.org/trac/coreboot/changeset/5837

Log:
Make SB600/SB700 more similar for easier diffs (trivial).

Also fixes random whitespace issues, typos, etc.

Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
Acked-by: Uwe Hermann <uwe at hermann-uwe.de>

Modified:
   trunk/src/southbridge/amd/rs690/chip.h
   trunk/src/southbridge/amd/rs780/chip.h
   trunk/src/southbridge/amd/sb600/sb600.c
   trunk/src/southbridge/amd/sb600/sb600_early_setup.c
   trunk/src/southbridge/amd/sb600/sb600_enable_usbdebug.c
   trunk/src/southbridge/amd/sb600/sb600_hda.c
   trunk/src/southbridge/amd/sb600/sb600_sm.c
   trunk/src/southbridge/amd/sb600/sb600_smbus.c
   trunk/src/southbridge/amd/sb700/sb700_early_setup.c
   trunk/src/southbridge/amd/sb700/sb700_enable_usbdebug.c
   trunk/src/southbridge/amd/sb700/sb700_ide.c
   trunk/src/southbridge/amd/sb700/sb700_smbus.c

Modified: trunk/src/southbridge/amd/rs690/chip.h
==============================================================================
--- trunk/src/southbridge/amd/rs690/chip.h	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/rs690/chip.h	Sat Sep 25 01:37:25 2010	(r5837)
@@ -20,13 +20,13 @@
 #ifndef RS690_CHIP_H
 #define RS690_CHIP_H
 
-/* Member variables are defined in Config.lb. */
+/* Member variables are defined in devicetree.cb. */
 struct southbridge_amd_rs690_config
 {
 	u8 gpp_configuration;	/* The configuration of General Purpose Port, A/B/C/D/E. */
 	u8 port_enable;		/* Which port is enabled? GFX(2,3), GPP(4,5,6,7) */
 	u8 gfx_dev2_dev3;	/* for GFX Core initialization REFCLK_SEL */
-	u8 gfx_dual_slot;		/* Is it dual graphics slots */
+	u8 gfx_dual_slot;	/* Is it dual graphics slots */
 	u8 gfx_lane_reversal;	/* Single/Dual slot lan reversal */
 	u8 gfx_tmds;		/* whether support TMDS? */
 	u8 gfx_compliance;	/* whether support compliance? */

Modified: trunk/src/southbridge/amd/rs780/chip.h
==============================================================================
--- trunk/src/southbridge/amd/rs780/chip.h	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/rs780/chip.h	Sat Sep 25 01:37:25 2010	(r5837)
@@ -20,7 +20,7 @@
 #ifndef RS780_CHIP_H
 #define RS780_CHIP_H
 
-/* Member variables are defined in Config.lb. */
+/* Member variables are defined in devicetree.cb. */
 struct southbridge_amd_rs780_config
 {
 	u8 gppsb_configuration;		/* The configuration of General Purpose Port, A/B/C/D/E. */

Modified: trunk/src/southbridge/amd/sb600/sb600.c
==============================================================================
--- trunk/src/southbridge/amd/sb600/sb600.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb600/sb600.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -120,21 +120,21 @@
 
 	printk(BIOS_DEBUG, "sb600_enable()\n");
 
-/*
-*	0:12.0  SATA	bit 8 of sm_dev 0xac : 1 - enable, default         + 32 * 3
-*	0:13.1  USB-1	bit 2 of sm_dev 0x68
-*	0:13.2  USB-2	bit 3 of sm_dev 0x68
-*	0:13.3  USB-3	bit 4 of sm_dev 0x68
-*	0:13.4  USB-4	bit 5 of sm_dev 0x68
-*	0:13.5  USB2	bit 0 of sm_dev 0x68 : 1 - enable, default
-*	0:14.0  SMBUS							0
-*	0:14.1  IDE							1
-*	0:14.2  HDA	bit 3 of pm_io 0x59 : 1 - enable, default	    + 32 * 4
-*	0:14.3  LPC	bit 20 of sm_dev 0x64 : 0 - disable, default  + 32 * 1
-*	0:14.4  PCI							4
-*	0:14.5  ACI	bit 0 of pm_io 0x59 : 0 - enable, default
-*	0:14.6  MCI	bit 1 of pm_io 0x59 : 0 - enable, default
-*/
+	/*
+	 * 0:12.0  SATA	bit 8 of sm_dev 0xac : 1 - enable, default         + 32 * 3
+	 * 0:13.1  USB-1	bit 2 of sm_dev 0x68
+	 * 0:13.2  USB-2	bit 3 of sm_dev 0x68
+	 * 0:13.3  USB-3	bit 4 of sm_dev 0x68
+	 * 0:13.4  USB-4	bit 5 of sm_dev 0x68
+	 * 0:13.5  USB2	bit 0 of sm_dev 0x68 : 1 - enable, default
+	 * 0:14.0  SMBUS							0
+	 * 0:14.1  IDE							1
+	 * 0:14.2  HDA	bit 3 of pm_io 0x59 : 1 - enable, default	    + 32 * 4
+	 * 0:14.3  LPC	bit 20 of sm_dev 0x64 : 0 - disable, default  + 32 * 1
+	 * 0:14.4  PCI							4
+	 * 0:14.5  ACI	bit 0 of pm_io 0x59 : 0 - enable, default
+	 * 0:14.6  MCI	bit 1 of pm_io 0x59 : 0 - enable, default
+	 */
 	if (dev->device == 0x0000) {
 		vendorid = pci_read_config32(dev, PCI_VENDOR_ID);
 		deviceid = (vendorid >> 16) & 0xffff;

Modified: trunk/src/southbridge/amd/sb600/sb600_early_setup.c
==============================================================================
--- trunk/src/southbridge/amd/sb600/sb600_early_setup.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb600/sb600_early_setup.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -25,7 +25,6 @@
 #define SMBUS_IO_BASE 0x1000	/* Is it a temporary SMBus I/O base address? */
 	 /*SIZE 0x40 */
 
-
 static void pmio_write(u8 reg, u8 value)
 {
 	outb(reg, PM_INDEX);
@@ -38,7 +37,7 @@
 	return inb(PM_INDEX + 1);
 }
 
-/* Get SB ASIC Revision.*/
+/* RPR 2.1: Get SB ASIC Revision. */
 static u8 get_sb600_revision(void)
 {
 	device_t dev;
@@ -131,22 +130,21 @@
 	return (dev >> 15) & 0x1f;
 }
 
-
 static u8 dual_core(void)
 {
 	return (pci_read_config32(PCI_DEV(0, 0x18, 3), 0xE8) & (0x3<<12)) != 0;
 }
 
 /*
-SB600 VFSMAF (VID/FID System Management Action Field)  is 010b by default.
-RPR 2.3.3 C-state and VID/FID change for the K8 platform.
+ * SB600 VFSMAF (VID/FID System Management Action Field) is 010b by default.
+ * RPR 2.3.3 C-state and VID/FID change for the K8 platform.
 */
 static void enable_fid_change_on_sb(u32 sbbusn, u32 sbdn)
 {
 	u8 byte;
 	byte = pmio_read(0x9a);
 	byte &= ~0x34;
-	if(dual_core())
+	if (dual_core())
 		byte |= 0x34;
 	else
 		byte |= 0x04;
@@ -170,11 +168,11 @@
 	byte |= 0x01;
 	pmio_write(0x7c, byte);
 
-	/*Must be 0 for K8 platform.*/
+	/* Must be 0 for K8 platform. */
 	byte = pmio_read(0x68);
 	byte &= ~0x01;
 	pmio_write(0x68, byte);
-	/*Must be 0 for K8 platform.*/
+	/* Must be 0 for K8 platform. */
 	byte = pmio_read(0x8d);
 	byte &= ~(1<<6);
 	pmio_write(0x8d, byte);
@@ -187,7 +185,7 @@
 	byte &= ~0x04;
 	pmio_write(0x42, byte);
 
-	if(get_sb600_revision() == 0x14) {
+	if (get_sb600_revision() == 0x14) {
 		pmio_write(0x89, 0x10);
 
 		byte = pmio_read(0x52);
@@ -196,7 +194,6 @@
 	}
 }
 
-
 void hard_reset(void)
 {
 	set_bios_reset();
@@ -277,7 +274,6 @@
 	pci_write_config8(dev, 0x4a, byte);
 }
 
-
 /* sbDevicesPorInitTable */
 static void sb600_devices_por_init(void)
 {

Modified: trunk/src/southbridge/amd/sb600/sb600_enable_usbdebug.c
==============================================================================
--- trunk/src/southbridge/amd/sb600/sb600_enable_usbdebug.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb600/sb600_enable_usbdebug.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -20,14 +20,12 @@
 #include <usbdebug.h>
 
 #ifndef SB600_DEVN_BASE
-
 #define SB600_DEVN_BASE 0
-
 #endif
 
-#define EHCI_BAR_INDEX 0x10
-#define EHCI_BAR 0xFEF00000
-#define EHCI_DEBUG_OFFSET 0xE0
+#define EHCI_BAR_INDEX		0x10
+#define EHCI_BAR		0xFEF00000
+#define EHCI_DEBUG_OFFSET	0xE0
 
 /* Required for successful build, but currently empty. */
 void set_debug_port(unsigned int port)
@@ -39,5 +37,5 @@
 	set_debug_port(port);
 	pci_write_config32(PCI_DEV(0, SB600_DEVN_BASE + 0x13, 5),
 			   EHCI_BAR_INDEX, EHCI_BAR);
-	pci_write_config8(PCI_DEV(0, SB600_DEVN_BASE + 0x13, 5), 0x04, 0x2);	/* mem space enabe */
+	pci_write_config8(PCI_DEV(0, SB600_DEVN_BASE + 0x13, 5), 0x04, 0x2);	/* mem space enable */
 }

Modified: trunk/src/southbridge/amd/sb600/sb600_hda.c
==============================================================================
--- trunk/src/southbridge/amd/sb600/sb600_hda.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb600/sb600_hda.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -151,6 +151,7 @@
 	0x01f71ec4,
 	0x01f71f01,
 };
+
 static u32 find_verb(u32 viddid, u32 ** verb)
 {
 	device_t azalia_dev = dev_find_slot(0, PCI_DEVFN(0x14, 2));
@@ -171,7 +172,6 @@
  *  Wait 50usec for for the codec to indicate it is ready
  *  no response would imply that the codec is non-operative
  */
-
 static int wait_for_ready(u32 base)
 {
 	/* Use a 50 usec timeout - the Linux kernel uses the
@@ -194,7 +194,6 @@
  *  the previous command.  No response would imply that the code
  *  is non-operative
  */
-
 static int wait_for_valid(u32 base)
 {
 	/* Use a 50 usec timeout - the Linux kernel uses the
@@ -301,7 +300,7 @@
 	if (!res)
 		return;
 
-	base = ((u32)res->base);
+	base = (u32)res->base;
 	printk(BIOS_DEBUG, "base = 0x%x\n", base);
 	codec_mask = codec_detect(base);
 

Modified: trunk/src/southbridge/amd/sb600/sb600_sm.c
==============================================================================
--- trunk/src/southbridge/amd/sb600/sb600_sm.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb600/sb600_sm.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -327,10 +327,10 @@
 	res->gran = 8;
 	res->flags = IORESOURCE_IO | IORESOURCE_FIXED;
 
-
 	compact_resources(dev);
 
 }
+
 static void sb600_sm_set_resources(struct device *dev)
 {
 	struct resource *res;
@@ -338,7 +338,6 @@
 
 	pci_dev_set_resources(dev);
 
-
 	/* rpr2.14: Make HPET MMIO decoding controlled by the memory enable bit in command register of LPC ISA bridage */
 	byte = pm_ioread(0x52);
 	byte |= 1 << 6;
@@ -357,6 +356,7 @@
 static struct pci_operations lops_pci = {
 	.set_subsystem = pci_dev_set_subsystem,
 };
+
 static struct device_operations smbus_ops = {
 	.read_resources = sb600_sm_read_resources,
 	.set_resources = sb600_sm_set_resources,
@@ -367,6 +367,7 @@
 	.ops_pci = &lops_pci,
 	.ops_smbus_bus = &lops_smbus_bus,
 };
+
 static const struct pci_driver smbus_driver __pci_driver = {
 	.ops = &smbus_ops,
 	.vendor = PCI_VENDOR_ID_ATI,

Modified: trunk/src/southbridge/amd/sb600/sb600_smbus.c
==============================================================================
--- trunk/src/southbridge/amd/sb600/sb600_smbus.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb600/sb600_smbus.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -114,8 +114,7 @@
 	return 0;
 }
 
-int do_smbus_read_byte(u32 smbus_io_base, u32 device,
-			      u32 address)
+int do_smbus_read_byte(u32 smbus_io_base, u32 device, u32 address)
 {
 	u8 byte;
 
@@ -145,8 +144,7 @@
 	return byte;
 }
 
-int do_smbus_write_byte(u32 smbus_io_base, u32 device,
-			       u32 address, u8 val)
+int do_smbus_write_byte(u32 smbus_io_base, u32 device, u32 address, u8 val)
 {
 	u8 byte;
 
@@ -176,8 +174,7 @@
 	return 0;
 }
 
-static void alink_ab_indx(u32 reg_space, u32 reg_addr,
-			  u32 mask, u32 val)
+static void alink_ab_indx(u32 reg_space, u32 reg_addr, u32 mask, u32 val)
 {
 	u32 tmp;
 

Modified: trunk/src/southbridge/amd/sb700/sb700_early_setup.c
==============================================================================
--- trunk/src/southbridge/amd/sb700/sb700_early_setup.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb700/sb700_early_setup.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -17,8 +17,8 @@
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
  */
 
-#ifndef  _SB700_EARLY_SETUP_C_
-#define  _SB700_EARLY_SETUP_C_
+#ifndef _SB700_EARLY_SETUP_C_
+#define _SB700_EARLY_SETUP_C_
 
 #include <reset.h>
 #include <arch/cpu.h>
@@ -40,7 +40,7 @@
 	return inb(PM_INDEX + 1);
 }
 
-/* RPR 2.28 Get SB ASIC Revision.*/
+/* RPR 2.28: Get SB ASIC Revision. */
 static u8 set_sb700_revision(void)
 {
 	device_t dev;
@@ -195,11 +195,11 @@
 	byte |= 0x03;
 	pmio_write(0x7c, byte);
 
-	/*Must be 0 for K8 platform.*/
+	/* Must be 0 for K8 platform. */
 	byte = pmio_read(0x68);
 	byte &= ~0x01;
 	pmio_write(0x68, byte);
-	/*Must be 0 for K8 platform.*/
+	/* Must be 0 for K8 platform. */
 	byte = pmio_read(0x8d);
 	byte &= ~(1<<6);
 	pmio_write(0x8d, byte);

Modified: trunk/src/southbridge/amd/sb700/sb700_enable_usbdebug.c
==============================================================================
--- trunk/src/southbridge/amd/sb700/sb700_enable_usbdebug.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb700/sb700_enable_usbdebug.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -20,14 +20,12 @@
 #include <usbdebug.h>
 
 #ifndef SB700_DEVN_BASE
-
 #define SB700_DEVN_BASE 0
-
 #endif
 
-#define EHCI_BAR_INDEX 0x10
-#define EHCI_BAR 0xFEF00000
-#define EHCI_DEBUG_OFFSET 0xE0
+#define EHCI_BAR_INDEX		0x10
+#define EHCI_BAR		0xFEF00000
+#define EHCI_DEBUG_OFFSET	0xE0
 
 /* Required for successful build, but currently empty. */
 void set_debug_port(unsigned int port)
@@ -39,5 +37,5 @@
 	set_debug_port(port);
 	pci_write_config32(PCI_DEV(0, SB700_DEVN_BASE + 0x13, 5),
 			   EHCI_BAR_INDEX, EHCI_BAR);
-	pci_write_config8(PCI_DEV(0, SB700_DEVN_BASE + 0x13, 5), 0x04, 0x2);	/* mem space enabe */
+	pci_write_config8(PCI_DEV(0, SB700_DEVN_BASE + 0x13, 5), 0x04, 0x2);	/* mem space enable */
 }

Modified: trunk/src/southbridge/amd/sb700/sb700_ide.c
==============================================================================
--- trunk/src/southbridge/amd/sb700/sb700_ide.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb700/sb700_ide.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -51,7 +51,7 @@
 	pci_write_config16(dev, 0x4, dword);
 
 	/* set ide as primary, if you want to boot from IDE, you'd better set it
-	 * in mainboard/Config.lb */
+	 * in $vendor/$mainboard/devicetree.cb */
 	if (conf->boot_switch_sata_ide == 1) {
 		byte = pci_read_config8(dev, 0xAD);
 		byte |= 1 << 4;

Modified: trunk/src/southbridge/amd/sb700/sb700_smbus.c
==============================================================================
--- trunk/src/southbridge/amd/sb700/sb700_smbus.c	Sat Sep 25 00:15:54 2010	(r5836)
+++ trunk/src/southbridge/amd/sb700/sb700_smbus.c	Sat Sep 25 01:37:25 2010	(r5837)
@@ -17,8 +17,8 @@
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301 USA
  */
 
-#ifndef  _SB700_SMBUS_C_
-#define  _SB700_SMBUS_C_
+#ifndef _SB700_SMBUS_C_
+#define _SB700_SMBUS_C_
 
 #include "sb700_smbus.h"
 
@@ -90,8 +90,7 @@
 	return byte;
 }
 
-int do_smbus_send_byte(u32 smbus_io_base, u32 device,
-			      u8 val)
+int do_smbus_send_byte(u32 smbus_io_base, u32 device, u8 val)
 {
 	u8 byte;
 
@@ -118,8 +117,7 @@
 	return 0;
 }
 
-int do_smbus_read_byte(u32 smbus_io_base, u32 device,
-			      u32 address)
+int do_smbus_read_byte(u32 smbus_io_base, u32 device, u32 address)
 {
 	u8 byte;
 
@@ -149,8 +147,7 @@
 	return byte;
 }
 
-int do_smbus_write_byte(u32 smbus_io_base, u32 device,
-			       u32 address, u8 val)
+int do_smbus_write_byte(u32 smbus_io_base, u32 device, u32 address, u8 val)
 {
 	u8 byte;
 
@@ -180,8 +177,7 @@
 	return 0;
 }
 
-static void alink_ab_indx(u32 reg_space, u32 reg_addr,
-			  u32 mask, u32 val)
+static void alink_ab_indx(u32 reg_space, u32 reg_addr, u32 mask, u32 val)
 {
 	u32 tmp;
 




More information about the coreboot mailing list