[U-Boot] [PATCH v2 1/2] power: ftpmu010: move drivers/power/ftpmu010.h to include/faraday
Po-Yu Chuang
ratbert.chuang at gmail.com
Fri Mar 18 07:15:10 CET 2011
Hi Macpaul,
On Fri, Mar 18, 2011 at 1:57 PM, Macpaul Lin <macpaul at gmail.com> wrote:
> 2011/2/18 Po-Yu Chuang <ratbert.chuang at gmail.com>
>> move include/ftpmu010.h to include/faraday/ftpmu010.h
>>
>> drivers/power/ftpmu010.c | 2 +-
>> drivers/power/ftpmu010.h | 146 ------------------------------------------
>> include/faraday/ftpmu010.h | 150 ++++++++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 151 insertions(+), 147 deletions(-)
>> delete mode 100644 drivers/power/ftpmu010.h
>> create mode 100644 include/faraday/ftpmu010.h
>
> Sorry I didn't notice that there are API declaration before in ftpmu010.h.
>
>> +void ftpmu010_32768osc_enable(void);
This is used by a320evb.
>> +void ftpmu010_dlldis_disable(void);
>> +void ftpmu010_sdram_clk_disable(unsigned int cr0);
These are not.
> Po-Yu, does these three function declaration is a necessity in a320evb?
>
> I'm using gcc 4.4 without these 3 function declaration seems okay.
Either adp-ag101 does not use any of them or you put the declarations
somewhere in board-specific code which is wrong.
> Because I'm going to refine another enhancement patch for using
> ftpmu010 in lowlevel_init.S.
> Hence I need to use #ifndef __ASSEMBLY__ to protect these three APIs.
No problem here. Please use __ASSEMBLY__.
> However, to compile adp-ag101 board (NDS32) doesn't require
> these 3 functions to be declared here. I need your opinion to clarify this.
Does your board use ftpmu010_dlldis_disable() or
ftpmu010_sdram_clk_disable() ?
Best regards,
Po-Yu Chuang
More information about the U-Boot
mailing list