Re: [PATCH 2/2] ARM: multi_v7_defconfig: Enable display on Trats2board

2015-05-23 Thread Arnd Bergmann
On Saturday 23 May 2015 11:18:58 Kukjin Kim wrote:
 On 05/22/15 18:11, Javier Martinez Canillas wrote:
  Hello Krzysztof,
  
  On 05/22/2015 02:48 AM, Krzysztof Kozlowski wrote:
  Enable the Exynos DSI and S6E8AA0 panel for full X11 display on Trats2.
 
  Signed-off-by: Krzysztof Kozlowski k.kozlow...@samsung.com
  ---
   arch/arm/configs/multi_v7_defconfig | 2 ++
   1 file changed, 2 insertions(+)
 
  diff --git a/arch/arm/configs/multi_v7_defconfig 
  b/arch/arm/configs/multi_v7_defconfig
  index 0848337a2a01..e9785020aab1 100644
  --- a/arch/arm/configs/multi_v7_defconfig
  +++ b/arch/arm/configs/multi_v7_defconfig
  @@ -413,10 +413,12 @@ CONFIG_DRM=y
   CONFIG_DRM_PTN3460=m
   CONFIG_DRM_PS8622=m
   CONFIG_DRM_EXYNOS=m
  +CONFIG_DRM_EXYNOS_DSI=y
   CONFIG_DRM_EXYNOS_FIMD=y
   CONFIG_DRM_EXYNOS_HDMI=y
   CONFIG_DRM_RCAR_DU=m
   CONFIG_DRM_TEGRA=y
  +CONFIG_DRM_PANEL_S6E8AA0=m
   CONFIG_DRM_PANEL_SIMPLE=y
   CONFIG_FB_ARMCLCD=y
   CONFIG_FB_WM8505=y
 
  
  Reviewed-by: Javier Martinez Canillas javier.marti...@collabora.co.uk
  
 Looks good to me and this would be handled by arm-soc guys directly 
 
 Acked-by: Kukjin Kim kg...@kernel.org
 

I'd actually prefer if you could put this into the same branch as your
other defconfig changes. We'll handle any conflicts when merging that in.

Arnd
--
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: exynos_defconfig: savedefconfig

2015-05-23 Thread Krzysztof Kozlowski
2015-05-23 12:34 GMT+09:00 Kukjin Kim kg...@kernel.org:
 This patch does 'savedefconfig' for exynos_defconfig to remove useless
 configs and check its dependencies. Sometimes some configs could be
 ignored because of the dependencies even they are enabled in defconfig
 file manually. So checking the dependencies would be better.

 Signed-off-by: Kukjin Kim kg...@kernel.org

Generated autoconf.h is almost the same as previously, so:
Reviewed-by: Krzysztof Kozlowski k.kozlow...@samsung.com

Best regards,
Krzysztof
--
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: [PATCHv5 2/5] ARM: dts: Odroid-XU3 Enable TMU at Exynos5422 base

