From mboxrd@z Thu Jan  1 00:00:00 1970
From: Heiko Schocher <hs@denx.de>
Date: Mon, 26 Oct 2015 11:39:44 +0100
Subject: [U-Boot] [PATCH 03/13] compat: Remove is_power_of_2() definition
In-Reply-To: <CAOMZO5C8HCReeXq_Z6s65O0u=_ZkSDmSwTXdjnzkFqXC8+C8VQ@mail.gmail.com>
References: <1445689693-21436-1-git-send-email-festevam@gmail.com>
 <1445689693-21436-3-git-send-email-festevam@gmail.com>
 <562DB619.5020504@denx.de>
 <CAOMZO5C8HCReeXq_Z6s65O0u=_ZkSDmSwTXdjnzkFqXC8+C8VQ@mail.gmail.com>
Message-ID: <562E02F0.7010505@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hello Fabio,

Am 26.10.2015 um 11:22 schrieb Fabio Estevam:
> Hi Heiko,
>
> On Mon, Oct 26, 2015 at 3:11 AM, Heiko Schocher <hs@denx.de> wrote:
>
>>> diff --git a/fs/ubifs/super.c b/fs/ubifs/super.c
>>> index 41763a1..eaae5fb 100644
>>> --- a/fs/ubifs/super.c
>>> +++ b/fs/ubifs/super.c
>>> @@ -31,7 +31,7 @@
>>>    #include <common.h>
>>>    #include <malloc.h>
>>>    #include <memalign.h>
>>> -#include <linux/compat.h>
>>> +#include <linux/log2.h>
>>>    #include <linux/stat.h>
>>>    #include <linux/err.h>
>>>    #include "ubifs.h"
>
>> It would be nice to ge rid of linux/compat.h at all in ubi/ubifs
>> code ... but does this really compile for all boards?
>
> I haven't seen a board failure because of this change on the several
> arch's/boards I have built.

Did you compilied all arm boards?

> Could you please provide a failure case?

I did not tried your patches, but:

fs/ubifs/super.c calls a lot of functions, which are "defined" in
include/linux/compat.h

for example:

kfree
kmem_cache_alloc
mutex_init
spin_lock_init
...

>> I think you should add here linux/log2.h, without deleting linux/compat.h
>
> My understanding is that linux/compat.h is currently used only for
> providing is_power_of_2() in this file.

I think there is more used from include/linux/compat.h

> Now that we have log2.h (which provides is_power_of_2()), we can
> simply use it instead.
>
> Am I missing anything here?

I think the aristainetos or aristainetos2 board will fail ... Could you
please try to compile them? If not, I try to find time to do it.

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany