On Tue, Nov 23, 2010 at 12:32:43AM +0800, mkl0301@xxxxxxxxx wrote: > From: Mac Lin <mkl0301@xxxxxxxxx> > > This patch add cns3xxx_pwr_clk_dis, and export thoes power management functions > that may be used by many other device drivers on CNS3XXX. > > Signed-off-by: Mac Lin <mkl0301@xxxxxxxxx> > --- > arch/arm/mach-cns3xxx/core.h | 4 ++-- > arch/arm/mach-cns3xxx/include/mach/pm.h | 19 +++++++++++++++++++ > arch/arm/mach-cns3xxx/pm.c | 17 +++++++++++++++++ > 3 files changed, 38 insertions(+), 2 deletions(-) > create mode 100644 arch/arm/mach-cns3xxx/include/mach/pm.h > > diff --git a/arch/arm/mach-cns3xxx/core.h b/arch/arm/mach-cns3xxx/core.h > index 6b33ec1..c6636bd 100644 > --- a/arch/arm/mach-cns3xxx/core.h > +++ b/arch/arm/mach-cns3xxx/core.h > @@ -11,13 +11,13 @@ > #ifndef __CNS3XXX_CORE_H > #define __CNS3XXX_CORE_H > > +#include <mach/pm.h> This isn't needed in this file. Instead, devices.c should include it. Plus, pm.c needs to include mach/pm.h, otherwise sparse gives the following warnings: CHECK arch/arm/mach-cns3xxx/pm.c pm.c:26:6: warning: symbol 'cns3xxx_pwr_clk_dis' was not declared. Should it be static? pm.c:35:6: warning: symbol 'cns3xxx_pwr_power_up' was not declared. Should it be static? pm.c:47:6: warning: symbol 'cns3xxx_pwr_power_down' was not declared. Should it be static? pm.c:121:10: warning: symbol 'usb_pwr_ref' was not declared. Should it be static? I've fixed it up, and applied the patch to cns3xxx tree. Thanks! -- Anton Vorontsov Email: cbouatmailru@xxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html