2015-05-23 Thread Anand Moon
On 21 May 2015 at 05:33, Krzysztof Kozlowski k.kozlow...@samsung.com wrote:
 On 20.05.2015 23:59, Anand Moon wrote:
 On 20 May 2015 at 08:57, Dongjin Kim tobet...@gmail.com wrote:
 Hello Kryzsztof,

 Are you able to check if TMU is under VDDI power domain from Exynos5422
 datasheet?
 If it is, XU3 use BUCK3 for TMU and more internal blocks.

 Thank you,
 Dongjin.

 On Tue, May 19, 2015 at 4:42 PM, Krzysztof Kozlowski
 k.kozlow...@samsung.com wrote:

 2015-05-19 16:28 GMT+09:00 Anand Moon linux.am...@gmail.com:
 On 15 May 2015 at 05:42, Krzysztof Kozlowski k.kozlow...@samsung.com
 wrote:
 2015-05-15 1:16 GMT+09:00 Anand Moon linux.am...@gmail.com:
 On 13 May 2015 at 14:02, Krzysztof Kozlowski k.kozlow...@samsung.com
 wrote:
 2015-05-13 17:21 GMT+09:00 Anand Moon linux.am...@gmail.com:
 On 13 May 2015 at 12:51, Krzysztof Kozlowski
 k.kozlow...@samsung.com wrote:
 2015-05-13 15:36 GMT+09:00 Anand Moon linux.am...@gmail.com:
 This changes enables TMU IP block on the Exynos5422 Odroid-XU3
 device.

 Tested-by: Markus Reichl m.rei...@fivetechno.de
 Acked-by: Lukasz Majewski l.majew...@samsung.com
 Signed-off-by: Anand Moon linux.am...@gmail.com
 ---
  arch/arm/boot/dts/exynos5422-odroidxu3.dts | 25
 +
  1 file changed, 25 insertions(+)

 diff --git a/arch/arm/boot/dts/exynos5422-odroidxu3.dts
 b/arch/arm/boot/dts/exynos5422-odroidxu3.dts
 index 9446e28..cd78816 100644
 --- a/arch/arm/boot/dts/exynos5422-odroidxu3.dts
 +++ b/arch/arm/boot/dts/exynos5422-odroidxu3.dts
 @@ -319,6 +319,31 @@
 #cooling-cells = 2;
 cooling-levels = 0 130 170 230;
 };
 +
 +   tmu@1006 {

 Here and for other overrides please use label notation, like:

 tmu_cpu0 {
   ...
 };

 +   vtmu-supply = ldo10_reg;

 I am curious, how did you find that LDO10 supplies TMU unit?

 Best regards,
 Krzysztof

 Hi Krzysztof,

 I have re-base my work on earlier Lukasz Majewski patches.

 https://patchwork.kernel.org/patch/5693201/

 NAK.
 I am sorry, but this is not sufficient explanation. Actually such
 explanation could mean that you did just blindly copied everything
 instead of developing it.

 You cannot use some regulator here just because some Exynos4 boards
 use it. You have to be sure that this regulator supplies this part of
 SoC or device.

 Hi Krzysztof,

 After going through the schematics, I came to understanding that their
 is
 missing regulator related to TEMP SE in the exynos5422-odroidxu3.dts.

 Below is the schematic of the board.

 http://dn.odroid.com/5422/ODROID-XU3/Schematics/XU3_MAIN_REV0.2.PDF

   ldo18_reg: LDO18 {
  regulator-name = vdd_ldo18;
  regulator-min-microvolt =
 180;
  regulator-max-microvolt =
 180;
  regulator-always-on;
};

 The output of LDO18 goes to VDD_EMMC_1V8. This is not regulator for
 TMU.

 I think the schematics are missing some of details but it can be
 deducted that:
 1. TEMP SE is supplied by VDD18_TS power domain. It consists of 5
 pairs of pins (XTSTEST_OUT[0-4], XTSEXT_RES[0-4]).
 2. The VDD18_TS01, VDD18_TS23 and VDD18_TS4 are wired to theL DO7 of
 S2MPS11 PMIC.
 3. I confirmed with the Exynos5422 datasheet that these
 VDD18_TS{01,23,4} supply the XTSTEST pins (OUT and RES).

 So the LDO7 it is... but before using it there is a caveat. The LDO7
 is also connected to VDD of MIPI, HDMI and few more. So when you use
 this regulator in TMU it may be turned off by TMU driver (e.g. during
 unbind). In such case these other blocks also should be tested and
 checked whether they take this regulator and enable it.

 hi Krzysztof,

 I tried to use the LDO7 regulator for TMU but it failed to register.

 [3.231329] ina2xx 0-0045: power monitor ina231 (Rshunt = 1 uOhm)
 [3.237691] thermal thermal_zone0: failed to read out thermal zone
 (-22)
 [3.243033] exynos-tmu 1006.tmu: Looking up vtmu-supply from
 device tree
 [3.243936] thermal thermal_zone1: failed to read out thermal zone
 (-22)
 [3.249791] exynos-tmu 10064000.tmu: Looking up vtmu-supply from
 device tree
 [3.250677] thermal thermal_zone2: failed to read out thermal zone
 (-22)
 [3.256410] exynos-tmu 10068000.tmu: Looking up vtmu-supply from
 device tree
 [3.257345] thermal thermal_zone3: failed to read out thermal zone
 (-22)
 [3.263050] exynos-tmu 1006c000.tmu: Looking up vtmu-supply from
 device tree
 [3.263984] thermal thermal_zone4: failed to read out thermal zone
 (-22)
 [3.269769] exynos-tmu 100a.tmu: Looking up vtmu-supply from
 device tree
 [3.270363] usb 5-1: New USB device found, idVendor=0424,
 idProduct=9514
 [3.276389] usb 5-1: New USB device strings: Mfr=0, Product=0,
 SerialNumber=0

 Indeed.


 Hi Krzysztof/Dongjin

 BUCK3 is option for TMU as suggested by Dongjin

 What do you mean 

Re: [PATCH v7 03/25] iommu: Init iommu-groups support earlier, in core_initcall

2015-05-23 Thread Laurent Pinchart
Hi Marek,

Thank you for the patch.

On Tuesday 19 May 2015 15:20:23 Marek Szyprowski wrote:
 iommu_group_alloc might be called very early in case of iommu controllers
 activated from of_iommu, so ensure that this part of subsystem is ready
 when devices are being populated from device-tree (core_initcall seems to
 be okay for this case).
 
 Signed-off-by: Marek Szyprowski m.szyprow...@samsung.com
 ---
  drivers/iommu/iommu.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c
 index d4f527e56679..37a6aa8f318b 100644
 --- a/drivers/iommu/iommu.c
 +++ b/drivers/iommu/iommu.c
 @@ -1207,7 +1207,7 @@ static int __init iommu_init(void)
 
   return 0;
  }
 -arch_initcall(iommu_init);
 +core_initcall(iommu_init);

