- flashrom-gerrit - mail.coreboot.org

Change in flashrom[master]: linux_mtd: Fix compilation errors
by David Hendricks (Code Review) 17 Nov '17

17 Nov '17

David Hendricks has uploaded this change for review. ( https://review.coreboot.org/22502 Change subject: linux_mtd: Fix compilation errors ...................................................................... linux_mtd: Fix compilation errors This fixes compilation errors from the initial import patch. Change-Id: I216fab281f7a20a5819de3adf610b770a0b3d37c Signed-off-by: David Hendricks <dhendricks(a)fb.com> --- M linux_mtd.c 1 file changed, 26 insertions(+), 37 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/02/22502/1 diff --git a/linux_mtd.c b/linux_mtd.c index e1b11cd..07db2e9 100644 --- a/linux_mtd.c +++ b/linux_mtd.c @@ -22,6 +22,7 @@ #include <fcntl.h> #include <inttypes.h> #include <libgen.h> +#include <limits.h> #include <stdio.h> #include <stdlib.h> #include <mtd/mtd-user.h> @@ -42,32 +43,12 @@ static int dev_fd = -1; -static char *mtd_device_name; static int mtd_device_is_writeable; /* Size info is presented in bytes in sysfs. */ static unsigned long int mtd_total_size; static unsigned long int mtd_numeraseregions; static unsigned long int mtd_erasesize; /* only valid if numeraseregions is 0 */ - -static int stat_mtd_files(char *dev_path, char *sysfs_path) -{ - struct stat s; - - errno = 0; - if (stat(dev_path, &s) < 0) { - msg_pdbg("Cannot stat \"%s\": %s\n", dev_path, strerror(errno)); - return 1; - } - - if (lstat(sysfs_path, &s) < 0) { - msg_pdbg("Cannot stat \"%s\" : %s\n", - sysfs_path, strerror(errno)); - return 1; - } - - return 0; -} /* read a string from a sysfs file and sanitize it */ static int read_sysfs_string(const char *filename, char *buf, int len) @@ -109,7 +90,6 @@ static int read_sysfs_int(const char *filename, unsigned long int *val) { - uint32_t tmp; char buf[32]; char *endptr; @@ -181,17 +161,20 @@ return 0; } -static int linux_mtd_probe(struct flashchip *flash) +static int linux_mtd_probe(struct flashctx *flash) { - flash->tested = TEST_OK_PREW; - flash->total_size = mtd_total_size / 1024; /* bytes -> kB */ - flash->block_erasers[0].eraseblocks[0].size = mtd_erasesize; - flash->block_erasers[0].eraseblocks[0].count = + flash->chip->tested.probe = OK; + flash->chip->tested.read = OK; + flash->chip->tested.erase = OK; + flash->chip->tested.write = OK; + flash->chip->total_size = mtd_total_size / 1024; /* bytes -> kB */ + flash->chip->block_erasers[0].eraseblocks[0].size = mtd_erasesize; + flash->chip->block_erasers[0].eraseblocks[0].count = mtd_total_size / mtd_erasesize; return 1; } -static int linux_mtd_read(struct flashchip *flash, uint8_t *buf, +static int linux_mtd_read(struct flashctx *flash, uint8_t *buf, unsigned int start, unsigned int len) { if (lseek(dev_fd, start, SEEK_SET) != start) { @@ -209,13 +192,13 @@ } /* this version assumes we must divide the write request into pages ourselves */ -static int linux_mtd_write(struct flashchip *flash, uint8_t *buf, +static int linux_mtd_write(struct flashctx *flash, const uint8_t *buf, unsigned int start, unsigned int len) { unsigned int page; unsigned int chunksize, page_size; - chunksize = page_size = flash->page_size; + chunksize = page_size = flash->chip->page_size; if (!mtd_device_is_writeable) return 1; @@ -246,10 +229,9 @@ return 0; } -static int linux_mtd_erase(struct flashchip *flash, +static int linux_mtd_erase(struct flashctx *flash, unsigned int start, unsigned int len) { - struct region_info_user region_info; uint32_t u; if (mtd_numeraseregions != 0) { @@ -272,7 +254,7 @@ return 0; } -static struct opaque_programmer programmer_linux_mtd = { +static struct opaque_master opaque_master_linux_mtd = { /* FIXME: Do we need to change these (especially write) as per * page size requirements? */ .max_data_read = MAX_DATA_UNSPECIFIED, @@ -318,8 +300,18 @@ msg_pdbg("%s: sysfs_path: \"%s\", dev_path: \"%s\"\n", __func__, sysfs_path, dev_path); - if (stat_mtd_files(dev_path, sysfs_path)) + struct stat s; + errno = 0; + if (stat(dev_path, &s) < 0) { + msg_pdbg("Cannot stat \"%s\": %s\n", dev_path, strerror(errno)); goto linux_mtd_setup_exit; + } + + if (lstat(sysfs_path, &s) < 0) { + msg_pdbg("Cannot stat \"%s\" : %s\n", + sysfs_path, strerror(errno)); + goto linux_mtd_setup_exit; + } if (get_mtd_info()) goto linux_mtd_setup_exit; @@ -351,9 +343,6 @@ int dev_num = -1; /* linux_mtd_setup will search if dev_num < 0 */ int ret = 1; - if (alias && alias->type != ALIAS_HOST) - return 1; - param = extract_programmer_param("dev"); if (param) { char *endptr; @@ -373,7 +362,7 @@ if (register_shutdown(linux_mtd_shutdown, NULL)) goto linux_mtd_init_exit; - register_opaque_programmer(&programmer_linux_mtd); + register_opaque_master(&opaque_master_linux_mtd); ret = 0; linux_mtd_init_exit: -- To view, visit https://review.coreboot.org/22502 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I216fab281f7a20a5819de3adf610b770a0b3d37c Gerrit-Change-Number: 22502 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com>

1 0

Change in flashrom[master]: linux_mtd: Suppress message if NOR device not found
by David Hendricks (Code Review) 17 Nov '17

17 Nov '17

Hello Shawn Nematbakhsh, I'd like you to do a code review. Please visit https://review.coreboot.org/22503 to review the following change. Change subject: linux_mtd: Suppress message if NOR device not found ...................................................................... linux_mtd: Suppress message if NOR device not found This just suppresses a message that might cause confusion for unsuspecting users. BUG=none BRANCH=none TEST=ran on veyron_mickey, "NOR type device not found" message no longer appears under normal circumstances. Signed-off-by: David Hendricks <dhendrix(a)chromium.org> Change-Id: I76d50847e6c4a431d30ad39bcd3994059679172d Reviewed-on: https://chromium-review.googlesource.com/302145 Commit-Ready: David Hendricks <dhendrix(a)chromium.org> Tested-by: David Hendricks <dhendrix(a)chromium.org> Reviewed-by: Shawn N <shawnn(a)chromium.org> --- M linux_mtd.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/03/22503/1 diff --git a/linux_mtd.c b/linux_mtd.c index 07db2e9..064065f 100644 --- a/linux_mtd.c +++ b/linux_mtd.c @@ -276,7 +276,7 @@ tmp = (char *)scanft(LINUX_MTD_SYSFS_ROOT, "type", "nor", 1); if (!tmp) { - msg_perr("%s: NOR type device not found.\n", __func__); + msg_pdbg("%s: NOR type device not found.\n", __func__); goto linux_mtd_setup_exit; } -- To view, visit https://review.coreboot.org/22503 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I76d50847e6c4a431d30ad39bcd3994059679172d Gerrit-Change-Number: 22503 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Shawn Nematbakhsh <shawnn(a)chromium.org>

1 0

Change in flashrom[master]: Initial MTD support
by David Hendricks (Code Review) 17 Nov '17

17 Nov '17

Hello Shawn Nematbakhsh, I'd like you to do a code review. Please visit https://review.coreboot.org/22501 to review the following change. Change subject: Initial MTD support ...................................................................... Initial MTD support (This patch was ported from chromiumos @cebee89a) This adds MTD support to flashrom so that we can read, erase, and write content on a NOR flash chip via MTD. BUG=chrome-os-partner:40208 BRANCH=none TEST=read, write, and erase works on Oak Signed-off-by: David Hendricks <dhendrix(a)chromium.org> Change-Id: Ib3b8963874722ea80299e9298101409408d6c253 Reviewed-on: https://chromium-review.googlesource.com/272983 Reviewed-by: Shawn N <shawnn(a)chromium.org> --- M Makefile M flashrom.c M internal.c A linux_mtd.c M programmer.h 5 files changed, 416 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/01/22501/1 diff --git a/Makefile b/Makefile index 26bf343..8b2f7c5 100644 --- a/Makefile +++ b/Makefile @@ -623,7 +623,8 @@ # Always enable Marvell SATA controllers for now. CONFIG_SATAMV ?= yes -# Enable Linux spidev interface by default. We disable it on non-Linux targets. +# Enable Linux spidev and MTD interfaces by default. We disable them on non-Linux targets. +CONFIG_LINUX_MTD ?= yes CONFIG_LINUX_SPI ?= yes # Always enable ITE IT8212F PATA controllers for now. @@ -901,6 +902,11 @@ NEED_LIBPCI += CONFIG_SATAMV endif +ifeq ($(CONFIG_LINUX_MTD), yes) +FEATURE_CFLAGS += -D'CONFIG_LINUX_MTD=1' +PROGRAMMER_OBJS += linux_mtd.o +endif + ifeq ($(CONFIG_LINUX_SPI), yes) # This is a totally ugly hack. FEATURE_CFLAGS += $(call debug_shell,grep -q "LINUX_SPI_SUPPORT := yes" .features && printf "%s" "-D'CONFIG_LINUX_SPI=1'") diff --git a/flashrom.c b/flashrom.c index 4fe1843..6329041 100644 --- a/flashrom.c +++ b/flashrom.c @@ -344,6 +344,17 @@ }, #endif +#if CONFIG_LINUX_MTD == 1 + { + .name = "linux_mtd", + .type = OTHER, + .init = linux_mtd_init, + .map_flash_region = fallback_map, + .unmap_flash_region = fallback_unmap, + .delay = internal_delay, + }, +#endif + #if CONFIG_LINUX_SPI == 1 { .name = "linux_spi", diff --git a/internal.c b/internal.c index 4062b64..5395178 100644 --- a/internal.c +++ b/internal.c @@ -243,6 +243,11 @@ if (pci_init_common() != 0) return 1; +#if CONFIG_LINUX_MTD == 1 + if (!linux_mtd_init()) + return 0; +#endif + if (processor_flash_enable()) { msg_perr("Processor detection/init failed.\n" "Aborting.\n"); diff --git a/linux_mtd.c b/linux_mtd.c new file mode 100644 index 0000000..e1b11cd --- /dev/null +++ b/linux_mtd.c @@ -0,0 +1,382 @@ +/* + * This file is part of the flashrom project. + * + * Copyright 2015 Google Inc. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; version 2 of the License. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA + */ + +#include <ctype.h> +#include <errno.h> +#include <fcntl.h> +#include <inttypes.h> +#include <libgen.h> +#include <stdio.h> +#include <stdlib.h> +#include <mtd/mtd-user.h> +#include <string.h> +#include <sys/ioctl.h> +#include <sys/stat.h> +#include <unistd.h> + +#include "file.h" +#include "flash.h" +#include "programmer.h" + +#define LINUX_DEV_ROOT "/dev" +#define LINUX_MTD_SYSFS_ROOT "/sys/class/mtd" + +/* enough space for LINUX_MTD_SYSFS_ROOT + directory name + filename */ +static char sysfs_path[PATH_MAX]; + +static int dev_fd = -1; + +static char *mtd_device_name; +static int mtd_device_is_writeable; + +/* Size info is presented in bytes in sysfs. */ +static unsigned long int mtd_total_size; +static unsigned long int mtd_numeraseregions; +static unsigned long int mtd_erasesize; /* only valid if numeraseregions is 0 */ + +static int stat_mtd_files(char *dev_path, char *sysfs_path) +{ + struct stat s; + + errno = 0; + if (stat(dev_path, &s) < 0) { + msg_pdbg("Cannot stat \"%s\": %s\n", dev_path, strerror(errno)); + return 1; + } + + if (lstat(sysfs_path, &s) < 0) { + msg_pdbg("Cannot stat \"%s\" : %s\n", + sysfs_path, strerror(errno)); + return 1; + } + + return 0; +} + +/* read a string from a sysfs file and sanitize it */ +static int read_sysfs_string(const char *filename, char *buf, int len) +{ + int fd, bytes_read, i; + char path[strlen(LINUX_MTD_SYSFS_ROOT) + 32]; + + snprintf(path, sizeof(path), "%s/%s", sysfs_path, filename); + + if ((fd = open(path, O_RDONLY)) < 0) { + msg_perr("Cannot open %s\n", path); + return 1; + } + + if ((bytes_read = read(fd, buf, len - 1)) < 0) { + msg_perr("Cannot read %s\n", path); + close(fd); + return 1; + } + + buf[bytes_read] = '\0'; + + /* + * Files from sysfs sometimes contain a newline or other garbage that + * can confuse functions like strtoul() and ruin formatting in print + * statements. Replace the first non-printable character (space is + * considered printable) with a proper string terminator. + */ + for (i = 0; i < len; i++) { + if (!isprint(buf[i])) { + buf[i] = '\0'; + break; + } + } + + close(fd); + return 0; +} + +static int read_sysfs_int(const char *filename, unsigned long int *val) +{ + uint32_t tmp; + char buf[32]; + char *endptr; + + if (read_sysfs_string(filename, buf, sizeof(buf))) + return 1; + + errno = 0; + *val = strtoul(buf, &endptr, 0); + if (endptr != &buf[strlen(buf)]) { + msg_perr("Error reading %s\n", filename); + return 1; + } + + if (errno) { + msg_perr("Error reading %s: %s\n", filename, strerror(errno)); + return 1; + } + + return 0; +} + +/* returns 0 to indicate success, non-zero to indicate error */ +static int get_mtd_info(void) +{ + unsigned long int tmp; + char mtd_device_name[32]; + + /* Flags */ + if (read_sysfs_int("flags", &tmp)) + return 1; + if (tmp & MTD_WRITEABLE) { + /* cache for later use by write function */ + mtd_device_is_writeable = 1; + } + + /* Device name */ + if (read_sysfs_string("name", mtd_device_name, sizeof(mtd_device_name))) + return 1; + + /* Total size */ + if (read_sysfs_int("size", &mtd_total_size)) + return 1; + if (__builtin_popcount(mtd_total_size) != 1) { + msg_perr("MTD size is not a power of 2\n"); + return 1; + } + + /* Erase size */ + if (read_sysfs_int("erasesize", &mtd_erasesize)) + return 1; + if (__builtin_popcount(mtd_erasesize) != 1) { + msg_perr("MTD erase size is not a power of 2\n"); + return 1; + } + + /* Erase regions */ + if (read_sysfs_int("numeraseregions", &mtd_numeraseregions)) + return 1; + if (mtd_numeraseregions != 0) { + msg_perr("Non-uniform eraseblock size is unsupported.\n"); + return 1; + } + + msg_pspew("%s: device_name: \"%s\", is_writeable: %d, " + "numeraseregions: %lu, total_size: %lu, erasesize: %lu\n", + __func__, mtd_device_name, mtd_device_is_writeable, + mtd_numeraseregions, mtd_total_size, mtd_erasesize); + + return 0; +} + +static int linux_mtd_probe(struct flashchip *flash) +{ + flash->tested = TEST_OK_PREW; + flash->total_size = mtd_total_size / 1024; /* bytes -> kB */ + flash->block_erasers[0].eraseblocks[0].size = mtd_erasesize; + flash->block_erasers[0].eraseblocks[0].count = + mtd_total_size / mtd_erasesize; + return 1; +} + +static int linux_mtd_read(struct flashchip *flash, uint8_t *buf, + unsigned int start, unsigned int len) +{ + if (lseek(dev_fd, start, SEEK_SET) != start) { + msg_perr("Cannot seek to 0x%06x: %s\n", start, strerror(errno)); + return 1; + } + + if (read(dev_fd, buf, len) != len) { + msg_perr("Cannot read 0x%06x bytes at 0x%06x: %s\n", + start, len, strerror(errno)); + return 1; + } + + return 0; +} + +/* this version assumes we must divide the write request into pages ourselves */ +static int linux_mtd_write(struct flashchip *flash, uint8_t *buf, + unsigned int start, unsigned int len) +{ + unsigned int page; + unsigned int chunksize, page_size; + + chunksize = page_size = flash->page_size; + + if (!mtd_device_is_writeable) + return 1; + + for (page = start / page_size; + page <= (start + len - 1) / page_size; page++) { + unsigned int i, starthere, lenhere; + + starthere = max(start, page * page_size); + lenhere = min(start + len, (page + 1) * page_size) - starthere; + for (i = 0; i < lenhere; i += chunksize) { + unsigned int towrite = min(chunksize, lenhere - i); + + if (lseek(dev_fd, starthere, SEEK_SET) != starthere) { + msg_perr("Cannot seek to 0x%06x: %s\n", + start, strerror(errno)); + return 1; + } + + if (write(dev_fd, &buf[starthere - start], towrite) != towrite) { + msg_perr("Cannot read 0x%06x bytes at 0x%06x: " + "%s\n", start, len, strerror(errno)); + return 1; + } + } + } + + return 0; +} + +static int linux_mtd_erase(struct flashchip *flash, + unsigned int start, unsigned int len) +{ + struct region_info_user region_info; + uint32_t u; + + if (mtd_numeraseregions != 0) { + /* TODO: Support non-uniform eraseblock size using + use MEMGETREGIONCOUNT/MEMGETREGIONINFO ioctls */ + } + + for (u = 0; u < len; u += mtd_erasesize) { + struct erase_info_user erase_info = { + .start = start + u, + .length = mtd_erasesize, + }; + + if (ioctl(dev_fd, MEMERASE, &erase_info) == -1) { + msg_perr("%s: ioctl: %s\n", __func__, strerror(errno)); + return 1; + } + } + + return 0; +} + +static struct opaque_programmer programmer_linux_mtd = { + /* FIXME: Do we need to change these (especially write) as per + * page size requirements? */ + .max_data_read = MAX_DATA_UNSPECIFIED, + .max_data_write = MAX_DATA_UNSPECIFIED, + .probe = linux_mtd_probe, + .read = linux_mtd_read, + .write = linux_mtd_write, + .erase = linux_mtd_erase, +}; + +/* Returns 0 if setup is successful, non-zero to indicate error */ +static int linux_mtd_setup(int dev_num) +{ + char dev_path[16]; /* "/dev/mtdN" */ + int ret = 1; + + if (dev_num < 0) { + char *tmp, *p; + + tmp = (char *)scanft(LINUX_MTD_SYSFS_ROOT, "type", "nor", 1); + if (!tmp) { + msg_perr("%s: NOR type device not found.\n", __func__); + goto linux_mtd_setup_exit; + } + + /* "tmp" should be something like "/sys/blah/mtdN/type" */ + p = tmp + strlen(LINUX_MTD_SYSFS_ROOT); + while (p[0] == '/') + p++; + + if (sscanf(p, "mtd%d", &dev_num) != 1) { + msg_perr("Can't obtain device number from \"%s\"\n", p); + free(tmp); + goto linux_mtd_setup_exit; + } + free(tmp); + } + + snprintf(sysfs_path, sizeof(sysfs_path), "%s/mtd%d", + LINUX_MTD_SYSFS_ROOT, dev_num); + snprintf(dev_path, sizeof(dev_path), "%s/mtd%d", + LINUX_DEV_ROOT, dev_num); + msg_pdbg("%s: sysfs_path: \"%s\", dev_path: \"%s\"\n", + __func__, sysfs_path, dev_path); + + if (stat_mtd_files(dev_path, sysfs_path)) + goto linux_mtd_setup_exit; + + if (get_mtd_info()) + goto linux_mtd_setup_exit; + + if ((dev_fd = open(dev_path, O_RDWR)) == -1) { + msg_pdbg("%s: failed to open %s: %s\n", __func__, + dev_path, strerror(errno)); + goto linux_mtd_setup_exit; + } + + ret = 0; +linux_mtd_setup_exit: + return ret; +} + +static int linux_mtd_shutdown(void *data) +{ + if (dev_fd != -1) { + close(dev_fd); + dev_fd = -1; + } + + return 0; +} + +int linux_mtd_init(void) +{ + char *param; + int dev_num = -1; /* linux_mtd_setup will search if dev_num < 0 */ + int ret = 1; + + if (alias && alias->type != ALIAS_HOST) + return 1; + + param = extract_programmer_param("dev"); + if (param) { + char *endptr; + + dev_num = strtol(param, &endptr, 0); + if ((param == endptr) || (dev_num < 0)) { + msg_perr("Invalid device number %s. Use flashrom -p " + "linux_mtd:dev=N where N is a valid MTD " + "device number\n", param); + goto linux_mtd_init_exit; + } + } + + if (linux_mtd_setup(dev_num)) + goto linux_mtd_init_exit; + + if (register_shutdown(linux_mtd_shutdown, NULL)) + goto linux_mtd_init_exit; + + register_opaque_programmer(&programmer_linux_mtd); + + ret = 0; +linux_mtd_init_exit: + msg_pdbg("%s: %s\n", __func__, ret == 0 ? "success." : "failed."); + return ret; +} diff --git a/programmer.h b/programmer.h index a98b713..d3a77bc 100644 --- a/programmer.h +++ b/programmer.h @@ -99,6 +99,9 @@ #if CONFIG_LINUX_SPI == 1 PROGRAMMER_LINUX_SPI, #endif +#if CONFIG_LINUX_MTD == 1 + PROGRAMMER_LINUX_MTD, +#endif #if CONFIG_USBBLASTER_SPI == 1 PROGRAMMER_USBBLASTER_SPI, #endif @@ -525,6 +528,11 @@ int buspirate_spi_init(void); #endif +/* linux_mtd.c */ +#if CONFIG_LINUX_MTD == 1 +int linux_mtd_init(void); +#endif + /* linux_spi.c */ #if CONFIG_LINUX_SPI == 1 int linux_spi_init(void); @@ -587,6 +595,9 @@ #if CONFIG_OGP_SPI == 1 || CONFIG_NICINTEL_SPI == 1 || CONFIG_RAYER_SPI == 1 || CONFIG_PONY_SPI == 1 || (CONFIG_INTERNAL == 1 && (defined(__i386__) || defined(__x86_64__))) SPI_CONTROLLER_BITBANG, #endif +#if CONFIG_LINUX_MTD == 1 + SPI_CONTROLLER_LINUX_MTD, +#endif #if CONFIG_LINUX_SPI == 1 SPI_CONTROLLER_LINUX, #endif -- To view, visit https://review.coreboot.org/22501 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: Ib3b8963874722ea80299e9298101409408d6c253 Gerrit-Change-Number: 22501 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Shawn Nematbakhsh <shawnn(a)chromium.org>

