Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-19 Thread Sylwester Nawrocki
On 19/11/15 10:16, Tomasz Figa wrote: > 2015-11-19 13:51 GMT+09:00 Krzysztof Kozlowski : >> > On 19.11.2015 13:18, Tomasz Figa wrote: >>> >> However, I don't think we can disable compilation of particular >>> >> 64-bit SoCs, so maybe there isn't much sense in splitting their >>> >> clock drivers

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-19 Thread Krzysztof Kozlowski
W dniu 19.11.2015 o 18:16, Tomasz Figa pisze: > 2015-11-19 13:51 GMT+09:00 Krzysztof Kozlowski : >> On 19.11.2015 13:18, Tomasz Figa wrote: >>> However, I don't think we can disable compilation of particular 64-bit >>> SoCs, so maybe there isn't much sense in splitting their clock drivers >>> into

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-19 Thread Tomasz Figa
2015-11-19 13:51 GMT+09:00 Krzysztof Kozlowski : > On 19.11.2015 13:18, Tomasz Figa wrote: >> However, I don't think we can disable compilation of particular 64-bit >> SoCs, so maybe there isn't much sense in splitting their clock drivers >> into separate symbols? > > To me it does not really

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-19 Thread Tomasz Figa
2015-11-19 13:51 GMT+09:00 Krzysztof Kozlowski : > On 19.11.2015 13:18, Tomasz Figa wrote: >> However, I don't think we can disable compilation of particular 64-bit >> SoCs, so maybe there isn't much sense in splitting their clock drivers >> into separate symbols? > > To

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-19 Thread Krzysztof Kozlowski
W dniu 19.11.2015 o 18:16, Tomasz Figa pisze: > 2015-11-19 13:51 GMT+09:00 Krzysztof Kozlowski : >> On 19.11.2015 13:18, Tomasz Figa wrote: >>> However, I don't think we can disable compilation of particular 64-bit >>> SoCs, so maybe there isn't much sense in splitting

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-19 Thread Sylwester Nawrocki
On 19/11/15 10:16, Tomasz Figa wrote: > 2015-11-19 13:51 GMT+09:00 Krzysztof Kozlowski : >> > On 19.11.2015 13:18, Tomasz Figa wrote: >>> >> However, I don't think we can disable compilation of particular >>> >> 64-bit SoCs, so maybe there isn't much sense in splitting

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-18 Thread Krzysztof Kozlowski
On 19.11.2015 13:18, Tomasz Figa wrote: > Hi Krzysztof, > > Good idea, just a couple of nits inline. Other than that: > > Acked-by: Tomasz Figa > > 2015-11-16 10:36 GMT+09:00 Krzysztof Kozlowski : >> Currently the Exynos5433 (ARMv8 SoC) clock driver depends on ARCH_EXYNOS >> so it is built

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-18 Thread Tomasz Figa
Hi Krzysztof, Good idea, just a couple of nits inline. Other than that: Acked-by: Tomasz Figa 2015-11-16 10:36 GMT+09:00 Krzysztof Kozlowski : > 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. >

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-18 Thread Tomasz Figa
Hi Krzysztof, Good idea, just a couple of nits inline. Other than that: Acked-by: Tomasz Figa 2015-11-16 10:36 GMT+09:00 Krzysztof Kozlowski : > Currently the Exynos5433 (ARMv8 SoC) clock driver depends on ARCH_EXYNOS > so it is built also on

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-18 Thread Krzysztof Kozlowski
On 19.11.2015 13:18, Tomasz Figa wrote: > Hi Krzysztof, > > Good idea, just a couple of nits inline. Other than that: > > Acked-by: Tomasz Figa > > 2015-11-16 10:36 GMT+09:00 Krzysztof Kozlowski : >> Currently the Exynos5433 (ARMv8 SoC) clock

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-17 Thread Sylwester Nawrocki
On 17/11/15 05:39, Krzysztof Kozlowski wrote: > On 17.11.2015 13:31, pankaj.dubey wrote: >> On Monday 16 November 2015 07:06 AM, 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

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-17 Thread Sylwester Nawrocki
On 17/11/15 05:39, Krzysztof Kozlowski wrote: > On 17.11.2015 13:31, pankaj.dubey wrote: >> On Monday 16 November 2015 07:06 AM, 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

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-16 Thread Krzysztof Kozlowski
On 17.11.2015 13:31, pankaj.dubey wrote: > > > On Monday 16 November 2015 07:06 AM, 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

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-16 Thread pankaj.dubey
On Monday 16 November 2015 07:06 AM, 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

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-16 Thread pankaj.dubey
On Monday 16 November 2015 07:06 AM, 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

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-16 Thread Krzysztof Kozlowski
On 17.11.2015 13:31, pankaj.dubey wrote: > > > On Monday 16 November 2015 07:06 AM, 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

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-15 Thread Chanwoo Choi
Hi, On 2015년 11월 16일 10:36, 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). >

[PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-15 Thread Krzysztof Kozlowski
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

[PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-15 Thread Krzysztof Kozlowski
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

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-15 Thread Chanwoo Choi
Hi, On 2015년 11월 16일 10:36, 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). >