Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-10-27 Thread Sylwester Nawrocki
Hi Abhilash,

On 24/10/14 16:32, Abhilash Kesavan wrote:
 On Wed, Oct 8, 2014 at 11:18 AM, Abhilash Kesavan
 kesavan.abhil...@gmail.com wrote:
 On Tue, Oct 7, 2014 at 9:46 AM, Kukjin Kim kg...@kernel.org wrote:
 Tomasz Figa wrote:
 On 30.09.2014 17:12, Abhilash Kesavan wrote:
 On Mon, Sep 22, 2014 at 2:22 PM, Tomasz Figa tomasz.f...@gmail.com 
 wrote:

 The clock patches look good to me, but since they are doing quite a lot
 of code moving I'd prefer to take them through clk tree. Based on the
 fact that there are no code dependencies between clock patches and
 remaining ones and Exynos7 is a new material for 3.18, I'm inclined to
 apply them to my tree if nobody minds.

 Will you be picking up the clock changes soon ?

 I'd like to do so. Kukjin, since clock changes are a part of this
 series, might I have your Ack for them to be applied separately?

 Sure, I'm fine so please go ahead with my ack on the changes.

 Acked-by: Kukjin Kim kgene@samsung.com

 Thanks for your asking.

 Can you please pick up the clock changes now that Kukjin has ack'ed it.
 
 Sylwester, you mentioned in another thread that Tomasz might not be
 available for the next few days. Could you please consider applying
 the clock changes in this series to samsung clk-next if they look OK ?
 
 Please take a look at clock changes at
 http://www.spinics.net/lists/arm-kernel/msg370673.html too.

I've picked up patches from the $subject series.  But please see my
comment the the second series you've pointed out with the link above.
If you decide to rearrange changes to exynos7-clk.h header file among
those two series I will drop that 4 patches.  Perhaps you could gather
all pending exynos7 clk patches, fix any issues and send to me in one
series ?
I'd like to avoid modifying indices assigned to specific clocks in
subsequent patches, DT binding should be defined in advance and ideally
be specifying exact indices for each possible clock.

--
Thanks,
Sylwester
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-10-24 Thread Abhilash Kesavan
+Sylwester

On Wed, Oct 8, 2014 at 11:18 AM, Abhilash Kesavan
kesavan.abhil...@gmail.com wrote:
 Hi Tomasz,

 On Tue, Oct 7, 2014 at 9:46 AM, Kukjin Kim kg...@kernel.org wrote:
 Tomasz Figa wrote:

 On 30.09.2014 17:12, Abhilash Kesavan wrote:
  Hi Tomasz,
 
  On Mon, Sep 22, 2014 at 2:22 PM, Tomasz Figa tomasz.f...@gmail.com 
  wrote:
  Hi Abhilash,
 
  On 22.09.2014 06:47, Abhilash Kesavan wrote:
  Changes since v4:
  - Fixed comments from Tomasz Figa:
- Changed the namespace prefix from exynos to samsung
- Defined bindings to take all input clocks
- Sorted the Kconfig entries alphabetically in clock Makefile
- Used consistent 1 tab line breaks across the clock file
- Statically initialized the samsung_cmu_info struct
  - Enabled exynos7 in the arm64 defconfig as per Catalin Marinas' 
  comment.
  - Added Kukjin Kim's ack along with Thomas Abraham's tested and 
  reviewed tags.
 
 
  The clock patches look good to me, but since they are doing quite a lot
  of code moving I'd prefer to take them through clk tree. Based on the
  fact that there are no code dependencies between clock patches and
  remaining ones and Exynos7 is a new material for 3.18, I'm inclined to
  apply them to my tree if nobody minds.
 
  Will you be picking up the clock changes soon ?

 I'd like to do so. Kukjin, since clock changes are a part of this
 series, might I have your Ack for them to be applied separately?

 Sure, I'm fine so please go ahead with my ack on the changes.

 Acked-by: Kukjin Kim kgene@samsung.com

 Thanks for your asking.

 Can you please pick up the clock changes now that Kukjin has ack'ed it.

Sylwester, you mentioned in another thread that Tomasz might not be
available for the next few days. Could you please consider applying
the clock changes in this series to samsung clk-next if they look OK ?

Please take a look at clock changes at
http://www.spinics.net/lists/arm-kernel/msg370673.html too.

