Add support to save boot count variable in a file in a FAT filesystem.

Signed-off-by: Vasileios Amoiridis <vassilisa...@gmail.com>
---
 doc/README.bootcount                          | 12 ++---
 drivers/bootcount/Kconfig                     | 53 +++++++++++++------
 drivers/bootcount/Makefile                    |  2 +-
 .../{bootcount_ext.c => bootcount_fs.c}       | 12 ++---
 4 files changed, 50 insertions(+), 29 deletions(-)
 rename drivers/bootcount/{bootcount_ext.c => bootcount_fs.c} (81%)

diff --git a/doc/README.bootcount b/doc/README.bootcount
index f6c5f82f..0f4ffb68 100644
--- a/doc/README.bootcount
+++ b/doc/README.bootcount
@@ -23,15 +23,15 @@ It is the responsibility of some application code 
(typically a Linux
 application) to reset the variable "bootcount" to 0 when the system booted
 successfully, thus allowing for more boot cycles.
 
-CONFIG_BOOTCOUNT_EXT
+CONFIG_BOOTCOUNT_FS
 --------------------
 
-This adds support for maintaining boot count in a file on an EXT filesystem.
-The file to use is defined by:
+This adds support for maintaining boot count in a file on a filesystem.
+Supported filesystems are FAT and EXT. The file to use is defined by:
 
-CONFIG_SYS_BOOTCOUNT_EXT_INTERFACE
-CONFIG_SYS_BOOTCOUNT_EXT_DEVPART
-CONFIG_SYS_BOOTCOUNT_EXT_NAME
+CONFIG_SYS_BOOTCOUNT_FS_INTERFACE
+CONFIG_SYS_BOOTCOUNT_FS_DEVPART
+CONFIG_SYS_BOOTCOUNT_FS_NAME
 
 The format of the file is:
 
diff --git a/drivers/bootcount/Kconfig b/drivers/bootcount/Kconfig
index 3c56253b..d3679eb5 100644
--- a/drivers/bootcount/Kconfig
+++ b/drivers/bootcount/Kconfig
@@ -25,10 +25,9 @@ config BOOTCOUNT_GENERIC
            Set to the address where the bootcount and bootcount magic
            will be stored.
 
-config BOOTCOUNT_EXT
-       bool "Boot counter on EXT filesystem"
-       depends on FS_EXT4
-       select EXT4_WRITE
+config BOOTCOUNT_FS
+       bool "Boot counter on a filesystem"
+       depends on FS_EXT4 || FS_FAT
        help
          Add support for maintaining boot count in a file on an EXT
          filesystem.
@@ -184,26 +183,48 @@ config SYS_BOOTCOUNT_SINGLEWORD
          This option enables packing boot count magic value and boot count
          into single word (32 bits).
 
-config SYS_BOOTCOUNT_EXT_INTERFACE
-       string "Interface on which to find boot counter EXT filesystem"
+if BOOTCOUNT_FS
+choice
+       prompt "Filesystem type"
+       default BOOTCOUNT_EXT
+
+config BOOTCOUNT_EXT
+       bool "Boot counter on EXT filesystem"
+       depends on FS_EXT4
+       select EXT4_WRITE
+       help
+         Add support for maintaing boot counter in a file on EXT filesystem"
+
+config BOOTCOUNT_FAT
+       bool "Boot counter on FAT filesystem"
+       depends on FS_FAT
+       select FAT_WRITE
+       help
+         Add support for maintaing boot counter in a file on FAT filesystem"
+
+endchoice
+endif
+
+config SYS_BOOTCOUNT_FS_INTERFACE
+       string "Interface on which to find boot counter filesystem"
        default "mmc"
-       depends on BOOTCOUNT_EXT
+       depends on BOOTCOUNT_FS
        help
          Set the interface to use when locating the filesystem to use for the
          boot counter.
 
-config SYS_BOOTCOUNT_EXT_DEVPART
-       string "Partition of the boot counter EXT filesystem"
+config SYS_BOOTCOUNT_FS_DEVPART
+       string "Partition of the boot counter filesystem"
        default "0:1"
-       depends on BOOTCOUNT_EXT
+       depends on BOOTCOUNT_FS
        help
          Set the partition to use when locating the filesystem to use for the
          boot counter.
 
-config SYS_BOOTCOUNT_EXT_NAME
-       string "Path and filename of the EXT filesystem based boot counter"
+config SYS_BOOTCOUNT_FS_NAME
+       string "Path and filename of the FS filesystem based boot counter"
        default "/boot/failures"
-       depends on BOOTCOUNT_EXT
+       depends on BOOTCOUNT_FS
        help
          Set the filename and path of the file used to store the boot counter.
 
@@ -211,18 +232,18 @@ config SYS_BOOTCOUNT_ADDR
        hex "RAM address used for reading and writing the boot counter"
        default 0x44E3E000 if BOOTCOUNT_AM33XX || BOOTCOUNT_AM33XX_NVMEM
        default 0xE0115FF8 if ARCH_LS1043A || ARCH_LS1021A
