Dear Sonic Zhang, In message <1361257266-3722-2-git-send-email-sonic....@gmail.com> you wrote: > > Add header files for blackfin new processor bf60x. ... > arch/blackfin/include/asm/mach-bf609/BF609_def.h | 3758 > +++++++++++++++++++++
This is a bit excessive, isn't it? Do we really all thse many thousands lines of defines? > diff --git a/arch/blackfin/include/asm/mach-bf609/BF609_def.h > b/arch/blackfin/include/asm/mach-bf609/BF609_def.h > new file mode 100644 > index 0000000..39b740f > --- /dev/null > +++ b/arch/blackfin/include/asm/mach-bf609/BF609_def.h > @@ -0,0 +1,3758 @@ > +/* DO NOT EDIT THIS FILE > + * Automatically generated by generate-def-headers.xsl > + * DO NOT EDIT THIS FILE > + */ > + > +#ifndef __BFIN_DEF_ADSP_BF609_proc__ > +#define __BFIN_DEF_ADSP_BF609_proc__ > + > +#include "../mach-common/ADSP-EDN-core_def.h" > + > +#define CNT_CFG 0xFFC00400 /* CNT0 Configuration > Register */ > +#define CNT_IMSK 0xFFC00404 /* CNT0 Interrupt > Mask Register */ > +#define CNT_STAT 0xFFC00408 /* CNT0 Status > Register */ > +#define CNT_CMD 0xFFC0040C /* CNT0 Command > Register */ > +#define CNT_DEBNCE 0xFFC00410 /* CNT0 Debounce > Register */ > +#define CNT_CNTR 0xFFC00414 /* CNT0 Counter > Register */ > +#define CNT_MAX 0xFFC00418 /* CNT0 Maximum Count > Register */ > +#define CNT_MIN 0xFFC0041C /* CNT0 Minimum Count > Register */ > + > +#define RSI_CLK_CONTROL 0xFFC00604 /* RSI0 Clock Control > Register */ > +#define RSI_ARGUMENT 0xFFC00608 /* RSI0 Argument > Register */ > +#define RSI_COMMAND 0xFFC0060C /* RSI0 Command > Register */ > +#define RSI_RESP_CMD 0xFFC00610 /* RSI0 Response > Command Register */ > +#define RSI_RESPONSE0 0xFFC00614 /* RSI0 Response 0 > Register */ > +#define RSI_RESPONSE1 0xFFC00618 /* RSI0 Response 1 > Register */ > +#define RSI_RESPONSE2 0xFFC0061C /* RSI0 Response 2 > Register */ > +#define RSI_RESPONSE3 0xFFC00620 /* RSI0 Response 3 > Register */ > +#define RSI_DATA_TIMER 0xFFC00624 /* RSI0 Data Timer > Register */ > +#define RSI_DATA_LGTH 0xFFC00628 /* RSI0 Data Length > Register */ > +#define RSI_DATA_CONTROL 0xFFC0062C /* RSI0 Data Control > Register */ > +#define RSI_DATA_CNT 0xFFC00630 /* RSI0 Data Count > Register */ > +#define RSI_STATUS 0xFFC00634 /* RSI0 Status > Register */ > +#define RSI_STATUSCL 0xFFC00638 /* RSI0 Status Clear > Register */ > +#define RSI_IMSK0 0xFFC0063C /* RSI0 Interrupt 0 > Mask Register */ > +#define RSI_IMSK1 0xFFC00640 /* RSI0 Interrupt 1 > Mask Register */ > +#define RSI_FIFO_CNT 0xFFC00648 /* RSI0 FIFO Counter > Register */ ... I think I mentioned before that device registers have to be described using C structs in U-Boot, and proper I/O accessors must be used to access these. In any case, please trim to the really needed definitions. > --- /dev/null > +++ b/arch/blackfin/include/asm/mach-bf609/anomaly.h > @@ -0,0 +1,103 @@ > +/* > + * DO NOT EDIT THIS FILE > + * This file is under version control at > + * > svn://sources.blackfin.uclinux.org/toolchain/trunk/proc-defs/header-frags/ > + * and can be replaced with that version at any time > + * DO NOT EDIT THIS FILE This makes no sense to me. > + * Copyright 2004-2012 Analog Devices Inc. > + * Licensed under the ADI BSD license. > + * https://docs.blackfin.uclinux.org/doku.php?id=adi_bsd > + */ I'm not sure if this has been discussed before - is this really GPL compatible? 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: w...@denx.de Harrison's Postulate: For every action, there is an equal and opposite criticism. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot