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

[Hans de Goede <hdegoede@redhat.com>]

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@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@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