From patchwork Tue Nov  9 09:17:07 2010
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: adjust some code/comment of sb700 sata init
Date: Tue, 09 Nov 2010 09:17:07 -0000
From: Jason Wang <wangqingpei@gmail.com>
X-Patchwork-Id: 2279
Message-Id: <AANLkTikWWmHun5nMT=P1w5ok+Zgzuph+7UAbD2JaAsp0@mail.gmail.com>
To: Coreboot <coreboot@coreboot.org>

the inline comment of sata_init function seems not placed correctly. rearrange
it.

Signed-off-by: Wang Qing Pei <wangqingpei@gmail.com>

Best wishes
Wang Qing Pei
Phone: 86+18930528086
Acked-by: Marc Jones <marcj303@gmail.com>


Index: src/southbridge/amd/sb700/sb700_sata.c
===================================================================
--- src/southbridge/amd/sb700/sb700_sata.c	(revision 6049)
+++ src/southbridge/amd/sb700/sb700_sata.c	(working copy)
@@ -91,14 +91,13 @@
 
 	device_t sm_dev;
 	/* SATA SMBus Disable */
-	/* sm_dev = pci_locate_device(PCI_ID(0x1002, 0x4385), 0); */
 	sm_dev = dev_find_slot(0, PCI_DEVFN(0x14, 0));
-	/* Disable SATA SMBUS */
+
 	byte = pci_read_config8(sm_dev, 0xad);
-	byte |= (1 << 1);
-	/* Enable SATA and power saving */
-	byte = pci_read_config8(sm_dev, 0xad);
+	/* Disable SATA SMBUS */
 	byte |= (1 << 0);
+	/* Enable SATA and power saving */
+	byte |= (1 << 1);
 	byte |= (1 << 5);
 	pci_write_config8(sm_dev, 0xad, byte);