Re: [PATCH v5 0/6] exynos: Move pmu driver to driver/soc folder and add exynos7 support
On Saturday 29 November 2014 19:45:02 Amit Daniel Kachhap wrote: This patch series [1 - 6] performs, 1) Moves pmu driver to driver/soc/samsung folder. This is needed as exynos7 is an arm64 based platform and hence PMU driver should be in driver folder. Some discussion happened about this in the v1 version. Finally adding it in driver/soc folder as it too SoC specific and not a general driver. Entire discussion can be found here (A). 2) Add exynos7 PMU support. 3) Enables the driver for 32bit arm exynos platforms. TODO: There is some suggestion to split the PMU data for 32 bit and 64 bit Exynos SOC. This can be implemented as a separate optimization patches. Submitting this version without much change from previous version as the merge window is near. I only now saw this series. I think it's a bad idea to have this code shared with arm64 at all, and I want to see a proper driver for the PMU. 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 v3 3/5] pinctrl: exynos: add exynos5410 SoC specific data
Hi Andreas, 2014-11-28 21:07 GMT+09:00 Andreas Färber afaer...@suse.de: Am 28.11.2014 um 12:59 schrieb Linus Walleij: On Sat, Nov 22, 2014 at 11:26 PM, Andreas Färber afaer...@suse.de wrote: From: Hakjoo Kim ruppi@hardkernel.com Add Samsung EXYNOS5410 SoC specific data to enable pinctrl support for all platforms based on EXYNOS5410. Signed-off-by: Hakjoo Kim ruppi@hardkernel.com [AF: Rebased onto Exynos5260 and irq_chip consolidation] Signed-off-by: Andreas Färber afaer...@suse.de --- v2 - v3: * Rebased (.svc, .{g,w}eint_{con,mask,pend} fields dropped) v1 - v2: * Filled in Sob from Hakjoo Kim Is this based on the pinctrl devel branch so I can apply it? It was based on the Samsung for-next tree. I can rebase if needed. Yes, please rebase (and please make sure that all the structs are const, which became possible after recent changes). Otherwise the patch looks good. 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 v2 3/5] pinctrl: exynos: Fix GPIO setup failure because domain clock being gated
Hi Krzysztof, 2014-11-28 23:08 GMT+09:00 Krzysztof Kozlowski k.kozlow...@samsung.com: On pią, 2014-11-28 at 15:04 +0100, Linus Walleij wrote: On Wed, Nov 26, 2014 at 3:24 PM, Krzysztof Kozlowski k.kozlow...@samsung.com wrote: The audio subsystem on Exynos 5420 has separate clocks and GPIO. To operate properly on GPIOs the main block clock 'mau_epll' must be enabled. This was observed on Peach Pi/Pit and Arndale Octa (after enabling i2s0) after introducing runtime PM to pl330 DMA driver. After that commit the 'mau_epll' was gated, because the amba clock was disabled and there were no more users of mau_epll. The system hang just before probing i2s0 because samsung_pinmux_setup() tried to access memory from audss block which was gated. Add a clock property to the pinctrl driver and enable the clock during GPIO setup. During normal GPIO operations (set, get, set_direction) the clock is not enabled. Could you make sure that possibility of gating this clock is worth the effort of adding gating code to all affected drivers? If there is no significant change in power consumption maybe it could be simply keep running all the time? Also isn't a similar problem happening due to power domains? I believe the whole maudio block is located in a separate power domain but somehow it doesn't get turned off? Could you investigate the above, please? 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 1/2] Input: add regulator haptic driver
Hi Mark, 2014년 11월 28일 21:50에 Mark Brown 이(가) 쓴 글: On Thu, Nov 20, 2014 at 10:31:39PM +0900, Jaewon Kim wrote: + haptic-regulator = devm_regulator_get(pdev-dev, haptic); + if (IS_ERR(haptic-regulator)) { + dev_err(pdev-dev, failed to get regulator\n); + return PTR_ERR(haptic-regulator); + } This needs to be _get_exclusive() - the driver will be broken if something else shares the regualtor since it relies on both enabling and disabling the regulator and on setting the voltage for effects. It's not like a power supply where leaving the device powered when it could be off is going to have no effect, if the power is left on then the haptic motor will continue to operate. Thanks to review my patch. Haptic regulator has to only be used in this driver. So, it is right to change like your advice. I will fix next version. Thanks Jaewon Kim. -- 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 v4 0/2] Add regulator-haptic driver
This patch series adds regulator-haptic driver. The regulator-haptic has haptic motor and it is controlled by voltage of regulator via force feedback framework. Changes in v4: - _regulator_get() - _regulator_get_exclusive() Changes in v3: - fix typo in Documentation - add define in header file Changes in v2: - remove driver owner - merge enable/disable function - support platform data - fix wrong suspends_state check in regulator_haptic_resume() Jaewon Kim (2): Input: add regulator haptic driver ARM: dts: Add regulator-haptic device node for exynos3250-rinato .../devicetree/bindings/input/regulator-haptic.txt | 21 ++ arch/arm/boot/dts/exynos3250-rinato.dts|7 + drivers/input/misc/Kconfig | 11 + drivers/input/misc/Makefile|1 + drivers/input/misc/regulator-haptic.c | 247 include/linux/input/regulator-haptic.h | 31 +++ 6 files changed, 318 insertions(+) create mode 100644 Documentation/devicetree/bindings/input/regulator-haptic.txt create mode 100644 drivers/input/misc/regulator-haptic.c create mode 100644 include/linux/input/regulator-haptic.h -- 1.7.9.5 -- 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 v4 2/2] ARM: dts: Add regulator-haptic device node for exynos3250-rinato
This patch adds regulator-haptic device node controlled by regulator. Signed-off-by: Jaewon Kim jaewon02@samsung.com Reviewed-by: Chanwoo Choi cw00.c...@samsung.com --- arch/arm/boot/dts/exynos3250-rinato.dts |7 +++ 1 file changed, 7 insertions(+) diff --git a/arch/arm/boot/dts/exynos3250-rinato.dts b/arch/arm/boot/dts/exynos3250-rinato.dts index 84380fa..da03005 100644 --- a/arch/arm/boot/dts/exynos3250-rinato.dts +++ b/arch/arm/boot/dts/exynos3250-rinato.dts @@ -104,6 +104,13 @@ }; }; }; + + haptics { + compatible = regulator-haptic; + haptic-supply = motor_reg; + min-microvolt = 110; + max-microvolt = 270; + }; }; adc { -- 1.7.9.5 -- 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 v4 1/2] Input: add regulator haptic driver
This patch adds support for haptic driver controlled by voltage of regulator. And this driver support for Force Feedback interface from input framework Signed-off-by: Jaewon Kim jaewon02@samsung.com Signed-off-by: Hyunhee Kim hyunhee@samsung.com Acked-by: Kyungmin Park kyungmin.p...@samsung.com Tested-by: Chanwoo Choi cw00.c...@samsung.com Reviewed-by: Chanwoo Choi cw00.c...@samsung.com Reviewed-by: Pankaj Dubey pankaj.du...@samsung.com --- .../devicetree/bindings/input/regulator-haptic.txt | 21 ++ drivers/input/misc/Kconfig | 11 + drivers/input/misc/Makefile|1 + drivers/input/misc/regulator-haptic.c | 247 include/linux/input/regulator-haptic.h | 31 +++ 5 files changed, 311 insertions(+) create mode 100644 Documentation/devicetree/bindings/input/regulator-haptic.txt create mode 100644 drivers/input/misc/regulator-haptic.c create mode 100644 include/linux/input/regulator-haptic.h diff --git a/Documentation/devicetree/bindings/input/regulator-haptic.txt b/Documentation/devicetree/bindings/input/regulator-haptic.txt new file mode 100644 index 000..3ed1c7e --- /dev/null +++ b/Documentation/devicetree/bindings/input/regulator-haptic.txt @@ -0,0 +1,21 @@ +* Regulator Haptic Device Tree Bindings + +Required Properties: + - compatible : Should be regulator-haptic + - haptic-supply : Power supply to the haptic motor. + [*] refer Documentation/devicetree/bindings/regulator/regulator.txt + + - max-microvolt : The maximum voltage value supplied to the haptic motor. + [The unit of the voltage is a micro] + + - min-microvolt : The minimum voltage value supplied to the haptic motor. + [The unit of the voltage is a micro] + +Example: + + haptics { + compatible = regulator-haptic; + haptic-supply = motor_regulator; + max-microvolt = 270; + min-microvolt = 110; + }; diff --git a/drivers/input/misc/Kconfig b/drivers/input/misc/Kconfig index 23297ab..e5e556d 100644 --- a/drivers/input/misc/Kconfig +++ b/drivers/input/misc/Kconfig @@ -394,6 +394,17 @@ config INPUT_CM109 To compile this driver as a module, choose M here: the module will be called cm109. +config INPUT_REGULATOR_HAPTIC + tristate regulator haptics support + select INPUT_FF_MEMLESS + help + This option enables device driver support for the haptic controlled + by regulator. This driver supports ff-memless interface + from input framework. + + To compile this driver as a module, choose M here: the + module will be called regulator-haptic. + config INPUT_RETU_PWRBUTTON tristate Retu Power button Driver depends on MFD_RETU diff --git a/drivers/input/misc/Makefile b/drivers/input/misc/Makefile index 19c7603..1f135af 100644 --- a/drivers/input/misc/Makefile +++ b/drivers/input/misc/Makefile @@ -53,6 +53,7 @@ obj-$(CONFIG_INPUT_PMIC8XXX_PWRKEY) += pmic8xxx-pwrkey.o obj-$(CONFIG_INPUT_POWERMATE) += powermate.o obj-$(CONFIG_INPUT_PWM_BEEPER) += pwm-beeper.o obj-$(CONFIG_INPUT_RB532_BUTTON) += rb532_button.o +obj-$(CONFIG_INPUT_REGULATOR_HAPTIC) += regulator-haptic.o obj-$(CONFIG_INPUT_RETU_PWRBUTTON) += retu-pwrbutton.o obj-$(CONFIG_INPUT_GPIO_ROTARY_ENCODER)+= rotary_encoder.o obj-$(CONFIG_INPUT_SGI_BTNS) += sgi_btns.o diff --git a/drivers/input/misc/regulator-haptic.c b/drivers/input/misc/regulator-haptic.c new file mode 100644 index 000..6bc8e45 --- /dev/null +++ b/drivers/input/misc/regulator-haptic.c @@ -0,0 +1,247 @@ +/* + * Regulator haptic driver + * + * Copyright (c) 2014 Samsung Electronics Co., Ltd. + * Author: Jaewon Kim jaewon02@samsung.com + * Author: Hyunhee Kim hyunhee@samsung.com + * + * 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. + */ + +#include linux/input.h +#include linux/input/regulator-haptic.h +#include linux/module.h +#include linux/of.h +#include linux/platform_device.h +#include linux/regulator/consumer.h +#include linux/slab.h + +#define MAX_MAGNITUDE_SHIFT16 + +struct regulator_haptic { + struct device *dev; + struct input_dev *input_dev; + struct regulator *regulator; + struct work_struct work; + + bool enabled; + bool suspend_state; + unsigned int max_volt; + unsigned int min_volt; + unsigned int intensity; + unsigned int magnitude; +}; + +static void regulator_haptic_enable(struct regulator_haptic *haptic, bool state) +{ + int error; + + if (haptic-enabled == state) + return; + + if (state) + error = regulator_enable(haptic-regulator); + else + error =
Re: [PATCH 16/19] arm64: dts: exynos: Add dts files for 64-bit Exynos5433 SoC
Dear Mark, On 11/28/2014 11:00 PM, Mark Rutland wrote: On Fri, Nov 28, 2014 at 01:18:25PM +, Chanwoo Choi wrote: Dear Mark, On 11/27/2014 08:18 PM, Mark Rutland wrote: On Thu, Nov 27, 2014 at 07:35:13AM +, Chanwoo Choi wrote: This patch adds new Exynos5433 dtsi to support 64-bit Exynos5433 SoC based on Octal core CPUs (quad Cortex-A57 and quad Cortex-A53). Cc: Kukjin Kim kgene@samsung.com Cc: Mark Rutland mark.rutl...@arm.com Cc: Arnd Bergmann a...@arndb.de Cc: Olof Johansson o...@lixom.net Cc: Catalin Marinas catalin.mari...@arm.com Cc: Will Deacon will.dea...@arm.com Signed-off-by: Chanwoo Choi cw00.c...@samsung.com Acked-by: Inki Dae inki@samsung.com Acked-by: Geunsik Lim geunsik@samsung.com --- arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi | 698 + arch/arm64/boot/dts/exynos/exynos5433.dtsi | 523 +++ 2 files changed, 1221 insertions(+) create mode 100644 arch/arm64/boot/dts/exynos/exynos5433-pinctrl.dtsi create mode 100644 arch/arm64/boot/dts/exynos/exynos5433.dtsi [...] + cpus { + #address-cells = 2; + #size-cells = 0; + + cpu0: cpu@100 { + device_type = cpu; + compatible = arm,cortex-a53, arm,armv8; + enable-method = psci; While the CPU nodes have enable-methods, I didn't spot a PSCI node anywhere, so this dts cannot possibly have been used to bring up an SMP system. How has this dts been tested? What PSCI revision have you implemented? Have have you tested it? My mistake, Exynos5433 supports PSCI v0.1. I'll add following PSCI nodes: I tested the boot of secondary cpu. psci { compatible = arm,psci; method = smc; cpu_off = 0x8402; cpu_on = 0xC403; }; Ok. I take it _any_ CPU may be hotplugged (including CPU0), given that you don't have MIGRATE_INFO_TYPE from PSCI 0.2 to tell you that this is not possible? If not, attempting to hotplug CPU0 will result in a BUG() and the kernel will explode. Has that been tested? I just tested secondary CPU on during kernel booting after added 'psci' dt node. So, I got the ON state of Octa CPUs. Maybe I need more time to implement CPU0 and secondary cpu hotplugged dynamically on runtime. Do all CPUs enter the kernel at EL2? I didn't consider EL2 for hypervisor mode. First role of this job, I'll implement CPU on/off and suspend by using PSCI. + soc: soc { + compatible = simple-bus; + #address-cells = 1; + #size-cells = 1; + ranges; + + fixed-rate-clocks { + #address-cells = 1; + #size-cells = 0; + + xusbxti: clock@0 { + compatible = fixed-clock; + clock-output-names = xusbxti; + #clock-cells = 0; + }; + }; Get rid of the fixed-rate-clocks container node. It's pointless and messy. Given you only have one there's no need for the bogus unit-address either. OK, I'll remove unneeded code and will add following dt node for fin_pll. fin_pll: xxti { compatible = fixed-clock; clock-output-names = fin_pll; #clock-cells = 0; }; That looks fine to me. [...] + mct@101c { + compatible = samsung,exynos4210-mct; + reg = 0x101c 0x800; + interrupts = 0 102 0, 0 103 0, 0 104 0, 0 105 0, + 0 106 0, 0 107 0, 0 108 0, 0 109, + 0 110 0, 0 111 0, 0 112 0, 0 113 0; + clocks = cmu_top CLK_FIN_PLL, cmu_peris CLK_PCLK_MCT; + clock-names = fin_pll, mct; + }; Hase this block had no changes whatsoever since its use in Exynos4210? Do we not need a samsung,exynos5433-mct comaptible string too? The type of Exynos5433's MCT(Multi-Core Timer) IP is the same with the type of Exynos4210 MCT. Just Exynos5433 have eight local timer for Octa cores. So samsung,exynos4210-mct should appear in the list. I'm just wondering if it's worth having: compatible = samsung,exynos5433-mct, samsung,exynos4210-mct; Just in case we need to special-case the 5433 MCT for some reason later. OK, I'll add samsung,exynos5433-mct compatible string in exynos5433.dtsi according to your comment. CPU0 CPU1 CPU2 CPU3 CPU4 CPU5 CPU6 CPU7 134: 0 0 0 0 0 0 0 0 GIC 134 mct_comp_irq 138: 3189 0 0 0 0 0
Re: [PATCH v5 2/6] drivers: soc: Add support for Exynos PMU driver
On Sun, Nov 30, 2014 at 4:54 AM, Kukjin Kim kg...@kernel.org wrote: Amit Daniel Kachhap wrote: This patch moves Exynos PMU driver implementation from arm/mach-exynos to drivers/soc/samsung. This driver is mainly used for setting misc bits of register from PMU IP of Exynos SoC which will be required to configure before Suspend/Resume. Currently all these settings are done in arch/arm/mach-exynos/pmu.c but moving ahead for ARM64 based SoC support, there is a need of this PMU driver in driver/* folder. This driver uses existing DT binding information and there should be no functionality change in the supported platforms. Signed-off-by: Pankaj Dubey pankaj.du...@samsung.com Signed-off-by: Amit Daniel Kachhap amit.dan...@samsung.com --- arch/arm/mach-exynos/Makefile |2 +- drivers/soc/Kconfig|1 + drivers/soc/Makefile |1 + drivers/soc/samsung/Kconfig| 20 drivers/soc/samsung/Makefile |1 + .../pmu.c = drivers/soc/samsung/exynos-pmu.c |0 6 files changed, 24 insertions(+), 1 deletion(-) create mode 100644 drivers/soc/samsung/Kconfig create mode 100644 drivers/soc/samsung/Makefile rename arch/arm/mach-exynos/pmu.c = drivers/soc/samsung/exynos-pmu.c (100%) diff --git a/arch/arm/mach-exynos/Makefile b/arch/arm/mach-exynos/Makefile index bcefb54..b91b382 100644 --- a/arch/arm/mach-exynos/Makefile +++ b/arch/arm/mach-exynos/Makefile @@ -9,7 +9,7 @@ ccflags-$(CONFIG_ARCH_MULTIPLATFORM) += -I$(srctree)/$(src)/include -I$(srctree) # Core -obj-$(CONFIG_ARCH_EXYNOS)+= exynos.o pmu.o exynos-smc.o firmware.o +obj-$(CONFIG_ARCH_EXYNOS)+= exynos.o exynos-smc.o firmware.o obj-$(CONFIG_EXYNOS_CPU_SUSPEND) += pm.o sleep.o obj-$(CONFIG_PM_SLEEP) += suspend.o diff --git a/drivers/soc/Kconfig b/drivers/soc/Kconfig index 76d6bd4..90f33b9 100644 --- a/drivers/soc/Kconfig +++ b/drivers/soc/Kconfig @@ -3,5 +3,6 @@ menu SOC (System On Chip) specific Drivers source drivers/soc/qcom/Kconfig source drivers/soc/ti/Kconfig source drivers/soc/versatile/Kconfig +source drivers/soc/samsung/Kconfig Alphabetical order? Ok, will add in the next version. [...] - 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 -- 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 3/6] driver: soc: exynos-pmu: Add an API to be called after wakeup
On Sun, Nov 30, 2014 at 4:56 AM, Kukjin Kim kg...@kernel.org wrote: Amit Daniel Kachhap wrote: This patch adds an API exynos_sys_powerup_conf to be called after system sleep wakeup. This will useful for exynos7 SoC to perform resume related initialisations. This is similar to currently existing API exynos_sys_powerdown_conf. Reviewed-by: Pankaj Dubey pankaj.du...@samsung.com Signed-off-by: Amit Daniel Kachhap amit.dan...@samsung.com In subject, driver: soc... - drivers: soc... ? Ok will change. - Kukjin --- drivers/soc/samsung/exynos-pmu.c |9 + include/linux/soc/samsung/exynos-pmu.h |1 + 2 files changed, 10 insertions(+) diff --git a/drivers/soc/samsung/exynos-pmu.c b/drivers/soc/samsung/exynos-pmu.c index 7eb5b0b..c3307eb 100644 --- a/drivers/soc/samsung/exynos-pmu.c +++ b/drivers/soc/samsung/exynos-pmu.c @@ -33,6 +33,7 @@ struct exynos_pmu_data { void (*pmu_init)(void); void (*powerdown_conf)(enum sys_powerdown); void (*powerdown_conf_extra)(enum sys_powerdown); + void (*powerup_conf)(enum sys_powerdown); }; struct exynos_pmu_context { @@ -768,6 +769,14 @@ void exynos_sys_powerdown_conf(enum sys_powerdown mode) } } +void exynos_sys_powerup_conf(enum sys_powerdown mode) +{ + const struct exynos_pmu_data *pmu_data = pmu_context-pmu_data; + + if (pmu_data-powerup_conf) + pmu_data-powerup_conf(mode); +} + static void exynos3250_pmu_init(void) { unsigned int value; diff --git a/include/linux/soc/samsung/exynos-pmu.h b/include/linux/soc/samsung/exynos-pmu.h index a2ab0d5..b497712 100644 --- a/include/linux/soc/samsung/exynos-pmu.h +++ b/include/linux/soc/samsung/exynos-pmu.h @@ -20,5 +20,6 @@ enum sys_powerdown { }; extern void exynos_sys_powerdown_conf(enum sys_powerdown mode); +extern void exynos_sys_powerup_conf(enum sys_powerdown mode); #endif /* __EXYNOS_PMU_H */ -- 1.7.9.5 -- 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 -- 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 5/6] drivers: soc: exynos-pmu: Add support for Exynos7
On Sun, Nov 30, 2014 at 5:05 AM, Kukjin Kim kg...@kernel.org wrote: Amit Daniel Kachhap wrote: Add PMU settings for exynos7. This is required for future suspend-to-ram, cpuidle and power domain support. Note: In this patch some static declarations lines are over 80 characters per line for easy redability. Reviewed-by: Pankaj Dubey pankaj.du...@samsung.com Signed-off-by: Eunseok Choi es10.c...@samsung.com Signed-off-by: Abhilash Kesavan a.kesa...@samsung.com Signed-off-by: Amit Daniel Kachhap amit.dan...@samsung.com --- drivers/soc/samsung/exynos-pmu.c| 427 ++- include/linux/soc/samsung/exynos-regs-pmu.h | 273 + 2 files changed, 699 insertions(+), 1 deletion(-) I'm not sure all of the definitions in exynos-regs-pmu are required?...too many, and I couldn't check the addresses are correct ;) Actually i added the PMU power down for almost all the IP's of Exynos7 as this is full system power down. I have taken care to add only used macros. And how about splitting to each SoC specific file in drivers/soc/samsung/ ? For example, exynos-pmu.c and exynos7-pmu.c like cpufreq...? Just considering. Pankaj posted a patch which takes care of this. Regards, Amit D. - 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 -- 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/6] exynos: Move pmu driver to driver/soc folder and add exynos7 support
Hi Amit, I'm so sorry. I used the name of other man on previous mail. If you possbile, I want to include me in the mailing list. Best Regards, Chanwoo Choi On 12/01/2014 01:30 PM, Chanwoo Choi wrote: Hi Pankaj, If you possible, could you add me to mailing list on the next patchset? because I'll support PMU for Exynos5433, too. Best Regards, Chanwoo Choi On 11/29/2014 11:15 PM, Amit Daniel Kachhap wrote: This patch series [1 - 6] performs, 1) Moves pmu driver to driver/soc/samsung folder. This is needed as exynos7 is an arm64 based platform and hence PMU driver should be in driver folder. Some discussion happened about this in the v1 version. Finally adding it in driver/soc folder as it too SoC specific and not a general driver. Entire discussion can be found here (A). 2) Add exynos7 PMU support. 3) Enables the driver for 32bit arm exynos platforms. TODO: There is some suggestion to split the PMU data for 32 bit and 64 bit Exynos SOC. This can be implemented as a separate optimization patches. Submitting this version without much change from previous version as the merge window is near. Change from V4: * Added a new configuration structure for u32 values in patch 4. * Rebased on Kukjin for-next tree. Changes from V3: * Fixed Kconfig as per Russell feedback * Rebased the series against Pankaj SoC restart consolidation patches (D) as per Kukjin request. * Link to V3 can be found here (C) Changes from V2: * Added review comment changes suggested by Pankaj. * Removed mfd client support in this patch series. This will be added later in the power domain patch series. * Link to V2 can be found here (B) Changes from V1: * Move pmu driver in driver/soc/samsung folder * Removed the power domain features. They will posted as a separate series. * Added exynos7 PMU support. * Link to v1 can be found here (A) This complete patch series is rebased on Kukjin for-next tree. (A) - http://www.spinics.net/lists/linux-samsung-soc/msg38442.html (B) - http://www.spinics.net/lists/arm-kernel/msg375910.html (C) - http://www.spinics.net/lists/linux-samsung-soc/msg39237.html (D) - http://www.spinics.net/lists/linux-samsung-soc/msg39095.html Amit Daniel Kachhap (6): ARM: EXYNOS: Move pmu specific header files under linux/soc/samsung ARM: EXYNOS: Move pmu specific headers under linux/soc/samsung drivers: soc: Add support for Exynos PMU driver driver: soc: exynos-pmu: Add an API to be called after wakeup driver: soc: exynos-pmu: Add a new structure to allow u32 conf data drivers: soc: exynos-pmu: Add support for Exynos7 arm: exynos: Select SOC_SAMSUNG config option arch/arm/mach-exynos/Kconfig |1 + arch/arm/mach-exynos/Makefile |2 +- arch/arm/mach-exynos/exynos.c |2 +- arch/arm/mach-exynos/mcpm-exynos.c |2 +- arch/arm/mach-exynos/platsmp.c |2 +- arch/arm/mach-exynos/pm.c |4 +- arch/arm/mach-exynos/suspend.c |4 +- drivers/soc/Kconfig|1 + drivers/soc/Makefile |1 + drivers/soc/samsung/Kconfig| 20 + drivers/soc/samsung/Makefile |1 + .../pmu.c = drivers/soc/samsung/exynos-pmu.c | 451 +++- .../linux/soc/samsung}/exynos-pmu.h|1 + .../linux/soc/samsung/exynos-regs-pmu.h| 273 14 files changed, 750 insertions(+), 15 deletions(-) create mode 100644 drivers/soc/samsung/Kconfig create mode 100644 drivers/soc/samsung/Makefile rename arch/arm/mach-exynos/pmu.c = drivers/soc/samsung/exynos-pmu.c (68%) rename {arch/arm/mach-exynos = include/linux/soc/samsung}/exynos-pmu.h (89%) rename arch/arm/mach-exynos/regs-pmu.h = include/linux/soc/samsung/exynos-regs-pmu.h (68%) -- 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 -- 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 5/6] drivers: soc: exynos-pmu: Add support for Exynos7
Hi, On 1 December 2014 at 09:59, amit daniel kachhap amit.dan...@samsung.com wrote: On Sun, Nov 30, 2014 at 5:05 AM, Kukjin Kim kg...@kernel.org wrote: Amit Daniel Kachhap wrote: Add PMU settings for exynos7. This is required for future suspend-to-ram, cpuidle and power domain support. Note: In this patch some static declarations lines are over 80 characters per line for easy redability. Reviewed-by: Pankaj Dubey pankaj.du...@samsung.com Signed-off-by: Eunseok Choi es10.c...@samsung.com Signed-off-by: Abhilash Kesavan a.kesa...@samsung.com Signed-off-by: Amit Daniel Kachhap amit.dan...@samsung.com --- drivers/soc/samsung/exynos-pmu.c| 427 ++- include/linux/soc/samsung/exynos-regs-pmu.h | 273 + 2 files changed, 699 insertions(+), 1 deletion(-) I'm not sure all of the definitions in exynos-regs-pmu are required?...too many, and I couldn't check the addresses are correct ;) Actually i added the PMU power down for almost all the IP's of Exynos7 as this is full system power down. I have taken care to add only used macros. And how about splitting to each SoC specific file in drivers/soc/samsung/ ? For example, exynos-pmu.c and exynos7-pmu.c like cpufreq...? Just considering. Pankaj posted a patch which takes care of this. It looks like something went wrong, I can't see my post on Samsung and ARM mailing list, but it appeared on LKML. here is link for it.. https://lkml.org/lkml/2014/11/30/175 Thanks, Pankaj Dubey Regards, Amit D. - 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 ___ linux-arm-kernel mailing list linux-arm-ker...@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- 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