summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMartin Roth <martin.roth@se-eng.com>2014-12-09 13:51:49 -0700
committerMartin Roth <gaumless@gmail.com>2014-12-17 02:30:12 +0100
commit226db05a15a0c6ee55a500117307bdcf119a1822 (patch)
treeea68081d47298de7ee49c23b95845cda649765a1
parent474313d1b6556ad8b3e6750a4096b5735cbfc846 (diff)
southbridge/sis: Spelling/comment fixes
Change-Id: I6a0f5406fb3bc3e8aa3a1111b1d702f530c9329b Signed-off-by: Martin Roth <martin.roth@se-eng.com> Reviewed-on: http://review.coreboot.org/7733 Tested-by: build bot (Jenkins) Reviewed-by: Ronald G. Minnich <rminnich@gmail.com>
-rw-r--r--src/southbridge/sis/sis966/lpc.c6
-rw-r--r--src/southbridge/sis/sis966/sis761.c8
-rw-r--r--src/southbridge/sis/sis966/sis966.c7
3 files changed, 13 insertions, 8 deletions
diff --git a/src/southbridge/sis/sis966/lpc.c b/src/southbridge/sis/sis966/lpc.c
index 0e11c6f9b7..a61883bd77 100644
--- a/src/southbridge/sis/sis966/lpc.c
+++ b/src/southbridge/sis/sis966/lpc.c
@@ -185,7 +185,7 @@ static void sis966_lpc_read_resources(device_t dev)
/**
* Enable resources for children devices.
*
- * @param dev The device whos children's resources are to be enabled.
+ * @param dev The device whose children's resources are to be enabled.
*/
static void sis966_lpc_enable_childrens_resources(device_t dev)
{
@@ -212,11 +212,11 @@ static void sis966_lpc_enable_childrens_resources(device_t dev)
reg |= (1<<0); break;
case 0x2f8: // COM2
reg |= (1<<1); break;
- case 0x378: // Parallal 1
+ case 0x378: // Parallel 1
reg |= (1<<24); break;
case 0x3f0: // FD0
reg |= (1<<20); break;
- case 0x220: // Aduio 0
+ case 0x220: // Audio 0
reg |= (1<<8); break;
case 0x300: // Midi 0
reg |= (1<<12); break;
diff --git a/src/southbridge/sis/sis966/sis761.c b/src/southbridge/sis/sis966/sis761.c
index 79a64f91bf..c92b666f2c 100644
--- a/src/southbridge/sis/sis966/sis761.c
+++ b/src/southbridge/sis/sis966/sis761.c
@@ -22,7 +22,7 @@
/*
* Turn off machine check triggers when reading
* pci space where there are no devices.
- * This is necessary when scaning the bus for
+ * This is necessary when scanning the bus for
* devices which is done by the kernel
*
* written in 2003 by Eric Biederman
@@ -57,13 +57,13 @@ static inline msr_t rdmsr(unsigned index)
/**
* Read resources for AGP aperture.
*
- * There is only one AGP aperture resource needed. The resoruce is added to
+ * There is only one AGP aperture resource needed. The resource is added to
* the northbridge of BSP.
*
* The same trick can be used to augment legacy VGA resources which can
* be detect by generic PCI resource allocator for VGA devices.
* BAD: it is more tricky than I think, the resource allocation code is
- * implemented in a way to NOT DOING legacy VGA resource allcation on
+ * implemented in a way to NOT DOING legacy VGA resource allocation on
* purpose :-(.
*
* @param dev TODO
@@ -74,7 +74,7 @@ static void sis761_read_resources(device_t dev)
printk(BIOS_DEBUG, "sis761_read_resources ------->\n");
pci_dev_read_resources(dev);
- /* If we are not the first processor don't allocate the gart apeture */
+ /* If we are not the first processor don't allocate the gart aperture */
if (dev->path.pci.devfn != PCI_DEVFN(0x0, 0)) {
printk(BIOS_DEBUG, "sis761_not_the_first_processor !!!\n");
return;
diff --git a/src/southbridge/sis/sis966/sis966.c b/src/southbridge/sis/sis966/sis966.c
index 68da9170fd..5fecd3cccc 100644
--- a/src/southbridge/sis/sis966/sis966.c
+++ b/src/southbridge/sis/sis966/sis966.c
@@ -180,7 +180,12 @@ void sis966_enable(device_t dev)
if (!dev->enabled) {
final_reg |= (1 << index);// disable it
- //The reason for using final_reg, if diable func 1, the func 2 will be func 1 so We need disable them one time.
+ /*
+ * The reason for using final_reg is that if func 1 is disabled,
+ * then func 2 will become func 1.
+ * Because of this, we need loop through disabling them all at
+ * the same time.
+ */
}
if(index == 9 ) { //NIC1 is the final, We need update final reg to 0xe8