[U-Boot] [PATCH 01/13] Blackfin: BF60x: new processor header files

Wolfgang Denk wd at denx.de
Sun Sep 2 16:10:04 CEST 2012


Dear Bob Liu,

In message <1345526833-10804-1-git-send-email-lliubbo at gmail.com> you wrote:
> Add header files for blackfin new processor bf60x.
> 
> Signed-off-by: Bob Liu <lliubbo at gmail.com>
> ---
>  arch/blackfin/include/asm/blackfin_cdef.h         |    3 +
>  arch/blackfin/include/asm/blackfin_def.h          |    5 +
>  arch/blackfin/include/asm/blackfin_local.h        |    3 +
>  arch/blackfin/include/asm/mach-bf609/BF609_cdef.h |  543 +++
>  arch/blackfin/include/asm/mach-bf609/BF609_def.h  | 3758 +++++++++++++++++++++
>  arch/blackfin/include/asm/mach-bf609/anomaly.h    |  128 +
>  arch/blackfin/include/asm/mach-bf609/def_local.h  |    5 +
>  arch/blackfin/include/asm/mach-bf609/portmux.h    |  257 ++
>  arch/blackfin/include/asm/mach-bf609/ports.h      |  103 +
>  arch/blackfin/include/asm/mach-common/bits/cgu.h  |   80 +
>  arch/blackfin/include/asm/mach-common/bits/dde.h  |   88 +
>  arch/blackfin/include/asm/mach-common/bits/mpu.h  |    6 +-
>  arch/blackfin/include/asm/mach-common/bits/pll.h  |    5 +
>  13 files changed, 4983 insertions(+), 1 deletion(-)
>  create mode 100644 arch/blackfin/include/asm/mach-bf609/BF609_cdef.h
>  create mode 100644 arch/blackfin/include/asm/mach-bf609/BF609_def.h
>  create mode 100644 arch/blackfin/include/asm/mach-bf609/anomaly.h
>  create mode 100644 arch/blackfin/include/asm/mach-bf609/def_local.h
>  create mode 100644 arch/blackfin/include/asm/mach-bf609/portmux.h
>  create mode 100644 arch/blackfin/include/asm/mach-bf609/ports.h
>  create mode 100644 arch/blackfin/include/asm/mach-common/bits/cgu.h
>  create mode 100644 arch/blackfin/include/asm/mach-common/bits/dde.h

Please make sure to have the string "PATCH" included with all patch
submissions, otherwise your patches are lost to patchwork, and most
likely to the respective custodian as well.

> --- /dev/null
> +++ b/arch/blackfin/include/asm/mach-bf609/BF609_cdef.h
...
> +#define bfin_read_CGU_STAT() bfin_read32(CGU_STAT)
> +#define bfin_read_CGU_CLKOUTSEL() bfin_read32(CGU_CLKOUTSEL)
> +#define bfin_read_CGU_CTL() bfin_read32(CGU_CTL)
> +#define bfin_write_CGU_CTL(val) bfin_write32(CGU_CTL, val)
> +#define bfin_read_CGU_DIV() bfin_read32(CGU_DIV)
> +#define bfin_write_CGU_DIV(val) bfin_write32(CGU_DIV, val)

We don't allow CamelCaps identifiers.  Please fix globally.


> +#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 */

We don't allow register access based on raw addresses or base address
plus offset.  Please define proper C structs to describe your
hardware.   Please fix globally.


> --- /dev/null
> +++ b/arch/blackfin/include/asm/mach-bf609/anomaly.h
> @@ -0,0 +1,128 @@
> +/*
> + * 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 is bullshit.  If you submit code to U-Boot, it gets maintained in
the U-Boot git repository.  Dump this.

> + * Copyright 2004-2010 Analog Devices Inc.
> + * Licensed under the ADI BSD license.
> + *   https://docs.blackfin.uclinux.org/doku.php?id=adi_bsd

Is this GPL compatible??

The link is dead and returns only

	This topic does not exist yet

> +#define ANOMALY_05000074 (1)

Please do not put parens around simple defines.  Please fix globally.

Checkpatch throws a ton of errors that all need fixing.

Also note that we don't allow fixed network parameters (like
CONFIG_ETHADDR) in board config files.

Review stops here (except for the last part with the licensing stuff).

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: wd at denx.de
All these theories, diverse as they are, have two things  in  common:
they explain the observed facts, and they are completeley and utterly
wrong.                       - Terry Pratchett, _The Light Fantastic_


More information about the U-Boot mailing list