I'll let Joerg comment on this, but this initcall ordering dance always makes 
me feel that something isn't quite right. Have you had a chance to look at the 
patch series I posted about a week ago to implement IOMMU probe deferral 
support ?

  int iommu_domain_get_attr(struct iommu_domain *domain,
 enum iommu_attr attr, void *data)

-- 
Regards,

Laurent Pinchart

--
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: [PATCHv5 2/5] ARM: dts: Odroid-XU3 Enable TMU at Exynos5422 base

2015-05-23 Thread Anand Moon
On 24 May 2015 at 10:03, Krzysztof Kozlowski k.kozlow...@samsung.com wrote:
 2015-05-24 2:35 GMT+09:00 Anand Moon linux.am...@gmail.com:
 Hi Krzysztof,

 I did some debugging on the this using powerdebug utility.
 Using  LDO7 to control TMU seams to me correct option compared to BUCK3.

 Setting LDO7 to control TMU I observed  following output on powerdebug.
 I observed that the power drawn by the board on Odroid show is much
 less compare to BUCK3 on the same setup.
 I can see the HDMI output on Odroid-V2 display screen.
 I have done some stress testing on this configuration and It worked 
 correctly.

 Powerdebug output using LDO7
 ---
 PowerDebug 0.7.3 Clocks  Regulators  Sensors  Gpio
 NameStatus  State   TypeUsers   Microvolts
  Min u-volts Max u-volts
 phy 0   0
  0   0
 phy 0   0
  0   0
 vdd_ldo1enabled voltage 0   100
  100 100
 LDO10   0
  0   0
 LDO2enabled voltage 0   180
  0   0
 vdd_ldo3enabled voltage 0   180
  180 180
 LDO30   0
  0   0
 LDO4enabled voltage 0   180
  0   0
 vdd_ldo5enabled voltage 0   180
  180 180
 LDO50   0
  0   0
 vdd_ldo6enabled voltage 0   100
  100 100
 hdmiokay0   0
  0   0
 hdmiokay0   0
  0   0
 LDO60   0
  0   0
 vdd_ldo7enabled voltage 0   180
  180 180
 hdmiokay0   0
  0   0
 tmu okay0   0
  0   0
 tmu okay0   0
  0   0
 tmu okay0   0
  0   0
 tmu okay0   0
  0   0
 tmu okay0   0
  0   0
 LDO70   0
  0   0
 vdd_ldo8enabled voltage 0   180
  180 180
 LDO80   0
  0   0
 vdd_ldo9enabled voltage 0   300
  300 300
 LDO90   0
  0   0

 Powerdebug output using BUCK3.
 --
 PowerDebug 0.7.3 Clocks  Regulators  Sensors  Gpio
 NameStatus  State   TypeUsers   Microvolts
  Min u-volts Max u-volts
 LDO21   disabledvoltage 0   180
  0   0
 LDO22   disabledvoltage 0   120
  0   0
 LDO23   enabled voltage 0   110
  0   0
 tsp_io  enabled voltage 0   280
  280 280
 LDO24   0   0
  0   0
 LDO25   disabledvoltage 0   180
  0   0
 vdd_ldo26   enabled voltage 0   300
  300 300
 LDO26   0   0
  0   0
 LDO27   enabled voltage 0   100
  0   0
 LDO28   disabledvoltage 0   330
  0   0
 LDO29   disabledvoltage 0   180
  0   0
 LDO30   disabledvoltage 0   180
  0   0
 LDO31   disabledvoltage 0   180
  0   0
 LDO32   disabledvoltage 0   180
  0   0
 LDO33   disabledvoltage 0   180
  0   0
 LDO34   disabledvoltage 0   300
  0   0
 LDO35   disabledvoltage 0   160
  0   0
 LDO36   disabledvoltage 0   180
  0   0
 LDO37   disabledvoltage 0   180
  0   0
 LDO38 

