commit 0dcbecbd254d89c2593005497fe98ef5b9901fe4
author Simon Glass <sjg@chromium.org> Thu Mar 22 21:38:55 2012 -0700
committer Gerrit <chrome-bot@google.com> Fri Mar 23 23:33:31 2012 -0700
tree 57e46d51bfaf2858111fda584dd8c4786451a7af
parent 66c1a9f5b73c03ebb6afd1bf927a0273bc99df3c

cros_bundle_firmware: Change flash destination to lower case

Having the first character upper case is a bit odd, and not very friendly
if we want to provide this function to the user.

Change to using lower case, so Sdmmc becomes sdmmc, etc.

BUG=chromium-os:28242
TEST=run and flash Seaboard SPI

Change-Id: I78ea0df67fc80e8fdea84d5f183f7b2e159b90f5
Reviewed-on: https://gerrit.chromium.org/gerrit/18932
Tested-by: Simon Glass <sjg@chromium.org>
Commit-Ready: Simon Glass <sjg@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>

host/lib/write_firmware.py

diff --git a/host/lib/write_firmware.py b/host/lib/write_firmware.py
index cf2eef3..fcd1dba 100644
--- a/host/lib/write_firmware.py
+++ b/host/lib/write_firmware.py
@@ -64,7 +64,7 @@
       payload_size: Size of payload in bytes.
       update: Use faster update algorithm rather then full device erase
       verify: Verify the write by doing a readback and CRC
-      boot_type: The source for bootdevice (Nand, Sdmmc, or Spi)
+      boot_type: The source for bootdevice (nand, sdmmc, or spi)
       checksum: The checksum of the payload (an integer)
       bus: The bus number
 
@@ -77,9 +77,9 @@
     """
     replace_me = 'zsHEXYla'
     page_size = 4096
-    if boot_type == 'Sdmmc':
+    if boot_type == 'sdmmc':
       page_size = 512
-    if boot_type != 'Spi':
+    if boot_type != 'spi':
       update = False
 
     cmds = [
@@ -91,14 +91,14 @@
             checksum,
         'setenv _clear  "echo Clearing RAM; mw.b     ${address} 0 ${length}"',
     ]
-    if boot_type == 'Nand':
+    if boot_type == 'nand':
       cmds.extend([
           'setenv _init   "echo Init NAND;  nand info"',
           'setenv _erase  "echo Erase NAND; nand erase            0 ${length}"',
           'setenv _write  "echo Write NAND; nand write ${address} 0 ${length}"',
           'setenv _read   "echo Read NAND;  nand read  ${address} 0 ${length}"',
       ])
-    elif boot_type == 'Sdmmc':
+    elif boot_type == 'sdmmc':
       cmds.extend([
           'setenv _init   "echo Init EMMC;  mmc rescan            0"',
           'setenv _erase  "echo Erase EMMC; "',
@@ -159,7 +159,7 @@
       payload: Full path to payload.
       update: Use faster update algorithm rather then full device erase
       verify: Verify the write by doing a readback and CRC
-      boot_type: the src for bootdevice (Nand, Sdmmc, or Spi)
+      boot_type: the src for bootdevice (nand, sdmmc, or spi)
 
     Returns:
       Filename of the flasher binary created.
@@ -236,7 +236,7 @@
     match = re.compile('DevType\[0\] = NvBootDevType_(?P<boot>([a-zA-Z])+);')
     bct_dumped = self._tools.Run('bct_dump', [bct]).splitlines()
     boot_type = filter(match.match, bct_dumped)
-    boot_type = match.match(boot_type[0]).group('boot')
+    boot_type = match.match(boot_type[0]).group('boot').lower()
 
     flasher = self.PrepareFlasher(uboot, payload, self.update, self.verify,
                                   boot_type, 0)