linaro-dev

linaro-dev@lists.linaro.org

4332 discussions

[PATCH 0/2] Add v4l2 subdev driver for S5K4ECGX sensor with embedded SoC ISP

by Sangwook Lee

The following 2 patches add driver for S5K4ECGX sensor with embedded ISP SoC, and minor v4l2 control API enhancement. S5K4ECGX is 5M CMOS Image sensor from Samsung. Currenlty ony preview mode is supported. (no capture mode/face detection) Sangwook Lee (2): v4l: Add factory register values form S5K4ECGX sensor v4l: Add v4l2 subdev driver for S5K4ECGX sensor drivers/media/video/Kconfig | 7 + drivers/media/video/Makefile | 1 + drivers/media/video/s5k4ecgx.c | 871 ++++++++++ drivers/media/video/s5k4ecgx_regs.h | 3121 +++++++++++++++++++++++++++++++++++ include/media/s5k4ecgx.h | 29 + 5 files changed, 4029 insertions(+) create mode 100644 drivers/media/video/s5k4ecgx.c create mode 100644 drivers/media/video/s5k4ecgx_regs.h create mode 100644 include/media/s5k4ecgx.h -- 1.7.9.5

14 years

Agenda posted for the linaro-android team meeting on 2012/7/18

by Zach Pfeffer

https://wiki.linaro.org/Platform/Android/Meetings/2012-07-18 1300 UTC in #linaro-meeting Come and enjoy! -- Zach Pfeffer Android Platform Team Lead, Linaro Platform Teams Linaro.org | Open source software for ARM SoCs Follow Linaro: http://www.facebook.com/pages/Linaro http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog

14 years

[GIT PULL] big-LITTLE-MP-V3

by Viresh Kumar

