RE: [PATCH v5 00/12] clk: exynos4: migrate to common clock framework

2012-12-30 Thread Kukjin Kim
Olof Johansson wrote:
 
 On Sat, Dec 29, 2012 at 04:33:29PM -0800, Thomas Abraham wrote:
  Changes since v4:
  - Rebased to linux-3.8-rc1.
 
  Changes since v3:
  - Includes changes suggested by Tomasz Figa tomasz.f...@gmail.com
 
  This patch series migrates the Samsung Exynos4 SoC clock code to adopt
 the
  common clock framework. The use of Samsung specific clock structures has
  been removed and all board support code has been updated. imx-style of
  clock registration and lookup has been adopted for device tree based
  exynos4 platforms.
 
 I'd prefer to see if exynos4 and 5 were kept common here, and both
 transitioned
 at the same time. Especially since there are no legacy boards for exynos5,
it
 would mean you could have a very clean transition there. What's the plan
to
 follow up with 5?
 
Yeah, I'm working on EXYNOS5250 common clk stuff and Thomas is working on
EXYNOS5440. So I think, we can move on exynos4 and 5 both common clk at the
same time. Note, EXYNOS5 common clk stuff will be submitted in the beginning
of Jan.

 What are the plans to remove legacy board files on exynos4 at this time
 and switch them to DT-only? You could do it gradually like Stephen Warren
 did on Tegra, with hooks that call out to some of the legacy code, but
 configure the board through device tree and do away with the classic
 machine descriptors, etc.
 
I had a plan to remove non-DT support on EXYNOS4 for v3.10, but if possible,
I will try to do it for v3.9. If any updates, let you know.

Thanks.

Happy New Year!

- 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] ARM: dts: add board dts file for EXYNOS4412 based Origen board

2012-12-30 Thread Kukjin Kim
Thomas Abraham wrote:
 
 Add a minimal board dts file for EXYNOS4412 based Insignal Origen board.
 
 Signed-off-by: Thomas Abraham thomas.abra...@linaro.org
 ---
  arch/arm/boot/dts/Makefile  |1 +
  arch/arm/boot/dts/exynos4412-origen.dts |   45
 +++
  2 files changed, 46 insertions(+), 0 deletions(-)
  create mode 100644 arch/arm/boot/dts/exynos4412-origen.dts
 
 diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
 index e44da40..c84f9f7 100644
 --- a/arch/arm/boot/dts/Makefile
 +++ b/arch/arm/boot/dts/Makefile
 @@ -43,6 +43,7 @@ dtb-$(CONFIG_ARCH_EXYNOS) += exynos4210-
 origen.dtb \
   exynos4210-smdkv310.dtb \
   exynos4210-trats.dtb \
   exynos4412-smdk4412.dtb \
 + exynos4412-origen.dtb \
   exynos5250-smdk5250.dtb \
   exynos5250-snow.dtb \
   exynos5440-ssdk5440.dtb
 diff --git a/arch/arm/boot/dts/exynos4412-origen.dts
 b/arch/arm/boot/dts/exynos4412-origen.dts
 new file mode 100644
 index 000..df880c4
 --- /dev/null
 +++ b/arch/arm/boot/dts/exynos4412-origen.dts
 @@ -0,0 +1,45 @@
 +/*
 + * Insignal's Exynos4412 based Origen board device tree source
 + *
 + * Copyright (c) 2012-2013 Samsung Electronics Co., Ltd.
 + *   http://www.samsung.com
 + *
 + * Device tree source file for Insignal's Origen board which is based on
 + * Samsung's Exynos4412 SoC.
 + *
 + * This program is free software; you can redistribute it and/or modify
 + * it under the terms of the GNU General Public License version 2 as
 + * published by the Free Software Foundation.
 +*/
 +
 +/dts-v1/;
 +/include/ exynos4412.dtsi
 +
 +/ {
 + model = Insignal Origen evaluation board based on Exynos4412;
 + compatible = insignal,origen4412, samsung,exynos4412;
 +
 + memory {
 + reg = 0x4000 0x4000;
 + };
 +
 + chosen {
 + bootargs =console=ttySAC2,115200;
 + };
 +
 + serial@1380 {
 + status = okay;
 + };
 +
 + serial@1381 {
 + status = okay;
 + };
 +
 + serial@1382 {
 + status = okay;
 + };
 +
 + serial@1383 {
 + status = okay;
 + };
 +};
 --
 1.7.5.4

