Dear Semih Hazar,

In message <[email protected]> you wrote:
> Nimbus Cloud is an AVR32 based single board computer with
> 256MiB NAND, 64MiB SDRAM, battery backed RTC, LCD/touch
> screen support, VGA Output, Ethernet and offers seamless
> integration with Indefia's Zigbee transceivers.
> 
> ---
>  CREDITS                                 |    4 +
>  MAINTAINERS                             |    4 +
>  MAKEALL                                 |    1 +
>  Makefile                                |    3 +
>  board/indefia/nimbuscloud/Makefile      |   40 +++++++
>  board/indefia/nimbuscloud/config.mk     |    3 +
>  board/indefia/nimbuscloud/nimbuscloud.c |  165 +++++++++++++++++++++++++++
>  board/indefia/nimbuscloud/nimbuscloud.h |    8 ++
>  board/indefia/nimbuscloud/u-boot.lds    |   72 ++++++++++++
>  include/configs/nimbuscloud.h           |  184 
> +++++++++++++++++++++++++++++++
>  10 files changed, 484 insertions(+), 0 deletions(-)

I am not sure how this patch addresses the previously discussed topic
of cached/uncached accesses to the NOR flash. It does not mention the
topic anywhere in the commit message or other comments, which I
consider a serious flaw of the patch.

>From the

        #define CONFIG_SYS_FLASH_BASE          0xa0000000

I understand that this is the version where the end user has to deal
with two different address ranges (i. e. "md 10000" would be used
to dump the 2nd 64kB-sector, while "era A0010000" would be needed to
erase the same sector.

Such behaviour is seems unacceptable to me. If my understanding is
corrent, I tend to reject this patch. [Please see explanations in
previous messages for other options to solve this issue.]

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [email protected]
The IQ of the group is the lowest IQ of a member of the group divided
by the number of people in the group.
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to