1 0

Change in flashrom[master]: file: add new helper functions to grok FDT nodes
by David Hendricks (Code Review) 17 Nov '17

17 Nov '17

Hello build bot (Jenkins), Brian Norris, I'd like you to reexamine a change. Please visit https://review.coreboot.org/22493 to look at the new patch set (#2). Change subject: file: add new helper functions to grok FDT nodes ...................................................................... file: add new helper functions to grok FDT nodes This file is intended to help find SPI devices using the FDT. Here's how it works: Under /proc/device-tree there is a node called "aliases" where each file is a device name with an instance (e.g. "spi0"). The content of the file is the controller's memory-mapped address. That info can be used to find the appropriate node directory to search under /proc/device-tree, where we can recursively search for "compatible" nodes and scan for the matching user-provided compatible string. For SPI devices, we need to look for the matching "compatible" node and then read the corresponding "reg" node to obtain chip- select number. BUG=chrome-os-partner:43453 BRANCH=none TEST=reads now work on danger without specifying a SPI device Signed-off-by: David Hendricks <dhendrix(a)chromium.org> Change-Id: I1f771bcca61de71daa54e358e058723d3fd37079 Reviewed-on: https://chromium-review.googlesource.com/310072 Commit-Ready: David Hendricks <dhendrix(a)chromium.org> Tested-by: David Hendricks <dhendrix(a)chromium.org> Reviewed-by: Brian Norris <briannorris(a)chromium.org> --- M file.c M file.h 2 files changed, 160 insertions(+), 2 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/93/22493/2 -- To view, visit https://review.coreboot.org/22493 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I1f771bcca61de71daa54e358e058723d3fd37079 Gerrit-Change-Number: 22493 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Brian Norris <briannorris(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: file: Re-implement file parsing function
by David Hendricks (Code Review) 17 Nov '17

