[U-Boot] [PATCH 01/15 V3] iMX28: Initial support for iMX28 CPU

Wolfgang Denk wd at denx.de
Fri Oct 14 23:01:23 CEST 2011


Dear Marek Vasut,

In message <1317375728-23802-1-git-send-email-marek.vasut at gmail.com> you wrote:
> This patch supports:
> - Timers
> - Debug UART
> - Clock
> 
> Signed-off-by: Marek Vasut <marek.vasut at gmail.com>
> Cc: Stefano Babic <sbabic at denx.de>
> Cc: Wolfgang Denk <wd at denx.de>
> Cc: Detlev Zundel <dzu at denx.de>
> ---
>  arch/arm/cpu/arm926ejs/mx28/Makefile          |   46 +
>  arch/arm/cpu/arm926ejs/mx28/clock.c           |  355 +++++++
>  arch/arm/cpu/arm926ejs/mx28/mx28.c            |  194 ++++
>  arch/arm/cpu/arm926ejs/mx28/timer.c           |  141 +++
>  arch/arm/include/asm/arch-mx28/clock.h        |   61 ++
>  arch/arm/include/asm/arch-mx28/imx-regs.h     |   38 +
>  arch/arm/include/asm/arch-mx28/regs-base.h    |   88 ++
>  arch/arm/include/asm/arch-mx28/regs-bch.h     |  230 +++++
>  arch/arm/include/asm/arch-mx28/regs-clkctrl.h |  312 ++++++
>  arch/arm/include/asm/arch-mx28/regs-common.h  |   66 ++
>  arch/arm/include/asm/arch-mx28/regs-gpmi.h    |  222 +++++
>  arch/arm/include/asm/arch-mx28/regs-i2c.h     |  207 ++++
>  arch/arm/include/asm/arch-mx28/regs-ocotp.h   |  173 ++++
>  arch/arm/include/asm/arch-mx28/regs-pinctrl.h | 1284 +++++++++++++++++++++++++
>  arch/arm/include/asm/arch-mx28/regs-power.h   |  413 ++++++++
>  arch/arm/include/asm/arch-mx28/regs-rtc.h     |  147 +++
>  arch/arm/include/asm/arch-mx28/regs-ssp.h     |  349 +++++++
>  arch/arm/include/asm/arch-mx28/regs-timrot.h  |  171 ++++
>  arch/arm/include/asm/arch-mx28/sys_proto.h    |   30 +
>  19 files changed, 4527 insertions(+), 0 deletions(-)
>  create mode 100644 arch/arm/cpu/arm926ejs/mx28/Makefile
>  create mode 100644 arch/arm/cpu/arm926ejs/mx28/clock.c
>  create mode 100644 arch/arm/cpu/arm926ejs/mx28/mx28.c
>  create mode 100644 arch/arm/cpu/arm926ejs/mx28/timer.c
>  create mode 100644 arch/arm/include/asm/arch-mx28/clock.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/imx-regs.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-base.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-bch.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-clkctrl.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-common.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-gpmi.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-i2c.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-ocotp.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-pinctrl.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-power.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-rtc.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-ssp.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/regs-timrot.h
>  create mode 100644 arch/arm/include/asm/arch-mx28/sys_proto.h
...
> +#include <common.h>
> +#include <asm/errno.h>
> +#include <asm/io.h>
> +#include <asm/arch/clock.h>
> +#include <asm/arch/gpio.h>
> +#include <asm/arch/imx-regs.h>
> +#include <asm/arch/sys_proto.h>
> +#include <asm/arch/iomux-mx28.h>

You include <asm/arch/iomux-mx28.h> here, but add this file only in
patch # 4 of your series.  That means your patches are not bisectable.

Please fix.

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
Man did not weave the web of life; he  is  merely  a  strand  in  it.
Whatever he does to the web, he does to himself.     - Seattle [1854]


More information about the U-Boot mailing list