* [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol
@ 2016-02-16 6:20 Krzysztof Kozlowski
2016-02-16 6:20 ` [PATCH v5 1/3] clk: samsung: Enable COMPILE_TEST for Samsung clocks Krzysztof Kozlowski
` (3 more replies)
0 siblings, 4 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2016-02-16 6:20 UTC (permalink / raw)
To: Kukjin Kim, Krzysztof Kozlowski, Catalin Marinas, Will Deacon,
Michael Turquette, Stephen Boyd, Sylwester Nawrocki, Tomasz Figa,
linux-arm-kernel, linux-samsung-soc, linux-kernel, devicetree,
linux-clk
Cc: Arnd Bergmann, Olof Johansson, Kevin Hilman, Alim Akhtar,
Pankaj Dubey, Chanwoo Choi
Hi,
Patchset attempts to remove the ARCH_EXYNOS7 symbol.
I rebased this on top of James':
- clk: Move vendor's Kconfig into CCF menu section
http://www.spinics.net/lists/devicetree/msg111953.html
The third patch depends on previous ones so one way of merging this
would be to prepare a tag from clk tree with first two patches
and the dependency. The last third patch would go through arm-soc.
Changes since v4:
=================
1. Rebased on "clk: Move vendor's Kconfig into CCF menu section"
existing in clock tree.
2. Re-work patch 2/3 (see specific changelog).
Changes since v3:
=================
1. Rebase on next-20160127 (s3c64xx became multiplatform).
Changes since v2:
=================
1. Follow Arnd's advice about not enabling silently the newly added
EXYNOS_ARM64_COMMON_CLK, when COMPILE_TEST is enabled. This was
also a trigger for some changes in placement of Samsung clocks
Kconfig entries...
2. ... which lead to introducing patch 1.
Rationale:
==========
We don't need ARCH_EXYNOS7 symbol because all ARMv8 platforms should
fall under generic ARCH_EXYNOS. In the same time there is no sense
in building clocks for these ARMv8 SoCs on ARMv7 builds.
Best regards,
Krzysztof
Krzysztof Kozlowski (3):
clk: samsung: Enable COMPILE_TEST for Samsung clocks
clk: samsung: Don't build ARMv8 clock drivers on ARMv7
arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS
arch/arm64/Kconfig.platforms | 11 ++---------
arch/arm64/boot/dts/exynos/Makefile | 2 +-
arch/arm64/configs/defconfig | 2 +-
drivers/clk/samsung/Kconfig | 18 +++++++++++++-----
drivers/clk/samsung/Makefile | 4 ++--
5 files changed, 19 insertions(+), 18 deletions(-)
--
2.5.0
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v5 1/3] clk: samsung: Enable COMPILE_TEST for Samsung clocks
2016-02-16 6:20 [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol Krzysztof Kozlowski
@ 2016-02-16 6:20 ` Krzysztof Kozlowski
2016-02-16 6:20 ` [PATCH v5 2/3] clk: samsung: Don't build ARMv8 clock drivers on ARMv7 Krzysztof Kozlowski
` (2 subsequent siblings)
3 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2016-02-16 6:20 UTC (permalink / raw)
To: Kukjin Kim, Krzysztof Kozlowski, Catalin Marinas, Will Deacon,
Michael Turquette, Stephen Boyd, Sylwester Nawrocki, Tomasz Figa,
linux-arm-kernel, linux-samsung-soc, linux-kernel, devicetree,
linux-clk
Cc: Arnd Bergmann, Olof Johansson, Kevin Hilman, Alim Akhtar,
Pankaj Dubey, Chanwoo Choi
Enable the COMPILE_TEST to get build coverage of some of Samsung clock
controller drivers. Still some of them will be built only if
appropriate SoC is chosen (like SOC_EXYNOS4415 or ARCH_S3C64XX).
Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
Tested-by: Javier Martinez Canillas <javier@osg.samsung.com>
Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
---
Changes since v4:
1. Rebase on current clk tree which already contains patch moving
Samsung clock drivers into the COMMON_CLK section.
(b9e65ebc654, "clk: Move vendor's Kconfig into CCF menu section")
Changes since v3:
1. Rebase on next-20160127 (s3c64xx became multiplatform)
Changes since v2:
1. New patch.
---
drivers/clk/samsung/Kconfig | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/clk/samsung/Kconfig b/drivers/clk/samsung/Kconfig
index b3fe5cb01afe..dd6093a33f84 100644
--- a/drivers/clk/samsung/Kconfig
+++ b/drivers/clk/samsung/Kconfig
@@ -1,8 +1,8 @@
config COMMON_CLK_SAMSUNG
- bool
+ bool "Samsung Exynos clock controller support" if COMPILE_TEST
config S3C2410_COMMON_CLK
- bool
+ bool "Samsung S3C2410 clock controller support" if COMPILE_TEST
select COMMON_CLK_SAMSUNG
help
Build the s3c2410 clock driver based on the common clock framework.
@@ -16,10 +16,10 @@ config S3C2410_COMMON_DCLK
framework.
config S3C2412_COMMON_CLK
- bool
+ bool "Samsung S3C2412 clock controller support" if COMPILE_TEST
select COMMON_CLK_SAMSUNG
config S3C2443_COMMON_CLK
- bool
+ bool "Samsung S3C2443 clock controller support" if COMPILE_TEST
select COMMON_CLK_SAMSUNG
--
2.5.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v5 2/3] clk: samsung: Don't build ARMv8 clock drivers on ARMv7
2016-02-16 6:20 [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol Krzysztof Kozlowski
2016-02-16 6:20 ` [PATCH v5 1/3] clk: samsung: Enable COMPILE_TEST for Samsung clocks Krzysztof Kozlowski
@ 2016-02-16 6:20 ` Krzysztof Kozlowski
2016-02-17 10:03 ` Sylwester Nawrocki
2016-02-16 6:20 ` [PATCH v5 3/3] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS Krzysztof Kozlowski
2016-02-16 22:31 ` [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol Michael Turquette
3 siblings, 1 reply; 8+ messages in thread
From: Krzysztof Kozlowski @ 2016-02-16 6:20 UTC (permalink / raw)
To: Kukjin Kim, Krzysztof Kozlowski, Catalin Marinas, Will Deacon,
Michael Turquette, Stephen Boyd, Sylwester Nawrocki, Tomasz Figa,
linux-arm-kernel, linux-samsung-soc, linux-kernel, devicetree,
linux-clk
Cc: Arnd Bergmann, Olof Johansson, Kevin Hilman, Alim Akhtar,
Pankaj Dubey, Chanwoo Choi
Currently the Exynos5433 (ARMv8 SoC) clock driver depends on ARCH_EXYNOS
so it is built also on ARMv7. This does not bring any kind of benefit.
There won't be a single kernel image for ARMv7 and ARMv8 SoCs (like
multi_v7 for ARMv7).
Instead build clock drivers only for respective SoC's architecture.
Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
---
Changes since v4:
1. Move ARM64 clock entry just before COMMON_CLK_SAMSUNG to make it more
consistent with selecting behavior. Newer platforms just select
COMMON_CLK_SAMSUNG. Only S3C24XX select specific clock symbols.
2. Re-work behaviour on COMPILE_TEST=y to fix ignored compilation
when COMPILE_TEST=y but COMMON_CLK_SAMSUNG=n.
The COMMON_CLK_SAMSUNG=y is necessary always to compile ARM64 clocks.
3. I received reviews and tests (Chanwoo, Javier, Andi, Stephen - thank
you guys!)... but the patch is now different so I decided to drop
them. I am kindly asking for review/testing (again...).
Changes since v3:
1. Rebase on next-20160127.
Changes since v2:
1. Follow Arnd's advice making EXYNOS_ARM64_COMMON_CLK user-visible when
COMPILE_TEST is enabled.
2. Dropped accumulated tags because the logic is now different.
Changes since v1:
1. Use one Kconfig symbol for clocks of all ARM64 SoCs (after
discussion): EXYNOS_ARM64_COMMON_CLK.
2. Minor cleanups after Tomasz's comments.
3. Added tags: Chanwoo's, Sylwester's, Alim's and Tomasz's (the tags
were given for v1 but assuming that change from 1. is not major
enough to require re-tagging).
---
drivers/clk/samsung/Kconfig | 10 +++++++++-
drivers/clk/samsung/Makefile | 4 ++--
2 files changed, 11 insertions(+), 3 deletions(-)
diff --git a/drivers/clk/samsung/Kconfig b/drivers/clk/samsung/Kconfig
index dd6093a33f84..20c5fe92ab4a 100644
--- a/drivers/clk/samsung/Kconfig
+++ b/drivers/clk/samsung/Kconfig
@@ -1,6 +1,15 @@
+# Recent Exynos platforms should just select COMMON_CLK_SAMSUNG:
config COMMON_CLK_SAMSUNG
bool "Samsung Exynos clock controller support" if COMPILE_TEST
+ # Clocks on ARM64 SoCs (e.g. Exynos5433, Exynos7) are chosen by
+ # EXYNOS_ARM64_COMMON_CLK to avoid building them on ARMv7:
+ select EXYNOS_ARM64_COMMON_CLK if ARM64 && ARCH_EXYNOS
+config EXYNOS_ARM64_COMMON_CLK
+ bool "Samsung Exynos ARMv8-family clock controller support" if COMPILE_TEST
+ depends on COMMON_CLK_SAMSUNG
+
+# For S3C24XX platforms, select following symbols:
config S3C2410_COMMON_CLK
bool "Samsung S3C2410 clock controller support" if COMPILE_TEST
select COMMON_CLK_SAMSUNG
@@ -22,4 +31,3 @@ config S3C2412_COMMON_CLK
config S3C2443_COMMON_CLK
bool "Samsung S3C2443 clock controller support" if COMPILE_TEST
select COMMON_CLK_SAMSUNG
-
diff --git a/drivers/clk/samsung/Makefile b/drivers/clk/samsung/Makefile
index 5f6833ea355d..fc367d4b2902 100644
--- a/drivers/clk/samsung/Makefile
+++ b/drivers/clk/samsung/Makefile
@@ -10,11 +10,11 @@ obj-$(CONFIG_SOC_EXYNOS5250) += clk-exynos5250.o
obj-$(CONFIG_SOC_EXYNOS5260) += clk-exynos5260.o
obj-$(CONFIG_SOC_EXYNOS5410) += clk-exynos5410.o
obj-$(CONFIG_SOC_EXYNOS5420) += clk-exynos5420.o
-obj-$(CONFIG_ARCH_EXYNOS) += clk-exynos5433.o
+obj-$(CONFIG_EXYNOS_ARM64_COMMON_CLK) += clk-exynos5433.o
obj-$(CONFIG_SOC_EXYNOS5440) += clk-exynos5440.o
obj-$(CONFIG_ARCH_EXYNOS) += clk-exynos-audss.o
obj-$(CONFIG_ARCH_EXYNOS) += clk-exynos-clkout.o
-obj-$(CONFIG_ARCH_EXYNOS7) += clk-exynos7.o
+obj-$(CONFIG_EXYNOS_ARM64_COMMON_CLK) += clk-exynos7.o
obj-$(CONFIG_S3C2410_COMMON_CLK)+= clk-s3c2410.o
obj-$(CONFIG_S3C2410_COMMON_DCLK)+= clk-s3c2410-dclk.o
obj-$(CONFIG_S3C2412_COMMON_CLK)+= clk-s3c2412.o
--
2.5.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH v5 3/3] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS
2016-02-16 6:20 [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol Krzysztof Kozlowski
2016-02-16 6:20 ` [PATCH v5 1/3] clk: samsung: Enable COMPILE_TEST for Samsung clocks Krzysztof Kozlowski
2016-02-16 6:20 ` [PATCH v5 2/3] clk: samsung: Don't build ARMv8 clock drivers on ARMv7 Krzysztof Kozlowski
@ 2016-02-16 6:20 ` Krzysztof Kozlowski
2016-02-16 22:31 ` [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol Michael Turquette
3 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2016-02-16 6:20 UTC (permalink / raw)
To: Kukjin Kim, Krzysztof Kozlowski, Catalin Marinas, Will Deacon,
Michael Turquette, Stephen Boyd, Sylwester Nawrocki, Tomasz Figa,
linux-arm-kernel, linux-samsung-soc, linux-kernel, devicetree,
linux-clk
Cc: Arnd Bergmann, Olof Johansson, Kevin Hilman, Alim Akhtar,
Pankaj Dubey, Chanwoo Choi
The ARMv8 Exynos family SoCs in Linux kernel are currently:
- Exynos5433 (controlled by ARCH_EXYNOS),
- Exynos7 (controlled by ARCH_EXYNOS7).
It duplicates Kconfig symbols unnecessarily, so consolidate them into
one ARCH_EXYNOS. Future SoCs could fall also under the ARCH_EXYNOS
symbol.
The commit should not bring any visible functional change.
Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Reviewed-by: Pankaj Dubey <pankaj.dubey@samsung.com>
Tested-by: Alim Akhtar <alim.akhtar@samsung.com>
Reviewed-by: Tomasz Figa <tomasz.figa@gmail.com>
Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
Reviewed-by: Andi Shyti <andi.shyti@samsung.com>
---
Changes since v4:
1. Added tags from: Javier, Andi.
Changes since v3:
1. Rebase on next-20160127.
Changes since v2:
1. None
Changes since v1:
1. Minor cleanups after Tomasz's comments.
2. Added tags: Chanwoo's, Pankaj's, Alim's and Tomasz's.
---
arch/arm64/Kconfig.platforms | 11 ++---------
arch/arm64/boot/dts/exynos/Makefile | 2 +-
arch/arm64/configs/defconfig | 2 +-
3 files changed, 4 insertions(+), 11 deletions(-)
diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index 21074f674bde..653662bbafeb 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -14,21 +14,14 @@ config ARCH_BERLIN
This enables support for Marvell Berlin SoC Family
config ARCH_EXYNOS
- bool
- help
- This enables support for Samsung Exynos SoC family
-
-config ARCH_EXYNOS7
- bool "ARMv8 based Samsung Exynos7"
- select ARCH_EXYNOS
+ bool "ARMv8 based Samsung Exynos SoC family"
select COMMON_CLK_SAMSUNG
select HAVE_S3C2410_WATCHDOG if WATCHDOG
select HAVE_S3C_RTC if RTC_CLASS
select PINCTRL
select PINCTRL_EXYNOS
-
help
- This enables support for Samsung Exynos7 SoC family
+ This enables support for ARMv8 based Samsung Exynos SoC family.
config ARCH_LAYERSCAPE
bool "ARMv8 based Freescale Layerscape SoC family"
diff --git a/arch/arm64/boot/dts/exynos/Makefile b/arch/arm64/boot/dts/exynos/Makefile
index 20310e5b6d6f..50c9b9383cfa 100644
--- a/arch/arm64/boot/dts/exynos/Makefile
+++ b/arch/arm64/boot/dts/exynos/Makefile
@@ -1,4 +1,4 @@
-dtb-$(CONFIG_ARCH_EXYNOS7) += exynos7-espresso.dtb
+dtb-$(CONFIG_ARCH_EXYNOS) += exynos7-espresso.dtb
always := $(dtb-y)
subdir-y := $(dts-dirs)
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 18ca9fb9e65f..331a843cf122 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -33,7 +33,7 @@ CONFIG_MODULE_UNLOAD=y
# CONFIG_IOSCHED_DEADLINE is not set
CONFIG_ARCH_BCM_IPROC=y
CONFIG_ARCH_BERLIN=y
-CONFIG_ARCH_EXYNOS7=y
+CONFIG_ARCH_EXYNOS=y
CONFIG_ARCH_LAYERSCAPE=y
CONFIG_ARCH_HISI=y
CONFIG_ARCH_MEDIATEK=y
--
2.5.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol
2016-02-16 6:20 [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol Krzysztof Kozlowski
` (2 preceding siblings ...)
2016-02-16 6:20 ` [PATCH v5 3/3] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS Krzysztof Kozlowski
@ 2016-02-16 22:31 ` Michael Turquette
3 siblings, 0 replies; 8+ messages in thread
From: Michael Turquette @ 2016-02-16 22:31 UTC (permalink / raw)
To: Krzysztof Kozlowski, Kukjin Kim, Krzysztof Kozlowski,
Catalin Marinas, Will Deacon, Stephen Boyd, Sylwester Nawrocki,
Tomasz Figa, linux-arm-kernel, linux-samsung-soc, linux-kernel,
devicetree, linux-clk
Cc: Arnd Bergmann, Olof Johansson, Kevin Hilman, Alim Akhtar,
Pankaj Dubey, Chanwoo Choi
Quoting Krzysztof Kozlowski (2016-02-15 22:20:29)
> Hi,
>
> Patchset attempts to remove the ARCH_EXYNOS7 symbol.
>
> I rebased this on top of James':
> - clk: Move vendor's Kconfig into CCF menu section
> http://www.spinics.net/lists/devicetree/msg111953.html
>
> The third patch depends on previous ones so one way of merging this
> would be to prepare a tag from clk tree with first two patches
> and the dependency. The last third patch would go through arm-soc.
I've done this locally and things look good. I'm just waiting for Acks
to patch #2 and I'll merge patches #1 and #2 and provide a stable tag
named clk-samsung-kconfig, based on v4.5-rc1.
These patches were not based on -rc1, but the fixups were trivial so I'm
fine with them.
Regards,
Mike
>
> Changes since v4:
> =================
> 1. Rebased on "clk: Move vendor's Kconfig into CCF menu section"
> existing in clock tree.
> 2. Re-work patch 2/3 (see specific changelog).
>
> Changes since v3:
> =================
> 1. Rebase on next-20160127 (s3c64xx became multiplatform).
>
> Changes since v2:
> =================
> 1. Follow Arnd's advice about not enabling silently the newly added
> EXYNOS_ARM64_COMMON_CLK, when COMPILE_TEST is enabled. This was
> also a trigger for some changes in placement of Samsung clocks
> Kconfig entries...
> 2. ... which lead to introducing patch 1.
>
>
> Rationale:
> ==========
> We don't need ARCH_EXYNOS7 symbol because all ARMv8 platforms should
> fall under generic ARCH_EXYNOS. In the same time there is no sense
> in building clocks for these ARMv8 SoCs on ARMv7 builds.
>
>
> Best regards,
> Krzysztof
>
> Krzysztof Kozlowski (3):
> clk: samsung: Enable COMPILE_TEST for Samsung clocks
> clk: samsung: Don't build ARMv8 clock drivers on ARMv7
> arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS
>
> arch/arm64/Kconfig.platforms | 11 ++---------
> arch/arm64/boot/dts/exynos/Makefile | 2 +-
> arch/arm64/configs/defconfig | 2 +-
> drivers/clk/samsung/Kconfig | 18 +++++++++++++-----
> drivers/clk/samsung/Makefile | 4 ++--
> 5 files changed, 19 insertions(+), 18 deletions(-)
>
> --
> 2.5.0
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v5 2/3] clk: samsung: Don't build ARMv8 clock drivers on ARMv7
2016-02-16 6:20 ` [PATCH v5 2/3] clk: samsung: Don't build ARMv8 clock drivers on ARMv7 Krzysztof Kozlowski
@ 2016-02-17 10:03 ` Sylwester Nawrocki
2016-02-22 18:26 ` Michael Turquette
0 siblings, 1 reply; 8+ messages in thread
From: Sylwester Nawrocki @ 2016-02-17 10:03 UTC (permalink / raw)
To: Krzysztof Kozlowski, Michael Turquette, linux-clk
Cc: Kukjin Kim, Catalin Marinas, Will Deacon, Stephen Boyd,
Tomasz Figa, linux-arm-kernel, linux-samsung-soc, linux-kernel,
devicetree, Arnd Bergmann, Olof Johansson, Kevin Hilman,
Alim Akhtar, Pankaj Dubey, Chanwoo Choi
On 16/02/16 07:20, Krzysztof Kozlowski wrote:
> Currently the Exynos5433 (ARMv8 SoC) clock driver depends on ARCH_EXYNOS
> so it is built also on ARMv7. This does not bring any kind of benefit.
> There won't be a single kernel image for ARMv7 and ARMv8 SoCs (like
> multi_v7 for ARMv7).
>
> Instead build clock drivers only for respective SoC's architecture.
>
> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v5 2/3] clk: samsung: Don't build ARMv8 clock drivers on ARMv7
2016-02-17 10:03 ` Sylwester Nawrocki
@ 2016-02-22 18:26 ` Michael Turquette
2016-02-23 0:14 ` Krzysztof Kozlowski
0 siblings, 1 reply; 8+ messages in thread
From: Michael Turquette @ 2016-02-22 18:26 UTC (permalink / raw)
To: Sylwester Nawrocki, Krzysztof Kozlowski, linux-clk
Cc: Kukjin Kim, Catalin Marinas, Will Deacon, Stephen Boyd,
Tomasz Figa, linux-arm-kernel, linux-samsung-soc, linux-kernel,
devicetree, Arnd Bergmann, Olof Johansson, Kevin Hilman,
Alim Akhtar, Pankaj Dubey, Chanwoo Choi
Quoting Sylwester Nawrocki (2016-02-17 02:03:36)
> On 16/02/16 07:20, Krzysztof Kozlowski wrote:
> > Currently the Exynos5433 (ARMv8 SoC) clock driver depends on ARCH_EXYNOS
> > so it is built also on ARMv7. This does not bring any kind of benefit.
> > There won't be a single kernel image for ARMv7 and ARMv8 SoCs (like
> > multi_v7 for ARMv7).
> >
> > Instead build clock drivers only for respective SoC's architecture.
> >
> > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>
> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
>
Thanks for the Ack.
I've rebased patches #1 and #2 against James's b9e65ebc654d and pushed
the clk-samsung branch to the clk tree. I have deleted the
clk-samsung-kconfig branch that was blowing up the kbuild robot.
clk-samsung will not be rebased and can be used as a dependency for
arm-soc. It has been merged into clk-next.
Regards,
Mike
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH v5 2/3] clk: samsung: Don't build ARMv8 clock drivers on ARMv7
2016-02-22 18:26 ` Michael Turquette
@ 2016-02-23 0:14 ` Krzysztof Kozlowski
0 siblings, 0 replies; 8+ messages in thread
From: Krzysztof Kozlowski @ 2016-02-23 0:14 UTC (permalink / raw)
To: Michael Turquette, Sylwester Nawrocki, linux-clk
Cc: Kukjin Kim, Catalin Marinas, Will Deacon, Stephen Boyd,
Tomasz Figa, linux-arm-kernel, linux-samsung-soc, linux-kernel,
devicetree, Arnd Bergmann, Olof Johansson, Kevin Hilman,
Alim Akhtar, Pankaj Dubey, Chanwoo Choi
On 23.02.2016 03:26, Michael Turquette wrote:
> Quoting Sylwester Nawrocki (2016-02-17 02:03:36)
>> On 16/02/16 07:20, Krzysztof Kozlowski wrote:
>>> Currently the Exynos5433 (ARMv8 SoC) clock driver depends on ARCH_EXYNOS
>>> so it is built also on ARMv7. This does not bring any kind of benefit.
>>> There won't be a single kernel image for ARMv7 and ARMv8 SoCs (like
>>> multi_v7 for ARMv7).
>>>
>>> Instead build clock drivers only for respective SoC's architecture.
>>>
>>> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>
>> Acked-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
>>
>
> Thanks for the Ack.
>
> I've rebased patches #1 and #2 against James's b9e65ebc654d and pushed
> the clk-samsung branch to the clk tree. I have deleted the
> clk-samsung-kconfig branch that was blowing up the kbuild robot.
> clk-samsung will not be rebased and can be used as a dependency for
> arm-soc. It has been merged into clk-next.
Thanks, pulled!
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2016-02-23 0:14 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-16 6:20 [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol Krzysztof Kozlowski
2016-02-16 6:20 ` [PATCH v5 1/3] clk: samsung: Enable COMPILE_TEST for Samsung clocks Krzysztof Kozlowski
2016-02-16 6:20 ` [PATCH v5 2/3] clk: samsung: Don't build ARMv8 clock drivers on ARMv7 Krzysztof Kozlowski
2016-02-17 10:03 ` Sylwester Nawrocki
2016-02-22 18:26 ` Michael Turquette
2016-02-23 0:14 ` Krzysztof Kozlowski
2016-02-16 6:20 ` [PATCH v5 3/3] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS Krzysztof Kozlowski
2016-02-16 22:31 ` [PATCH v5 0/3] arm64/clk: EXYNOS: Consolidate Exynos7 symbol Michael Turquette
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).