[U-Boot] [PATCH v2 1/2] ftpmu010: support faraday ftpmu010 driver

Po-Yu Chuang ratbert.chuang at gmail.com
Wed Feb 16 10:10:10 CET 2011


Hi Macpaul,

On Wed, Feb 16, 2011 at 3:58 PM, Macpaul Lin <macpaul at gmail.com> wrote:
>
> As you can see, the include of PMU header has been replaced to a correct path.
> [U-Boot,2/3] fttmr010: move fttmr010 controller to drivers/timer folder
> http://patchwork.ozlabs.org/patch/71952/
>
> However, I cannot found the part of previous patch [U-Boot,1/3]
> You can find that I have replace a correct path to the file
> "arch/arm/cpu/arm920t/a320/timer.c" which haven't been applied.
> http://www.mail-archive.com/u-boot@lists.denx.de/msg41320.html
>
> Wolfgang, could you please check if something have been missing in the
> last git apply?
> It looks like that we have something missed in the current tree while
> this patch have been already applied
> "http://www.mail-archive.com/u-boot@lists.denx.de/msg41320.html".
>
> The missing part is the following patch.
> diff --git a/arch/arm/cpu/arm920t/a320/timer.c
> b/arch/arm/cpu/arm920t/a320/timer.c
> index d2e316f..4718ae6 100644
> --- a/arch/arm/cpu/arm920t/a320/timer.c
> +++ b/arch/arm/cpu/arm920t/a320/timer.c
> @@ -19,7 +19,7 @@
>
>  #include <common.h>
>  #include <asm/io.h>
> -#include <asm/arch/ftpmu010.h>
> +#include "../../../../../drivers/power/ftpmu010.h"
>  #include <asm/arch/fttmr010.h>
>
>  static ulong timestamp;

If I remember correctly, this patch was not applied
because Wolfgang does not like the include "../../../..".

> Po-Yu,
> Yes, there are also another patches waiting to be review and be applied.
> ftpmu010: fix relocation and enhance features
> http://patchwork.ozlabs.org/patch/77704/

This patch doesn't seem to affect a320.

Since I have a patch to fix the current (compilation) problem,
I'll send it to RFC.

regards,
Po-Yu Chuang


More information about the U-Boot mailing list