Re: [PATCHv5 2/5] ARM: dts: Odroid-XU3 Enable TMU at Exynos5422 base

2015-05-23 Thread Krzysztof Kozlowski
2015-05-24 2:35 GMT+09:00 Anand Moon linux.am...@gmail.com:
 Hi Krzysztof,

 I did some debugging on the this using powerdebug utility.
 Using  LDO7 to control TMU seams to me correct option compared to BUCK3.

 Setting LDO7 to control TMU I observed  following output on powerdebug.
 I observed that the power drawn by the board on Odroid show is much
 less compare to BUCK3 on the same setup.
 I can see the HDMI output on Odroid-V2 display screen.
 I have done some stress testing on this configuration and It worked correctly.

 Powerdebug output using LDO7
 ---
 PowerDebug 0.7.3 Clocks  Regulators  Sensors  Gpio
 NameStatus  State   TypeUsers   Microvolts
  Min u-volts Max u-volts
 phy 0   0
  0   0
 phy 0   0
  0   0
 vdd_ldo1enabled voltage 0   100
  100 100
 LDO10   0
  0   0
 LDO2enabled voltage 0   180
  0   0
 vdd_ldo3enabled voltage 0   180
  180 180
 LDO30   0
  0   0
 LDO4enabled voltage 0   180
  0   0
 vdd_ldo5enabled voltage 0   180
  180 180
 LDO50   0
  0   0
 vdd_ldo6enabled voltage 0   100
  100 100
 hdmiokay0   0
  0   0
 hdmiokay0   0
  0   0
 LDO60   0
  0   0
 vdd_ldo7enabled voltage 0   180
  180 180
 hdmiokay0   0
  0   0
 tmu okay0   0
  0   0
 tmu okay0   0
  0   0
 tmu okay0   0
  0   0
 tmu okay0   0
  0   0
 tmu okay0   0
  0   0
 LDO70   0
  0   0
 vdd_ldo8enabled voltage 0   180
  180 180
 LDO80   0
  0   0
 vdd_ldo9enabled voltage 0   300
  300 300
 LDO90   0
  0   0

 Powerdebug output using BUCK3.
 --
 PowerDebug 0.7.3 Clocks  Regulators  Sensors  Gpio
 NameStatus  State   TypeUsers   Microvolts
  Min u-volts Max u-volts
 LDO21   disabledvoltage 0   180
  0   0
 LDO22   disabledvoltage 0   120
  0   0
 LDO23   enabled voltage 0   110
  0   0
 tsp_io  enabled voltage 0   280
  280 280
 LDO24   0   0
  0   0
 LDO25   disabledvoltage 0   180
  0   0
 vdd_ldo26   enabled voltage 0   300
  300 300
 LDO26   0   0
  0   0
 LDO27   enabled voltage 0   100
  0   0
 LDO28   disabledvoltage 0   330
  0   0
 LDO29   disabledvoltage 0   180
  0   0
 LDO30   disabledvoltage 0   180
  0   0
 LDO31   disabledvoltage 0   180
  0   0
 LDO32   disabledvoltage 0   180
  0   0
 LDO33   disabledvoltage 0   180
  0   0
 LDO34   disabledvoltage 0   300
  0   0
 LDO35   disabledvoltage 0   160
  0   0
 LDO36   disabledvoltage 0   180
  0   0
 LDO37   disabledvoltage 0   180
  0   0
 LDO38   disabledvoltage 0   280
  0   0
 vdd_mif