Hi,
On 25-08-15 13:00, Heiko Schocher wrote:
Hello Hans,
Am 22.08.2015 um 20:04 schrieb Hans de Goede:
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 <hdego...@redhat.com>
---
common/cmd_ubifs.c | 14 +++--------
fs/ubifs/ubifs.c | 70 ++++++++++++++++++++++++++++++++++++++++-----------
fs/ubifs/ubifs.h | 6 +----
include/ubifs_uboot.h | 29 +++++++++++++++++++++
4 files changed, 89 insertions(+), 30 deletions(-)
create mode 100644 include/ubifs_uboot.h
Only one nitpick, beside of this, you can add my:
Reviewed-by: Heiko Schocher <h...@denx.de>
diff --git a/common/cmd_ubifs.c b/common/cmd_ubifs.c
index 8e9a4e5..0a3dd24 100644
--- a/common/cmd_ubifs.c
+++ b/common/cmd_ubifs.c
@@ -15,11 +15,10 @@
#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;
+int ubifs_mounted;
later you add "extern int ubifs_mounted" in include/ubifs_uboot.h
Maybe you want to add a function, which returns the state
of this var and let the var static?
Yes that would be cleaner, I'll fix the patch-set to do
things that way.
Thanks for the reviews.
So when the time come comes (when v2015.10 is out), how do
we merge these 3, shall I take them upstream through the
linux-sunxi tree, or do you want me to send a v2 with this fixed,
and then you take them upstream ?
Regards,
Hans
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
