ubifs: Modify ubifs u-boot wrapper function prototypes for generic fs use

Modify the ubifs u-boot wrapper function prototypes for generic fs use,
and give them their own header file.

This is a preparation patch for adding ubifs support to the generic fs
code from fs/fs.c.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Reviewed-by: Heiko Schocher <hs@denx.de>

diff --git a/common/cmd_ubifs.c b/common/cmd_ubifs.c
index 8e9a4e5038d87c19cfda9850dab0dbfa3d7446b0..5e9d357cd7d6f0a3d03d7e6634b7a1ad41060dfa 100644 (file)
--- a/common/cmd_ubifs.c
+++ b/common/cmd_ubifs.c
@@ -15,8 +15,7 @@
 #include <common.h>
 #include <config.h>
 #include <command.h>
-
-#include "../fs/ubifs/ubifs.h"
+#include <ubifs_uboot.h>
 
 static int ubifs_initialized;
 static int ubifs_mounted;
@@ -54,14 +53,7 @@ int ubifs_is_mounted(void)
 
 void cmd_ubifs_umount(void)
 {
-
-       if (ubifs_sb) {
-               printf("Unmounting UBIFS volume %s!\n",
-                      ((struct ubifs_info *)(ubifs_sb->s_fs_info))->vi.name);
-               ubifs_umount(ubifs_sb->s_fs_info);
-       }
-
-       ubifs_sb = NULL;
+       uboot_ubifs_umount();
        ubifs_mounted = 0;
        ubifs_initialized = 0;
 }

diff --git a/fs/ubifs/ubifs.c b/fs/ubifs/ubifs.c
index f7a084747e692ac0506cc8649b93058351e71f11..0a7a7bff09f2c73cc89bf89d65220d497912a4c4 100644 (file)
--- a/fs/ubifs/ubifs.c
+++ b/fs/ubifs/ubifs.c
@@ -570,7 +570,7 @@ static unsigned long ubifs_findfile(struct super_block *sb, char *filename)
        return 0;
 }
 
