[U-Boot] [PATCH] Add support for Indefia Nimbus Cloud Board

Semih Hazar semih.hazar at indefia.com
Thu Jan 21 14:13:35 CET 2010


Dear Wolfgang,

Please find my answers below.
>
> Is ther e a chance to do without this file and use the CFI driver
> instead?
>
>   
There's currently a problem with the CFI driver in AVR32 platform. There
was a long discussion about it, but a solution was not available. If
that issue is solved, I can submit a patch to change this to use CFI.
>
>> +	// First 8 sectors are 8k
>>     
> Please do not use C++ comments.
>
>   
These are fixed.
>
>> +	/* Enable GPIO on config swicthes */
>> +	portmux_select_gpio(PORTMUX_PORT_B, (1 << 13) | (1 << 14) | (1 << 15) | (1 << 16),
>>     
> Line too long. Please fix globally.
>
>   
Fixed.
> Line too long. And please use strmhz() to print clock frequencies.
>
> Um... this print does not belong here at all. CPU init messages and
> board initi messages have no place in initdram() at all.
>
>   
Removed it.
>> diff --git a/board/indefia/nimbuscloud/u-boot.lds b/board/indefia/nimbuscloud/u-boot.lds
>> new file mode 100644
>> index 0000000..a7243f2
>> --- /dev/null
>> +++ b/board/indefia/nimbuscloud/u-boot.lds
>>     
> Do you really need a board specific linker script?
>
>
>   
The README file says to put a u-boot.lds file for a new board and I did
so. I also looked at the other AVR32 boards and they all also have it.
If there's another way, I can change it.

I'm sending the revised patch in another mail.

Best regards,
Semih Hazar



More information about the U-Boot mailing list