17 Nov '17

Hello build bot (Jenkins), Brian Norris, I'd like you to reexamine a change. Please visit https://review.coreboot.org/22492 to look at the new patch set (#2). Change subject: file: Re-implement file parsing function ...................................................................... file: Re-implement file parsing function This re-does the original file parsing function to be a little bit more robust. The old version just copied the file content and did a string compare. The new version will read the file character-by- character, comparing with the specified string as we go along, until a match is found. BUG=none BRANCH=none TEST=Works as expected on veyron_mickey to grok thru sysfs. Also tested with an upcoming FDT-related patch to search thru procfs. Signed-off-by: David Hendricks <dhendrix(a)chromium.org> Change-Id: I6e3811acf352d85c3f43192f87388833a997dd2c Reviewed-on: https://chromium-review.googlesource.com/310653 Reviewed-by: Brian Norris <briannorris(a)chromium.org> --- M file.c 1 file changed, 58 insertions(+), 28 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/92/22492/2 -- To view, visit https://review.coreboot.org/22492 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I6e3811acf352d85c3f43192f87388833a997dd2c Gerrit-Change-Number: 22492 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Brian Norris <briannorris(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: avoid unnecessary syscalls in scanft()
by David Hendricks (Code Review) 17 Nov '17

17 Nov '17

