2015-05-15 12:49:55

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH 0/4] ARM: multi_v7_defconfig: Stuff for Exynos

Dear Kukjin,

The patchset enables various config options on multi_v7 config
for Exynos boards. The first two patches are actually resend.

Arnd suggested [0] that this can go through your tree.

Patchset is rebased on next-20150515 and Javier's patchset [1]
(to avoid conflicts around regulators and clocks).

Please let me know if this should be rebased on other commit.

[0] http://www.spinics.net/lists/kernel/msg1991518.html
[1] http://www.spinics.net/lists/kernel/msg1990767.html

Best regards,
Krzysztof


Krzysztof Kozlowski (4):
ARM: multi_v7_defconfig: Enable CPU idle for Exynos
ARM: multi_v7_defconfig: Enable PMIC and MUIC drivers for Exynos
boards
ARM: multi_v7_defconfig: Enable TMU for Exynos
ARM: multi_v7_defconfig: Enable OHCI on Exynos

arch/arm/configs/multi_v7_defconfig | 14 ++++++++++++++
1 file changed, 14 insertions(+)

--
2.1.4


2015-05-15 12:50:05

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH 1/4] ARM: multi_v7_defconfig: Enable CPU idle for Exynos

From: Krzysztof Kozlowski <[email protected]>

Current Exynos CPU idle driver supports entering AFTR (Arm Off, Top
Running) mode on Exynos3250, Exynos4210 (coupled), Exynos4x12 and
Exynos5250. Enable it in default configuration to reduce energy
consumption.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/configs/multi_v7_defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 76cbb81cbaa3..d524d2e9633c 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -117,6 +117,7 @@ CONFIG_CPU_FREQ=y
CONFIG_CPU_FREQ_STAT_DETAILS=y
CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND=y
CONFIG_CPU_IDLE=y
+CONFIG_ARM_EXYNOS_CPUIDLE=y
CONFIG_NEON=y
CONFIG_KERNEL_MODE_NEON=y
CONFIG_ARM_ZYNQ_CPUIDLE=y
--
2.1.4

2015-05-15 12:50:58

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH 2/4] ARM: multi_v7_defconfig: Enable PMIC and MUIC drivers for Exynos boards

From: Krzysztof Kozlowski <[email protected]>

Enable drivers for PMICs and MUICs present on Exynos-based devices:
- max14577: charger, fuel gauge (max17040), regulator,
used on: Gear 1, Gear 2,
- max7693: charger, fuel gauge (max17042),
used on: Trats2,
- s5m/s2mps: RTC, clock,
used on: Arndale, Arndale Octa, Gear 1, Gear 2

This allows full usage of charging stack on these devices along with RTC
and 32 kHz clocks.

The I2C_GPIO is necessary for MAX77693 on Trats2 because the device is
wired to Exynos4412 over GPIO.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/configs/multi_v7_defconfig | 11 +++++++++++
1 file changed, 11 insertions(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index d524d2e9633c..6a6a292c5bf8 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -280,6 +280,7 @@ CONFIG_I2C_DESIGNWARE_PLATFORM=y
CONFIG_I2C_DIGICOLOR=m
CONFIG_I2C_GPIO=m
CONFIG_I2C_EXYNOS5=y
+CONFIG_I2C_GPIO=y
CONFIG_I2C_MV64XXX=y
CONFIG_I2C_RIIC=y
CONFIG_I2C_S3C2410=y
@@ -328,7 +329,11 @@ CONFIG_GPIO_SYSCON=y
CONFIG_GPIO_TPS6586X=y
CONFIG_GPIO_TPS65910=y
CONFIG_BATTERY_SBS=y
+CONFIG_BATTERY_MAX17040=y
+CONFIG_BATTERY_MAX17042=y
+CONFIG_CHARGER_MAX14577=y
CONFIG_CHARGER_TPS65090=y
+CONFIG_CHARGER_MAX77693=y
CONFIG_POWER_RESET_AS3722=y
CONFIG_POWER_RESET_GPIO=y
CONFIG_POWER_RESET_GPIO_RESTART=y
@@ -357,8 +362,10 @@ CONFIG_MFD_AXP20X=y
CONFIG_MFD_CROS_EC=y
CONFIG_MFD_CROS_EC_I2C=m
CONFIG_MFD_CROS_EC_SPI=y
+CONFIG_MFD_MAX14577=y
CONFIG_MFD_MAX77686=y
CONFIG_MFD_MAX8907=y
+CONFIG_MFD_MAX77693=y
CONFIG_MFD_SEC_CORE=y
CONFIG_MFD_STMPE=y
CONFIG_MFD_PALMAS=y
@@ -374,9 +381,11 @@ CONFIG_REGULATOR_DA9210=y
CONFIG_REGULATOR_GPIO=y
CONFIG_MFD_SYSCON=y
CONFIG_POWER_RESET_SYSCON=y
+CONFIG_REGULATOR_MAX14577=y
CONFIG_REGULATOR_MAX8907=y
CONFIG_REGULATOR_MAX8973=y
CONFIG_REGULATOR_MAX77686=y
+CONFIG_REGULATOR_MAX77693=y
CONFIG_REGULATOR_MAX77802=m
CONFIG_REGULATOR_PALMAS=y
CONFIG_REGULATOR_S2MPS11=y
@@ -529,6 +538,7 @@ CONFIG_RTC_DRV_SUN6I=y
CONFIG_RTC_DRV_SUNXI=y
CONFIG_RTC_DRV_MV=y
CONFIG_RTC_DRV_TEGRA=y
+CONFIG_RTC_DRV_S5M=y
CONFIG_DMADEVICES=y
CONFIG_DW_DMAC=y
CONFIG_MV_XOR=y
@@ -558,6 +568,7 @@ CONFIG_QCOM_GSBI=y
CONFIG_COMMON_CLK_QCOM=y
CONFIG_COMMON_CLK_MAX77686=y
CONFIG_COMMON_CLK_MAX77802=m
+CONFIG_COMMON_CLK_S2MPS11=y
CONFIG_APQ_MMCC_8084=y
CONFIG_MSM_GCC_8660=y
CONFIG_MSM_MMCC_8960=y
--
2.1.4

2015-05-15 12:50:17

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH 3/4] ARM: multi_v7_defconfig: Enable TMU for Exynos

