diff options
author | Jeremy Kerr <jk@ozlabs.org> | 2016-04-27 19:28:03 +0800 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2016-04-28 14:10:06 +1000 |
commit | d2c32593ca991da4083c3c2386d19e2fb29cf5e3 (patch) | |
tree | 1c229c443a7e33228b0d7050c15d419764d53891 /core | |
parent | 7eeb1cc1f4d9b4a183ef8a16ad20930ed9dba6d6 (diff) | |
download | skiboot-d2c32593ca991da4083c3c2386d19e2fb29cf5e3.zip skiboot-d2c32593ca991da4083c3c2386d19e2fb29cf5e3.tar.gz skiboot-d2c32593ca991da4083c3c2386d19e2fb29cf5e3.tar.bz2 |
core/flash: Add fwts olog annotations
This change adds a few fwts patterns to the flash code, mostly as an
example of the annotations we have.
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'core')
-rw-r--r-- | core/flash.c | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/core/flash.c b/core/flash.c index 746db6f..734e0fa 100644 --- a/core/flash.c +++ b/core/flash.c @@ -161,6 +161,13 @@ static int flash_nvram_probe(struct flash *flash, struct ffs_handle *ffs) rc = ffs_part_info(ffs, part, NULL, &start, &size, NULL, NULL); if (rc) { + /** + * @fwts-label NVRAMNoPartition + * @fwts-advice OPAL could not find an NVRAM partition + * on the system flash. Check that the system flash + * has a valid partition table, and that the firmware + * build process has added a NVRAM partition. + */ prlog(PR_ERR, "FLASH: Can't parse ffs info for NVRAM\n"); return OPAL_HARDWARE; } @@ -202,12 +209,25 @@ static void setup_system_flash(struct flash *flash, struct dt_node *node, char *path; if (system_flash) { + /** + * @fwts-label SystemFlashDuplicate + * @fwts-advice More than one flash device was registered + * as the system flash device. Check for duplicate calls + * to flash_register(..., true). + */ prlog(PR_WARNING, "FLASH: attempted to register a second " "system flash device %s\n", name); return; } if (!ffs) { + /** + * @fwts-label SystemFlashNoPartitionTable + * @fwts-advice OPAL Could not read a partition table on + * system flash. Since we've still booted the machine (which + * requires flash), check that we're registering the proper + * system flash device. + */ prlog(PR_WARNING, "FLASH: attempted to register system flash " "%s, wwhich has no partition info\n", name); return; |