Hi Andrey, V3 contains following updates: - per-cpu-thread-hotplug-v3 instead of v2 - All branches rebased on v3.5-rc7 They are compile tested. The following changes since commit 84a1caf1453c3d44050bd22db958af4a7f99315c: Linux 3.5-rc7 (2012-07-14 15:40:28 -0700) are available in the git repository at: git://git.linaro.org/arm/big.LITTLE/mp.git big-LITTLE-MP-v3 for you to fetch changes up to dc56b26b3ea05693552b85253db6bfdd62c8f763: Merge branches 'arm-asymmetric-support-v3', 'cpuidle-next-v4', 'per-cpu-thread-hotplug-v3', 'task-placement-v1' and 'config-fragments' into big-LITTLE-MP-v3 (2012-07-16 16:10:14 +0100) ---------------------------------------------------------------- Ben Segall (1): sched: maintain per-rq runnable averages Colin Cross (4): cpuidle: refactor out cpuidle_enter_state cpuidle: fix error handling in __cpuidle_register_device cpuidle: add support for states that affect multiple cpus cpuidle: coupled: add parallel barrier function Morten Rasmussen (5): sched: Add ftrace events for entity load-tracking sched: entity load-tracking load_avg_ratio sched: load-tracking driven wakeup migration for HMP platforms sched: Forced migration of high load task on HMP platforms sched: Add HMP forced task migration ftrace event Paul E. McKenney (1): rcu: Use smp_hotplug_thread facility for RCUs per-CPU kthread Paul Turner (15): sched: track the runnable average on a per-task entitiy basis sched: aggregate load contributed by task entities on parenting cfs_rq sched: maintain the load contribution of blocked entities sched: add an rq migration call-back to sched_class sched: account for blocked load waking back up sched: aggregate total task_group load sched: compute load contribution by a group entity sched: normalize tg load contributions against runnable time sched: maintain runnable averages across throttled periods sched: replace update_shares weight distribution with per-entity computation sched: refactor update_shares_cpu() -> update_blocked_avgs() sched: update_cfs_shares at period edge sched: make __update_entity_runnable_avg() fast sched: implement usage tracking sched: introduce temporary FAIR_GROUP_SCHED dependency for load-tracking Peter Zijlstra (1): sched, x86: Remove broken power estimation Thomas Gleixner (6): rcu: Yield simpler kthread: Implement park/unpark facility smpboot: Provide infrastructure for percpu hotplug threads softirq: Use hotplug thread infrastructure watchdog: Use hotplug thread infrastructure infiniband: ehca: Use hotplug thread infrastructure Vincent Guittot (4): ARM: topology: Add arch_scale_freq_power function ARM: topology: factorize the update of sibling masks ARM: topology: Update cpu_power according to DT information sched: cpu_power: enable ARCH_POWER Viresh Kumar (1): Merge branches 'arm-asymmetric-support-v3', 'cpuidle-next-v4', 'per-cpu-thread-hotplug-v3', 'task-placement-v1' and 'config-fragments' into big-LITTLE-MP-v3 arch/arm/Kconfig | 29 + arch/arm/kernel/topology.c | 209 +++- arch/x86/kernel/cpu/Makefile | 2 +- arch/x86/kernel/cpu/sched.c | 55 - drivers/cpuidle/Kconfig | 3 + drivers/cpuidle/Makefile | 1 + drivers/cpuidle/coupled.c | 715 +++++++++++ drivers/cpuidle/cpuidle.c | 68 +- drivers/cpuidle/cpuidle.h | 32 + drivers/infiniband/hw/ehca/ehca_irq.c | 253 ++-- drivers/infiniband/hw/ehca/ehca_irq.h | 6 +- include/linux/cpuidle.h | 11 + include/linux/kthread.h | 11 +- include/linux/sched.h | 19 + include/linux/smpboot.h | 43 + include/trace/events/sched.h | 151 +++ kernel/cpu.c | 10 +- kernel/kthread.c | 185 ++- kernel/rcutree.c | 12 +- kernel/rcutree.h | 15 +- kernel/rcutree_plugin.h | 403 ++----- kernel/rcutree_trace.c | 3 +- kernel/sched/core.c | 5 + kernel/sched/debug.c | 39 +- kernel/sched/fair.c | 1105 ++++++++++++++--- kernel/sched/features.h | 2 +- kernel/sched/sched.h | 60 +- kernel/smpboot.c | 229 ++++ kernel/smpboot.h | 4 + kernel/softirq.c | 107 +- kernel/watchdog.c | 263 ++-- linaro/configs/android.conf | 30 + linaro/configs/big-LITTLE-MP.conf | 9 + linaro/configs/linaro-base.conf | 84 ++ linaro/configs/ubuntu-minimal.conf | 24 + linaro/configs/ubuntu.conf | 2133 +++++++++++++++++++++++++++++++++ 36 files changed, 5244 insertions(+), 1086 deletions(-) delete mode 100644 arch/x86/kernel/cpu/sched.c create mode 100644 drivers/cpuidle/coupled.c create mode 100644 include/linux/smpboot.h create mode 100644 linaro/configs/android.conf create mode 100644 linaro/configs/big-LITTLE-MP.conf create mode 100644 linaro/configs/linaro-base.conf create mode 100644 linaro/configs/ubuntu-minimal.conf create mode 100644 linaro/configs/ubuntu.conf

14 years, 1 month

Gator fix for 3.5 kernels

by Jon Medhurst (Tixy)

We now have a fix for Gator to stop it deadlocking the system on 3.5 kernels [0]. This fix will find it's way into linux-linaro-core-tracking (llct) when Andrey next updates that with the ARM LT topics. For people managing 3.5 kernels with in-tree Gator code they didn't via llct then then they need to manually re-pull our topic branch [1]. For Android builds which don't use an in-tree version of Gator, I have pushed the patch to the external git branch these Android builds use [2], so they will automatically pick up the fix. -- Tixy [0] http://git.linaro.org/gitweb?p=arm/ds5/gator.git;a=commit;h=2ccfda6ba471ba1… [1] http://git.linaro.org/gitweb?p=landing-teams/working/arm/kernel.git;a=short… [2] http://git.linaro.org/gitweb?p=arm/ds5/gator.git;a=shortlog;h=refs/heads/an…

14 years, 1 month

Help with testing "flash-kernel" for stable release update (SRU)

by David Cullen