Looks OK to me, applied.

Thanks.

- 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] ARM: dts: add board dts file for EXYNOS4412 based Origen board

2012-12-30 Thread Kukjin Kim
Kukjin Kim wrote:
 
 Thomas Abraham wrote:
 
  Add a minimal board dts file for EXYNOS4412 based Insignal Origen board.
 
  Signed-off-by: Thomas Abraham thomas.abra...@linaro.org
  ---
   arch/arm/boot/dts/Makefile  |1 +
   arch/arm/boot/dts/exynos4412-origen.dts |   45
  +++
   2 files changed, 46 insertions(+), 0 deletions(-)
   create mode 100644 arch/arm/boot/dts/exynos4412-origen.dts
 
  diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
  index e44da40..c84f9f7 100644
  --- a/arch/arm/boot/dts/Makefile
  +++ b/arch/arm/boot/dts/Makefile
  @@ -43,6 +43,7 @@ dtb-$(CONFIG_ARCH_EXYNOS) += exynos4210-
  origen.dtb \
  exynos4210-smdkv310.dtb \
  exynos4210-trats.dtb \
  exynos4412-smdk4412.dtb \
  +   exynos4412-origen.dtb \
  exynos5250-smdk5250.dtb \
  exynos5250-snow.dtb \
  exynos5440-ssdk5440.dtb
  diff --git a/arch/arm/boot/dts/exynos4412-origen.dts
  b/arch/arm/boot/dts/exynos4412-origen.dts
  new file mode 100644
  index 000..df880c4
  --- /dev/null
  +++ b/arch/arm/boot/dts/exynos4412-origen.dts
  @@ -0,0 +1,45 @@
  +/*
  + * Insignal's Exynos4412 based Origen board device tree source
  + *
  + * Copyright (c) 2012-2013 Samsung Electronics Co., Ltd.
  + * http://www.samsung.com
  + *
  + * Device tree source file for Insignal's Origen board which is based
on
  + * Samsung's Exynos4412 SoC.
  + *
  + * This program is free software; you can redistribute it and/or modify
  + * it under the terms of the GNU General Public License version 2 as
  + * published by the Free Software Foundation.
  +*/
  +
  +/dts-v1/;
  +/include/ exynos4412.dtsi
  +
  +/ {
  +   model = Insignal Origen evaluation board based on Exynos4412;
  +   compatible = insignal,origen4412, samsung,exynos4412;
  +
  +   memory {
  +   reg = 0x4000 0x4000;
  +   };
  +
  +   chosen {
  +   bootargs =console=ttySAC2,115200;
  +   };
  +
  +   serial@1380 {
  +   status = okay;
  +   };
  +
  +   serial@1381 {
  +   status = okay;
  +   };
  +
  +   serial@1382 {
  +   status = okay;
  +   };
  +
  +   serial@1383 {
  +   status = okay;
  +   };
  +};
  --
  1.7.5.4
 
 Looks OK to me, applied.
 
Ah, one more.
BTW, I think, need to sort out the memory configuration and bootargs

And compatible name? origen4412 is right? Probably, origen-quad?

Thanks.

- 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


[PATCH RESEND] ARM: dts: Add EHCI device tree node for Exynos4

2012-12-30 Thread Dongjin Kim
This patch adds EHCI device node on device tree for Exynos4 and defines its
default platform data, s5p_usb_phy_init and s5p_usb_phy_exit, so that those
function can be called from the driver.

Signed-off-by: Dongjin Kim tobet...@gmail.com
---
 arch/arm/boot/dts/exynos4.dtsi |7 +++
 arch/arm/mach-exynos/mach-exynos4-dt.c |9 +
 2 files changed, 16 insertions(+)

