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

Semih Hazar
2010-01-21T13:13:35Z

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