Hello, I submitted a patch for "flash-kernel" to Ubuntu to solve this problem: > Kernel /boot/vmlinuz-3.4.0-1-linaro-lt-omap does not match your subarchitecture > omap4, therefore not writing it to flash. The problem has been reported as https://bugs.launchpad.net/ubuntu/+source/flash-kernel/+bug/1023952 I submitted a patch, but for the patch to be included in a "stable release update" (aka SRU), it needs to be tested on a BeagleBoard-xM. If you are running Ubuntu on a BeagleBoard-xM, will you help me test this patch? -- Thank you, David Cullen

14 years, 1 month

Re: [PATCH] mfd: Implement devicetree support for AB8500 Btemp

by Rajanikanth HV

> Date: Tue, 10 Jul 2012 18:20:13 +0200 > From: Lee Jones <lee.jones(a)linaro.org> > To: "Rajanikanth H.V" <rajanikanth.hv(a)stericsson.com> > Cc: STEricsson_nomadik_linux <STEricsson_nomadik_linux(a)list.st.com>, > linaro-dev(a)lists.linaro.org, linux-kernel(a)vger.kernel.org, > linux-arm-kernel(a)lists.infradead.org, patches(a)linaro.org > Subject: Re: [PATCH] mfd: Implement devicetree support for AB8500 > Btemp > Message-ID: <4FFC563D.2080807(a)linaro.org> > Content-Type: text/plain; charset=UTF-8; format=flowed > > Some of my comments are picky, but if it's going into Mainline, it > should at least look professional. > > You didn't CC the ST-Ericsson internal mailing list. > > Address is STEricsson_nomadik_linux(a)list.st.com > > I'll do it for now, but please do so on your next submission. > > On 10/07/12 15:23, Rajanikanth H.V wrote: >> From: "Rajanikanth H.V" <rajanikanth.hv(a)stericsson.com> >> >> This patch adds device tree support for >> battery temperature monitor driver >> >> Signed-off-by: Rajanikanth H.V <rajanikanth.hv(a)stericsson.com> >> --- >> .../bindings/power_supply/ab8500/btemp.txt | 54 ++ >> arch/arm/boot/dts/db8500.dtsi | 17 + >> arch/arm/mach-ux500/Makefile | 4 +- >> arch/arm/mach-ux500/board-mop500-bm.c | 532 ++++++++++++++++++++ >> arch/arm/mach-ux500/include/mach/board-mop500-bm.h | 24 + >> drivers/mfd/ab8500-core.c | 1 + >> drivers/power/Kconfig | 8 +- >> drivers/power/ab8500_btemp.c | 79 ++- >> include/linux/mfd/ab8500/pwmleds.h | 20 + >> 9 files changed, 722 insertions(+), 17 deletions(-) >> create mode 100644 Documentation/devicetree/bindings/power_supply/ab8500/btemp.txt >> create mode 100644 arch/arm/mach-ux500/board-mop500-bm.c >> create mode 100644 arch/arm/mach-ux500/include/mach/board-mop500-bm.h >> create mode 100644 include/linux/mfd/ab8500/pwmleds.h >> >> diff --git a/Documentation/devicetree/bindings/power_supply/ab8500/btemp.txt b/Documentation/devicetree/bindings/power_supply/ab8500/btemp.txt >> new file mode 100644 >> index 0000000..8543ed1 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/power_supply/ab8500/btemp.txt >> @@ -0,0 +1,54 @@ >> +AB8500 Battery Termperature Monitor Driver > > Spelling. > >> + >> +AB8500 is a mixed signal multimedia and power management >> +device comprising: power and energy management module, >> +WalliCharger and USB charger interface, audio, general >> +purpose ADC TVOut, clock management and SIM card Interface. > > Mixture of capitalised and otherwise device names. > >> + >> +Battery temperature monitoring support is part of 'energy >> +management module', the other components of this module >> +are: 'main and USB Combo charger' and fuel guage. > > Spelling. Mixed use of ''. > >> +The properties below describes the node for battery >> +temperature monitor driver. >> + >> +Required Properties: >> +- compatible = "stericsson,ab8500-btemp" >> + >> +interrupts: >> + Four battery temperature ranges are be defined >> + which results in interrupt events as: >> + - Btemp >> + - BtempLow >> + - BtempMedium >> + - BtempHigh >> + >> + >> +Supplied-to: >> + This shall be power supply class dependency where in the runtime battery >> + properties will be shared across fuel guage and charging algorithm driver. > > Spelling. > >> + >> +Thermister-interface: >> + 'btemp' and 'batctrl' are the pins interfaced for battery temperature >> + measurement, btemp is used when NTC(Negative Termperature coefficient) > > Spelling. > >> + resister is interfaced external to battery and batctrl is used when >> + NTC resister is internal to battery. >> + >> +example: >> +ab8500-btemp { >> + compatible = "stericsson,ab8500-btemp"; >> + >> + interrupt-names = >> + "BAT_CTRL_INDB", /* battery removal indicator */ >> + "BTEMP_LOW", /* Btemp < BtempLow, if battery temperature is lower than -10?C */ >> + "BTEMP_HIGH", /* BtempLow < Btemp < BtempMedium, >> + if battery temperature is between -10 and 0?C */ >> + "BTEMP_LOW_MEDIUM", /* BtempMedium < Btemp < BtempHigh, >> + if battery temperature is between 0?C and ?MaxTemp?.*/ >> + "BTEMP_MEDIUM_HIGH";/* Btemp > BtempHigh, >> + if battery temperature is higher than ?MaxTemp?. */ >> + >> + supplied-to = "ab8500_chargalg", "ab8500_fg"; >> + >> + thermister-internal-to-battery = <1>; >> +}; >> diff --git a/arch/arm/boot/dts/db8500.dtsi b/arch/arm/boot/dts/db8500.dtsi >> index 7279165..527fdc3 100644 >> --- a/arch/arm/boot/dts/db8500.dtsi >> +++ b/arch/arm/boot/dts/db8500.dtsi >> @@ -330,6 +330,23 @@ >> vddadc-supply = <&ab8500_ldo_tvout_reg>; >> }; >> >> + ab8500-btemp { >> + compatible = "stericsson,ab8500-btemp"; >> + interrupts = <20 0x04 >> + 80 0x04 >> + 81 0x04 >> + 82 0x04 >> + 83 0x04>; > > Odd tabbing. > >> + interrupt-names = "BAT_CTRL_INDB", >> + "BTEMP_LOW", >> + "BTEMP_HIGH", >> + "BTEMP_LOW_MEDIUM", >> + "BTEMP_MEDIUM_HIGH"; > > As above. > >> + supplied_to = "ab8500_chargalg", "ab8500_fg"; >> + num_supplicants = <2>; >> + battery_term_on_batctrl = <1>; >> + }; >> + >> ab8500-usb { >> compatible = "stericsson,ab8500-usb"; >> interrupts = < 90 0x4 >> diff --git a/arch/arm/mach-ux500/Makefile b/arch/arm/mach-ux500/Makefile >> index 026086f..b474917 100644 >> --- a/arch/arm/mach-ux500/Makefile >> +++ b/arch/arm/mach-ux500/Makefile >> @@ -12,6 +12,8 @@ obj-$(CONFIG_MACH_MOP500) += board-mop500.o board-mop500-sdi.o \ >> board-mop500-uib.o board-mop500-stuib.o \ >> board-mop500-u8500uib.o \ >> board-mop500-pins.o \ >> - board-mop500-msp.o >> + board-mop500-msp.o \ >> + board-mop500-bm.o >> + > > Unrelated line change. > >> obj-$(CONFIG_SMP) += platsmp.o headsmp.o >> obj-$(CONFIG_HOTPLUG_CPU) += hotplug.o > >> diff --git a/arch/arm/mach-ux500/board-mop500-bm.c b/arch/arm/mach-ux500/board-mop500-bm.c >> diff --git a/arch/arm/mach-ux500/include/mach/board-mop500-bm.h b/arch/arm/mach-ux500/include/mach/board-mop500-bm.h > > It would be better if you can find a way to not upstream these. > > I think this data would be better suited for an include header file. > >> diff --git a/drivers/mfd/ab8500-core.c b/drivers/mfd/ab8500-core.c >> index 738b9c5..402f630 100644 >> --- a/drivers/mfd/ab8500-core.c >> +++ b/drivers/mfd/ab8500-core.c >> @@ -1046,6 +1046,7 @@ static struct mfd_cell __devinitdata ab8500_bm_devs[] = { >> }, >> { >> .name = "ab8500-btemp", >> + .of_compatible = "stericsson,ab8500-btemp", >> .num_resources = ARRAY_SIZE(ab8500_btemp_resources), >> .resources = ab8500_btemp_resources, >> }, >> diff --git a/drivers/power/Kconfig b/drivers/power/Kconfig >> index e3a3b49..00dec0f 100644 >> --- a/drivers/power/Kconfig >> +++ b/drivers/power/Kconfig >> @@ -303,10 +303,10 @@ config AB8500_BM >> help >> Say Y to include support for AB5500 battery management. >> >> -config AB8500_BATTERY_THERM_ON_BATCTRL >> - bool "Thermistor connected on BATCTRL ADC" >> +config AB8500_9100_LI_ION_BATTERY >> + bool "Enable support of the 9100 Li-ion battery charging" >> depends on AB8500_BM >> help >> - Say Y to enable battery temperature measurements using >> - thermistor connected on BATCTRL ADC. >> + Say Y to enable support of the 9100 Li-ion battery charging. >> + > > Random formatting. > >> endif # POWER_SUPPLY >> diff --git a/drivers/power/ab8500_btemp.c b/drivers/power/ab8500_btemp.c >> index bba3cca..1272bba 100644 >> --- a/drivers/power/ab8500_btemp.c >> +++ b/drivers/power/ab8500_btemp.c >> @@ -16,6 +16,7 @@ >> #include <linux/interrupt.h> >> #include <linux/delay.h> >> #include <linux/slab.h> >> +#include <linux/of.h> >> #include <linux/platform_device.h> >> #include <linux/power_supply.h> >> #include <linux/completion.h> >> @@ -25,6 +26,7 @@ >> #include <linux/mfd/abx500/ab8500-bm.h> >> #include <linux/mfd/abx500/ab8500-gpadc.h> >> #include <linux/jiffies.h> >> +#include <mach/board-mop500-bm.h> >> >> #define VTVOUT_V 1800 >> >> @@ -964,11 +966,13 @@ static int __devinit ab8500_btemp_probe(struct platform_device *pdev) >> { >> int irq, i, ret = 0; >> u8 val; >> - struct abx500_bm_plat_data *plat_data = pdev->dev.platform_data; > > I already told you about this. your previous comment was: "No, it's meant to work with _both_ platform and Device Tree registation." Why is this required when platform specific information/file is removed? > >> + struct device_node *np = pdev->dev.of_node; >> struct ab8500_btemp *di; >> + u32 pval; >> + const char *sup_val; >> >> - if (!plat_data) { >> - dev_err(&pdev->dev, "No platform data\n"); > > And this. Check the last review I provided. > >> + if (!np) { >> + dev_err(&pdev->dev, "No DT node for btemp found\n"); >> return -EINVAL; >> } >> >> @@ -981,21 +985,64 @@ static int __devinit ab8500_btemp_probe(struct platform_device *pdev) >> di->parent = dev_get_drvdata(pdev->dev.parent); >> di->gpadc = ab8500_gpadc_get("ab8500-gpadc.0"); >> >> - /* get btemp specific platform data */ >> - di->pdata = plat_data->btemp; >> - if (!di->pdata) { >> - dev_err(di->dev, "no btemp platform data supplied\n"); > > We still need to support registering from platform code. > >> + di->pdata = >> + kzalloc(sizeof(struct abx500_btemp_platform_data), GFP_KERNEL); > > Use devm_kzalloc instead. > >> + if (di->pdata == NULL) { >> + ret = -ENOMEM; >> + goto free_device_info; >> + } >> + /* get battery specific platform data */ >> + ret = of_property_read_u32(np, "num_supplicants", &pval); >> + if (ret) { >> + dev_err(di->dev, "missing property num_supplicants\n"); >> + kfree(di->pdata); > > Then remove this line. > >> + ret = -EINVAL; >> + goto free_device_info; >> + } >> + di->pdata->num_supplicants = pval; >> + di->pdata->supplied_to = >> + kzalloc(di->pdata->num_supplicants * >> + sizeof(const char *), GFP_KERNEL); >> + if (di->pdata->supplied_to == NULL) { >> + kfree(di->pdata); >> ret = -EINVAL; >> goto free_device_info; >> } >> >> - /* get battery specific platform data */ >> - di->bat = plat_data->battery; > > Don't remove this, check for it. > >> + for (val = 0; val < di->pdata->num_supplicants; ++val) >> + if (of_property_read_string_index >> + (np, "supplied_to", val, &sup_val) == 0) >> + *(di->pdata->supplied_to + val) = (char *)sup_val; >> + else { >> + dev_err(di->dev, "insufficient number of supplied_to data found\n"); >> + kfree(di->pdata); >> + kfree(di->pdata->supplied_to); >> + ret = -EINVAL; >> + goto free_device_info; >> + } >> + >> + di->bat = kzalloc(sizeof(struct abx500_bm_data), GFP_KERNEL); >> if (!di->bat) { >> - dev_err(di->dev, "no battery platform data supplied\n"); >> - ret = -EINVAL; >> + kfree(di->pdata->supplied_to); >> + kfree(di->pdata); >> + ret = -ENOMEM; >> goto free_device_info; >> } >> + dev_dbg(di->dev, "getting battery information\n"); > > Is this really necessary? > >> + di->bat = &ab8500_bm_data; >> + ret = of_property_read_u32(np, "battery_term_on_batctrl", &pval); >> + if (ret) { >> + dev_err(di->dev, "missing property battery_term_on_batctrl\n"); >> + kfree(di->pdata->supplied_to); >> + kfree(di->pdata); >> + kfree(di->bat); >> + ret = -ENOMEM; >> + goto free_device_info; >> + } >> + if (pval == 0) { >> + di->bat->bat_type = bat_type_ext_thermister; >> + di->bat->adc_therm = ABx500_ADC_THERM_BATTEMP; >> + } >> >> /* BTEMP supply */ >> di->btemp_psy.name = "ab8500_btemp"; >> @@ -1008,7 +1055,6 @@ static int __devinit ab8500_btemp_probe(struct platform_device *pdev) >> di->btemp_psy.external_power_changed = >> ab8500_btemp_external_power_changed; >> >> - > > Unrelated change. > >> /* Create a work queue for the btemp */ >> di->btemp_wq = >> create_singlethread_workqueue("ab8500_btemp_wq"); >> @@ -1090,14 +1136,22 @@ free_irq: >> irq = platform_get_irq_byname(pdev, ab8500_btemp_irq[i].name); >> free_irq(irq, di); >> } >> + > > As above. > >> free_btemp_wq: >> destroy_workqueue(di->btemp_wq); >> + > > As above. > >> free_device_info: >> kfree(di); >> >> return ret; >> } >> >> +static const struct of_device_id ab8500_btemp_match[] = { >> + {.compatible = "stericsson,ab8500-btemp",}, > > Spacing is not consistent with previous submissions. > >> + {}, >> +}; >> + >> + >> static struct platform_driver ab8500_btemp_driver = { >> .probe = ab8500_btemp_probe, >> .remove = __devexit_p(ab8500_btemp_remove), >> @@ -1106,6 +1160,7 @@ static struct platform_driver ab8500_btemp_driver = { >> .driver = { >> .name = "ab8500-btemp", >> .owner = THIS_MODULE, >> + .of_match_table = ab8500_btemp_match, >> }, >> }; >> >> diff --git a/include/linux/mfd/ab8500/pwmleds.h b/include/linux/mfd/ab8500/pwmleds.h >> new file mode 100644 >> index 0000000..e316582 >> --- /dev/null >> +++ b/include/linux/mfd/ab8500/pwmleds.h >> @@ -0,0 +1,20 @@ >> +/* >> + * Copyright ST-Ericsson 2012. >> + * >> + * Author: Naga Radhesh <naga.radheshy(a)stericsson.com> >> + * Licensed under GPLv2. >> + */ >> +#ifndef _AB8500_PWMLED_H >> +#define _AB8500_PWMLED_H >> + >> +struct ab8500_led_pwm { >> + int pwm_id; >> + int blink_en; >> +}; >> + >> +struct ab8500_pwmled_platform_data { >> + int num_pwm; >> + struct ab8500_led_pwm *leds; >> +}; >> + >> +#endif >> > > > -- > Lee Jones > Linaro ST-Ericsson Landing Team Lead > M: +44 77 88 633 515 > Linaro.org ? Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog > > > > > > ------------------------------ > > _______________________________________________ > linaro-dev mailing list > linaro-dev(a)lists.linaro.org > http://lists.linaro.org/mailman/listinfo/linaro-dev > > > End of linaro-dev Digest, Vol 26, Issue 38 > ****************************************** >

