* [PATCH 03/11] ARM: EXYNOS: local regs-pmu.h header file
@ 2013-12-17 11:52 kgene at kernel.org
2013-12-28 23:09 ` Olof Johansson
0 siblings, 1 reply; 3+ messages in thread
From: kgene at kernel.org @ 2013-12-17 11:52 UTC (permalink / raw)
To: linux-arm-kernel
From: Kukjin Kim <kgene.kim@samsung.com>
This moves regs-pmu.h file into mach-exynos directory.
Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>
---
arch/arm/mach-exynos/common.c | 3 ++-
arch/arm/mach-exynos/cpuidle.c | 2 +-
arch/arm/mach-exynos/hotplug.c | 2 +-
arch/arm/mach-exynos/mach-exynos5-dt.c | 2 +-
arch/arm/mach-exynos/platsmp.c | 2 +-
arch/arm/mach-exynos/pm.c | 2 +-
arch/arm/mach-exynos/pm_domains.c | 3 ++-
arch/arm/mach-exynos/pmu.c | 2 +-
arch/arm/mach-exynos/{include/mach => }/regs-pmu.h | 0
9 files changed, 10 insertions(+), 8 deletions(-)
rename arch/arm/mach-exynos/{include/mach => }/regs-pmu.h (100%)
diff --git a/arch/arm/mach-exynos/common.c b/arch/arm/mach-exynos/common.c
index 61d2906..1fc0424 100644
--- a/arch/arm/mach-exynos/common.c
+++ b/arch/arm/mach-exynos/common.c
@@ -38,13 +38,14 @@
#include <asm/cacheflush.h>
#include <mach/regs-irq.h>
-#include <mach/regs-pmu.h>
#include <plat/cpu.h>
#include <plat/pm.h>
#include <plat/regs-serial.h>
#include "common.h"
+#include "regs-pmu.h"
+
#define L2_AUX_VAL 0x7C470001
#define L2_AUX_MASK 0xC200ffff
diff --git a/arch/arm/mach-exynos/cpuidle.c b/arch/arm/mach-exynos/cpuidle.c
index ddbfe87..6ba13ea 100644
--- a/arch/arm/mach-exynos/cpuidle.c
+++ b/arch/arm/mach-exynos/cpuidle.c
@@ -23,12 +23,12 @@
#include <asm/unified.h>
#include <asm/cpuidle.h>
#include <mach/regs-clock.h>
-#include <mach/regs-pmu.h>
#include <plat/cpu.h>
#include <plat/pm.h>
#include "common.h"
+#include "regs-pmu.h"
#define REG_DIRECTGO_ADDR (samsung_rev() == EXYNOS4210_REV_1_1 ? \
S5P_INFORM7 : (samsung_rev() == EXYNOS4210_REV_1_0 ?
\
diff --git a/arch/arm/mach-exynos/hotplug.c b/arch/arm/mach-exynos/hotplug.c
index af90cfa..5eead53 100644
--- a/arch/arm/mach-exynos/hotplug.c
+++ b/arch/arm/mach-exynos/hotplug.c
@@ -19,10 +19,10 @@
#include <asm/cp15.h>
#include <asm/smp_plat.h>
-#include <mach/regs-pmu.h>
#include <plat/cpu.h>
#include "common.h"
+#include "regs-pmu.h"
static inline void cpu_enter_lowpower_a9(void)
{
diff --git a/arch/arm/mach-exynos/mach-exynos5-dt.c
b/arch/arm/mach-exynos/mach-exynos5-dt.c
index 1fe075a..65a4646 100644
--- a/arch/arm/mach-exynos/mach-exynos5-dt.c
+++ b/arch/arm/mach-exynos/mach-exynos5-dt.c
@@ -14,10 +14,10 @@
#include <linux/io.h>
#include <asm/mach/arch.h>
-#include <mach/regs-pmu.h>
#include <plat/mfc.h>
#include "common.h"
+#include "regs-pmu.h"
static void __init exynos5_dt_machine_init(void)
{
diff --git a/arch/arm/mach-exynos/platsmp.c b/arch/arm/mach-exynos/platsmp.c
index 58b43e6..b5b34f8 100644
--- a/arch/arm/mach-exynos/platsmp.c
+++ b/arch/arm/mach-exynos/platsmp.c
@@ -28,11 +28,11 @@
#include <mach/hardware.h>
#include <mach/regs-clock.h>
-#include <mach/regs-pmu.h>
#include <plat/cpu.h>
#include "common.h"
+#include "regs-pmu.h"
extern void exynos4_secondary_startup(void);
diff --git a/arch/arm/mach-exynos/pm.c b/arch/arm/mach-exynos/pm.c
index c679db5..717c6a6 100644
--- a/arch/arm/mach-exynos/pm.c
+++ b/arch/arm/mach-exynos/pm.c
@@ -31,10 +31,10 @@
#include <mach/regs-irq.h>
#include <mach/regs-clock.h>
-#include <mach/regs-pmu.h>
#include <mach/pm-core.h>
#include "common.h"
+#include "regs-pmu.h"
static struct sleep_save exynos4_set_clksrc[] = {
{ .reg = EXYNOS4_CLKSRC_MASK_TOP , .val = 0x00000001,
},
diff --git a/arch/arm/mach-exynos/pm_domains.c
b/arch/arm/mach-exynos/pm_domains.c
index 1703593..ff50836 100644
--- a/arch/arm/mach-exynos/pm_domains.c
+++ b/arch/arm/mach-exynos/pm_domains.c
@@ -22,9 +22,10 @@
#include <linux/of_platform.h>
#include <linux/sched.h>
-#include <mach/regs-pmu.h>
#include <plat/devs.h>
+#include "regs-pmu.h"
+
/*
* Exynos specific wrapper around the generic power domain
*/
diff --git a/arch/arm/mach-exynos/pmu.c b/arch/arm/mach-exynos/pmu.c
index 9062b58..0116d5b 100644
--- a/arch/arm/mach-exynos/pmu.c
+++ b/arch/arm/mach-exynos/pmu.c
@@ -14,9 +14,9 @@
#include <linux/bug.h>
#include <mach/regs-clock.h>
-#include <mach/regs-pmu.h>
#include "common.h"
+#include "regs-pmu.h"
static struct exynos_pmu_conf *exynos_pmu_config;
diff --git a/arch/arm/mach-exynos/include/mach/regs-pmu.h
b/arch/arm/mach-exynos/regs-pmu.h
similarity index 100%
rename from arch/arm/mach-exynos/include/mach/regs-pmu.h
rename to arch/arm/mach-exynos/regs-pmu.h
--
1.7.10.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH 03/11] ARM: EXYNOS: local regs-pmu.h header file
2013-12-17 11:52 [PATCH 03/11] ARM: EXYNOS: local regs-pmu.h header file kgene at kernel.org
@ 2013-12-28 23:09 ` Olof Johansson
2014-01-04 2:20 ` Kukjin Kim
0 siblings, 1 reply; 3+ messages in thread
From: Olof Johansson @ 2013-12-28 23:09 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On Tue, Dec 17, 2013 at 3:52 AM, <kgene@kernel.org> wrote:
> From: Kukjin Kim <kgene.kim@samsung.com>
>
> This moves regs-pmu.h file into mach-exynos directory.
>
> Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>
> ---
> arch/arm/mach-exynos/common.c | 3 ++-
> arch/arm/mach-exynos/cpuidle.c | 2 +-
> arch/arm/mach-exynos/hotplug.c | 2 +-
> arch/arm/mach-exynos/mach-exynos5-dt.c | 2 +-
> arch/arm/mach-exynos/platsmp.c | 2 +-
> arch/arm/mach-exynos/pm.c | 2 +-
> arch/arm/mach-exynos/pm_domains.c | 3 ++-
> arch/arm/mach-exynos/pmu.c | 2 +-
> arch/arm/mach-exynos/{include/mach => }/regs-pmu.h | 0
> 9 files changed, 10 insertions(+), 8 deletions(-)
> rename arch/arm/mach-exynos/{include/mach => }/regs-pmu.h (100%)
This looks OK, but when you committed, you did the move of the include
file in another commit (e44de221d3ddb99387b04cfd49483bcd05daa6fb).
That means that between that commit and this patch, there's build
breakage. Please try to avoid that in the future since if something
needs to be bisected near here it can cause problems.
-Olof
^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH 03/11] ARM: EXYNOS: local regs-pmu.h header file
2013-12-28 23:09 ` Olof Johansson
@ 2014-01-04 2:20 ` Kukjin Kim
0 siblings, 0 replies; 3+ messages in thread
From: Kukjin Kim @ 2014-01-04 2:20 UTC (permalink / raw)
To: linux-arm-kernel
Olof Johansson wrote:
>
> Hi,
>
Hi,
>
> On Tue, Dec 17, 2013 at 3:52 AM, <kgene@kernel.org> wrote:
> > From: Kukjin Kim <kgene.kim@samsung.com>
> >
> > This moves regs-pmu.h file into mach-exynos directory.
> >
> > Signed-off-by: Kukjin Kim <kgene.kim@samsung.com>
> > ---
> > arch/arm/mach-exynos/common.c | 3 ++-
> > arch/arm/mach-exynos/cpuidle.c | 2 +-
> > arch/arm/mach-exynos/hotplug.c | 2 +-
> > arch/arm/mach-exynos/mach-exynos5-dt.c | 2 +-
> > arch/arm/mach-exynos/platsmp.c | 2 +-
> > arch/arm/mach-exynos/pm.c | 2 +-
> > arch/arm/mach-exynos/pm_domains.c | 3 ++-
> > arch/arm/mach-exynos/pmu.c | 2 +-
> > arch/arm/mach-exynos/{include/mach => }/regs-pmu.h | 0
> > 9 files changed, 10 insertions(+), 8 deletions(-)
> > rename arch/arm/mach-exynos/{include/mach => }/regs-pmu.h (100%)
>
> This looks OK, but when you committed, you did the move of the include
> file in another commit (e44de221d3ddb99387b04cfd49483bcd05daa6fb).
> That means that between that commit and this patch, there's build
> breakage. Please try to avoid that in the future since if something
> needs to be bisected near here it can cause problems.
>
Yeah, you're right. Sorry and I'll try to avoid that per your suggestion.
Thanks,
Kukjin
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-01-04 2:20 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-17 11:52 [PATCH 03/11] ARM: EXYNOS: local regs-pmu.h header file kgene at kernel.org
2013-12-28 23:09 ` Olof Johansson
2014-01-04 2:20 ` Kukjin Kim
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).