-       depends on BOOTCOUNT_AM33XX || BOOTCOUNT_GENERIC || BOOTCOUNT_EXT || \
+       depends on BOOTCOUNT_AM33XX || BOOTCOUNT_GENERIC || BOOTCOUNT_FS || \
                   BOOTCOUNT_AM33XX_NVMEM
        help
          Set the address used for reading and writing the boot counter.
 
 config SYS_BOOTCOUNT_MAGIC
        hex "Magic value for the boot counter"
-       default 0xB001C041 if BOOTCOUNT_GENERIC || BOOTCOUNT_EXT || \
+       default 0xB001C041 if BOOTCOUNT_GENERIC || BOOTCOUNT_FS || \
                              BOOTCOUNT_AM33XX || BOOTCOUNT_ENV || \
                              BOOTCOUNT_RAM || BOOTCOUNT_AT91 || DM_BOOTCOUNT
        default 0xB0 if BOOTCOUNT_AM33XX_NVMEM
-       depends on BOOTCOUNT_GENERIC || BOOTCOUNT_EXT || \
+       depends on BOOTCOUNT_GENERIC || BOOTCOUNT_FS || \
                   BOOTCOUNT_AM33XX || BOOTCOUNT_ENV || \
                   BOOTCOUNT_RAM || BOOTCOUNT_AT91 || DM_BOOTCOUNT || \
                   BOOTCOUNT_AM33XX_NVMEM
diff --git a/drivers/bootcount/Makefile b/drivers/bootcount/Makefile
index e7771f5b..245f8796 100644
--- a/drivers/bootcount/Makefile
+++ b/drivers/bootcount/Makefile
@@ -6,7 +6,7 @@ obj-$(CONFIG_BOOTCOUNT_AT91)    += bootcount_at91.o
 obj-$(CONFIG_BOOTCOUNT_AM33XX) += bootcount_davinci.o
 obj-$(CONFIG_BOOTCOUNT_RAM)    += bootcount_ram.o
 obj-$(CONFIG_BOOTCOUNT_ENV)    += bootcount_env.o
-obj-$(CONFIG_BOOTCOUNT_EXT)    += bootcount_ext.o
+obj-$(CONFIG_BOOTCOUNT_FS)     += bootcount_fs.o
 obj-$(CONFIG_BOOTCOUNT_AM33XX_NVMEM)   += bootcount_nvmem.o
 
 obj-$(CONFIG_DM_BOOTCOUNT)      += bootcount-uclass.o
diff --git a/drivers/bootcount/bootcount_ext.c 
b/drivers/bootcount/bootcount_fs.c
similarity index 81%
rename from drivers/bootcount/bootcount_ext.c
rename to drivers/bootcount/bootcount_fs.c
index 9639e638..569592d8 100644
--- a/drivers/bootcount/bootcount_ext.c
+++ b/drivers/bootcount/bootcount_fs.c
@@ -25,8 +25,8 @@ void bootcount_store(ulong a)
        loff_t len;
        int ret;
 
-       if (fs_set_blk_dev(CONFIG_SYS_BOOTCOUNT_EXT_INTERFACE,
-                          CONFIG_SYS_BOOTCOUNT_EXT_DEVPART, FS_TYPE_EXT)) {
+       if (fs_set_blk_dev(CONFIG_SYS_BOOTCOUNT_FS_INTERFACE,
+                          CONFIG_SYS_BOOTCOUNT_FS_DEVPART, FS_TYPE_ANY)) {
                puts("Error selecting device\n");
                return;
        }
@@ -42,7 +42,7 @@ void bootcount_store(ulong a)
        buf->upgrade_available = upgrade_available;
        unmap_sysmem(buf);
 
-       ret = fs_write(CONFIG_SYS_BOOTCOUNT_EXT_NAME,
+       ret = fs_write(CONFIG_SYS_BOOTCOUNT_FS_NAME,
                       CONFIG_SYS_BOOTCOUNT_ADDR, 0, sizeof(bootcount_ext_t),
                       &len);
        if (ret != 0)
@@ -55,13 +55,13 @@ ulong bootcount_load(void)
        loff_t len_read;
        int ret;
 
-       if (fs_set_blk_dev(CONFIG_SYS_BOOTCOUNT_EXT_INTERFACE,
-                          CONFIG_SYS_BOOTCOUNT_EXT_DEVPART, FS_TYPE_EXT)) {
+       if (fs_set_blk_dev(CONFIG_SYS_BOOTCOUNT_FS_INTERFACE,
+                          CONFIG_SYS_BOOTCOUNT_FS_DEVPART, FS_TYPE_ANY)) {
                puts("Error selecting device\n");
                return 0;
        }
 
-       ret = fs_read(CONFIG_SYS_BOOTCOUNT_EXT_NAME, CONFIG_SYS_BOOTCOUNT_ADDR,
+       ret = fs_read(CONFIG_SYS_BOOTCOUNT_FS_NAME, CONFIG_SYS_BOOTCOUNT_ADDR,
                      0, sizeof(bootcount_ext_t), &len_read);
        if (ret != 0 || len_read != sizeof(bootcount_ext_t)) {
                puts("Error loading bootcount\n");
-- 
2.25.1

Reply via email to