linaro-kernel July 2013

linaro-kernel@lists.linaro.org

73 participants
90 discussions

[PATCH 1/4] ARM: exynos: cpuidle: Fallback to WFI, when AFTR is selected

by Daniel Lezcano

When there are several cpus online, the AFTR state does not work. The AFTR must be selected only when there is one cpu online. The previous code was already handling this case. Simplified the code, especially the init routine to have the same init pattern than the other drivers. Signed-off-by: Daniel Lezcano <daniel.lezcano(a)linaro.org> --- arch/arm/mach-exynos/cpuidle.c | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/arch/arm/mach-exynos/cpuidle.c b/arch/arm/mach-exynos/cpuidle.c index 17a18ff..cc4b097 100644 --- a/arch/arm/mach-exynos/cpuidle.c +++ b/arch/arm/mach-exynos/cpuidle.c @@ -147,16 +147,11 @@ static int exynos4_enter_lowpower(struct cpuidle_device *dev, struct cpuidle_driver *drv, int index) { - int new_index = index; - /* This mode only can be entered when other core's are offline */ if (num_online_cpus() > 1) - new_index = drv->safe_state_index; + return drv->states[0].enter(dev, drv, 0); - if (new_index == 0) - return arm_cpuidle_simple_enter(dev, drv, new_index); - else - return exynos4_enter_core0_aftr(dev, drv, new_index); + return exynos4_enter_core0_aftr(dev, drv, index); } static void __init exynos5_core_down_clk(void) @@ -209,10 +204,6 @@ static int __init exynos4_init_cpuidle(void) device = &per_cpu(exynos4_cpuidle_device, cpu_id); device->cpu = cpu_id; - /* Support IDLE only */ - if (cpu_id != 0) - device->state_count = 1; - ret = cpuidle_register_device(device); if (ret) { printk(KERN_ERR "CPUidle register device failed\n"); -- 1.7.9.5

10 years, 9 months

Re: [PATCH 1/3 v6] cpufreq: Add debugfs directory for cpufreq

by Viresh Kumar

On 18 July 2013 16:47, Chanwoo Choi <cw00.choi(a)samsung.com> wrote: > +#ifdef CONFIG_CPU_FREQ_STAT > +/* The cpufreq_debugfs is used to create debugfs root directory for CPUFreq. */ > +static struct dentry *cpufreq_debugfs; > + > +static int cpufreq_create_debugfs_dir(struct cpufreq_policy *policy, > + struct device *dev) > +{ > + char name[CPUFREQ_NAME_LEN]; > + unsigned int cpus, size, idx; > + > + if (!cpufreq_debugfs) > + return -EINVAL; > + > + cpus = cpumask_weight(policy->cpus); I remember I told you not to use policy->cpus for this purpose?? But related_cpus. > + idx = cpus > 1 ? policy->cpu : 0; > + policy->cpu_debugfs[idx] = debugfs_create_dir(name, cpufreq_debugfs); This is broken. A policy may contain cpus 9,10 only.. You will allocate array for 2 cpus and try to access cpu_debugfs[9] :) > + if (!policy->cpu_debugfs[idx]) { > + pr_err("creating debugfs directory failed\n"); > + return -ENODEV; > + } > + > + return 0; > +} > + > +static int cpufreq_create_debugfs_symlink(struct cpufreq_policy *policy, > + unsigned int src_cpu, > + unsigned int dest_cpu) Only use policy and cpu for which symlink has to be created as param to this routine. And create link to policy->cpu. > +{ > + char symlink_name[CPUFREQ_NAME_LEN]; > + char target_name[CPUFREQ_NAME_LEN]; > + > + if (!cpufreq_debugfs) > + return -EINVAL; > + > + if (!policy->cpu_debugfs[src_cpu]) > + return -EINVAL; > + > + sprintf(symlink_name, "cpu%d", dest_cpu); > + sprintf(target_name, "./cpu%d", src_cpu); > + policy->cpu_debugfs[dest_cpu] = debugfs_create_symlink( > + symlink_name, > + cpufreq_debugfs, > + target_name); > + if (!policy->cpu_debugfs[dest_cpu]) { > + pr_err("creating debugfs symlink failed\n"); > + return -ENODEV; > + } > + > + return 0; > +} > + > +static void cpufreq_remove_debugfs_dir(struct cpufreq_policy *policy, > + unsigned int cpu) > +{ > + unsigned int idx = cpumask_weight(policy->cpus) > 1 ? cpu : 0; > + > + if (!policy->cpu_debugfs[idx]) > + return; > + > + debugfs_remove_recursive(policy->cpu_debugfs[idx]); Whey do we need recursive here? And what exactly does recursive will do? > +} > + same problem here too. > +static void cpufreq_move_debugfs_dir(struct cpufreq_policy *policy, > + unsigned int new_cpu) > +{ > + struct dentry *old_entry, *new_entry; > + char new_dir_name[CPUFREQ_NAME_LEN]; > + unsigned int j, old_cpu = policy->cpu; > + > + if (!policy->cpu_debugfs[new_cpu]) > + return; > + > + /* > + * Remove symbolic link of debugfs directory except for debugfs > + * directory of old_cpu. > + */ > + for_each_present_cpu(j) { > + if (old_cpu == j) > + continue; > + > + debugfs_remove(policy->cpu_debugfs[j]); Why you need this? We aren't removing the earlier dentry at all here. > + } > + > + /* > + * Change debugfs directory name from as following: > + * - old debugfs dir name : /sys/kernel/debugfs/cpufreq/cpu${old_cpu} > + * - new debugfs dir name : /sys/kernel/debugfs/cpufreq/cpu${new_cpu} > + */ > + sprintf(new_dir_name, "cpu%d", new_cpu); > + old_entry = policy->cpu_debugfs[old_cpu]; > + new_entry = debugfs_rename(cpufreq_debugfs, old_entry, > + cpufreq_debugfs, new_dir_name); This routine returns old_entry only.. and so you can simply create a single routine with name dentry. > + if (!new_entry) { > + pr_err("changing debugfs directory name failed\n"); > + goto err_rename; > + } > + > + policy->cpu_debugfs[new_cpu] = new_entry; > + policy->cpu_debugfs[old_cpu] = NULL; > + > + /* Create again symbolic link of debugfs directory */ > + for_each_present_cpu(j) { present_cpu?? We discussed this before.. You will break multi cluster systems. > + if (new_cpu == j) > + continue; > + > + cpufreq_create_debugfs_symlink(policy, new_cpu, j); > + } > + > + return; > + > +err_rename: > + for_each_present_cpu(j) { > + if (old_cpu == j) > + continue; > + > + cpufreq_create_debugfs_symlink(policy, old_cpu, j); > + } > +} > + > +static int cpufreq_create_debugfs(void) > +{ > + cpufreq_debugfs = debugfs_create_dir("cpufreq", NULL); > + if (!cpufreq_debugfs) { > + pr_err("creating debugfs root failed\n"); > + return -ENODEV; > + } > + > + return 0; > +} > + > +static void cpufreq_remove_debugfs(void) > +{ > + if (cpufreq_debugfs) > + debugfs_remove_recursive(cpufreq_debugfs); > +} > +#else > +static int cpufreq_create_debugfs_dir(struct cpufreq_policy *policy, > + struct device *dev) { return 0; } > +static int cpufreq_create_debugfs_symlink(struct cpufreq_policy *policy, > + unsigned int src_cpu, > + unsigned int dest_cpu) { return 0;} > +static void cpufreq_remove_debugfs_dir(struct cpufreq_policy *policy, > + unsigned int cpu) { } > +static void cpufreq_move_debugfs_dir(struct cpufreq_policy *policy, > + unsigned int new_cpu) { } > +static int cpufreq_create_debugfs(void) { return 0; } > +static void cpufreq_remove_debugfs(void) { } make all above #else part routines inline. > +#endif > + > /* symlink affected CPUs */ > static int cpufreq_add_dev_symlink(unsigned int cpu, > struct cpufreq_policy *policy) > @@ -726,6 +885,8 @@ static int cpufreq_add_dev_symlink(unsigned int cpu, > cpufreq_cpu_put(managed_policy); > return ret; > } > + > + cpufreq_create_debugfs_symlink(policy, cpu, j); > } > return ret; > } > @@ -777,6 +938,9 @@ static int cpufreq_add_dev_interface(unsigned int cpu, > } > write_unlock_irqrestore(&cpufreq_driver_lock, flags); > > + /* prepare interface data for debugfs */ > + cpufreq_create_debugfs_dir(policy, dev); > + > ret = cpufreq_add_dev_symlink(cpu, policy); > if (ret) > goto err_out_kobj_put; > @@ -839,6 +1003,8 @@ static int cpufreq_add_policy_cpu(unsigned int cpu, unsigned int sibling, > return ret; > } > > + cpufreq_create_debugfs_symlink(policy, sibling, cpu); > + > return 0; > } > #endif > @@ -1046,6 +1212,7 @@ static int __cpufreq_remove_dev(struct device *dev, struct subsys_interface *sif > > if (cpu != data->cpu) { > sysfs_remove_link(&dev->kobj, "cpufreq"); > + cpufreq_remove_debugfs_dir(data, cpu); > } else if (cpus > 1) { > /* first sibling now owns the new sysfs dir */ > cpu_dev = get_cpu_device(cpumask_first(data->cpus)); > @@ -1068,6 +1235,8 @@ static int __cpufreq_remove_dev(struct device *dev, struct subsys_interface *sif > return -EINVAL; > } > > + cpufreq_move_debugfs_dir(data, cpu_dev->id); > + > WARN_ON(lock_policy_rwsem_write(cpu)); > update_policy_cpu(data, cpu_dev->id); > unlock_policy_rwsem_write(cpu); > @@ -1089,6 +1258,8 @@ static int __cpufreq_remove_dev(struct device *dev, struct subsys_interface *sif > unlock_policy_rwsem_read(cpu); > kobject_put(kobj); > > + cpufreq_remove_debugfs_dir(data, cpu); > + > /* we need to make sure that the underlying kobj is actually > * not referenced anymore by anybody before we proceed with > * unloading. > @@ -1894,6 +2065,8 @@ int cpufreq_register_driver(struct cpufreq_driver *driver_data) > cpufreq_driver = driver_data; > write_unlock_irqrestore(&cpufreq_driver_lock, flags); > > + cpufreq_create_debugfs(); Why you moved this to register_driver? It was fine at cpufreq_core_init() > ret = subsys_interface_register(&cpufreq_interface); > if (ret) > goto err_null_driver; > @@ -1918,12 +2091,14 @@ int cpufreq_register_driver(struct cpufreq_driver *driver_data) > } > > register_hotcpu_notifier(&cpufreq_cpu_notifier); > + unrelated change. > pr_debug("driver %s up and running\n", driver_data->name); > > return 0; > err_if_unreg: > subsys_interface_unregister(&cpufreq_interface); > err_null_driver: > + cpufreq_remove_debugfs(); > write_lock_irqsave(&cpufreq_driver_lock, flags); > cpufreq_driver = NULL; > write_unlock_irqrestore(&cpufreq_driver_lock, flags); > @@ -1949,6 +2124,8 @@ int cpufreq_unregister_driver(struct cpufreq_driver *driver) > > pr_debug("unregistering driver %s\n", driver->name); > > + cpufreq_remove_debugfs(); And so you don't need this. > subsys_interface_unregister(&cpufreq_interface); > unregister_hotcpu_notifier(&cpufreq_cpu_notifier); > > diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h > index 037d36a..825f379 100644 > --- a/include/linux/cpufreq.h > +++ b/include/linux/cpufreq.h > @@ -115,6 +115,7 @@ struct cpufreq_policy { > > struct kobject kobj; > struct completion kobj_unregister; > + struct dentry **cpu_debugfs; > }; > > #define CPUFREQ_ADJUST (0) > -- > 1.8.0 >

10 years, 9 months

[PATCH 00/11] USB: OHCI:Properly handle ohci_suspend()routine in bus glue

by Manjunath Goudar

Suspend scenario in case of ohci bus glue was not properly handled as it was not suspending generic part of ohci controller. Calling explicitly the ohci_suspend()routine will ensure proper handling of suspend scenario. V2: -Incase ohci_suspend() fails, return right away without executing further. V3: -New patch 1/11 added, for generic ohci-hcd suspend code. V4: -Properly aligned "do_wakeup" and "ret" variables. Manjunath Goudar (11): USB: OHCI: Properly handle OHCI controller suspend USB: OHCI: Properly handle ohci-at91 suspend USB: OHCI: Properly handle ohci-s3c2410 suspend USB: OHCI: Properly handle ohci-da8xx suspend USB: OHCI: Properly handle ohci-ep93xx suspend USB: OHCI: Properly handle ohci-exynos suspend USB: OHCI: Properly handle ohci-omap suspend USB: OHCI: Properly handle ohci-platform suspend USB: OHCI: Properly handle ohci-pxa27x suspend USB: OHCI: Properly handle ohci-sm501 suspend USB: OHCI: Properly handle ohci-spear suspend drivers/usb/host/ohci-at91.c | 10 ++++++++-- drivers/usb/host/ohci-da8xx.c | 15 +++++++++++---- drivers/usb/host/ohci-ep93xx.c | 9 ++++++++- drivers/usb/host/ohci-exynos.c | 20 +++++--------------- drivers/usb/host/ohci-hcd.c | 9 ++++++++- drivers/usb/host/ohci-omap.c | 13 ++++++++++--- drivers/usb/host/ohci-platform.c | 9 ++++++++- drivers/usb/host/ohci-pxa27x.c | 8 +++++++- drivers/usb/host/ohci-s3c2410.c | 19 ++++--------------- drivers/usb/host/ohci-sm501.c | 11 +++++++++-- drivers/usb/host/ohci-spear.c | 12 +++++++++--- 11 files changed, 87 insertions(+), 48 deletions(-) -- 1.7.9.5

10 years, 9 months

[PATCH 1/4] ARM: dts: Hook up IRQ for PMIC on Arndale

by Mark Brown

From: Mark Brown <broonie(a)linaro.org> The out of tree code configures a pullup on the line indicating that it is an active low interrupt. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- arch/arm/boot/dts/exynos5250-arndale.dts | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/boot/dts/exynos5250-arndale.dts b/arch/arm/boot/dts/exynos5250-arndale.dts index abc7272..dab40ae 100644 --- a/arch/arm/boot/dts/exynos5250-arndale.dts +++ b/arch/arm/boot/dts/exynos5250-arndale.dts @@ -11,6 +11,7 @@ /dts-v1/; #include "exynos5250.dtsi" +#include <dt-bindings/interrupt-controller/irq.h> / { model = "Insignal Arndale evaluation board based on EXYNOS5250"; @@ -37,6 +38,8 @@ s5m8767_pmic@66 { compatible = "samsung,s5m8767-pmic"; reg = <0x66>; + interrupt-parent = <&gpx3>; + interrupts = <2 IRQ_TYPE_LEVEL_LOW>; s5m8767,pmic-buck2-dvs-voltage = <1300000>; s5m8767,pmic-buck3-dvs-voltage = <1100000>; -- 1.8.3.2

10 years, 9 months

[PATCH] regulator: ab8500: Provide map_voltage() functions

by Mark Brown

From: Mark Brown <broonie(a)linaro.org> Since all the voltage tables for ab8500 are sorted in ascending order we can improve performance by using regulator_map_voltage_ascend(). A couple of the regulators can actually use linear maps which would be even better. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- drivers/regulator/ab8500.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/drivers/regulator/ab8500.c b/drivers/regulator/ab8500.c index 603f192..43cb354 100644 --- a/drivers/regulator/ab8500.c +++ b/drivers/regulator/ab8500.c @@ -651,6 +651,7 @@ static struct regulator_ops ab8500_regulator_volt_mode_ops = { .get_voltage_sel = ab8500_regulator_get_voltage_sel, .set_voltage_sel = ab8500_regulator_set_voltage_sel, .list_voltage = regulator_list_voltage_table, + .map_voltage = regulator_map_voltage_ascend, }; static struct regulator_ops ab8540_aux3_regulator_volt_mode_ops = { @@ -663,6 +664,7 @@ static struct regulator_ops ab8540_aux3_regulator_volt_mode_ops = { .get_voltage_sel = ab8540_aux3_regulator_get_voltage_sel, .set_voltage_sel = ab8540_aux3_regulator_set_voltage_sel, .list_voltage = regulator_list_voltage_table, + .map_voltage = regulator_map_voltage_ascend, }; static struct regulator_ops ab8500_regulator_volt_ops = { @@ -672,6 +674,7 @@ static struct regulator_ops ab8500_regulator_volt_ops = { .get_voltage_sel = ab8500_regulator_get_voltage_sel, .set_voltage_sel = ab8500_regulator_set_voltage_sel, .list_voltage = regulator_list_voltage_table, + .map_voltage = regulator_map_voltage_ascend, }; static struct regulator_ops ab8500_regulator_mode_ops = { @@ -682,6 +685,7 @@ static struct regulator_ops ab8500_regulator_mode_ops = { .set_mode = ab8500_regulator_set_mode, .get_mode = ab8500_regulator_get_mode, .list_voltage = regulator_list_voltage_table, + .map_voltage = regulator_map_voltage_ascend, }; static struct regulator_ops ab8500_regulator_ops = { @@ -689,6 +693,7 @@ static struct regulator_ops ab8500_regulator_ops = { .disable = ab8500_regulator_disable, .is_enabled = ab8500_regulator_is_enabled, .list_voltage = regulator_list_voltage_table, + .map_voltage = regulator_map_voltage_ascend, }; static struct regulator_ops ab8500_regulator_anamic_mode_ops = { @@ -698,6 +703,7 @@ static struct regulator_ops ab8500_regulator_anamic_mode_ops = { .set_mode = ab8500_regulator_set_mode, .get_mode = ab8500_regulator_get_mode, .list_voltage = regulator_list_voltage_table, + .map_voltage = regulator_map_voltage_ascend, }; /* AB8500 regulator information */ -- 1.8.3.1

10 years, 9 months

[PATCH] ASoC: samsung: Enable COMPILE_TEST builds

by Mark Brown

From: Mark Brown <broonie(a)linaro.org> Platform using OF and some others don't have any dependencies on arch/arm so can be built on any platform. Enable that for better compile coverage when COMPILE_TEST is enabled. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/samsung/Kconfig | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/sound/soc/samsung/Kconfig b/sound/soc/samsung/Kconfig index 9855dfc..df28dc8 100644 --- a/sound/soc/samsung/Kconfig +++ b/sound/soc/samsung/Kconfig @@ -1,6 +1,6 @@ config SND_SOC_SAMSUNG tristate "ASoC support for Samsung" - depends on PLAT_SAMSUNG + depends on PLAT_SAMSUNG || COMPILE_TEST select S3C64XX_DMA if ARCH_S3C64XX select S3C2410_DMA if ARCH_S3C24XX help @@ -150,7 +150,7 @@ config SND_SOC_SMARTQ config SND_SOC_GONI_AQUILA_WM8994 tristate "SoC I2S Audio support for AQUILA/GONI - WM8994" - depends on SND_SOC_SAMSUNG && (MACH_GONI || MACH_AQUILA) + depends on SND_SOC_SAMSUNG && (MACH_GONI || MACH_AQUILA || COMPILE_TEST) depends on I2C=y && GENERIC_HARDIRQS select SND_SAMSUNG_I2S select MFD_WM8994 @@ -186,7 +186,7 @@ config SND_SOC_SMDK_WM8994_PCM config SND_SOC_SPEYSIDE tristate "Audio support for Wolfson Speyside" - depends on SND_SOC_SAMSUNG && MACH_WLF_CRAGG_6410 + depends on SND_SOC_SAMSUNG && (MACH_WLF_CRAGG_6410 || COMPILE_TEST) select SND_SAMSUNG_I2S select SND_SOC_WM8996 select SND_SOC_WM9081 @@ -195,13 +195,13 @@ config SND_SOC_SPEYSIDE config SND_SOC_TOBERMORY tristate "Audio support for Wolfson Tobermory" - depends on SND_SOC_SAMSUNG && MACH_WLF_CRAGG_6410 + depends on SND_SOC_SAMSUNG && (MACH_WLF_CRAGG_6410 || COMPILE_TEST) select SND_SAMSUNG_I2S select SND_SOC_WM8962 config SND_SOC_BELLS tristate "Audio support for Wolfson Bells" - depends on SND_SOC_SAMSUNG && MACH_WLF_CRAGG_6410 + depends on SND_SOC_SAMSUNG && (MACH_WLF_CRAGG_6410 || COMPILE_TEST) select SND_SAMSUNG_I2S select SND_SOC_WM5102 select SND_SOC_WM5110 @@ -211,14 +211,14 @@ config SND_SOC_BELLS config SND_SOC_LOWLAND tristate "Audio support for Wolfson Lowland" - depends on SND_SOC_SAMSUNG && MACH_WLF_CRAGG_6410 + depends on SND_SOC_SAMSUNG && (MACH_WLF_CRAGG_6410 || COMPILE_TEST) select SND_SAMSUNG_I2S select SND_SOC_WM5100 select SND_SOC_WM9081 config SND_SOC_LITTLEMILL tristate "Audio support for Wolfson Littlemill" - depends on SND_SOC_SAMSUNG && MACH_WLF_CRAGG_6410 + depends on SND_SOC_SAMSUNG && (MACH_WLF_CRAGG_6410 || COMPILE_TEST) select SND_SAMSUNG_I2S select MFD_WM8994 select SND_SOC_WM8994 -- 1.8.3.2

10 years, 9 months

[PATCH] ASoC: mxs: Depends on COMMON_CLK

by Mark Brown

From: Mark Brown <broonie(a)linaro.org> The SAIF driver is a clock provider so specifically needs the common clock implementedation. Reported-by: Fengguang Wu <fengguang.wu(a)intel.com> Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/mxs/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/sound/soc/mxs/Kconfig b/sound/soc/mxs/Kconfig index 7daf860..219235c 100644 --- a/sound/soc/mxs/Kconfig +++ b/sound/soc/mxs/Kconfig @@ -1,6 +1,7 @@ menuconfig SND_MXS_SOC tristate "SoC Audio for Freescale MXS CPUs" depends on ARCH_MXS || COMPILE_TEST + depends on COMMON_CLK select SND_SOC_GENERIC_DMAENGINE_PCM help Say Y or M if you want to add support for codecs attached to -- 1.8.3.2

10 years, 9 months

[PATCH] ASoC: mxs: Remove unneeded mach-types.h inclusions

by Mark Brown

From: Mark Brown <broonie(a)linaro.org> Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/mxs/mxs-saif.c | 1 - sound/soc/mxs/mxs-sgtl5000.c | 1 - 2 files changed, 2 deletions(-) diff --git a/sound/soc/mxs/mxs-saif.c b/sound/soc/mxs/mxs-saif.c index 54511c5..b56b8a0 100644 --- a/sound/soc/mxs/mxs-saif.c +++ b/sound/soc/mxs/mxs-saif.c @@ -31,7 +31,6 @@ #include <sound/pcm.h> #include <sound/pcm_params.h> #include <sound/soc.h> -#include <asm/mach-types.h> #include "mxs-saif.h" diff --git a/sound/soc/mxs/mxs-sgtl5000.c b/sound/soc/mxs/mxs-sgtl5000.c index 1b134d7..b2e372d 100644 --- a/sound/soc/mxs/mxs-sgtl5000.c +++ b/sound/soc/mxs/mxs-sgtl5000.c @@ -25,7 +25,6 @@ #include <sound/soc.h> #include <sound/jack.h> #include <sound/soc-dapm.h> -#include <asm/mach-types.h> #include "../codecs/sgtl5000.h" #include "mxs-saif.h" -- 1.8.3.2

10 years, 9 months

[PATCH V3 0/6] USB: OHCI: more bus glues as separate modules

by Manjunath Goudar

These patches are for separating the SOC On-Chip ohci host controller from ohci-hcd host code into its own driver module. This work is part of enabling multi-platform kernels on ARM; it would be nice to have in 3.11. V2: In patch 5/6 and 6/6: -Set non-standard fields in hc_driver manually, rather than relying on an expanded struct ohci_driver_overrides. -Save orig_ohci_hub_control and orig_ohci_hub_status_data rather than relying on ohci_hub_control and hub_status_data being exported. In patch 1/6 to 4/6 -ohci_setup() has been removed because it is called in .reset member of the ohci_hc_driver structure. V3: In patch 5/6 and 6/6: -ohci_setup() has been removed because it is called in .reset member of the ohci_hc_driver structure. In patch 5/6: -The ohci_restart() function is not required in current scenario, only discarding connection state of integrated transceivers is sufficient, for this directly handling ohci->hc_control. In patch 2/6 : -rewritten if (config->otg || config->rwc) block statements into two separate 'if blocks' to handle below scenarios 1. config->otg set scenario. 2. if any of these (config->otg, config->rwc) are set, this scenario should be handled only after ohci_setup() In patch 1/6 and 4/6: No change. Manjunath Goudar (6): USB: OHCI: make ohci-exynos a separate driver USB: OHCI: make ohci-omap a separate driver USB: OHCI: make ohci-omap3 a separate driver USB: OHCI: make ohci-spear a separate driver USB: OHCI: make ohci-at91 a separate driver USB: OHCI: make ohci-s3c2410 a separate driver drivers/usb/host/Kconfig | 30 ++++++- drivers/usb/host/Makefile | 6 ++ drivers/usb/host/ohci-at91.c | 151 ++++++++++++++++------------------- drivers/usb/host/ohci-exynos.c | 167 ++++++++++++++++----------------------- drivers/usb/host/ohci-hcd.c | 108 ------------------------- drivers/usb/host/ohci-omap.c | 157 +++++++++++++----------------------- drivers/usb/host/ohci-omap3.c | 113 ++++++++------------------ drivers/usb/host/ohci-s3c2410.c | 128 +++++++++++++----------------- drivers/usb/host/ohci-spear.c | 140 +++++++++++++------------------- 9 files changed, 369 insertions(+), 631 deletions(-) -- 1.7.9.5

10 years, 9 months

[PATCH] ASoC: twl6040: Use power efficient workqueue

by Mark Brown

From: Mark Brown <broonie(a)linaro.org> The accessory detect debounce work is not performance sensitive so let the scheduler run it wherever is most efficient rather than in a per CPU workqueue by using the system power efficient workqueue. Signed-off-by: Mark Brown <broonie(a)linaro.org> --- sound/soc/codecs/twl6040.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/sound/soc/codecs/twl6040.c b/sound/soc/codecs/twl6040.c index 44621dd..caf8784 100644 --- a/sound/soc/codecs/twl6040.c +++ b/sound/soc/codecs/twl6040.c @@ -429,7 +429,8 @@ static irqreturn_t twl6040_audio_handler(int irq, void *data) struct snd_soc_codec *codec = data; struct twl6040_data *priv = snd_soc_codec_get_drvdata(codec); - schedule_delayed_work(&priv->hs_jack.work, msecs_to_jiffies(200)); + queue_delayed_work(system_power_efficient_wq, + &priv->hs_jack.work, msecs_to_jiffies(200)); return IRQ_HANDLED; } -- 1.8.3.2

10 years, 9 months

← Newer
1
2
3
4
5
6
7
8
9
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

linaro-kernel July 2013