Hello Julius Werner, build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/22491 to look at the new patch set (#2). Change subject: avoid unnecessary syscalls in scanft() ...................................................................... avoid unnecessary syscalls in scanft() This adds a test to see if the path given to scanft() is a symlink or a directory before calling opendir() and readdir(). This cuts down on syscalls which can be very slow. BUG=none BRANCH=none TEST="flashrom -p host -i RW_ELOG:/tmp/elog.bin -r -V" ran on pit a few hundred milliseconds faster (without other optimizations applied) Change-Id: Ib067cfda1ba3edbc0b1bba604fa5eb57c64501f2 Reviewed-on: https://chromium-review.googlesource.com/176580 Commit-Queue: David Hendricks <dhendrix(a)chromium.org> Tested-by: David Hendricks <dhendrix(a)chromium.org> Reviewed-by: Julius Werner <jwerner(a)chromium.org> --- M file.c 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/91/22491/2 -- To view, visit https://review.coreboot.org/22491 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ib067cfda1ba3edbc0b1bba604fa5eb57c64501f2 Gerrit-Change-Number: 22491 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: Add a file utility library
by David Hendricks (Code Review) 17 Nov '17

17 Nov '17

Hello Louis Yung-Chieh Lo, build bot (Jenkins), I'd like you to reexamine a change. Please visit https://review.coreboot.org/22490 to look at the new patch set (#2). Change subject: Add a file utility library ...................................................................... Add a file utility library This adds a library for BSD-licensed helper functions for dealing with files. Currently the library only has scanft() which was ported over from mosys. This function recursively scans files in a filesystem tree and can optionally match the beginning content. This is particularly useful for searching for devices in sysfs, which may be infinitely recursive (due to symlinks) and typically contain single values, such as a module alias. BUG=none TEST=used in follow-up CLs to find I2C and SPI buses and addresses Example test output using /sys/bus/i2c/devices/ as root path, following 2 levels of symbolic links, physical i2c bus is 4 but got shifted to 6: gec_i2c_probe_programmer: probing for GEC on I2C... gec_i2c_probe_programmer: chromeos-ec found in \ "/sys/bus/i2c/devices/i2c-6/6-001e/name" gec_i2c_probe_programmer: bus: 6, addr: 0x1e Change-Id: I6c86ad3a50748e6bdbaa6675b05e75ef714b3994 Reviewed-on: https://gerrit.chromium.org/gerrit/23466 Reviewed-by: Yung-Chieh Lo <yjlou%chromium.org(a)gtempaccount.com> Commit-Ready: David Hendricks <dhendrix(a)chromium.org> Tested-by: David Hendricks <dhendrix(a)chromium.org> --- M Makefile A file.c A file.h 3 files changed, 175 insertions(+), 1 deletion(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/90/22490/2 -- To view, visit https://review.coreboot.org/22490 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I6c86ad3a50748e6bdbaa6675b05e75ef714b3994 Gerrit-Change-Number: 22490 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Louis Yung-Chieh Lo <yjlou(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: file: read in entire file (up to 4K) instead of stat()'ing for size
by David Hendricks (Code Review) 17 Nov '17

17 Nov '17

Hello build bot (Jenkins), Brian Norris, I'd like you to reexamine a change. Please visit https://review.coreboot.org/22494 to look at the new patch set (#2). Change subject: file: read in entire file (up to 4K) instead of stat()'ing for size ...................................................................... file: read in entire file (up to 4K) instead of stat()'ing for size We use find_string() to check files in both /proc (e.g., /proc/device-tree/...) and /sys (e.g., /sys/bus/spi/devices/...). find_string() currently tries to iteratively read pieces of these files, using stat() to get the file size. However, this size is inaccurate on sysfs, as these files are often dynamically generated, so it reports a size of PAGE_SIZE (i.e., 4096). Instead of trying to pre-determine the file size, let's read in the whole file (up to 4KiB - 1) and do a series of string searches to find any matching strings. The 4KiB limitation should be OK, since we're only looking for things like modalias and compatible strings. As a side effect, this eliminates a lot of unnecessary strlen() calls, and allows us to again avoid hand-rolling strstr(). BUG=chrome-os-partner:51651 BRANCH=none TEST=`flashrom --wp-status` doesn't show log spam TEST=`flashrom --read /tmp/flashX` gives same results before/after Change-Id: I2d053b18ba624f4ce3ebc1956b9e7945b693b0bc Signed-off-by: Brian Norris <briannorris(a)chromium.org> Reviewed-on: https://chromium-review.googlesource.com/335353 Reviewed-by: David Hendricks <dhendrix(a)chromium.org> --- M file.c 1 file changed, 41 insertions(+), 44 deletions(-) git pull ssh://review.coreboot.org:29418/flashrom refs/changes/94/22494/2 -- To view, visit https://review.coreboot.org/22494 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I2d053b18ba624f4ce3ebc1956b9e7945b693b0bc Gerrit-Change-Number: 22494 Gerrit-PatchSet: 2 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Brian Norris <briannorris(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>

1 0

Change in flashrom[master]: file: read in entire file (up to 4K) instead of stat()'ing for size
by build bot (Jenkins) (Code Review) 17 Nov '17

17 Nov '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22494 ) Change subject: file: read in entire file (up to 4K) instead of stat()'ing for size ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom_gerrit/773/ : SUCCESS https://qa.coreboot.org/job/flashrom-customrules/884/ : SUCCESS -- To view, visit https://review.coreboot.org/22494 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2d053b18ba624f4ce3ebc1956b9e7945b693b0bc Gerrit-Change-Number: 22494 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Brian Norris <briannorris(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 17 Nov 2017 05:51:11 +0000 Gerrit-HasComments: No

1 0

Change in flashrom[master]: avoid unnecessary syscalls in scanft()
by build bot (Jenkins) (Code Review) 17 Nov '17

17 Nov '17

build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/22491 ) Change subject: avoid unnecessary syscalls in scanft() ...................................................................... Patch Set 1: Verified+1 Build Successful https://qa.coreboot.org/job/flashrom_gerrit/771/ : SUCCESS https://qa.coreboot.org/job/flashrom-customrules/882/ : SUCCESS -- To view, visit https://review.coreboot.org/22491 To unsubscribe, visit https://review.coreboot.org/settings Gerrit-Project: flashrom Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ib067cfda1ba3edbc0b1bba604fa5eb57c64501f2 Gerrit-Change-Number: 22491 Gerrit-PatchSet: 1 Gerrit-Owner: David Hendricks <david.hendricks(a)gmail.com> Gerrit-Reviewer: Julius Werner <jwerner(a)chromium.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org> Gerrit-Comment-Date: Fri, 17 Nov 2017 05:50:59 +0000 Gerrit-HasComments: No

1 0

2024

2023

2022

2021

2020

2019

2018

2017

flashrom-gerrit