Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 1 | /* |
| 2 | * This file is part of the flashrom project. |
| 3 | * |
| 4 | * Copyright (C) 2000 Silicon Integrated System Corporation |
| 5 | * Copyright (C) 2004 Tyan Corp <yhlu@tyan.com> |
| 6 | * Copyright (C) 2005-2008 coresystems GmbH |
| 7 | * Copyright (C) 2008,2009,2010 Carl-Daniel Hailfinger |
| 8 | * |
| 9 | * This program is free software; you can redistribute it and/or modify |
| 10 | * it under the terms of the GNU General Public License as published by |
| 11 | * the Free Software Foundation; either version 2 of the License, or |
| 12 | * (at your option) any later version. |
| 13 | * |
| 14 | * This program is distributed in the hope that it will be useful, |
| 15 | * but WITHOUT ANY WARRANTY; without even the implied warranty of |
| 16 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
| 17 | * GNU General Public License for more details. |
| 18 | * |
| 19 | * You should have received a copy of the GNU General Public License |
| 20 | * along with this program; if not, write to the Free Software |
| 21 | * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA |
| 22 | */ |
| 23 | |
Carl-Daniel Hailfinger | 831e8f4 | 2010-05-30 22:24:40 +0000 | [diff] [blame] | 24 | #include <stdio.h> |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 25 | #include <fcntl.h> |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 26 | #include <sys/stat.h> |
| 27 | #include <string.h> |
| 28 | #include <stdlib.h> |
| 29 | #include <getopt.h> |
| 30 | #include "flash.h" |
| 31 | #include "flashchips.h" |
Carl-Daniel Hailfinger | 5b997c3 | 2010-07-27 22:41:39 +0000 | [diff] [blame] | 32 | #include "programmer.h" |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 33 | |
Carl-Daniel Hailfinger | ad3cc55 | 2010-07-03 11:02:10 +0000 | [diff] [blame] | 34 | static void cli_classic_usage(const char *name) |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 35 | { |
Carl-Daniel Hailfinger | 4e3391f | 2012-07-22 12:01:43 +0000 | [diff] [blame] | 36 | printf("Usage: flashrom [-h|-R|-L|" |
Carl-Daniel Hailfinger | 7112772 | 2010-05-31 15:27:27 +0000 | [diff] [blame] | 37 | #if CONFIG_PRINT_WIKI == 1 |
Carl-Daniel Hailfinger | 4e3391f | 2012-07-22 12:01:43 +0000 | [diff] [blame] | 38 | "-z|" |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 39 | #endif |
Carl-Daniel Hailfinger | 4e3391f | 2012-07-22 12:01:43 +0000 | [diff] [blame] | 40 | "-p <programmername>[:<parameters>]\n" |
| 41 | " [-E|-r <file>|-w <file>|-v <file>] [-c <chipname>]\n" |
| 42 | " [-l <file> [-i <image>]] [-n] [-f]]\n" |
| 43 | " [-V[V[V]]] [-o <logfile>]\n\n"); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 44 | |
| 45 | printf("Please note that the command line interface for flashrom has " |
| 46 | "changed between\n" |
Carl-Daniel Hailfinger | 4e3391f | 2012-07-22 12:01:43 +0000 | [diff] [blame] | 47 | "0.9.5 and 0.9.6 and will change again before flashrom 1.0.\n" |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 48 | "Do not use flashrom in scripts or other automated tools " |
| 49 | "without checking\n" |
| 50 | "that your flashrom version won't interpret options in a " |
| 51 | "different way.\n\n"); |
| 52 | |
| 53 | printf(" -h | --help print this help text\n" |
| 54 | " -R | --version print version (release)\n" |
| 55 | " -r | --read <file> read flash and save to " |
| 56 | "<file>\n" |
| 57 | " -w | --write <file> write <file> to flash\n" |
| 58 | " -v | --verify <file> verify flash against " |
| 59 | "<file>\n" |
| 60 | " -E | --erase erase flash device\n" |
| 61 | " -V | --verbose more verbose output\n" |
| 62 | " -c | --chip <chipname> probe only for specified " |
| 63 | "flash chip\n" |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 64 | " -f | --force force specific operations " |
| 65 | "(see man page)\n" |
| 66 | " -n | --noverify don't auto-verify\n" |
| 67 | " -l | --layout <file> read ROM layout from " |
| 68 | "<file>\n" |
| 69 | " -i | --image <name> only flash image <name> " |
| 70 | "from flash layout\n" |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 71 | " -o | --output <name> log to file <name>\n" |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 72 | " -L | --list-supported print supported devices\n" |
Carl-Daniel Hailfinger | 7112772 | 2010-05-31 15:27:27 +0000 | [diff] [blame] | 73 | #if CONFIG_PRINT_WIKI == 1 |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 74 | " -z | --list-supported-wiki print supported devices " |
| 75 | "in wiki syntax\n" |
| 76 | #endif |
| 77 | " -p | --programmer <name>[:<param>] specify the programmer " |
Carl-Daniel Hailfinger | a73fb49 | 2010-10-06 23:48:34 +0000 | [diff] [blame] | 78 | "device\n"); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 79 | |
Carl-Daniel Hailfinger | a73fb49 | 2010-10-06 23:48:34 +0000 | [diff] [blame] | 80 | list_programmers_linebreak(37, 80, 1); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 81 | printf("\nYou can specify one of -h, -R, -L, " |
Carl-Daniel Hailfinger | 7112772 | 2010-05-31 15:27:27 +0000 | [diff] [blame] | 82 | #if CONFIG_PRINT_WIKI == 1 |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 83 | "-z, " |
| 84 | #endif |
| 85 | "-E, -r, -w, -v or no operation.\n" |
| 86 | "If no operation is specified, flashrom will only probe for " |
| 87 | "flash chips.\n\n"); |
| 88 | } |
| 89 | |
Carl-Daniel Hailfinger | ad3cc55 | 2010-07-03 11:02:10 +0000 | [diff] [blame] | 90 | static void cli_classic_abort_usage(void) |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 91 | { |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 92 | printf("Please run \"flashrom --help\" for usage info.\n"); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 93 | exit(1); |
| 94 | } |
| 95 | |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 96 | static int check_filename(char *filename, char *type) |
| 97 | { |
| 98 | if (!filename || (filename[0] == '\0')) { |
| 99 | fprintf(stderr, "Error: No %s file specified.\n", type); |
| 100 | return 1; |
| 101 | } |
| 102 | /* Not an error, but maybe the user intended to specify a CLI option instead of a file name. */ |
| 103 | if (filename[0] == '-') |
| 104 | fprintf(stderr, "Warning: Supplied %s file name starts with -\n", type); |
| 105 | return 0; |
| 106 | } |
| 107 | |
Uwe Hermann | 394ee78 | 2011-08-20 14:14:22 +0000 | [diff] [blame] | 108 | int main(int argc, char *argv[]) |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 109 | { |
| 110 | unsigned long size; |
| 111 | /* Probe for up to three flash chips. */ |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 112 | const struct flashchip *chip = NULL; |
| 113 | struct flashctx flashes[3] = {{0}}; |
Carl-Daniel Hailfinger | 63fd902 | 2011-12-14 22:25:15 +0000 | [diff] [blame] | 114 | struct flashctx *fill_flash; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 115 | const char *name; |
Carl-Daniel Hailfinger | c40cff7 | 2011-12-20 00:19:29 +0000 | [diff] [blame] | 116 | int namelen, opt, i, j; |
Carl-Daniel Hailfinger | b428e97 | 2012-02-16 20:31:25 +0000 | [diff] [blame] | 117 | int startchip = -1, chipcount = 0, option_index = 0, force = 0; |
Carl-Daniel Hailfinger | 7112772 | 2010-05-31 15:27:27 +0000 | [diff] [blame] | 118 | #if CONFIG_PRINT_WIKI == 1 |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 119 | int list_supported_wiki = 0; |
| 120 | #endif |
Carl-Daniel Hailfinger | 082c8b5 | 2011-08-15 19:54:20 +0000 | [diff] [blame] | 121 | int read_it = 0, write_it = 0, erase_it = 0, verify_it = 0; |
| 122 | int dont_verify_it = 0, list_supported = 0, operation_specified = 0; |
Carl-Daniel Hailfinger | 2e68160 | 2011-09-08 00:00:29 +0000 | [diff] [blame] | 123 | enum programmer prog = PROGRAMMER_INVALID; |
Carl-Daniel Hailfinger | 082c8b5 | 2011-08-15 19:54:20 +0000 | [diff] [blame] | 124 | int ret = 0; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 125 | |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 126 | static const char optstring[] = "r:Rw:v:nVEfc:l:i:p:Lzho:"; |
Mathias Krause | a60faab | 2011-01-17 07:50:42 +0000 | [diff] [blame] | 127 | static const struct option long_options[] = { |
Uwe Hermann | 91f4afa | 2011-07-28 08:13:25 +0000 | [diff] [blame] | 128 | {"read", 1, NULL, 'r'}, |
| 129 | {"write", 1, NULL, 'w'}, |
| 130 | {"erase", 0, NULL, 'E'}, |
| 131 | {"verify", 1, NULL, 'v'}, |
| 132 | {"noverify", 0, NULL, 'n'}, |
| 133 | {"chip", 1, NULL, 'c'}, |
Uwe Hermann | 91f4afa | 2011-07-28 08:13:25 +0000 | [diff] [blame] | 134 | {"verbose", 0, NULL, 'V'}, |
| 135 | {"force", 0, NULL, 'f'}, |
| 136 | {"layout", 1, NULL, 'l'}, |
| 137 | {"image", 1, NULL, 'i'}, |
| 138 | {"list-supported", 0, NULL, 'L'}, |
| 139 | {"list-supported-wiki", 0, NULL, 'z'}, |
| 140 | {"programmer", 1, NULL, 'p'}, |
| 141 | {"help", 0, NULL, 'h'}, |
| 142 | {"version", 0, NULL, 'R'}, |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 143 | {"output", 1, NULL, 'o'}, |
Uwe Hermann | 91f4afa | 2011-07-28 08:13:25 +0000 | [diff] [blame] | 144 | {NULL, 0, NULL, 0}, |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 145 | }; |
| 146 | |
| 147 | char *filename = NULL; |
Carl-Daniel Hailfinger | 4628445 | 2012-01-11 02:10:11 +0000 | [diff] [blame] | 148 | char *layoutfile = NULL; |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 149 | char *logfile = NULL; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 150 | char *tempstr = NULL; |
Carl-Daniel Hailfinger | ad3cc55 | 2010-07-03 11:02:10 +0000 | [diff] [blame] | 151 | char *pparam = NULL; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 152 | |
| 153 | print_version(); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 154 | print_banner(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 155 | |
| 156 | if (selfcheck()) |
| 157 | exit(1); |
| 158 | |
| 159 | setbuf(stdout, NULL); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 160 | /* FIXME: Delay all operation_specified checks until after command |
| 161 | * line parsing to allow --help overriding everything else. |
| 162 | */ |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 163 | while ((opt = getopt_long(argc, argv, optstring, |
| 164 | long_options, &option_index)) != EOF) { |
| 165 | switch (opt) { |
| 166 | case 'r': |
| 167 | if (++operation_specified > 1) { |
| 168 | fprintf(stderr, "More than one operation " |
| 169 | "specified. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 170 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 171 | } |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 172 | filename = strdup(optarg); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 173 | read_it = 1; |
| 174 | break; |
| 175 | case 'w': |
| 176 | if (++operation_specified > 1) { |
| 177 | fprintf(stderr, "More than one operation " |
| 178 | "specified. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 179 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 180 | } |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 181 | filename = strdup(optarg); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 182 | write_it = 1; |
| 183 | break; |
| 184 | case 'v': |
| 185 | //FIXME: gracefully handle superfluous -v |
| 186 | if (++operation_specified > 1) { |
| 187 | fprintf(stderr, "More than one operation " |
| 188 | "specified. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 189 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 190 | } |
| 191 | if (dont_verify_it) { |
| 192 | fprintf(stderr, "--verify and --noverify are" |
| 193 | "mutually exclusive. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 194 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 195 | } |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 196 | filename = strdup(optarg); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 197 | verify_it = 1; |
| 198 | break; |
| 199 | case 'n': |
| 200 | if (verify_it) { |
| 201 | fprintf(stderr, "--verify and --noverify are" |
| 202 | "mutually exclusive. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 203 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 204 | } |
| 205 | dont_verify_it = 1; |
| 206 | break; |
| 207 | case 'c': |
| 208 | chip_to_probe = strdup(optarg); |
| 209 | break; |
| 210 | case 'V': |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 211 | verbose_screen++; |
| 212 | if (verbose_screen > MSG_DEBUG2) |
| 213 | verbose_logfile = verbose_screen; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 214 | break; |
| 215 | case 'E': |
| 216 | if (++operation_specified > 1) { |
| 217 | fprintf(stderr, "More than one operation " |
| 218 | "specified. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 219 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 220 | } |
| 221 | erase_it = 1; |
| 222 | break; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 223 | case 'f': |
| 224 | force = 1; |
| 225 | break; |
| 226 | case 'l': |
Carl-Daniel Hailfinger | 4628445 | 2012-01-11 02:10:11 +0000 | [diff] [blame] | 227 | if (layoutfile) { |
| 228 | fprintf(stderr, "Error: --layout specified " |
| 229 | "more than once. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 230 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | 4628445 | 2012-01-11 02:10:11 +0000 | [diff] [blame] | 231 | } |
| 232 | layoutfile = strdup(optarg); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 233 | break; |
| 234 | case 'i': |
| 235 | tempstr = strdup(optarg); |
Louis Yung-Chieh Lo | 9bcf268 | 2011-12-25 09:12:16 +0000 | [diff] [blame] | 236 | if (register_include_arg(tempstr)) |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 237 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 238 | break; |
| 239 | case 'L': |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 240 | if (++operation_specified > 1) { |
| 241 | fprintf(stderr, "More than one operation " |
| 242 | "specified. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 243 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 244 | } |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 245 | list_supported = 1; |
| 246 | break; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 247 | case 'z': |
Carl-Daniel Hailfinger | 7112772 | 2010-05-31 15:27:27 +0000 | [diff] [blame] | 248 | #if CONFIG_PRINT_WIKI == 1 |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 249 | if (++operation_specified > 1) { |
| 250 | fprintf(stderr, "More than one operation " |
| 251 | "specified. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 252 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 253 | } |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 254 | list_supported_wiki = 1; |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 255 | #else |
| 256 | fprintf(stderr, "Error: Wiki output was not compiled " |
| 257 | "in. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 258 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 259 | #endif |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 260 | break; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 261 | case 'p': |
Carl-Daniel Hailfinger | 2e68160 | 2011-09-08 00:00:29 +0000 | [diff] [blame] | 262 | if (prog != PROGRAMMER_INVALID) { |
| 263 | fprintf(stderr, "Error: --programmer specified " |
| 264 | "more than once. You can separate " |
| 265 | "multiple\nparameters for a programmer " |
| 266 | "with \",\". Please see the man page " |
| 267 | "for details.\n"); |
| 268 | cli_classic_abort_usage(); |
| 269 | } |
| 270 | for (prog = 0; prog < PROGRAMMER_INVALID; prog++) { |
| 271 | name = programmer_table[prog].name; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 272 | namelen = strlen(name); |
| 273 | if (strncmp(optarg, name, namelen) == 0) { |
| 274 | switch (optarg[namelen]) { |
| 275 | case ':': |
Carl-Daniel Hailfinger | ad3cc55 | 2010-07-03 11:02:10 +0000 | [diff] [blame] | 276 | pparam = strdup(optarg + namelen + 1); |
| 277 | if (!strlen(pparam)) { |
| 278 | free(pparam); |
| 279 | pparam = NULL; |
Carl-Daniel Hailfinger | 2702376 | 2010-04-28 15:22:14 +0000 | [diff] [blame] | 280 | } |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 281 | break; |
| 282 | case '\0': |
| 283 | break; |
| 284 | default: |
| 285 | /* The continue refers to the |
| 286 | * for loop. It is here to be |
| 287 | * able to differentiate between |
| 288 | * foo and foobar. |
| 289 | */ |
| 290 | continue; |
| 291 | } |
| 292 | break; |
| 293 | } |
| 294 | } |
Carl-Daniel Hailfinger | 2e68160 | 2011-09-08 00:00:29 +0000 | [diff] [blame] | 295 | if (prog == PROGRAMMER_INVALID) { |
Carl-Daniel Hailfinger | 4e3391f | 2012-07-22 12:01:43 +0000 | [diff] [blame] | 296 | fprintf(stderr, "Error: Unknown programmer \"%s\". Valid choices are:\n", |
| 297 | optarg); |
| 298 | list_programmers_linebreak(0, 80, 0); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 299 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 300 | } |
| 301 | break; |
| 302 | case 'R': |
| 303 | /* print_version() is always called during startup. */ |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 304 | if (++operation_specified > 1) { |
| 305 | fprintf(stderr, "More than one operation " |
| 306 | "specified. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 307 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 308 | } |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 309 | exit(0); |
| 310 | break; |
| 311 | case 'h': |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 312 | if (++operation_specified > 1) { |
| 313 | fprintf(stderr, "More than one operation " |
| 314 | "specified. Aborting.\n"); |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 315 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 316 | } |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 317 | cli_classic_usage(argv[0]); |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 318 | exit(0); |
| 319 | break; |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 320 | case 'o': |
| 321 | #ifdef STANDALONE |
| 322 | fprintf(stderr, "Log file not supported in standalone mode. Aborting.\n"); |
| 323 | cli_classic_abort_usage(); |
| 324 | #else /* STANDALONE */ |
| 325 | logfile = strdup(optarg); |
| 326 | if (logfile[0] == '\0') { |
| 327 | fprintf(stderr, "No log filename specified.\n"); |
| 328 | cli_classic_abort_usage(); |
| 329 | } |
| 330 | #endif /* STANDALONE */ |
| 331 | break; |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 332 | default: |
Uwe Hermann | 2db77a0 | 2010-06-04 17:07:39 +0000 | [diff] [blame] | 333 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 334 | break; |
| 335 | } |
| 336 | } |
| 337 | |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 338 | if (optind < argc) { |
| 339 | fprintf(stderr, "Error: Extra parameter found.\n"); |
| 340 | cli_classic_abort_usage(); |
| 341 | } |
| 342 | |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 343 | if ((read_it | write_it | verify_it) && check_filename(filename, "image")) { |
| 344 | cli_classic_abort_usage(); |
| 345 | } |
| 346 | if (layoutfile && check_filename(layoutfile, "layout")) { |
| 347 | cli_classic_abort_usage(); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 348 | } |
| 349 | |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 350 | #ifndef STANDALONE |
| 351 | if (logfile && check_filename(logfile, "log")) |
| 352 | cli_classic_abort_usage(); |
| 353 | if (logfile && open_logfile(logfile)) |
| 354 | return 1; |
| 355 | #endif /* !STANDALONE */ |
| 356 | |
Carl-Daniel Hailfinger | 7112772 | 2010-05-31 15:27:27 +0000 | [diff] [blame] | 357 | #if CONFIG_PRINT_WIKI == 1 |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 358 | if (list_supported_wiki) { |
| 359 | print_supported_wiki(); |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 360 | goto out; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 361 | } |
| 362 | #endif |
| 363 | |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 364 | if (list_supported) { |
Niklas Söderlund | ede2fa4 | 2012-10-23 13:06:46 +0000 | [diff] [blame] | 365 | if (print_supported()) |
| 366 | ret = 1; |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 367 | goto out; |
| 368 | } |
Carl-Daniel Hailfinger | 4628445 | 2012-01-11 02:10:11 +0000 | [diff] [blame] | 369 | |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 370 | #ifndef STANDALONE |
| 371 | start_logging(); |
| 372 | #endif /* !STANDALONE */ |
| 373 | |
| 374 | print_buildinfo(); |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 375 | msg_gdbg("Command line (%i args):", argc - 1); |
| 376 | for (i = 0; i < argc; i++) { |
| 377 | msg_gdbg(" %s", argv[i]); |
| 378 | } |
| 379 | msg_gdbg("\n"); |
| 380 | |
| 381 | if (layoutfile && read_romlayout(layoutfile)) { |
| 382 | ret = 1; |
| 383 | goto out; |
| 384 | } |
| 385 | if (process_include_args()) { |
| 386 | ret = 1; |
| 387 | goto out; |
| 388 | } |
Carl-Daniel Hailfinger | 63fd902 | 2011-12-14 22:25:15 +0000 | [diff] [blame] | 389 | /* Does a chip with the requested name exist in the flashchips array? */ |
Carl-Daniel Hailfinger | 2702376 | 2010-04-28 15:22:14 +0000 | [diff] [blame] | 390 | if (chip_to_probe) { |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 391 | for (chip = flashchips; chip && chip->name; chip++) |
| 392 | if (!strcmp(chip->name, chip_to_probe)) |
Carl-Daniel Hailfinger | 2702376 | 2010-04-28 15:22:14 +0000 | [diff] [blame] | 393 | break; |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 394 | if (!chip || !chip->name) { |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 395 | msg_cerr("Error: Unknown chip '%s' specified.\n", chip_to_probe); |
| 396 | msg_gerr("Run flashrom -L to view the hardware supported in this flashrom version.\n"); |
| 397 | ret = 1; |
| 398 | goto out; |
Carl-Daniel Hailfinger | 2702376 | 2010-04-28 15:22:14 +0000 | [diff] [blame] | 399 | } |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 400 | /* Keep chip around for later usage in case a forced read is requested. */ |
Carl-Daniel Hailfinger | 2702376 | 2010-04-28 15:22:14 +0000 | [diff] [blame] | 401 | } |
Carl-Daniel Hailfinger | 8841d3e | 2010-05-15 15:04:37 +0000 | [diff] [blame] | 402 | |
Carl-Daniel Hailfinger | 4e3391f | 2012-07-22 12:01:43 +0000 | [diff] [blame] | 403 | if (prog == PROGRAMMER_INVALID) { |
Stefan Tauner | fd0d413 | 2012-09-25 21:24:55 +0000 | [diff] [blame] | 404 | if (CONFIG_DEFAULT_PROGRAMMER != PROGRAMMER_INVALID) { |
| 405 | prog = CONFIG_DEFAULT_PROGRAMMER; |
| 406 | msg_pinfo("Using default programmer \"%s\".\n", |
| 407 | programmer_table[CONFIG_DEFAULT_PROGRAMMER].name); |
| 408 | } else { |
| 409 | msg_perr("Please select a programmer with the --programmer parameter.\n" |
| 410 | "Valid choices are:\n"); |
| 411 | list_programmers_linebreak(0, 80, 0); |
| 412 | ret = 1; |
| 413 | goto out; |
| 414 | } |
Carl-Daniel Hailfinger | 4e3391f | 2012-07-22 12:01:43 +0000 | [diff] [blame] | 415 | } |
Carl-Daniel Hailfinger | 2e68160 | 2011-09-08 00:00:29 +0000 | [diff] [blame] | 416 | |
Carl-Daniel Hailfinger | 4988420 | 2010-05-22 07:10:46 +0000 | [diff] [blame] | 417 | /* FIXME: Delay calibration should happen in programmer code. */ |
| 418 | myusec_calibrate_delay(); |
| 419 | |
Carl-Daniel Hailfinger | 2e68160 | 2011-09-08 00:00:29 +0000 | [diff] [blame] | 420 | if (programmer_init(prog, pparam)) { |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 421 | msg_perr("Error: Programmer initialization failed.\n"); |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 422 | ret = 1; |
| 423 | goto out_shutdown; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 424 | } |
Carl-Daniel Hailfinger | c40cff7 | 2011-12-20 00:19:29 +0000 | [diff] [blame] | 425 | tempstr = flashbuses_to_text(get_buses_supported()); |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 426 | msg_pdbg("The following protocols are supported: %s.\n", tempstr); |
Carl-Daniel Hailfinger | eaacd2d | 2011-11-09 23:40:00 +0000 | [diff] [blame] | 427 | free(tempstr); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 428 | |
Carl-Daniel Hailfinger | c40cff7 | 2011-12-20 00:19:29 +0000 | [diff] [blame] | 429 | for (j = 0; j < registered_programmer_count; j++) { |
| 430 | startchip = 0; |
Michael Karcher | 222bf10 | 2011-12-22 23:27:03 +0000 | [diff] [blame] | 431 | while (chipcount < ARRAY_SIZE(flashes)) { |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 432 | startchip = probe_flash(®istered_programmers[j], startchip, &flashes[chipcount], 0); |
Carl-Daniel Hailfinger | c40cff7 | 2011-12-20 00:19:29 +0000 | [diff] [blame] | 433 | if (startchip == -1) |
| 434 | break; |
| 435 | chipcount++; |
| 436 | startchip++; |
| 437 | } |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 438 | } |
| 439 | |
Carl-Daniel Hailfinger | 4c82318 | 2011-05-04 00:39:50 +0000 | [diff] [blame] | 440 | if (chipcount > 1) { |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 441 | msg_cinfo("Multiple flash chips were detected: \"%s\"", flashes[0].chip->name); |
Stefan Tauner | 716e098 | 2011-07-25 20:38:52 +0000 | [diff] [blame] | 442 | for (i = 1; i < chipcount; i++) |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 443 | msg_cinfo(", \"%s\"", flashes[i].chip->name); |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 444 | msg_cinfo("\nPlease specify which chip to use with the -c <chipname> option.\n"); |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 445 | ret = 1; |
| 446 | goto out_shutdown; |
Carl-Daniel Hailfinger | 4c82318 | 2011-05-04 00:39:50 +0000 | [diff] [blame] | 447 | } else if (!chipcount) { |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 448 | msg_cinfo("No EEPROM/flash device found.\n"); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 449 | if (!force || !chip_to_probe) { |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 450 | msg_cinfo("Note: flashrom can never write if the flash chip isn't found " |
| 451 | "automatically.\n"); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 452 | } |
| 453 | if (force && read_it && chip_to_probe) { |
Carl-Daniel Hailfinger | b428e97 | 2012-02-16 20:31:25 +0000 | [diff] [blame] | 454 | struct registered_programmer *pgm; |
| 455 | int compatible_programmers = 0; |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 456 | msg_cinfo("Force read (-f -r -c) requested, pretending the chip is there:\n"); |
Carl-Daniel Hailfinger | b428e97 | 2012-02-16 20:31:25 +0000 | [diff] [blame] | 457 | /* This loop just counts compatible controllers. */ |
| 458 | for (j = 0; j < registered_programmer_count; j++) { |
| 459 | pgm = ®istered_programmers[j]; |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 460 | /* chip is still set from the chip_to_probe earlier in this function. */ |
| 461 | if (pgm->buses_supported & chip->bustype) |
Carl-Daniel Hailfinger | b428e97 | 2012-02-16 20:31:25 +0000 | [diff] [blame] | 462 | compatible_programmers++; |
| 463 | } |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 464 | if (!compatible_programmers) { |
| 465 | msg_cinfo("No compatible controller found for the requested flash chip.\n"); |
| 466 | ret = 1; |
| 467 | goto out_shutdown; |
| 468 | } |
Carl-Daniel Hailfinger | b428e97 | 2012-02-16 20:31:25 +0000 | [diff] [blame] | 469 | if (compatible_programmers > 1) |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 470 | msg_cinfo("More than one compatible controller found for the requested flash " |
| 471 | "chip, using the first one.\n"); |
Carl-Daniel Hailfinger | b428e97 | 2012-02-16 20:31:25 +0000 | [diff] [blame] | 472 | for (j = 0; j < registered_programmer_count; j++) { |
| 473 | pgm = ®istered_programmers[j]; |
| 474 | startchip = probe_flash(pgm, 0, &flashes[0], 1); |
| 475 | if (startchip != -1) |
| 476 | break; |
| 477 | } |
Carl-Daniel Hailfinger | 4c82318 | 2011-05-04 00:39:50 +0000 | [diff] [blame] | 478 | if (startchip == -1) { |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 479 | // FIXME: This should never happen! Ask for a bug report? |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 480 | msg_cinfo("Probing for flash chip '%s' failed.\n", chip_to_probe); |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 481 | ret = 1; |
| 482 | goto out_shutdown; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 483 | } |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 484 | msg_cinfo("Please note that forced reads most likely contain garbage.\n"); |
| 485 | ret = read_flash_to_file(&flashes[0], filename); |
| 486 | goto out_shutdown; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 487 | } |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 488 | ret = 1; |
| 489 | goto out_shutdown; |
Stefan Tauner | 1d94763 | 2011-09-11 22:08:58 +0000 | [diff] [blame] | 490 | } else if (!chip_to_probe) { |
| 491 | /* repeat for convenience when looking at foreign logs */ |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 492 | tempstr = flashbuses_to_text(flashes[0].chip->bustype); |
Stefan Tauner | 1d94763 | 2011-09-11 22:08:58 +0000 | [diff] [blame] | 493 | msg_gdbg("Found %s flash chip \"%s\" (%d kB, %s).\n", |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 494 | flashes[0].chip->vendor, flashes[0].chip->name, flashes[0].chip->total_size, tempstr); |
Stefan Tauner | 1d94763 | 2011-09-11 22:08:58 +0000 | [diff] [blame] | 495 | free(tempstr); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 496 | } |
| 497 | |
Carl-Daniel Hailfinger | 4c82318 | 2011-05-04 00:39:50 +0000 | [diff] [blame] | 498 | fill_flash = &flashes[0]; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 499 | |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 500 | check_chip_supported(fill_flash->chip); |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 501 | |
Carl-Daniel Hailfinger | 5a7cb84 | 2012-08-25 01:17:58 +0000 | [diff] [blame] | 502 | size = fill_flash->chip->total_size * 1024; |
| 503 | if (check_max_decode(fill_flash->pgm->buses_supported & fill_flash->chip->bustype, size) && (!force)) { |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 504 | msg_cerr("Chip is too big for this programmer (-V gives details). Use --force to override.\n"); |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 505 | ret = 1; |
| 506 | goto out_shutdown; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 507 | } |
| 508 | |
| 509 | if (!(read_it | write_it | verify_it | erase_it)) { |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 510 | msg_ginfo("No operations were specified.\n"); |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 511 | goto out_shutdown; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 512 | } |
| 513 | |
| 514 | /* Always verify write operations unless -n is used. */ |
| 515 | if (write_it && !dont_verify_it) |
| 516 | verify_it = 1; |
| 517 | |
Carl-Daniel Hailfinger | 9ad4255 | 2010-09-15 10:20:16 +0000 | [diff] [blame] | 518 | /* FIXME: We should issue an unconditional chip reset here. This can be |
| 519 | * done once we have a .reset function in struct flashchip. |
| 520 | * Give the chip time to settle. |
| 521 | */ |
| 522 | programmer_delay(100000); |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 523 | ret |= doit(fill_flash, force, filename, read_it, write_it, erase_it, verify_it); |
| 524 | /* Note: doit() already calls programmer_shutdown(). */ |
| 525 | goto out; |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 526 | |
| 527 | out_shutdown: |
| 528 | programmer_shutdown(); |
Carl-Daniel Hailfinger | 901a3ba | 2012-05-14 22:54:58 +0000 | [diff] [blame] | 529 | out: |
Carl-Daniel Hailfinger | 1c15548 | 2012-06-06 09:17:06 +0000 | [diff] [blame] | 530 | #ifndef STANDALONE |
| 531 | ret |= close_logfile(); |
| 532 | #endif /* !STANDALONE */ |
Carl-Daniel Hailfinger | d566014 | 2011-07-15 23:47:45 +0000 | [diff] [blame] | 533 | return ret; |
Carl-Daniel Hailfinger | a84835a | 2010-01-07 03:24:05 +0000 | [diff] [blame] | 534 | } |