14 years, 1 month

Re: [PATCH] mfd: Implement devicetree support for AB8500 Btemp

by Rajanikanth HV

> Date: Tue, 10 Jul 2012 14:12:01 +0000 > From: Arnd Bergmann <arnd(a)arndb.de> > To: linux-arm-kernel(a)lists.infradead.org > Cc: linaro-dev(a)lists.linaro.org, linux-kernel(a)vger.kernel.org, > "Rajanikanth H.V" <rajanikanth.hv(a)stericsson.com>, patches(a)linaro.org > Subject: Re: [PATCH] mfd: Implement devicetree support for AB8500 > Btemp > Message-ID: <201207101412.01561.arnd(a)arndb.de> > Content-Type: Text/Plain; charset="utf-8" > > On Tuesday 10 July 2012, Rajanikanth H.V wrote: > >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/power_supply/ab8500/btemp.txt >> @@ -0,0 +1,54 @@ >> +AB8500 Battery Termperature Monitor Driver >> + >> +AB8500 is a mixed signal multimedia and power management >> +device comprising: power and energy management module, >> +WalliCharger and USB charger interface, audio, general >> +purpose ADC TVOut, clock management and SIM card Interface. >> + >> +Battery temperature monitoring support is part of 'energy >> +management module', the other components of this module >> +are: 'main and USB Combo charger' and fuel guage. >> + >> +The properties below describes the node for battery >> +temperature monitor driver. >> + >> +Required Properties: >> +- compatible = "stericsson,ab8500-btemp" >> + >> +interrupts: >> + Four battery temperature ranges are be defined >> + which results in interrupt events as: >> + - Btemp >> + - BtempLow >> + - BtempMedium >> + - BtempHigh >> + > > These names do not match the five interrupts in the example or in the > code. When you provide an "interrupt-names" property you have to define > the exact strings that are permissible for them in the binding. > >> +Supplied-to: >> + This shall be power supply class dependency where in the runtime battery >> + properties will be shared across fuel guage and charging algorithm driver. > > I probably don't understand enough of this, but shouldn't the other devices > that are supplied by this have a reference to this node rather than doing > it this way around? Why use strings here instead of phandles? This is a logical binding w.r.t power supply event change across energy-management-module drivers where in runtime battery properties are shared along with uevent notification. ref: di->btemp_psy.external_power_changed = ab8500_btemp_external_power_changed; ref: ab8500_btemp.c Need for this property: btemp, fg and charger updates power-supply properties based on the events listed above. Event handler invokes power supply change notifier which in-turn invokes registered power supply class call-back based on the 'supplied_to' string. ref: power_supply_changed_work(..) ./drivers/power/power_supply_core.c In this case how to approach through phandle? > > You are also not listing some of the properties that are in the device > tree here, like the "interrupts" property itself. > >> diff --git a/arch/arm/mach-ux500/board-mop500-bm.c b/arch/arm/mach-ux500/board-mop500-bm.c >> new file mode 100644 >> index 0000000..3349ceb >> --- /dev/null >> +++ b/arch/arm/mach-ux500/board-mop500-bm.c > > Didn't we conclude that this file has no board-specific information in it? > Either explain why it's still here, or move it into the driver itself. > >> +/* >> + * Note that the batres_vs_temp table must be strictly sorted by falling >> + * temperature values to work. >> + */ >> +#ifdef CONFIG_AB8500_9100_LI_ION_BATTERY >> +#define BATRES 180 >> +#else >> +#define BATRES 300 >> +#endif > > I think I mentioned before that you need to get rid of the > CONFIG_AB8500_9100_LI_ION_BATTERY symbol. If you have exclusive > compile-time options, it is impossible to build a kernel that > runs on all system, so this has to be a run-time option. > > Arnd > > >

