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

[Macpaul Lin]

Hi Po-Yu,

2011/2/16 Po-Yu Chuang <ratbert.chuang at gmail.com>:
> 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 "../../../..".

Yes.
However, I also found there exists similar include format in other SoC.
Even you put drivers for all common IP still have the same problem.

For example, like some SoCs will use the drivers in "board/Marvell/common/".
If you want to use the API inside "board/Marvell/common/",
the code will looks like "include ../../board/Marvell/common".

>> 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.

Just one comment,
I think you can put it into folder "include/faraday"
I have discussed with Wolfgang about collecting the faraday header
files together before.
This should be better for future driver releasement.

> regards,
> Po-Yu Chuang
>



-- 
Best regards,
Macpaul Lin