-int ubifs_ls(char *filename)
+int ubifs_ls(const char *filename)
 {
        struct ubifs_info *c = ubifs_sb->s_fs_info;
        struct file *file;
@@ -581,7 +581,7 @@ int ubifs_ls(char *filename)
        int ret = 0;
 
        c->ubi = ubi_open_volume(c->vi.ubi_num, c->vi.vol_id, UBI_READONLY);
-       inum = ubifs_findfile(ubifs_sb, filename);
+       inum = ubifs_findfile(ubifs_sb, (char *)filename);
        if (!inum) {
                ret = -1;
                goto out;
@@ -787,7 +787,8 @@ error:
        return err;
 }
 
-int ubifs_load(char *filename, u32 addr, u32 size)
+int ubifs_read(const char *filename, void *buf, loff_t offset,
+              loff_t size, loff_t *actread)
 {
        struct ubifs_info *c = ubifs_sb->s_fs_info;
        unsigned long inum;
@@ -798,10 +799,18 @@ int ubifs_load(char *filename, u32 addr, u32 size)
        int count;
        int last_block_size = 0;
 
+       *actread = 0;
+
+       if (offset & (PAGE_SIZE - 1)) {
+               printf("ubifs: Error offset must be a multple of %d\n",
+                      PAGE_SIZE);
+               return -1;
+       }
+
        c->ubi = ubi_open_volume(c->vi.ubi_num, c->vi.vol_id, UBI_READONLY);
        /* ubifs_findfile will resolve symlinks, so we know that we get
         * the real file here */
-       inum = ubifs_findfile(ubifs_sb, filename);
+       inum = ubifs_findfile(ubifs_sb, (char *)filename);
        if (!inum) {
                err = -1;
                goto out;
@@ -817,19 +826,24 @@ int ubifs_load(char *filename, u32 addr, u32 size)
                goto out;
        }
 
+       if (offset > inode->i_size) {
+               printf("ubifs: Error offset (%lld) > file-size (%lld)\n",
+                      offset, size);
+               err = -1;
+               goto put_inode;
+       }
+
        /*
         * If no size was specified or if size bigger than filesize
         * set size to filesize
         */
-       if ((size == 0) || (size > inode->i_size))
-               size = inode->i_size;
+       if ((size == 0) || (size > (inode->i_size - offset)))
+               size = inode->i_size - offset;
 
        count = (size + UBIFS_BLOCK_SIZE - 1) >> UBIFS_BLOCK_SHIFT;
-       printf("Loading file '%s' to addr 0x%08x with size %d (0x%08x)...\n",
-              filename, addr, size, size);
 
-       page.addr = (void *)addr;
-       page.index = 0;
+       page.addr = buf;
+       page.index = offset / PAGE_SIZE;
        page.inode = inode;
        for (i = 0; i < count; i++) {
                /*
@@ -846,16 +860,44 @@ int ubifs_load(char *filename, u32 addr, u32 size)
                page.index++;
        }
 
-       if (err)
+       if (err) {
                printf("Error reading file '%s'\n", filename);
-       else {
-               setenv_hex("filesize", size);
-               printf("Done\n");
+               *actread = i * PAGE_SIZE;
+       } else {
+               *actread = size;
        }
 
+put_inode:
        ubifs_iput(inode);
 
 out:
        ubi_close_volume(c->ubi);
        return err;
 }
+
+/* Compat wrappers for common/cmd_ubifs.c */
+int ubifs_load(char *filename, u32 addr, u32 size)
+{
+       loff_t actread;
+       int err;
+
+       printf("Loading file '%s' to addr 0x%08x...\n", filename, addr);
+
+       err = ubifs_read(filename, (void *)addr, 0, size, &actread);
+       if (err == 0) {
+               setenv_hex("filesize", actread);
+               printf("Done\n");
+       }
+
+       return err;
+}
+
+void uboot_ubifs_umount(void)
+{
+       if (ubifs_sb) {
+               printf("Unmounting UBIFS volume %s!\n",
+                      ((struct ubifs_info *)(ubifs_sb->s_fs_info))->vi.name);
+               ubifs_umount(ubifs_sb->s_fs_info);
+               ubifs_sb = NULL;
+       }
+}

diff --git a/fs/ubifs/ubifs.h b/fs/ubifs/ubifs.h
index a51b2376d22839cee21d5566ff4628063e1b3acc..225965c95d0226dd946f08666045c5eb34caceff 100644 (file)
--- a/fs/ubifs/ubifs.h
+++ b/fs/ubifs/ubifs.h
@@ -34,6 +34,7 @@
 #include <asm/atomic.h>
 #include <asm-generic/atomic-long.h>
 #include <ubi_uboot.h>
+#include <ubifs_uboot.h>
 
 #include <linux/ctype.h>
 #include <linux/time.h>
@@ -2379,11 +2380,6 @@ int ubifs_decompress(const void *buf, int len, void *out, int *out_len,
 #include "key.h"
 
 #ifdef __UBOOT__
-/* these are used in cmd_ubifs.c */
-int ubifs_init(void);
-int uboot_ubifs_mount(char *vol_name);
 void ubifs_umount(struct ubifs_info *c);
-int ubifs_ls(char *dir_name);
-int ubifs_load(char *filename, u32 addr, u32 size);
 #endif
 #endif /* !__UBIFS_H__ */

diff --git a/include/ubifs_uboot.h b/include/ubifs_uboot.h
new file mode 100644 (file)
index 0000000..3e0cd72
--- /dev/null
+++ b/include/ubifs_uboot.h
@@ -0,0 +1,28 @@
+/*
+ * UBIFS u-boot wrapper functions header
+ *
+ * Copyright (C) 2006-2008 Nokia Corporation
+ *
+ * (C) Copyright 2008-2009
+ * Stefan Roese, DENX Software Engineering, sr@denx.de.
+ *
+ * SPDX-License-Identifier:    GPL-2.0+
+ *
+ * Authors: Artem Bityutskiy (Битюцкий Артём)
+ *          Adrian Hunter
+ */
+
+#ifndef __UBIFS_UBOOT_H__
+#define __UBIFS_UBOOT_H__
+
+int ubifs_init(void);
+int uboot_ubifs_mount(char *vol_name);
+void uboot_ubifs_umount(void);
+int ubifs_is_mounted(void);
+int ubifs_load(char *filename, u32 addr, u32 size);
+
+int ubifs_ls(const char *dir_name);
+int ubifs_read(const char *filename, void *buf, loff_t offset,
+              loff_t size, loff_t *actread);
+
+#endif /* __UBIFS_UBOOT_H__ */