14 years, 1 month

Build failure for Omap

by Viresh Kumar

Hi, Sorry if I am missing something very obvious, I am building OMAP for the first time. I fetched latest kernel from: http://git.linaro.org/gitweb?p=kernel/linux-linaro-tracking.git;a=shortlog;… tried to do following: make ARCH=arm O=../panda omap2plus_defconfig (After this, OMAP 2/3/4 none is selected in .config) make ARCH=arm O=../panda uImage LD drivers/usb/built-in.o LD drivers/video/console/built-in.o LD drivers/video/built-in.o LD drivers/built-in.o LINK vmlinux LD vmlinux.o MODPOST vmlinux.o GEN .version CHK include/generated/compile.h UPD include/generated/compile.h CC init/version.o LD init/built-in.o arch/arm/mach-vexpress/built-in.o: In function `v2m_dt_timer_init': /root/viresh/work/linux/kernel/arch/arm/mach-vexpress/v2m.c:663: undefined reference to `arch_timer_broken_for_sched_clock' make[1]: *** [vmlinux] Error 1 make: *** [sub-make] Error 2 Something is wrong: why vexpress?? -- viresh

14 years, 1 month

[PATCH] pm-qa : Fix global variable with sigatomic_t

by Daniel Lezcano

Signed-off-by: Daniel Lezcano <daniel.lezcano(a)linaro.org> --- utils/cpucycle.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/utils/cpucycle.c b/utils/cpucycle.c index 5ad36bc..f896815 100644 --- a/utils/cpucycle.c +++ b/utils/cpucycle.c @@ -35,11 +35,11 @@ #include <sys/time.h> #include <sys/resource.h> -static bool intr; +volatile sig_atomic_t intr = 0; void sigalarm(int sig) { - intr = true; + intr = 1; } int main(int argc, char *argv[]) @@ -97,10 +97,10 @@ int main(int argc, char *argv[]) /* warmup */ alarm(1); - for (counter = 0, intr = false; !intr ; counter++); + for (counter = 0, intr = 0; !intr ; counter++); alarm(1); - for (counter = 0, intr = false; !intr ; counter++); + for (counter = 0, intr = 0; !intr ; counter++); printf("%ld\n", counter); -- 1.7.5.4

