sys/log, fs/fcb2: Remove fcb2 support

Author: vrahane

fs/fcb2/include/fcb/fcb2.h

@@ -196,19 +196,6 @@ int fcb2_walk(struct fcb2 *fcb, int sector, fcb2_walk_cb cb, void *cb_arg);
  */
 int fcb2_getnext(struct fcb2 *fcb, struct fcb2_entry *loc);
 
-/**
- * Get first entry in the provided flash area
- *
- * @param fcb Pointer to FCB
- * @param range Optional range in flash sector
- * @param loc Pointer to first FCB entry in the provided flash area
- *
- * @return 0 on success, non-zero on failure
- */
-int
-fcb2_getnext_in_area(struct fcb2 *fcb, struct flash_sector_range *range,
-                     struct fcb2_entry *loc);
-
 /**
  * Walk through entries within FCB from newest to oldest.
  * fcb_getprev() finds the previous valid entry backwards from loc, and fills in
@@ -317,17 +304,6 @@ int fcb2_clear(struct fcb2 *fcb);
  */
 int fcb2_area_info(struct fcb2 *fcb, int sector, int *elemsp, int *bytesp);
 
-/**
- * Returns the next sector flash range, given current entry
- *
- * @param fcb            Pointer to the FCB
- * @param loc            Pointer to the location
- *
- * @return Flash sector range of the next sector
- */
-struct flash_sector_range *
-fcb2_getnext_range(struct fcb2 *fcb, struct fcb2_entry *loc);
-
 #ifdef __cplusplus
 }
 

fs/fcb2/src/fcb.c

@@ -88,7 +88,7 @@ fcb2_init(struct fcb2 *fcb)
     fcb->f_active_id = newest;
 
     while (1) {
-        rc = fcb2_getnext_in_area(fcb, NULL, &fcb->f_active);
+        rc = fcb2_getnext_in_area(fcb, &fcb->f_active);
         if (rc == FCB2_ERR_NOVAR) {
             rc = FCB2_OK;
             break;

fs/fcb2/src/fcb_getnext.c

@@ -22,23 +22,13 @@
 #include "fcb_priv.h"
 
 int
-fcb2_getnext_in_area(struct fcb2 *fcb, struct flash_sector_range *range,
-                     struct fcb2_entry *loc)
+fcb2_getnext_in_area(struct fcb2 *fcb, struct fcb2_entry *loc)
 {
     int rc = FCB2_ERR_CRC;
     int len;
     int next_data_offset;
     int next_entry_offset;
 
-    /* If a flash range is specified, find first entry in that area */
-    if (range) {
-        loc->fe_range = range;
-        loc->fe_data_off = fcb2_len_in_flash(range, sizeof(struct fcb2_disk_area));
-        loc->fe_entry_num = 0;
-        loc->fe_data_len = 0;
-        loc->fe_sector = 0;
-    }
-
     while (rc == FCB2_ERR_CRC) {
         len = loc->fe_data_len;
         /* Next data offset in sector */
@@ -62,17 +52,6 @@ fcb2_getnext_in_area(struct fcb2 *fcb, struct flash_sector_range *range,
     return rc;
 }
 
-struct flash_sector_range *
-fcb2_getnext_range(struct fcb2 *fcb, struct fcb2_entry *loc)
-{
-    loc->fe_entry_num = 0;
-    loc->fe_data_len = 0;
-    loc->fe_sector = fcb2_getnext_sector(fcb, loc->fe_sector);
-    loc->fe_range = fcb2_get_sector_range(fcb, loc->fe_sector);
-
-    return loc->fe_range;
-}
-
 int
 fcb2_getnext_nolock(struct fcb2 *fcb, struct fcb2_entry *loc)
 {
@@ -92,7 +71,7 @@ fcb2_getnext_nolock(struct fcb2 *fcb, struct fcb2_entry *loc)
         loc->fe_entry_num = 1;
         rc = fcb2_elem_info(loc);
     } else {
-        rc = fcb2_getnext_in_area(fcb, NULL, loc);
+        rc = fcb2_getnext_in_area(fcb, loc);
     }
     switch (rc) {
     case 0:
@@ -103,7 +82,7 @@ fcb2_getnext_nolock(struct fcb2 *fcb, struct fcb2_entry *loc)
         goto next_sector;
     }
     while (rc == FCB2_ERR_CRC) {
-        rc = fcb2_getnext_in_area(fcb, NULL, loc);
+        rc = fcb2_getnext_in_area(fcb, loc);
         if (rc == 0) {
             return 0;
         }

fs/fcb2/src/fcb_priv.h

@@ -49,6 +49,8 @@ fcb2_len_in_flash(const struct flash_sector_range *range, uint16_t len)
     return (len + (range->fsr_align - 1)) & ~(range->fsr_align - 1);
 }
 
+int fcb2_getnext_in_area(struct fcb2 *fcb, struct fcb2_entry *loc);
+
 static inline int
 fcb2_getnext_sector(struct fcb2 *fcb, int sector)
 {

sys/log/full/src/log_fcb2.c

@@ -173,7 +173,7 @@ log_fcb2_start_append(struct log *log, int len, struct fcb2_entry *loc)
         }
 #endif
 
-#if MYNEWT_VAL(LOG_FCB_BOOKMARKS) && !MYNEWT_VAL(LOG_FCB_SECTOR_BOOKMARKS)
+#if MYNEWT_VAL(LOG_FCB_BOOKMARKS)
         /* The FCB needs to be rotated. */
         log_fcb_rotate_bmarks(fcb_log);
 #endif
@@ -183,18 +183,6 @@ log_fcb2_start_append(struct log *log, int len, struct fcb2_entry *loc)
             goto err;
         }
 
-#if MYNEWT_VAL(LOG_FCB_SECTOR_BOOKMARKS)
-        /* The FCB needs to be rotated, reinit previously allocated
-         * bookmarks
-         */
-        rc = log_fcb_init_bmarks(fcb_log, fcb_log->fl_bset.lfs_bmarks,
-                                 fcb_log->fl_bset.lfs_cap,
-                                 fcb_log->fl_bset.lfs_en_sect_bmarks);
-        if (rc) {
-            goto err;
-        }
-#endif
-
 #if MYNEWT_VAL(LOG_STORAGE_WATERMARK)
         /*
          * FCB was rotated successfully so let's check if watermark was within