[U-Boot] [PATCH v2 1/4] ubifs: Modify ubifs u-boot wrapper function prototypes for generic fs use

[Tom Rini]

On Tue, Aug 25, 2015 at 01:32:05PM +0200, Hans de Goede wrote:
> 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 <hdegoede at 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 <hs at 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 ?

I can just take 'em all into master :)

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150828/dabc5046/attachment.sig>