Enable support for Thermal Monitoring Unit present on Exynos SoCs. This
allows detection of overheat and handling this gracefully.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/configs/multi_v7_defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 6a6a292c5bf8..6ee2c24dfbc7 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -343,6 +343,7 @@ CONFIG_SENSORS_LM90=y
CONFIG_SENSORS_LM95245=y
CONFIG_THERMAL=y
CONFIG_CPU_THERMAL=y
+CONFIG_EXYNOS_THERMAL=y
CONFIG_RCAR_THERMAL=y
CONFIG_ARMADA_THERMAL=y
CONFIG_DAVINCI_WATCHDOG
--
2.1.4

2015-05-15 12:50:13

by Krzysztof Kozłowski

[permalink] [raw]
Subject: [PATCH 4/4] ARM: multi_v7_defconfig: Enable OHCI on Exynos

Enable the USB OHCI driver for Exynos SoCs: S5PV210, Exynos4 family,
Exynos5250, Exynos5440 and Exynos542x/5800 family.

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/configs/multi_v7_defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 6ee2c24dfbc7..b3573c01f34c 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -456,6 +456,7 @@ CONFIG_USB_ISP1760_HCD=y
CONFIG_USB_OHCI_HCD=y
CONFIG_USB_OHCI_HCD_STI=y
CONFIG_USB_OHCI_HCD_PLATFORM=y
+CONFIG_USB_OHCI_EXYNOS=y
CONFIG_USB_R8A66597_HCD=m
CONFIG_USB_RENESAS_USBHS=m
CONFIG_USB_STORAGE=y
--
2.1.4

2015-05-15 14:12:30

by Javier Martinez Canillas

[permalink] [raw]
Subject: Re: [PATCH 0/4] ARM: multi_v7_defconfig: Stuff for Exynos

Hello Krzysztof,

On 05/15/2015 02:48 PM, Krzysztof Kozlowski wrote:
> Dear Kukjin,
>
> The patchset enables various config options on multi_v7 config
> for Exynos boards. The first two patches are actually resend.
>
> Arnd suggested [0] that this can go through your tree.
>
> Patchset is rebased on next-20150515 and Javier's patchset [1]
> (to avoid conflicts around regulators and clocks).
>
> Please let me know if this should be rebased on other commit.
>
> [0] http://www.spinics.net/lists/kernel/msg1991518.html
> [1] http://www.spinics.net/lists/kernel/msg1990767.html
>
> Best regards,
> Krzysztof
>
>
> Krzysztof Kozlowski (4):
> ARM: multi_v7_defconfig: Enable CPU idle for Exynos
> ARM: multi_v7_defconfig: Enable PMIC and MUIC drivers for Exynos
> boards
> ARM: multi_v7_defconfig: Enable TMU for Exynos
> ARM: multi_v7_defconfig: Enable OHCI on Exynos
>
> arch/arm/configs/multi_v7_defconfig | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>

On a first look, I agree with all the Kconfig symbols but I noticed that
you are adding them to be built-in while the current policy for multi_v7
is to build as much as possible as a module.

Could you please add as =m for all the tristate symbols that can be build
as a module?

Best regards,
Javier

2015-05-16 00:33:17

by Krzysztof Kozłowski

[permalink] [raw]
Subject: Re: [PATCH 0/4] ARM: multi_v7_defconfig: Stuff for Exynos

2015-05-15 23:12 GMT+09:00 Javier Martinez Canillas
<[email protected]>:
> Hello Krzysztof,
>
> On 05/15/2015 02:48 PM, Krzysztof Kozlowski wrote:
>> Dear Kukjin,
>>
>> The patchset enables various config options on multi_v7 config
>> for Exynos boards. The first two patches are actually resend.
>>
>> Arnd suggested [0] that this can go through your tree.
>>
>> Patchset is rebased on next-20150515 and Javier's patchset [1]
>> (to avoid conflicts around regulators and clocks).
>>
>> Please let me know if this should be rebased on other commit.
>>
>> [0] http://www.spinics.net/lists/kernel/msg1991518.html
>> [1] http://www.spinics.net/lists/kernel/msg1990767.html
>>
>> Best regards,
>> Krzysztof
>>
>>
>> Krzysztof Kozlowski (4):
>> ARM: multi_v7_defconfig: Enable CPU idle for Exynos
>> ARM: multi_v7_defconfig: Enable PMIC and MUIC drivers for Exynos
>> boards
>> ARM: multi_v7_defconfig: Enable TMU for Exynos
>> ARM: multi_v7_defconfig: Enable OHCI on Exynos
>>
>> arch/arm/configs/multi_v7_defconfig | 14 ++++++++++++++
>> 1 file changed, 14 insertions(+)
>>
>
> On a first look, I agree with all the Kconfig symbols but I noticed that
> you are adding them to be built-in while the current policy for multi_v7
> is to build as much as possible as a module.
>
> Could you please add as =m for all the tristate symbols that can be build
> as a module?

Your request makes sense. For many Exynos boards we got used to
compiling it into the kernel, not as a module, but I think only reason
behind this is convenience. I'll try to switch to modules.

Best regards,
Krzysztof