Message ID | 1359135165-32108-4-git-send-email-gregory.clement@free-electrons.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Fri, Jan 25, 2013 at 06:32:43PM +0100, Gregory CLEMENT wrote: > Now that we have support for local timers, enable it by default > > Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com> > --- > arch/arm/configs/mvebu_defconfig | 1 - > 1 file changed, 1 deletion(-) Applied patches 1, 2, and 3 of this series to mvebu/boards. In the future, please put driver changes first (patch #2) with few if any dependencies. In this case, since these patches depended on SMP support added to mvebu/boards, I had to apply them to mvebu/boards to prevent a circular dependency. thx, Jason.
On 02/15/2013 10:56 PM, Jason Cooper wrote: > On Fri, Jan 25, 2013 at 06:32:43PM +0100, Gregory CLEMENT wrote: >> Now that we have support for local timers, enable it by default >> >> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com> >> --- >> arch/arm/configs/mvebu_defconfig | 1 - >> 1 file changed, 1 deletion(-) > > Applied patches 1, 2, and 3 of this series to mvebu/boards. In the > future, please put driver changes first (patch #2) with few if any > dependencies. > > In this case, since these patches depended on SMP support added to > mvebu/boards, I had to apply them to mvebu/boards to prevent a circular > dependency. Well the patch #2 can't work without the #1 (local interrupt support), so if the order is changed then we loose the "bissecatiblity". That was the reason of this order. I hope Olof talk at ELC will help to understand better this workflow! > > thx, > > Jason. >
On Fri, Feb 15, 2013 at 11:04:14PM +0100, Gregory CLEMENT wrote: > On 02/15/2013 10:56 PM, Jason Cooper wrote: > > On Fri, Jan 25, 2013 at 06:32:43PM +0100, Gregory CLEMENT wrote: > >> Now that we have support for local timers, enable it by default > >> > >> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com> > >> --- > >> arch/arm/configs/mvebu_defconfig | 1 - > >> 1 file changed, 1 deletion(-) > > > > Applied patches 1, 2, and 3 of this series to mvebu/boards. In the > > future, please put driver changes first (patch #2) with few if any > > dependencies. > > > > In this case, since these patches depended on SMP support added to > > mvebu/boards, I had to apply them to mvebu/boards to prevent a circular > > dependency. > > Well the patch #2 can't work without the #1 (local interrupt support), so if > the order is changed then we loose the "bissecatiblity". That was the reason of > this order. Yes, that's why I kept them in order and didn't put them in mvebu/drivers. thx, Jason.
diff --git a/arch/arm/configs/mvebu_defconfig b/arch/arm/configs/mvebu_defconfig index b5bc96c..28c1e38 100644 --- a/arch/arm/configs/mvebu_defconfig +++ b/arch/arm/configs/mvebu_defconfig @@ -14,7 +14,6 @@ CONFIG_MACH_ARMADA_XP=y # CONFIG_CACHE_L2X0 is not set # CONFIG_SWP_EMULATE is not set CONFIG_SMP=y -# CONFIG_LOCAL_TIMERS is not set CONFIG_AEABI=y CONFIG_HIGHMEM=y # CONFIG_COMPACTION is not set
Now that we have support for local timers, enable it by default Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com> --- arch/arm/configs/mvebu_defconfig | 1 - 1 file changed, 1 deletion(-)