From mboxrd@z Thu Jan 1 00:00:00 1970 From: mturquette@linaro.org (Mike Turquette) Date: Fri, 24 Aug 2012 17:21:51 -0700 Subject: [PATCH V5 0/5] clk: mmp: add clock framework for mmp In-Reply-To: <1345431315-8037-1-git-send-email-xiechao.mail@gmail.com> References: <1345431315-8037-1-git-send-email-xiechao.mail@gmail.com> Message-ID: <20120825002151.18366.99434@nucleus> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Quoting Chao Xie (2012-08-19 19:55:10) > From: Chao Xie > arch/arm/mach-mmp/Kconfig | 3 + > drivers/clk/Makefile | 3 + > drivers/clk/mmp/Makefile | 9 + > drivers/clk/mmp/clk-apbc.c | 152 ++++++++++++++ > drivers/clk/mmp/clk-apmu.c | 97 +++++++++ > drivers/clk/mmp/clk-frac.c | 153 ++++++++++++++ > drivers/clk/mmp/clk-mmp2.c | 449 ++++++++++++++++++++++++++++++++++++++++++ > drivers/clk/mmp/clk-pxa168.c | 346 ++++++++++++++++++++++++++++++++ > drivers/clk/mmp/clk-pxa910.c | 320 ++++++++++++++++++++++++++++++ > drivers/clk/mmp/clk.h | 35 ++++ Looks like you are not removing your arch/arm/mach-mmp/clock.c. Is that intentional? When I apply your series against v3.6-rc3 I find that compilation breaks with mmp2_defconfig due to conflicting definitions for the clk api (clk_enable, clk_set_rate, etc). This is not surprising since your legacy clock code is neither deleted nor removed from compilation conditionally by checking for CONFIG_COMMON_CLK. Did I somehow manage to misapply your patches or should your patches have removed the arch-specific clock framework as well? Regards, Mike > 10 files changed, 1567 insertions(+), 0 deletions(-) > create mode 100644 drivers/clk/mmp/Makefile > create mode 100644 drivers/clk/mmp/clk-apbc.c > create mode 100644 drivers/clk/mmp/clk-apmu.c > create mode 100644 drivers/clk/mmp/clk-frac.c > create mode 100644 drivers/clk/mmp/clk-mmp2.c > create mode 100644 drivers/clk/mmp/clk-pxa168.c > create mode 100644 drivers/clk/mmp/clk-pxa910.c > create mode 100644 drivers/clk/mmp/clk.h