diff --git a/arch/arm/boot/dts/exynos4.dtsi b/arch/arm/boot/dts/exynos4.dtsi
index 92bca86..df1a9f0 100644
--- a/arch/arm/boot/dts/exynos4.dtsi
+++ b/arch/arm/boot/dts/exynos4.dtsi
@@ -140,6 +140,13 @@
status = disabled;
};
 
+   ehci@1258 {
+   compatible = samsung,exynos-ehci;
+   reg = 0x1258 0x100;
+   interrupts = 0 70 0;
+   status = disabled;
+   };
+
serial@1380 {
compatible = samsung,exynos4210-uart;
reg = 0x1380 0x100;
diff --git a/arch/arm/mach-exynos/mach-exynos4-dt.c 
b/arch/arm/mach-exynos/mach-exynos4-dt.c
index 92757ff..c8a23f0 100644
--- a/arch/arm/mach-exynos/mach-exynos4-dt.c
+++ b/arch/arm/mach-exynos/mach-exynos4-dt.c
@@ -13,6 +13,7 @@
 
 #include linux/of_platform.h
 #include linux/serial_core.h
+#include linux/platform_data/usb-ehci-s5p.h
 
 #include asm/mach/arch.h
 #include asm/hardware/gic.h
@@ -20,9 +21,15 @@
 
 #include plat/cpu.h
 #include plat/regs-serial.h
+#include plat/usb-phy.h
 
 #include common.h
 
+static struct s5p_ehci_platdata s5p_ehci_platdata = {
+   .phy_init = s5p_usb_phy_init,
+   .phy_exit = s5p_usb_phy_exit,
+};
+
 /*
  * The following lookup table is used to override device names when devices
  * are registered from device tree. This is temporarily added to enable
@@ -80,6 +87,8 @@ static const struct of_dev_auxdata exynos4_auxdata_lookup[] 
__initconst = {
OF_DEV_AUXDATA(arm,pl330, EXYNOS4_PA_MDMA1, dma-pl330.2, NULL),
OF_DEV_AUXDATA(samsung,exynos4210-tmu, EXYNOS4_PA_TMU,
exynos-tmu, NULL),
+   OF_DEV_AUXDATA(samsung,exynos-ehci, EXYNOS4_PA_EHCI,
+   s5p-ehci, s5p_ehci_platdata),
{},
 };
 
-- 
1.7.10.4

--
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 0/2] Add missing fixed regulators on Origen board

2012-12-30 Thread Tushar Behera
On 12/30/2012 11:55 AM, Olof Johansson wrote:
 Hi,
 
 On Fri, Dec 28, 2012 at 12:34 AM, Tushar Behera
 tushar.beh...@linaro.org wrote:
 This patchset adds vmmc and hdmi-en regulators for Origen board.

 Tushar Behera (2):
   ARM: EXYNOS: Add vmmc regulator for Origen board
   ARM: EXYNOS: Add hdmi-en regulator for Origen board
 
 The vmmc one seems to already be there in the dts file for origen. It
 looks like the HDMI one is still needed though.
 

DT enablement patches for HDMI driver were posted sometime back, but
they have not yet been merged in trunk. So HDMI regulator entry for DT
will be added after that.

 Please move to using DT board files instead of expanding the legacy board 
 files.
 

Sure, the focus would be on adding DT support.

Legacy board support for Origen is pretty much complete with support for
various multimedia drivers, so it provides a good environment for
testing different features. It would be good to get these patches in for
enabling HDMI support, which otherwise would be broken if
REGULATOR_DUMMY is not defined.

 
 -Olof
 


-- 
Tushar Behera
--
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 RESEND] ARM: dts: Add EHCI device tree node for Exynos4

2012-12-30 Thread Tushar Behera
On 12/31/2012 08:15 AM, Dongjin Kim wrote:
 This patch adds EHCI device node on device tree for Exynos4 and defines its
 default platform data, s5p_usb_phy_init and s5p_usb_phy_exit, so that those
 function can be called from the driver.
 
 Signed-off-by: Dongjin Kim tobet...@gmail.com
 ---
  arch/arm/boot/dts/exynos4.dtsi |7 +++
  arch/arm/mach-exynos/mach-exynos4-dt.c |9 +
  2 files changed, 16 insertions(+)

[ ... ]

  
 +static struct s5p_ehci_platdata s5p_ehci_platdata = {
 + .phy_init = s5p_usb_phy_init,
 + .phy_exit = s5p_usb_phy_exit,
 +};
 +

Going by the recent development on Samsung USBPHY driver, we won't need
these hooks once that patch gets accepted. [1]

Patch for adding EHCI support for EXYNOS5-DT can be found here.[2]

[1] https://patchwork.kernel.org/patch/1794651/
[2] https://patchwork.kernel.org/patch/1875491/


  /*
   * The following lookup table is used to override device names when devices
   * are registered from device tree. This is temporarily added to enable
 @@ -80,6 +87,8 @@ static const struct of_dev_auxdata exynos4_auxdata_lookup[] 
 __initconst = {
   OF_DEV_AUXDATA(arm,pl330, EXYNOS4_PA_MDMA1, dma-pl330.2, NULL),
   OF_DEV_AUXDATA(samsung,exynos4210-tmu, EXYNOS4_PA_TMU,
   exynos-tmu, NULL),
 + OF_DEV_AUXDATA(samsung,exynos-ehci, EXYNOS4_PA_EHCI,
 + s5p-ehci, s5p_ehci_platdata),
   {},
  };
  
 


-- 
Tushar Behera
--
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 v3 0/5] ARM: Exynos: Enable device tree support for MCT controller

2012-12-30 Thread Thomas Abraham
On 29 December 2012 21:24, Olof Johansson o...@lixom.net wrote:
 On Sat, Dec 29, 2012 at 12:50:33PM -0800, Thomas Abraham wrote:
 Changes since v2:
 - Rebased to linux 3.8-rc1

 Changes since v1:
 - Includes changes suggested by Sylwester Nawrocki 
 sylvester.nawro...@gmail.com

 This patch series adds device tree support for Exynos4/5 MCT controller.
 This patch series has been tested on Exynos4210 based Origen board, 
 Exynos4412
 based Origen board and Exynos5250 based SMDK board.

 Hi,

 Nice cleanups -- no specific comments on those at this time. But
 this is a good time to move the MCT driver out of arch/arm and into
 drivers/clocksource. Can you please do that in this series as well,
 please?

Hi Olof,

Ok. I will move the MCT diver to drivers/clocksource and repost this
series. Thanks for your suggestion.

Regards,
Thomas.



 Thanks,

 -Olof
--
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 00/12] clk: exynos4: migrate to common clock framework

2012-12-30 Thread Thomas Abraham
On 29 December 2012 21:29, Olof Johansson o...@lixom.net wrote:
 On Sat, Dec 29, 2012 at 04:33:29PM -0800, Thomas Abraham wrote:
 Changes since v4:
 - Rebased to linux-3.8-rc1.

 Changes since v3:
 - Includes changes suggested by Tomasz Figa tomasz.f...@gmail.com

 This patch series migrates the Samsung Exynos4 SoC clock code to adopt the
 common clock framework. The use of Samsung specific clock structures has
 been removed and all board support code has been updated. imx-style of
 clock registration and lookup has been adopted for device tree based
 exynos4 platforms.

 I'd prefer to see if exynos4 and 5 were kept common here, and both 
 transitioned
 at the same time. Especially since there are no legacy boards for exynos5, it
 would mean you could have a very clean transition there. What's the plan to
 follow up with 5?

Ok. I have been looking into Exynos4 since Mr. Kim was already onto
Exynos5 common clock. Sure, we could do a cleaner Exynos4/5 common
clock series for v3.9. That would let both exynos4/5 to be built
together which otherwise this patch series would not allow.


 What are the plans to remove legacy board files on exynos4 at this time
 and switch them to DT-only? You could do it gradually like Stephen Warren
 did on Tegra, with hooks that call out to some of the legacy code, but
 configure the board through device tree and do away with the classic
 machine descriptors, etc.

I did try this sometime back for exynos4210 based origen board. But
did not make much progress since certain portions require the older
samsung gpiolib support and exynos4210 mainline device tree support
has migrated to newer pinctrl framework. There is some work here to do
and I will work with Mr. Kim on this.

Thanks,
Thomas.




 -Olof
--
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