Message ID | 1349220069-4051-1-git-send-email-swarren@wwwdotorg.org |
---|---|
State | Rejected, archived |
Headers | show |
On Wed, 2012-10-03 at 07:21 +0800, Stephen Warren wrote: > From: Stephen Warren <swarren@nvidia.com> > > sleep.S doesn't provide anything useful any more; commit b4e395b "ARM: > tegra: Remove flow controller programming" removed the last public code > here, so that everyting that was left was just macros or #includes. > > Signed-off-by: Stephen Warren <swarren@nvidia.com> > --- Hi Stephen, Just let you know. I may update this file later for CPU idle LP2 support. Anyway, it's OK to me to re-add this file when introducing CPU idle LP2 support as well. Thanks, Joseph -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 10/02/2012 08:06 PM, Joseph Lo wrote: > On Wed, 2012-10-03 at 07:21 +0800, Stephen Warren wrote: >> From: Stephen Warren <swarren@nvidia.com> >> >> sleep.S doesn't provide anything useful any more; commit b4e395b "ARM: >> tegra: Remove flow controller programming" removed the last public code >> here, so that everyting that was left was just macros or #includes. >> >> Signed-off-by: Stephen Warren <swarren@nvidia.com> >> --- > > Hi Stephen, > > Just let you know. > I may update this file later for CPU idle LP2 support. Anyway, it's OK > to me to re-add this file when introducing CPU idle LP2 support as well. Wouldn't the code go in sleep-t20.S, sleep-t30.S, or cpuidle.c anyway? But yes, I think removing the file until there's actually content should be fine. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 10/03/2012 09:25 AM, Stephen Warren wrote: > On 10/02/2012 08:06 PM, Joseph Lo wrote: >> On Wed, 2012-10-03 at 07:21 +0800, Stephen Warren wrote: >>> From: Stephen Warren <swarren@nvidia.com> >>> >>> sleep.S doesn't provide anything useful any more; commit b4e395b "ARM: >>> tegra: Remove flow controller programming" removed the last public code >>> here, so that everyting that was left was just macros or #includes. >>> >>> Signed-off-by: Stephen Warren <swarren@nvidia.com> >>> --- >> >> Hi Stephen, >> >> Just let you know. >> I may update this file later for CPU idle LP2 support. Anyway, it's OK >> to me to re-add this file when introducing CPU idle LP2 support as well. > > Wouldn't the code go in sleep-t20.S, sleep-t30.S, or cpuidle.c anyway? > > But yes, I think removing the file until there's actually content should > be fine. Since the cpuidle patches have now shown up and are likely to be merged in 3.8, I'll hold off on applying this patch to avoid any churn. -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/mach-tegra/Makefile b/arch/arm/mach-tegra/Makefile index 9aa653b..c623752 100644 --- a/arch/arm/mach-tegra/Makefile +++ b/arch/arm/mach-tegra/Makefile @@ -9,7 +9,6 @@ obj-y += flowctrl.o obj-y += powergate.o obj-y += apbio.o obj-$(CONFIG_CPU_IDLE) += cpuidle.o -obj-$(CONFIG_CPU_IDLE) += sleep.o obj-$(CONFIG_ARCH_TEGRA_2x_SOC) += tegra20_clocks.o obj-$(CONFIG_ARCH_TEGRA_2x_SOC) += tegra20_clocks_data.o obj-$(CONFIG_ARCH_TEGRA_2x_SOC) += tegra2_emc.o diff --git a/arch/arm/mach-tegra/sleep.S b/arch/arm/mach-tegra/sleep.S deleted file mode 100644 index ea81554..0000000 --- a/arch/arm/mach-tegra/sleep.S +++ /dev/null @@ -1,33 +0,0 @@ -/* - * arch/arm/mach-tegra/sleep.S - * - * Copyright (c) 2010-2011, NVIDIA Corporation. - * Copyright (c) 2011, Google, Inc. - * - * Author: Colin Cross <ccross@android.com> - * Gary King <gking@nvidia.com> - * - * This program is free software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License as published by - * the Free Software Foundation; either version 2 of the License, or - * (at your option) any later version. - * - * This program is distributed in the hope that it will be useful, but WITHOUT - * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or - * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for - * more details. - * - * You should have received a copy of the GNU General Public License along - * with this program; if not, write to the Free Software Foundation, Inc., - * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. - */ - -#include <linux/linkage.h> - -#include <asm/assembler.h> - -#include <mach/iomap.h> - -#include "flowctrl.h" -#include "sleep.h" -