Regards,
Abhilash
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-10-07 Thread Abhilash Kesavan
Hi Arnd and Olof,

On Tue, Sep 30, 2014 at 9:20 PM, Catalin Marinas
catalin.mari...@arm.com wrote:
 On Tue, Sep 30, 2014 at 04:15:21PM +0100, Abhilash Kesavan wrote:
 On Tue, Sep 23, 2014 at 2:18 PM, Abhilash Kesavan
 kesavan.abhil...@gmail.com wrote:
  This patchset supports the new Exynos7 Samsung SoC based on Cortex-A57.
  Exynos7 is a System-On-Chip (SoC) that is based on 64-bit ARMv8 RISC
  processor.
 
  The following patches are tested based on linux-next tree (20140919).
  https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/
 
  Following patches are required for this series:
  1- tty/serial: fix config dependencies for samsung serial
 
  https://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg36208.html
  2- dts, kbuild: Implement support for dtb vendor subdirs patchset
 http://comments.gmane.org/gmane.linux.kbuild.devel/12131
 
  Thanks for your comments. I have mentioned Robert's patchset as being
  a dependency for my series here.

 Do you have any comments on the arch and dts changes ?

 The arch changes look fine to me (not much in there, just Kconfig,
 defconfig, Makefile). Regarding the dts, I'd like to see some acks from
 the DT or arm-soc maintainers.

 BTW, the arch/arm64/ patches in your series should be merged via the
 arm-soc tree (Arnd/Olof, cc'ing them now).

Do you have any comments on this series ?

Abhilash

 --
 Catalin
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-10-07 Thread Abhilash Kesavan
Hi Tomasz,

On Tue, Oct 7, 2014 at 9:46 AM, Kukjin Kim kg...@kernel.org wrote:
 Tomasz Figa wrote:

 On 30.09.2014 17:12, Abhilash Kesavan wrote:
  Hi Tomasz,
 
  On Mon, Sep 22, 2014 at 2:22 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
  Hi Abhilash,
 
  On 22.09.2014 06:47, Abhilash Kesavan wrote:
  Changes since v4:
  - Fixed comments from Tomasz Figa:
- Changed the namespace prefix from exynos to samsung
- Defined bindings to take all input clocks
- Sorted the Kconfig entries alphabetically in clock Makefile
- Used consistent 1 tab line breaks across the clock file
- Statically initialized the samsung_cmu_info struct
  - Enabled exynos7 in the arm64 defconfig as per Catalin Marinas' comment.
  - Added Kukjin Kim's ack along with Thomas Abraham's tested and reviewed 
  tags.
 
 
  The clock patches look good to me, but since they are doing quite a lot
  of code moving I'd prefer to take them through clk tree. Based on the
  fact that there are no code dependencies between clock patches and
  remaining ones and Exynos7 is a new material for 3.18, I'm inclined to
  apply them to my tree if nobody minds.
 
  Will you be picking up the clock changes soon ?

 I'd like to do so. Kukjin, since clock changes are a part of this
 series, might I have your Ack for them to be applied separately?

 Sure, I'm fine so please go ahead with my ack on the changes.

 Acked-by: Kukjin Kim kgene@samsung.com

 Thanks for your asking.

Can you please pick up the clock changes now that Kukjin has ack'ed it.

Thanks,
Abhilash

 - Kukjin

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-10-06 Thread Kukjin Kim
Tomasz Figa wrote:
 
 On 30.09.2014 17:12, Abhilash Kesavan wrote:
  Hi Tomasz,
 
  On Mon, Sep 22, 2014 at 2:22 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
  Hi Abhilash,
 
  On 22.09.2014 06:47, Abhilash Kesavan wrote:
  Changes since v4:
  - Fixed comments from Tomasz Figa:
- Changed the namespace prefix from exynos to samsung
- Defined bindings to take all input clocks
- Sorted the Kconfig entries alphabetically in clock Makefile
- Used consistent 1 tab line breaks across the clock file
- Statically initialized the samsung_cmu_info struct
  - Enabled exynos7 in the arm64 defconfig as per Catalin Marinas' comment.
  - Added Kukjin Kim's ack along with Thomas Abraham's tested and reviewed 
  tags.
 
 
  The clock patches look good to me, but since they are doing quite a lot
  of code moving I'd prefer to take them through clk tree. Based on the
  fact that there are no code dependencies between clock patches and
  remaining ones and Exynos7 is a new material for 3.18, I'm inclined to
  apply them to my tree if nobody minds.
 
  Will you be picking up the clock changes soon ?
 
 I'd like to do so. Kukjin, since clock changes are a part of this
 series, might I have your Ack for them to be applied separately?
 
Sure, I'm fine so please go ahead with my ack on the changes.

Acked-by: Kukjin Kim kgene@samsung.com

Thanks for your asking.

- Kukjin

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-09-30 Thread Catalin Marinas
On Tue, Sep 30, 2014 at 04:15:21PM +0100, Abhilash Kesavan wrote:
 On Tue, Sep 23, 2014 at 2:18 PM, Abhilash Kesavan
 kesavan.abhil...@gmail.com wrote:
  This patchset supports the new Exynos7 Samsung SoC based on Cortex-A57.
  Exynos7 is a System-On-Chip (SoC) that is based on 64-bit ARMv8 RISC
  processor.
 
  The following patches are tested based on linux-next tree (20140919).
  https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/
 
  Following patches are required for this series:
  1- tty/serial: fix config dependencies for samsung serial
 
  https://www.mail-archive.com/linux-samsung-soc@vger.kernel.org/msg36208.html
  2- dts, kbuild: Implement support for dtb vendor subdirs patchset
 http://comments.gmane.org/gmane.linux.kbuild.devel/12131
 
  Thanks for your comments. I have mentioned Robert's patchset as being
  a dependency for my series here.
 
 Do you have any comments on the arch and dts changes ?

The arch changes look fine to me (not much in there, just Kconfig,
defconfig, Makefile). Regarding the dts, I'd like to see some acks from
the DT or arm-soc maintainers.

BTW, the arch/arm64/ patches in your series should be merged via the
arm-soc tree (Arnd/Olof, cc'ing them now).

-- 
Catalin
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-09-30 Thread Tomasz Figa
On 30.09.2014 17:12, Abhilash Kesavan wrote:
 Hi Tomasz,
 
 On Mon, Sep 22, 2014 at 2:22 PM, Tomasz Figa tomasz.f...@gmail.com wrote:
 Hi Abhilash,

 On 22.09.2014 06:47, Abhilash Kesavan wrote:
 Changes since v4:
 - Fixed comments from Tomasz Figa:
   - Changed the namespace prefix from exynos to samsung
   - Defined bindings to take all input clocks
   - Sorted the Kconfig entries alphabetically in clock Makefile
   - Used consistent 1 tab line breaks across the clock file
   - Statically initialized the samsung_cmu_info struct
 - Enabled exynos7 in the arm64 defconfig as per Catalin Marinas' comment.
 - Added Kukjin Kim's ack along with Thomas Abraham's tested and reviewed 
 tags.


 The clock patches look good to me, but since they are doing quite a lot
 of code moving I'd prefer to take them through clk tree. Based on the
 fact that there are no code dependencies between clock patches and
 remaining ones and Exynos7 is a new material for 3.18, I'm inclined to
 apply them to my tree if nobody minds.
 
 Will you be picking up the clock changes soon ?

I'd like to do so. Kukjin, since clock changes are a part of this
series, might I have your Ack for them to be applied separately?

Best regards,
Tomasz
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


RE: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-09-23 Thread Chanho Park
Hi,

 -Original Message-
 From: linux-arm-kernel [mailto:linux-arm-kernel-
 boun...@lists.infradead.org] On Behalf Of Abhilash Kesavan
 Sent: Monday, September 22, 2014 1:47 PM
 To: linux-samsung-soc@vger.kernel.org; linux-arm-
 ker...@lists.infradead.org; devicet...@vger.kernel.org;
 catalin.mari...@arm.com
 Cc: naveenkrishna...@gmail.com; kesavan.abhil...@gmail.com;
 tomasz.f...@gmail.com
 Subject: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7
 SoC
 
 Changes since v4:
 - Fixed comments from Tomasz Figa:
   - Changed the namespace prefix from exynos to samsung
   - Defined bindings to take all input clocks
   - Sorted the Kconfig entries alphabetically in clock Makefile
   - Used consistent 1 tab line breaks across the clock file
   - Statically initialized the samsung_cmu_info struct
 - Enabled exynos7 in the arm64 defconfig as per Catalin Marinas' comment.
 - Added Kukjin Kim's ack along with Thomas Abraham's tested and reviewed
 tags.
 
 Changes since v3:
 - Removed aliases for serial controllers from dtsi file and moved it
   into board specific dts file as suggested by Arnd.
 - Based this series on Robert Richter's patches for adding vendor
   device tree sub-directories for arm64.
   http://comments.gmane.org/gmane.linux.kbuild.devel/12131
 
 This patchset supports the new Exynos7 Samsung SoC based on Cortex-A57.
 Exynos7 is a System-On-Chip (SoC) that is based on 64-bit ARMv8 RISC
 processor.
 
 The following patches are tested based on linux-next tree (20140919).
 https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/
 
 Following patches are required for this series:
 1- tty/serial: fix config dependencies for samsung serial
https://www.mail-archive.com/linux-samsung-soc at
 vger.kernel.org/msg36208.html
 2- dts, kbuild: Implement support for dtb vendor subdirs patchset
http://comments.gmane.org/gmane.linux.kbuild.devel/12131

Maybe you make this patch on top of the Robert's patch.
As I know, Robert's patch is not yet merged in mainline.
You should specify this dependency in mail thread.

[1]: https://lkml.org/lkml/2014/9/5/64

Best Regards,
Chanho Park

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-09-23 Thread Abhilash Kesavan
Hi Chanho,

On Tue, Sep 23, 2014 at 1:20 PM, Chanho Park chanho61.p...@samsung.com wrote:
 Hi,

 -Original Message-
 From: linux-arm-kernel [mailto:linux-arm-kernel-
 boun...@lists.infradead.org] On Behalf Of Abhilash Kesavan
 Sent: Monday, September 22, 2014 1:47 PM
 To: linux-samsung-soc@vger.kernel.org; linux-arm-
 ker...@lists.infradead.org; devicet...@vger.kernel.org;
 catalin.mari...@arm.com
 Cc: naveenkrishna...@gmail.com; kesavan.abhil...@gmail.com;
 tomasz.f...@gmail.com
 Subject: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7
 SoC

 Changes since v4:
 - Fixed comments from Tomasz Figa:
   - Changed the namespace prefix from exynos to samsung
   - Defined bindings to take all input clocks
   - Sorted the Kconfig entries alphabetically in clock Makefile
   - Used consistent 1 tab line breaks across the clock file
   - Statically initialized the samsung_cmu_info struct
 - Enabled exynos7 in the arm64 defconfig as per Catalin Marinas' comment.
 - Added Kukjin Kim's ack along with Thomas Abraham's tested and reviewed
 tags.

 Changes since v3:
 - Removed aliases for serial controllers from dtsi file and moved it
   into board specific dts file as suggested by Arnd.
 - Based this series on Robert Richter's patches for adding vendor
   device tree sub-directories for arm64.
   http://comments.gmane.org/gmane.linux.kbuild.devel/12131

 This patchset supports the new Exynos7 Samsung SoC based on Cortex-A57.
 Exynos7 is a System-On-Chip (SoC) that is based on 64-bit ARMv8 RISC
 processor.

 The following patches are tested based on linux-next tree (20140919).
 https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/

 Following patches are required for this series:
 1- tty/serial: fix config dependencies for samsung serial
https://www.mail-archive.com/linux-samsung-soc at
 vger.kernel.org/msg36208.html
 2- dts, kbuild: Implement support for dtb vendor subdirs patchset
http://comments.gmane.org/gmane.linux.kbuild.devel/12131

Thanks for your comments. I have mentioned Robert's patchset as being
a dependency for my series here.

Regards,
Abhilash


 Maybe you make this patch on top of the Robert's patch.
 As I know, Robert's patch is not yet merged in mainline.
 You should specify this dependency in mail thread.

 [1]: https://lkml.org/lkml/2014/9/5/64

 Best Regards,
 Chanho Park

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-09-22 Thread Tomasz Figa
Hi Abhilash,

On 22.09.2014 06:47, Abhilash Kesavan wrote:
 Changes since v4:
 - Fixed comments from Tomasz Figa:
   - Changed the namespace prefix from exynos to samsung
   - Defined bindings to take all input clocks
   - Sorted the Kconfig entries alphabetically in clock Makefile
   - Used consistent 1 tab line breaks across the clock file
   - Statically initialized the samsung_cmu_info struct
 - Enabled exynos7 in the arm64 defconfig as per Catalin Marinas' comment.
 - Added Kukjin Kim's ack along with Thomas Abraham's tested and reviewed tags.
 

The clock patches look good to me, but since they are doing quite a lot
of code moving I'd prefer to take them through clk tree. Based on the
fact that there are no code dependencies between clock patches and
remaining ones and Exynos7 is a new material for 3.18, I'm inclined to
apply them to my tree if nobody minds.

Best regards,
Tomasz
--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v5 0/8] arch: arm64: Enable support for Samsung Exynos7 SoC

2014-09-21 Thread Abhilash Kesavan
Changes since v4:
- Fixed comments from Tomasz Figa:
- Changed the namespace prefix from exynos to samsung
- Defined bindings to take all input clocks
- Sorted the Kconfig entries alphabetically in clock Makefile
- Used consistent 1 tab line breaks across the clock file
- Statically initialized the samsung_cmu_info struct
- Enabled exynos7 in the arm64 defconfig as per Catalin Marinas' comment.
- Added Kukjin Kim's ack along with Thomas Abraham's tested and reviewed tags.

Changes since v3:
- Removed aliases for serial controllers from dtsi file and moved it
  into board specific dts file as suggested by Arnd.
- Based this series on Robert Richter's patches for adding vendor
  device tree sub-directories for arm64.
  http://comments.gmane.org/gmane.linux.kbuild.devel/12131

This patchset supports the new Exynos7 Samsung SoC based on Cortex-A57.
Exynos7 is a System-On-Chip (SoC) that is based on 64-bit ARMv8 RISC
processor.

The following patches are tested based on linux-next tree (20140919).
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/

Following patches are required for this series:
1- tty/serial: fix config dependencies for samsung serial
   https://www.mail-archive.com/linux-samsung-soc at 
vger.kernel.org/msg36208.html
2- dts, kbuild: Implement support for dtb vendor subdirs patchset 
   http://comments.gmane.org/gmane.linux.kbuild.devel/12131

Alim Akhtar (2):
  arm64: exynos7: Enable ARMv8 based Exynos7 (SoC) support
  arm64: Enable Exynos7 SOC in the defconfig

Naveen Krishna Ch (5):
  clk: samsung: add support for 145xx and 1460x PLLs
  clk: samsung: Factor out the common code to clk.c
  clk: samsung: Add fixed_factor_clocks field to struct exynos_cmu_info
  clk: samsung: add initial clock support for Exynos7 SoC
  arm64: dts: Add initial device tree support for EXYNOS7

Pankaj Dubey (1):
  arm64: dts: add dt-bindings/ symlink

 .../devicetree/bindings/clock/exynos7-clock.txt|  67 
 arch/arm64/Kconfig |  12 +
 arch/arm64/boot/dts/Makefile   |   1 +
 arch/arm64/boot/dts/exynos/Makefile|   5 +
 arch/arm64/boot/dts/exynos/exynos7-espresso.dts|  39 ++
 arch/arm64/boot/dts/exynos/exynos7.dtsi| 183 +
 arch/arm64/boot/dts/include/dt-bindings|   1 +
 arch/arm64/configs/defconfig   |   4 +
 drivers/clk/samsung/Makefile   |   1 +
 drivers/clk/samsung/clk-exynos5260.c   | 185 ++---
 drivers/clk/samsung/clk-exynos7.c  | 425 +
 drivers/clk/samsung/clk-pll.c  |  25 +-
 drivers/clk/samsung/clk-pll.h  |   4 +
 drivers/clk/samsung/clk.c  |  98 +
 drivers/clk/samsung/clk.h  |  37 ++
 include/dt-bindings/clock/exynos7-clk.h|  49 +++
 16 files changed, 972 insertions(+), 164 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/clock/exynos7-clock.txt
 create mode 100644 arch/arm64/boot/dts/exynos/Makefile
 create mode 100644 arch/arm64/boot/dts/exynos/exynos7-espresso.dts
 create mode 100644 arch/arm64/boot/dts/exynos/exynos7.dtsi
 create mode 12 arch/arm64/boot/dts/include/dt-bindings
 create mode 100644 drivers/clk/samsung/clk-exynos7.c
 create mode 100644 include/dt-bindings/clock/exynos7-clk.h

-- 
2.1.0

--
To unsubscribe from this list: send the line unsubscribe linux-samsung-soc in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html