14 years, 1 month

Config check fails for linux-linaro-lt-omap-3.4-3.4.0

by David Cullen

Hello, linaro-dev, I am using the instructions in the "adjust kernel config" section at this link: > https://wiki.linaro.org/KenWerner/Sandbox/CreateCustomKernelDeb When I run # fakeroot debian/rules editconfigs I get the the following errors: > Running config-check for all configurations ... > > check-config: /tmp/tmp.Sv8FblIPLa/CONFIGS/armel-config.flavour.linaro-lt-omap: loading config > check-config: /home/work/linux-linaro-lt-omap-3.4-3.4.0/debian.linaro/config/enforce: loading checks > check-config: FAIL: value CONFIG_INIT_PASS_ALL_PARAMS y > check-config: 43/44 checks passed -- exit 1 > check-config: /tmp/tmp.Sv8FblIPLa/CONFIGS/armhf-config.flavour.linaro-lt-omap: loading config > check-config: /home/work/linux-linaro-lt-omap-3.4-3.4.0/debian.linaro/config/enforce: loading checks > check-config: FAIL: value CONFIG_INIT_PASS_ALL_PARAMS y > check-config: 43/44 checks passed -- exit 1 > > *** ERROR: 2 config-check failures detected I am using config-3.4.0-1-linaro-lt-omap as my initial .config. A search for INIT_PASS_ALL_PARAMS in the Kconfig files produces no results: find . -name Kconfig -exec grep -H INIT_PASS_ALL_PARAMS '{}' ';' When I extract the source to a temporary directory and search the "kernel_build" subdirectory (that John Rigby told me to delete), I find it in ./init/Kconfig:config INIT_PASS_ALL_PARAMS So, now I am really confused. Is the real source in the "kernel_build" subdirectory? -- Thank you, David Cullen

14 years, 1 month

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

linaro-dev