[PATCH 2/2] at91sam9260/9g20ek: remove #ifdef for ek_usb_add_device_mci()

Fabio Porcedda fabio.porcedda at gmail.com
Fri Feb 15 06:14:29 EST 2013


On Fri, Feb 15, 2013 at 11:52 AM, Fabio Porcedda
<fabio.porcedda at gmail.com> wrote:
> On Fri, Feb 15, 2013 at 11:44 AM, Jean-Christophe PLAGNIOL-VILLARD
> <plagnioj at jcrosoft.com> wrote:
>> On 11:41 Fri 15 Feb     , Fabio Porcedda wrote:
>>> Remove #ifdef for ek_usb_add_device_mci() because there is already
>>> the #ifdef inside at91_add_device_mci(), this way the compiler can check
>>> always the code without any penality.
>>  it's not the struct will be keep
>
> The struct isn't keep
> without this patch:
>   text       data        bss        dec        hex    filename
>     877        192          0       1069        42d
> arch/arm/boards/at91sam9260ek/init.o
> with this patch:
>  text       data        bss        dec        hex    filename
>     877        192          0       1069        42d
> arch/arm/boards/at91sam9260ek/init.o


I'm sorry, you are right, i've made i mistake when checking.

  text       data        bss        dec        hex    filename
    861        176          0       1037        40d
arch/arm/boards/at91sam9260ek/init.o

  text       data        bss        dec        hex    filename
    877        192          0       1069        42d
arch/arm/boards/at91sam9260ek/init.o


>>  keep the ifdef
>> Best Regards,
>> J.
>
> Best regards
> --
> Fabio Porcedda



--
Fabio Porcedda



More information about the barebox mailing list