[PATCH v2 1/3] ARM: Add CLPS711X architecture

[Sascha Hauer]

On Tue, Oct 16, 2012 at 10:09:47AM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 22:56 Mon 15 Oct     , Alexander Shiyan wrote:
> > On Mon, 15 Oct 2012 19:54:14 +0200
> > Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com> wrote:
> > 
> > > On 20:26 Mon 15 Oct     , Alexander Shiyan wrote:
> > > > This patch adds new architecture (CLPS711X) into barebox.
> > > > The core-logic functionality of the device is built around an ARM720T
> > > > processor running at clock speeds up to 90 MHz.
> > > > 
> > > > Signed-off-by: Alexander Shiyan <shc_work at mail.ru>
> > ...
> > > > +#include <linux/clkdev.h>
> > > split the code in 2
> > >  - clock
> > >  - clocksource in drivers/clocksource
> > >  and cs as a driver
> > drivers/clocksource is missing in master repository and as far I know we are
> > made patches for "master" but not for "next".
> so base your code on next

Generally basing code on next causes maintenance burden. I really prefer
patches on master. In this case it means that the location for the
clocksource driver is fine. What we can look at is that this files
really only contains the clocksource so that it can simply be moved
later.

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |