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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxxx> 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@xxxxxxx>
> > ...
> > > > +#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 |

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox


[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux