Status report in detail in
https://wiki.linaro.org/OfficeofCTO/WeeklyReport
Last weekly meeting:
https://wiki.linaro.org/OfficeofCTO/2011-11-29
Highlights:
- ARMHF (also tracked via https://launchpad.net/linaro-octo-armhf
Launchpad project)
+ Debian builds are now churning
+ Ubuntu ARMHF now happening - there were some problems unearthed (eg
https://bugs.launchpad.net/ubuntu/+source/python2.7/+bug/898172) but
moving on. Bugs which will appear will be tackled
+ Bug https://bugs.launchpad.net/offspring/+bug/892240 - offspring
does not currently support armhf. This will impede efforts to get armhf
benchmarks running on LAVA. The issue is being now tackled by infra, a
machine will be set to handle putting together *unsupported*
experimental armhf Debian images as a precursor to getting eventually
Ubuntu armhf builds supported by offspring.
- ARM Server: Preparation work is ongoing, for the roadmap and
requirements as well as initial planning for the first steps. A server
LEB will also be discussed from next week onwards.
- A number of patches sent already to deal with reducing ticks as a way
to reduce power consumption. Check patches from
http://patches.linaro.org/team/linaro-octo/
- Finally a short account of memory management:
+ CMA has seen a new version of the patches as result of the
discussion on Kernel Summit
+ there is ongoing work to enable testing of CMA via LAVA
+ getting dma-buf sharing improved based on the review comments, and
also pushing implementation of v4l2 as a user of dmabuf
+ we have a first implementation of dri2video for nouveau driver, and
also coming soon for omap
All questions, comments, are welcome,
--
Ilias Biris ilias.biris(a)linaro.org
Project Manager, Linaro
M: +358504839608, IRC: ibiris Skype: ilias_biris
Linaro.org│ Open source software for ARM SoCs
On Thu, Dec 01, 2011 at 03:31:38PM +0700, tknv wrote:
> I could not find that WERROR override code at my kernels...
>
> Sorry delay it, I did make V=1.
> There are three kinds test.
> All targets are ARM.
> All Makefiles(top of kernel tree)-KBUILD_CFLAGS are same.
> KBUILD_CFLAGS := -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs \
> -fno-strict-aliasing -fno-common \
> -Wno-format-security \
> -fno-delete-null-pointer-checks
>
> I think linaro Werror behaviour is different or I need to add FLAGS more.
>
> Compile CM-kernel with android SDK
> make -f scripts/Makefile.build obj=drivers/net/wireless/bcm4329
> rm -f drivers/net/wireless/bcm4329/built-in.o;
> /home/tknv/android/Oxygen/prebuil
> t/linux-x86/toolchain/arm-eabi-4.4.3/bin/arm-eabi-ar rcs
> drivers/net/wireless/bcm43 29/built-in.o
> /home/tknv/android/Oxygen/prebuilt/linux-x86/toolchain/arm-eabi-4.4.3/bin/arm-eab
> i-gcc -Wp,-MD,drivers/net/wireless/bcm4329/.dhd_linux.o.d
> -nostdinc -isystem /home
> /tknv/android/Oxygen/prebuilt/linux-x86/toolchain/arm-eabi-4.4.3/bin/../lib/gcc/arm
> -eabi/4.4.3/include
> -I/home/tknv/android/SuperSonic/kernels/CM-kernel/arch/arm/incl
> ude -Iinclude -include include/generated/autoconf.h
> -D__KERNEL__ -mlittle-endian - Iarch/arm/mach-msm/include
> -Wall -Wundef -Wstrict-prototypes -Wno-trigraphs -fno-st
> rict-aliasing -fno-common -Wno-format-security
> -fno-delete-null-pointer-checks -Os -marm -mabi=aapcs-linux
> -mno-thumb-interwork -funwind-tables -D__LINUX_ARM_ARCH__=7
> -march=armv7-a -msoft-float -Uarm -Wframe-larger-than=1024
> -fno-stack-protector -f omit-frame-pointer
> -Wdeclaration-after-statement -Wno-pointer-sign
> -fno-strict-over flow -fno-dwarf2-cfi-asm -fconserve-stack
> -DLINUX -DBCMDRIVER -DBCMDONGLEHOST -DDHD THREAD -DBCMWPA2
> -DUNRELEASEDCHIP -Dlinux -DDHD_SDALIGN=64 -DMAX_HDR_READ=64
> -DDHD_ FIRSTREAD=64 -DDHD_GPL -DDHD_SCHED -DBDC -DTOE
> -DDHD_BCMEVENTS -DSHOW_EVENTS -DBCMS DIO -DDHD_GPL
> -DBCMLXSDMMC -DBCMPLATFORM_BUS -Wall -Wstrict-prototypes
> -Werror -DOO B_INTR_ONLY -DCUSTOMER_HW2 -DDHD_USE_STATIC_BUF
> -DMMC_SDIO_ABORT -DDHD_DEBUG_TRAP - DSOFTAP
> -DEMBEDDED_PLATFORM -DARP_OFFLOAD_SUPPORT -DPKT_FILTER_SUPPORT
> -DGET_CUSTOM _MAC_ENABLE -DSET_RANDOM_MAC_SOFTAP -DHW_OOB
> -Idrivers/net/wireless/bcm4329 -Idrive
> rs/net/wireless/bcm4329/include -DMODULE -D"KBUILD_STR(s)=#s"
> -D"KBUILD_BASENAME=K BUILD_STR(dhd_linux)"
> -D"KBUILD_MODNAME=KBUILD_STR(bcm4329)" -c -o drivers/net/wi
> reless/bcm4329/dhd_linux.o
> drivers/net/wireless/bcm4329/dhd_linux.
> ...
> [1]+ Done make V=1 > build.log 2>&1
>
> Compile CM-kernel with linaro.
> make -f scripts/Makefile.build obj=drivers/net/wireless/bcm4329
> rm -f drivers/net/wireless/bcm4329/built-in.o;
> /usr/bin/arm-linux-gnueabi-ar rcs
> drivers/net/wireless/bcm4329/built-in.o
> /usr/bin/arm-linux-gnueabi-gcc
> -Wp,-MD,drivers/net/wireless/bcm4329/.dhd_linux.o.d -nostdinc
> -isystem /usr/lib/gcc/arm-linux-gnueabi/4.6.1/include
> -I/home/tknv/android/SuperSonic/kernels/CM-kernel/arch/arm/include
> -Iinclude -include include/generated/autoconf.h -D__KERNEL__
> -mlittle-endian -Iarch/arm/mach-msm/include -Wall -Wundef
> -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing -fno-common
> -Wno-format-security -fno-delete-null-pointer-checks -Os -marm
> -mabi=aapcs-linux -mno-thumb-interwork -funwind-tables
> -D__LINUX_ARM_ARCH__=7 -march=armv7-a -msoft-float -Uarm
> -Wframe-larger-than=1024 -fno-stack-protector -fomit-frame-pointer
> -Wdeclaration-after-statement -Wno-pointer-sign
> -fno-strict-overflow -fno-dwarf2-cfi-asm -fconserve-stack -DLINUX
> -DBCMDRIVER -DBCMDONGLEHOST -DDHDTHREAD -DBCMWPA2 -DUNRELEASEDCHIP
> -Dlinux -DDHD_SDALIGN=64 -DMAX_HDR_READ=64 -DDHD_FIRSTREAD=64
> -DDHD_GPL -DDHD_SCHED -DBDC -DTOE -DDHD_BCMEVENTS -DSHOW_EVENTS
> -DBCMSDIO -DDHD_GPL -DBCMLXSDMMC -DBCMPLATFORM_BUS -Wall
> -Wstrict-prototypes -Werror -DOOB_INTR_ONLY -DCUSTOMER_HW2
> -DDHD_USE_STATIC_BUF -DMMC_SDIO_ABORT -DDHD_DEBUG_TRAP -DSOFTAP
> -DEMBEDDED_PLATFORM -DARP_OFFLOAD_SUPPORT -DPKT_FILTER_SUPPORT
> -DGET_CUSTOM_MAC_ENABLE -DSET_RANDOM_MAC_SOFTAP -DHW_OOB
> -Idrivers/net/wireless/bcm4329
> -Idrivers/net/wireless/bcm4329/include -DMODULE
> -D"KBUILD_STR(s)=#s" -D"KBUILD_BASENAME=KBUILD_STR(dhd_linux)"
> -D"KBUILD_MODNAME=KBUILD_STR(bcm4329)" -c -o
> drivers/net/wireless/bcm4329/dhd_linux.o
> drivers/net/wireless/bcm4329/dhd_linux.c
> drivers/net/wireless/bcm4329/dhd_linux.c: In function ‘
> dhd_rx_frame’:
> drivers/net/wireless/bcm4329/dhd_linux.c:1282:24: error: variable
> ‘save_pktbuf’ set but not used [-Werror=unused-but-set-variable]
> drivers/net/wireless/bcm4329/dhd_linux.c: In function ‘
> dhd_os_wd_timer’:
> drivers/net/wireless/bcm4329/dhd_linux.c:2737:14: error: variable
> ‘save_dhd_watchdog_ms’ set but not used
> [-Werror=unused-but-set-variable]
> cc1: all warnings being treated as errors
>
> make[4]: *** [drivers/net/wireless/bcm4329/dhd_linux.o] Error 1
> make[3]: *** [drivers/net/wireless/bcm4329] Error 2
> make[2]: *** [drivers/net/wireless] Error 2
> make[1]: *** [drivers/net] Error 2
> make: *** [drivers] Error 2
> ^C[1]+ Exit 2 make V=1 > build.log 2>&1
>
> Compile tiamatat-kernel with linaro.
> make -f scripts/Makefile.build obj=drivers/net/wireless/bcm4329
> /usr/bin/arm-linux-gnueabi-gcc
> -Wp,-MD,drivers/net/wireless/bcm4329/.dhd_linux.o.d -nostdinc
> -isystem /usr/lib/gcc/arm-linux-gnueabi/4.6.1/include
> -I/home/tknv/android/SuperSonic/kernels/tiamat-kernel/arch/arm/include
> -Iinclude -include include/generated/autoconf.h -D__KERNEL__
> -mlittle-endian -Iarch/arm/mach-msm/include -Wall -Wundef
> -Wstrict-prototypes -Wno-trigraphs -fno-strict-aliasing -fno-common
> -Wno-format-security -fno-delete-null-pointer-checks -Os -marm
> -fno-dwarf2-cfi-asm -fno-omit-frame-pointer -mapcs -mno-sched-prolog
> -mabi=aapcs-linux -mno-thumb-interwork -D__LINUX_ARM_ARCH__=7
> -march=armv7-a -mhard-float -Uarm -Wframe-larger-than=1024
> -fno-stack-protector -fno-omit-frame-pointer
> -fno-optimize-sibling-calls -Wdeclaration-after-statement
> -Wno-pointer-sign -fno-strict-overflow -fconserve-stack
> -DCC_HAVE_ASM_GOTO -DLINUX -DBCMDRIVER -DBCMDONGLEHOST -DDHDTHREAD
> -DBCMWPA2 -DUNRELEASEDCHIP -Dlinux -DDHD_SDALIGN=64 -DMAX_HDR_READ=64
> -DDHD_FIRSTREAD=64 -DDHD_GPL -DDHD_SCHED -DBDC -DTOE -DDHD_BCMEVENTS
> -DSHOW_EVENTS -DBCMSDIO -DDHD_GPL -DBCMLXSDMMC -DBCMPLATFORM_BUS -Wall
> -Wstrict-prototypes -Werror -DOOB_INTR_ONLY -DCUSTOMER_HW2
> -DDHD_USE_STATIC_BUF -DMMC_SDIO_ABORT -DDHD_DEBUG_TRAP -DSOFTAP
> -DEMBEDDED_PLATFORM -DARP_OFFLOAD_SUPPORT -DPKT_FILTER_SUPPORT
> -DGET_CUSTOM_MAC_ENABLE -DSET_RANDOM_MAC_SOFTAP -DCSCAN -DHW_OOB
> -DKEEP_ALIVE -DPNO_SUPPORT -Idrivers/net/wireless/bcm4329
> -Idrivers/net/wireless/bcm4329/include -DMODULE -DMODULE
> -march=armv7-a -mfpu=vfpv3 -ftree-vectorize -D"KBUILD_STR(s)=#s"
> -D"KBUILD_BASENAME=KBUILD_STR(dhd_linux)"
> -D"KBUILD_MODNAME=KBUILD_STR(bcm4329)" -c -o
> drivers/net/wireless/bcm4329/dhd_linux.o
> drivers/net/wireless/bcm4329/dhd_linux.c
> drivers/net/wireless/bcm4329/dhd_linux.c: In function ‘dhd_rx_frame’
> :
> drivers/net/wireless/bcm4329/dhd_linux.c:1256:24: error: variable ‘
> save_pktbuf’ set but not used [-Werror=unused-but-set-variable]
> cc1: all warnings being treated as errors
>
> make[4]: *** [drivers/net/wireless/bcm4329/dhd_linux.o] Error 1
> make[3]: *** [drivers/net/wireless/bcm4329] Error 2
> make[2]: *** [drivers/net/wireless] Error 2
> make[1]: *** [drivers/net] Error 2
> make: *** [drivers] Error 2
> ^C[1]+ Exit 2 make V=1 > build.log 2>&1
OK, so the reason why you can't turn -Werror off is because
drivers/net/wireless/bcm4329/Makefile adds it again:
DHDCFLAGS = -DLINUX -DBCMDRIVER -DBCMDONGLEHOST -DDHDTHREAD -DBCMWPA2 \
-DUNRELEASEDCHIP -Dlinux -DDHD_SDALIGN=64 -DMAX_HDR_READ=64 \
-DDHD_FIRSTREAD=64 -DDHD_GPL -DDHD_SCHED -DBDC -DTOE -DDHD_BCMEVENTS \
-DSHOW_EVENTS -DBCMSDIO -DDHD_GPL -DBCMLXSDMMC -DBCMPLATFORM_BUS \
-Wall -Wstrict-prototypes -Werror -DOOB_INTR_ONLY -DCUSTOMER_HW2 \
-DDHD_USE_STATIC_BUF -DMMC_SDIO_ABORT -DDHD_DEBUG_TRAP -DSOFTAP \
-DEMBEDDED_PLATFORM -DARP_OFFLOAD_SUPPORT -DPKT_FILTER_SUPPORT \
-DGET_CUSTOM_MAC_ENABLE -DSET_RANDOM_MAC_SOFTAP -DCSCAN -DHW_OOB \
-DKEEP_ALIVE -DPNO_SUPPORT \
-Idrivers/net/wireless/bcm4329 -Idrivers/net/wireless/bcm4329/include
EXTRA_CFLAGS = $(DHDCFLAGS)
Kbuild doesn't allow you to override such things from the top level.
Adding -Werror overrides any preceding -Wno-error flag.
-Werror really should only be added in subdirectory Makefiles if
the code in that directory is super-clean. The exact behaviour of GCC
with -Werror *will* change between different compiler versions -- this
is not a bug. It explicitly allows GCC to fail to compile code which
might be correct.
There seems to be no good way to override -Werror in a subdirectory
Makefile from higher-level Makefiles. You may be able to remove the
error behaviour for individual warnings with -Wno-error=<warning>, but
it's not really a good idea to do this at the top level because it will
affect the whole kernel.
So far as I can see, the warnings/errors you get probably indicate real
issues in the code. It doesn't look like a compiler bug, or a problem
with Kbuild -- it's just an effect of the way the writers of that
subdirectory wrote their Makefile.
If you have not already done so, you should check for newer versions of
the code which may fix the problems. If there is no newer version, it
would be a good idea to fix the source code issues and propose those
changes for merging so that the problem gets solved for everyone.
You can also simply remove -Werror from DHDCFLAGS in your local
build tree. This will allow you to build the kernel, but it won't
prevent other people from encountering the same problem.
Since this appears to be an issue with an Android kernel tree, I suggest
if you have more questions on this you should reply via
linaro-dev(a)lists.linaro.org, or ask questions on the #linaro or
#linaro-android channels on IRC at chat.freenode.net if you need
additional help.
Cheers
---Dave
Add pm_caps into platform data and update it in SD/MMC driver
This field can be used with SDIO device such as SDIO WLAN.
Sangwook Lee (2):
ARM: SAMSUNG: Add pm_caps into platform data
mmc: sdhci-s3c: Add pm_caps into SD/MMC host
arch/arm/plat-samsung/dev-hsmmc3.c | 2 ++
arch/arm/plat-samsung/include/plat/sdhci.h | 2 ++
drivers/mmc/host/sdhci-s3c.c | 3 +++
3 files changed, 7 insertions(+), 0 deletions(-)
--
1.7.4.1
Hello.
Here is the weekly status report from the Android Platform Team.
Key Points for wider discussion
===============================
* Android 4.0 ICS boots with gcc 4.6. Still some workarounds required.
* Android 4.0 ICS up and running on iMX53.
* 11.12 Pre-release 1 (ICS) has been validated on all major boards.
Team Highlights
===============================
* 11.11 Post Mortem status, meeting and lessons learned completed.
* Blueprints for 11.12 are ready and rolling.
* Demos and articles posted to YouTube, G+ and other sites.
* Support questions answered.
* Good progress on adding Gator (for DS-5) to the ICS Builds.
* The kernel rebuild script has been tested for all boards.
* Good progress on bringing up hardware accelerated graphics on Origen.
Bugs fixed
===============================
871757, 883307, 892881, 892899, 893325, 893150, 893151, 894488, 896467,
896468, 844890, 862327
===============================
* Hackbox ran out of memory when building Android ICS.
Blueprints
===============================
https://launchpad.net/linaro-android/+milestone/11.12
Greetings,
Here is the summary for Linaro release 11.11 postmortem and lessons learned.
For a detailed release review please visit
https://wiki.linaro.org/Cycles/1111/Release/Review
Highlights and Key Successes
============================
This month's release was highlighted by the release from the Android team of
Ice Cream Sandwich images. Uploaded to youtube, the demonstration of these
images running on the Snowball platform gave Linaro some great visibility and
publicity.
http://www.youtube.com/watch?v=MQOKPLg3ARE
The Android LEB is now based off of Android 2.3.7 with a first release of
Versatile Express, using the Linaro ARM LT kernel, a NEON optimized libpng,
preliminary DS-5 support and a host of bug fixes and enhancements.
The Ubuntu LEB now officially supports the PandaBoard ES and features
PandaBoard's USB booting with U-Boot USB-SPL. Developers have access to
updated ARM DS-5 packages, and both source and debug packages for all
Linaro's kernels. Also, Firefox can now be cross-built using multi-
arch. Instructions are available from
http://wiki.linaro.org/Platform/DevPlatform/CrossCompile/FirefoxCrossCompile.
For more information on release highlights please see our release page:
https://wiki.linaro.org/Cycles/1111/Release
Postmortem and Lessons Learned
==============================
The overall sentiment from the development teams regarding this release is
positive. Even though the cycle was relatively short and included a first
week at Linaro Connect, planning went smoothly and the items delivered were
interesting and substantive.
Communication is an issue that has been re-occurring through the past few
release cycles. Platform and Landing teams have proposed a plan to facilitate
communications between the teams.
Testing is another issue that has high visibility. Testing internal
deliveries before hand off is crucial for delivering quality in the final
release and will minimize the number of times that images need to be
re-spun.
Teams are still adjusting to the monthly release cycle. Work estimating is
being fine-tuned so that the amount of effort per team per cycle is
appropriate. Progress has been made and most of the work scheduled for this
cycle was delivered, even with the addition of ICS.
There are still a couple of groups that have not entered postmortem comments
in the release review wiki page.
The issues have generated several lessons learned:
* A meeting to determine a communications plan was held and resulted in the
following recommendations:
* Schedule a meeting once a month to discuss platform/landing team
issues.
* Schedule a weekly meeting to discuss common bugs for landing teams and
platform teams has been scheduled.
* On the last Friday, Monday and Tuesday of the cycle there will a stand-up
meeting with platform and landing teams to discuss any last minute
issues.
--
David Zinman
Linaro Release Manager | Project Manager
Linaro.org | Open source software for ARM SoCs
All,
We just deleted the #linaro-mentors channel a few minutes ago. This
channel was not getting much traffic and we felt that there are better
ways to get help including:
* asking on #linaro
* asking on this mailing list
* using ask.linaro.org
-andy
On Thu, Nov 17, 2011 at 6:13 PM, Donggeun Kim <dg77.kim(a)samsung.com> wrote:
> Because battery health monitoring should be done even when suspended,
> it needs to wake up and suspend periodically. Thus, userspace battery
> monitoring may incur too much overhead; every device and task is woken
> up periodically. Charger Manager uses suspend-again to provide
> in-suspend monitoring.
>
> This patch allows to monitor battery health in-suspend state.
>
> Signed-off-by: Donggeun Kim <dg77.kim(a)samsung.com>
> Signed-off-by: MyungJoo Ham <myungjoo.ham(a)samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park(a)samsung.com>
> ---
> Documentation/power/charger-manager.txt | 150 ++++++
> drivers/power/Kconfig | 9 +
> drivers/power/Makefile | 1 +
> drivers/power/charger-manager.c | 771 +++++++++++++++++++++++++++++++
> include/linux/power/charger-manager.h | 131 ++++++
> 5 files changed, 1062 insertions(+), 0 deletions(-)
> create mode 100644 Documentation/power/charger-manager.txt
> create mode 100644 drivers/power/charger-manager.c
> create mode 100644 include/linux/power/charger-manager.h
>
> diff --git a/Documentation/power/charger-manager.txt b/Documentation/power/charger-manager.txt
> new file mode 100644
> index 0000000..99630e5
> --- /dev/null
> +++ b/Documentation/power/charger-manager.txt
> @@ -0,0 +1,150 @@
> +Charger Manager
> + (C) 2011 MyungJoo Ham <myungjoo.ham(a)samsung.com>, GPL
> +
> +Charger Manager provides in-kernel battery charger management that
> +requires temperature monitoring during suspend-to-RAM state
> +and where each battery may have multiple chargers attached and the userland
> +wants to look at the aggregated information of the multiple chargers.
> +
> +Charger Manager is a platform_driver with power-supply-class entries.
> +An instance of Charger Manager (a platform-device created with Charger-Manager)
> +represents an independent battery with chargers. If there are multiple
> +batteries with their own chargers acting independently in a system,
> +the system may need multiple instances of Charger Manager.
> +
> +1. Introduction
> +===============
> +
> +Charger Manager supports the following:
> +
> +* Support for multiple chargers (e.g., a device with USB, AC, and solar panels)
> + A system may have multiple chargers (or power sources) and some of
> + they may be activated at the same time. Each charger may have its
> + own power-supply-class and each power-supply-class can provide
> + different information about the battery status. This framework
> + aggregates charger-related information from multiple sources and
> + shows combined information as a single power-supply-class.
> +
> +* Support for in suspend-to-RAM polling (with suspend_again callback)
> + While the battery is being charged and the system is in suspend-to-RAM,
> + we may need to monitor the battery health by looking at the ambient or
> + battery temperature. We can accomplish this by waking up the system
> + periodically. However, such a method wakes up devices unncessary for
> + monitoring the battery health and tasks, and user processes that are
> + supposed to be kept suspended. That, in turn, incurs unnecessary power
> + consumption and slow down charging process. Or even, such peak power
> + consumption can stop chargers in the middle of charging
> + (external power input < device power consumption), which not
> + only affects the charging time, but the lifespan of the battery.
> +
> + Charger Manager provides a function "cm_suspend_again" that can be
> + used as suspend_again callback of platform_suspend_ops. If the platform
> + requires tasks other than cm_suspend_again, it may implement its own
> + suspend_again callback that calls cm_suspend_again in the middle.
> + Normally, the platform will need to resume and suspend some devices
> + that are used by Charger Manager.
> +
> +2. Global Charger-Manager Data related with suspend_again
> +========================================================
> +In order to setup Charger Manager with suspend-again feature
> +(in-suspend monitoring), the user should provide charger_global_desc
> +with setup_charger_manager(struct charger_global_desc *).
> +This charger_global_desc data for in-suspend monitoring is global
> +as the name suggests. Thus, the user needs to provide only once even
> +if there are multiple batteries. If there are multiple batteries, the
> +multiple instances of Charger Manager share the same charger_global_desc
> +and it will manage in-suspend monitoring for all instances of Charger Manager.
> +
> +The user needs to provide all the three entries properly in order to activate
> +in-suspend monitoring:
> +
> +struct charger_global_desc {
> +
> +char *rtc;
> + : The name of rtc (e.g., "rtc0") used to wakeup the system from
> + suspend for Charger Manager. The alarm interrupt (AIE) of the rtc
> + should be able to wake up the system from suspend. Charger Manager
> + saves and restores the alarm value and use the previously-defined
> + alarm if it is going to go off earlier than Charger Manager so that
> + Charger Manager does not interfere with previously-defined alarms.
> +
> +bool (*is_rtc_only_wakeup_reason)(void);
> + : This callback should let CM know whether
> + the wakeup-from-suspend is caused only by the alarm of "rtc" in the
> + same struct. If there is any other wakeup source triggered the
> + wakeup, it should return false. If the "rtc" is the only wakeup
> + reason, it should return true.
> +};
> +
> +3. How to setup suspend_again
> +=============================
> +Charger Manager provides a function "extern bool cm_suspend_again(void)".
> +When cm_suspend_again is called, it monitors every battery. The suspend_ops
> +callback of the system's platform_suspend_ops can call cm_suspend_again
> +function to know whether Charger Manager wants to suspend again or not.
> +If there are no other devices or tasks that want to use suspend_again
> +feature, the platform_suspend_ops may directly refer to cm_suspend_again
> +for its suspend_again callback.
> +
> +The cm_suspend_again() returns true (meaning "I want to suspend again")
> +if the system was woken up by Charger Manager and the polling
> +(in-suspend monitoring) results in "normal".
> +
> +4. Charger-Manager Data (struct charger_desc)
> +=============================================
> +For each battery charged independently from other batteries (if a series of
> +batteries are charged by a single charger, they are counted as one independent
> +battery), an instance of Charger Manager is attached to it.
> +
> +struct charger_desc {
> +
> +enum polling_modes polling_mode;
> + : CM_POLL_DISABLE: do not poll this battery.
> + CM_POLL_ALWAYS: always poll this battery.
> + CM_POLL_EXTERNAL_POWER_ONLY: poll this battery if and only if
> + an external power source is attached.
> + CM_POLL_CHARGING_ONLY: poll this battery if and only if the
> + battery is being charged.
> +
> +unsigned int polling_interval_ms;
> + : Required polling interval in ms. Charger Manager will poll
> + this battery every polling_interval_ms or more frequently.
> +
> +enum data_source battery_present;
> + CM_FUEL_GAUGE: get battery presence information from fuel gauge.
> + CM_CHARGER_STAT: get battery presence from chargers.
> +
> +char **psy_charger_stat;
> + : An array ending with NULL that has power-supply-class names of
> + chargers. Each power-supply-class should provide "PRESENT" (if
> + battery_present is "CM_CHARGER_STAT"), "ONLINE" (shows whether an
> + external power source is attached or not), and "STATUS" (shows whether
> + the battery is {"FULL" or not FULL} or {"FULL", "Charging",
> + "Discharging", "NotCharging"}).
> +
> +int num_charger_regulators;
> +struct regulator_bulk_data *charger_regulators;
> + : Regulators representing the chargers in the form for
> + regulator framework's bulk functions.
> +
> +char *psy_fuel_gauge;
> + : Power-supply-class name of the fuel gauge.
> +
> +int (*is_temperature_error)(int *mC);
> + : This callback returns 0 if the temperature is safe for charging,
> + a positive number if it is too hot to charge, and a negative number
> + if it is too cold to charge. With the variable mC, the callback returns
> + the temperature in 1/1000 of centigrade.
> +};
> +
> +5. Other Considerations
> +=======================
> +
> +At the charger/battery-related events such as battery-pulled-out,
> +charger-pulled-out, charger-inserted, DCIN-over/under-voltage, charger-stopped,
> +and others critical to chargers, the system should be configured to wake up.
> +At least the following should wake up the system from a suspend:
> +a) charger-on/off b) external-power-in/out c) battery-in/out (while charging)
> +
> +It is usually accomplished by configuring the PMIC as a wakeup source.
> +
> diff --git a/drivers/power/Kconfig b/drivers/power/Kconfig
> index 9f88641..eb96b40 100644
> --- a/drivers/power/Kconfig
> +++ b/drivers/power/Kconfig
> @@ -236,6 +236,15 @@ config CHARGER_GPIO
> This driver can be build as a module. If so, the module will be
> called gpio-charger.
>
> +config CHARGER_MANAGER
> + tristate "Battery charger manager for multiple chargers"
> + help
> + Say Y to enable charger-manager support, which allows multiple
> + chargers attached to a battery and multiple batteries attached to a
> + system. The charger-manager also can monitor charging status in
> + runtime and in suspend-to-RAM by waking up the system periodically
> + with help of suspend_again support.
> +
> config CHARGER_MAX8997
> tristate "Maxim MAX8997/MAX8966 PMIC battery charger driver"
> depends on MFD_MAX8997 && REGULATOR_MAX8997
> diff --git a/drivers/power/Makefile b/drivers/power/Makefile
> index b4af13d..d3b24e9 100644
> --- a/drivers/power/Makefile
> +++ b/drivers/power/Makefile
> @@ -36,5 +36,6 @@ obj-$(CONFIG_CHARGER_ISP1704) += isp1704_charger.o
> obj-$(CONFIG_CHARGER_MAX8903) += max8903_charger.o
> obj-$(CONFIG_CHARGER_TWL4030) += twl4030_charger.o
> obj-$(CONFIG_CHARGER_GPIO) += gpio-charger.o
> +obj-$(CONFIG_CHARGER_MANAGER) += charger-manager.o
> obj-$(CONFIG_CHARGER_MAX8997) += max8997_charger.o
> obj-$(CONFIG_CHARGER_MAX8998) += max8998_charger.o
> diff --git a/drivers/power/charger-manager.c b/drivers/power/charger-manager.c
> new file mode 100644
> index 0000000..5006756
> --- /dev/null
> +++ b/drivers/power/charger-manager.c
> @@ -0,0 +1,771 @@
> +/* linux/drivers/power/charger-manager.c
> + *
> + * Copyright (C) 2011 Samsung Electronics Co., Ltd.
> + * MyungJoo Ham <myungjoo.ham(a)samsung.com>
> + *
> + * This driver enables to monitor battery health and control charger
> + * during suspend-to-mem.
> + * Charger manager depends on other devices. register this later than
> + * the depending devices.
> + *
> + * 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/io.h>
> +#include <linux/module.h>
> +#include <linux/irq.h>
> +#include <linux/interrupt.h>
> +#include <linux/rtc.h>
> +#include <linux/slab.h>
> +#include <linux/workqueue.h>
> +#include <linux/platform_device.h>
> +#include <linux/power/charger-manager.h>
> +#include <linux/regulator/consumer.h>
> +
> +/*
> + * Regard CM_JIFFIES_SMALL jiffies is small enough to ignore for
> + * delayed works so that we can run delayed works with CM_JIFFIES_SMALL
> + * without any delays.
> + */
> +#define CM_JIFFIES_SMALL (2)
> +
> +/* If y is valid (> 0) and smaller than x, do x = y */
> +#define CM_MIN_VALID(x, y) x = (((y > 0) && ((x) > (y))) ? (y) : (x))
> +
> +/*
> + * Regard CM_RTC_SMALL (sec) is small enough to ignore error in invoking
> + * rtc alarm. It should be 2 or larger
> + */
> +#define CM_RTC_SMALL (2)
> +
> +#define UEVENT_BUF_SIZE 32
> +
> +static LIST_HEAD(cm_list);
> +static DEFINE_MUTEX(cm_list_mtx);
> +
> +/* About in-suspend (suspend-again) monitoring */
> +static struct rtc_device *rtc_dev;
> +static struct rtc_wkalrm rtc_wkalarm_save; /* Backup RTC alarm */
> +static unsigned long rtc_wkalarm_save_; /* 0 if not available */
> +static bool cm_suspended;
> +static bool cm_rtc_set;
> +static unsigned long cm_suspend_duration_ms;
> +
> +/* Global charger-manager description */
> +static struct charger_global_desc *g_desc; /* init with setup_charger_manager */
> +
> +/**
> + * is_batt_present - See if the battery presents in place.
> + * @cm: the Charger Manager representing the battery.
> + */
> +static bool is_batt_present(struct charger_manager *cm)
> +{
> + union power_supply_propval val;
> + bool present = false;
> + int i, ret;
> +
> + switch (cm->desc->battery_present) {
> + case CM_FUEL_GAUGE:
> + ret = cm->fuel_gauge->get_property(cm->fuel_gauge,
> + POWER_SUPPLY_PROP_PRESENT, &val);
> + if (ret == 0 && val.intval)
> + present = true;
> + break;
> + case CM_CHARGER_STAT:
> + for (i = 0; cm->charger_stat[i]; i++) {
> + ret = cm->charger_stat[i]->get_property(
> + cm->charger_stat[i],
> + POWER_SUPPLY_PROP_PRESENT, &val);
> + if (ret == 0 && val.intval) {
> + present = true;
> + break;
> + }
> + }
> + break;
> + }
> +
> + return present;
> +}
> +
> +/**
> + * is_ext_pwr_online - See if an external power source is attached to charge
> + * @cm: the Charger Manager representing the battery.
> + *
> + * Returns true if at least one of the chargers of the battery has an external
> + * power source attached to charge the battery regardless of whether it is
> + * actually charging or not.
> + */
> +static bool is_ext_pwr_online(struct charger_manager *cm)
> +{
> + union power_supply_propval val;
> + bool online = false;
> + int i, ret;
> +
> + /* If at least one of them has one, it's yes. */
> + for (i = 0; cm->charger_stat[i]; i++) {
> + ret = cm->charger_stat[i]->get_property(
> + cm->charger_stat[i],
> + POWER_SUPPLY_PROP_ONLINE, &val);
> + if (ret == 0 && val.intval) {
> + online = true;
> + break;
> + }
> + }
> +
> + return online;
> +}
> +
> +/**
> + * is_charging - Returns true if the battery is being charged.
> + * @cm: the Charger Manager representing the battery.
> + */
> +static bool is_charging(struct charger_manager *cm)
> +{
> + int i, ret;
> + bool charging = false;
> + union power_supply_propval val;
> +
> + /* If there is no battery, it cannot be charged */
> + if (!is_batt_present(cm))
> + return false;
> +
> + /* If at least one of the charger is charging, return yes */
> + for (i = 0; cm->charger_stat[i]; i++) {
> + /* 1. The charger sholuld not be DISABLED */
> + if (cm->emergency_stop)
> + continue;
> + if (!cm->charger_enabled)
> + continue;
> +
> + /* 2. The charger should be online (ext-power) */
> + ret = cm->charger_stat[i]->get_property(
> + cm->charger_stat[i],
> + POWER_SUPPLY_PROP_ONLINE, &val);
> + if (ret) {
> + dev_warn(cm->dev, "Cannot read ONLINE value from %s.\n",
> + cm->desc->psy_charger_stat[i]);
> + continue;
> + }
> + if (val.intval == 0)
> + continue;
> +
> + /*
> + * 3. The charger should not be FULL, DISCHARGING,
> + * or NOT_CHARGING.
> + */
> + ret = cm->charger_stat[i]->get_property(
> + cm->charger_stat[i],
> + POWER_SUPPLY_PROP_STATUS, &val);
> + if (ret) {
> + dev_warn(cm->dev, "Cannot read STATUS value from %s.\n",
> + cm->desc->psy_charger_stat[i]);
> + continue;
> + }
> + if (val.intval == POWER_SUPPLY_STATUS_FULL ||
> + val.intval == POWER_SUPPLY_STATUS_DISCHARGING ||
> + val.intval == POWER_SUPPLY_STATUS_NOT_CHARGING)
> + continue;
> +
> + /* Then, this is charging. */
> + charging = true;
> + break;
> + }
> +
> + return charging;
> +}
> +
> +/**
> + * is_polling_required - Return true if need to continue polling for this CM.
> + * @cm: the Charger Manager representing the battery.
> + */
> +static bool is_polling_required(struct charger_manager *cm)
> +{
> + switch (cm->desc->polling_mode) {
> + case CM_POLL_DISABLE:
> + return false;
> + case CM_POLL_ALWAYS:
> + return true;
> + case CM_POLL_EXTERNAL_POWER_ONLY:
> + return is_ext_pwr_online(cm);
> + case CM_POLL_CHARGING_ONLY:
> + return is_charging(cm);
> + default:
> + dev_warn(cm->dev, "Incorrect polling_mode (%d)\n",
> + cm->desc->polling_mode);
> + }
> +
> + return false;
> +}
> +
> +/**
> + * try_charger_enable - Enable/Disable chargers altogether
> + * @cm: the Charger Manager representing the battery.
> + * @enable: true: enable / false: disable
> + *
> + * Note that Charger Manager keeps the charger enabled regardless whether
> + * the charger is charging or not (because battery is full or no external
> + * power source exists) except when CM needs to disable chargers forcibly
> + * bacause of emergency causes; when the battery is overheated or too cold.
> + */
> +static int try_charger_enable(struct charger_manager *cm, bool enable)
> +{
> + int err = 0, i;
> + struct charger_desc *desc = cm->desc;
> +
> + /* Ignore if it's redundent command */
> + if (enable && cm->charger_enabled)
> + return 0;
> + if (!enable && !cm->charger_enabled)
> + return 0;
> +
> + if (enable) {
> + if (cm->emergency_stop)
> + return -EAGAIN;
> + err = regulator_bulk_enable(desc->num_charger_regulators,
> + desc->charger_regulators);
> + } else {
> + /*
> + * Abnormal battery state - Stop charging forcibly,
> + * even if charger was enabled at the other places
> + */
> + err = regulator_bulk_disable(desc->num_charger_regulators,
> + desc->charger_regulators);
> +
> + for (i = 0; i < desc->num_charger_regulators; i++) {
> + if (regulator_is_enabled(
> + desc->charger_regulators[i].consumer)) {
> + regulator_force_disable(
> + desc->charger_regulators[i].consumer);
> + dev_warn(cm->dev,
> + "Disable regulator(%s) forcibly.\n",
> + desc->charger_regulators[i].supply);
> + }
> + }
> + }
> +
> + if (!err)
> + cm->charger_enabled = enable;
> +
> + return err;
> +}
> +
> +/**
> + * uevent_notify - Let users know something has changed.
> + * @cm: the Charger Manager representing the battery.
> + * @event: the event string.
> + *
> + * If @event is null, it implies that uevent_notify is called
> + * by resume function. When called in the resume function, cm_suspended
> + * should be already reset to false in order to let uevent_notify
> + * notify the recent event during the suspend to users. While
> + * suspended, uevent_notify does not notify users, but tracks
> + * events so that uevent_notify can notify users later after resumed.
> + */
> +static void uevent_notify(struct charger_manager *cm, const char *event)
> +{
> + static char env_str[UEVENT_BUF_SIZE + 1] = "";
> + static char env_str_save[UEVENT_BUF_SIZE + 1] = "";
> +
> + if (cm_suspended) {
> + /* Nothing in suspended-event buffer */
> + if (env_str_save[0] == 0) {
> + if (!strncmp(env_str, event, UEVENT_BUF_SIZE))
> + return; /* status not changed */
> + strncpy(env_str_save, event, UEVENT_BUF_SIZE);
> + return;
> + }
> +
> + if (!strncmp(env_str_save, event, UEVENT_BUF_SIZE))
> + return; /* Duplicated. */
> + else
> + strncpy(env_str_save, event, UEVENT_BUF_SIZE);
> +
> + return;
> + }
> +
> + if (event == NULL) {
> + /* No messages pending */
> + if (!env_str_save[0])
> + return;
> +
> + strncpy(env_str, env_str_save, UEVENT_BUF_SIZE);
> + kobject_uevent(&cm->dev->kobj, KOBJ_CHANGE);
> + env_str_save[0] = 0;
> +
> + return;
> + }
> +
> + /* status not changed */
> + if (!strncmp(env_str, event, UEVENT_BUF_SIZE))
> + return;
> +
> + /* save the status and notify the update */
> + strncpy(env_str, event, UEVENT_BUF_SIZE);
> + kobject_uevent(&cm->dev->kobj, KOBJ_CHANGE);
> +
> + dev_info(cm->dev, event);
> +}
> +
> +/**
> + * _cm_monitor - Monitor the temperature and return true for exceptions.
> + * @cm: the Charger Manager representing the battery.
> + *
> + * Returns true if there is an event to notify for the battery.
> + * (True if the status of "emergency_stop" changes)
> + */
> +static bool _cm_monitor(struct charger_manager *cm)
> +{
> + struct charger_desc *desc = cm->desc;
> + int temp = desc->is_temperature_error(&cm->last_temp_mC);
> +
> + dev_dbg(cm->dev, "monitoring (%2.2d.%3.3dC)\n",
> + cm->last_temp_mC / 1000, cm->last_temp_mC % 1000);
> +
> + /* It has been stopped already */
> + if (temp && cm->emergency_stop)
> + return false;
> +
> + /* It has been charging already */
> + if (!temp && !cm->emergency_stop)
> + return false;
> +
> + if (temp) {
> + cm->emergency_stop = temp;
> + if (!try_charger_enable(cm, false)) {
> + if (temp > 0)
> + uevent_notify(cm, "OVERHEAT");
> + else
> + uevent_notify(cm, "COLD");
> + }
> + } else {
> + cm->emergency_stop = 0;
> + if (!try_charger_enable(cm, true))
> + uevent_notify(cm, "CHARGING");
> + }
> +
> + return true;
> +}
> +
> +/**
> + * cm_monitor - Monitor every battery.
> + *
> + * Returns true if there is an event to notify from any of the batteries.
> + * (True if the status of "emergency_stop" changes)
> + */
> +static bool cm_monitor(void)
> +{
> + bool stop = false;
> + struct charger_manager *cm;
> +
> + mutex_lock(&cm_list_mtx);
> +
> + list_for_each_entry(cm, &cm_list, entry)
> + stop |= _cm_monitor(cm);
> +
> + mutex_unlock(&cm_list_mtx);
> +
> + return stop;
> +}
> +
> +/**
> + * cm_setup_timer - For in-suspend monitoring setup wakeup alarm
> + * for suspend_again.
> + *
> + * Returns true if the alarm is set for Charger Manager to use.
> + * Returns false if
> + * cm_setup_timer fails to set an alarm,
> + * cm_setup_timer does not need to set an alarm for Charger Manager,
> + * or an alarm previously configured is to be used.
> + */
> +static bool cm_setup_timer(void)
> +{
> + struct charger_manager *cm;
> + unsigned int wakeup_ms = UINT_MAX;
> + bool ret = false;
> +
> + mutex_lock(&cm_list_mtx);
> +
> + list_for_each_entry(cm, &cm_list, entry) {
> + /* Skip if polling is not required for this CM */
> + if (!is_polling_required(cm) && !cm->emergency_stop)
> + continue;
> + if (cm->desc->polling_interval_ms == 0)
> + continue;
> + CM_MIN_VALID(wakeup_ms, cm->desc->polling_interval_ms);
> + }
> +
> + mutex_unlock(&cm_list_mtx);
> +
> + if (wakeup_ms < UINT_MAX && wakeup_ms > 0) {
> + pr_info("Charger Manager wakeup timer: %u ms.\n", wakeup_ms);
> + if (rtc_dev) {
> + struct rtc_wkalrm tmp;
> + unsigned long time, now;
> + unsigned long add = DIV_ROUND_UP(wakeup_ms, 1000);
> +
> + /*
> + * Set alarm with the polling interval (wakeup_ms)
> + * except when rtc_wkalarm_save comes first.
> + * However, the alarm time should be NOW +
> + * CM_RTC_SMALL or later.
> + */
> + tmp.enabled = 1;
> + rtc_read_time(rtc_dev, &tmp.time);
> + rtc_tm_to_time(&tmp.time, &now);
> + if (add < CM_RTC_SMALL)
> + add = CM_RTC_SMALL;
> + time = now + add;
> +
> + ret = true;
> +
> + if (rtc_wkalarm_save.enabled && rtc_wkalarm_save_ &&
> + rtc_wkalarm_save_ < time) {
> + if (rtc_wkalarm_save_ < now + CM_RTC_SMALL)
> + time = now + CM_RTC_SMALL;
> + else
> + time = rtc_wkalarm_save_;
> +
> + /* The timer is not appointed by CM */
> + ret = false;
> + }
> +
> + pr_info("Waking up after %lu secs.\n",
> + time - now);
> +
> + rtc_time_to_tm(time, &tmp.time);
> + rtc_set_alarm(rtc_dev, &tmp);
> + cm_suspend_duration_ms += wakeup_ms;
> + return ret;
> + }
> + }
> +
> + if (rtc_dev)
> + rtc_set_alarm(rtc_dev, &rtc_wkalarm_save);
> + return false;
> +}
> +
> +/**
> + * cm_suspend_again - Determine whether suspend again or not
> + *
> + * Returns true if the system should be suspended again
> + * Returns false if the system should be woken up
> + */
> +bool cm_suspend_again(void)
> +{
> + struct charger_manager *cm;
> + bool ret = false;
> +
> + if (!g_desc)
> + return false;
> + if (!g_desc->is_rtc_only_wakeup_reason)
> + return false;
> + if (!g_desc->is_rtc_only_wakeup_reason())
> + return false;
> + if (!cm_rtc_set)
> + return false;
> +
> + if (cm_monitor())
> + goto out;
> +
> + ret = true;
> + mutex_lock(&cm_list_mtx);
> + list_for_each_entry(cm, &cm_list, entry) {
> + if (cm->status_save_ext_pwr_inserted != is_ext_pwr_online(cm) ||
> + cm->status_save_batt != is_batt_present(cm))
> + ret = false;
> + }
> + mutex_unlock(&cm_list_mtx);
> +
> + cm_rtc_set = cm_setup_timer();
> +out:
> + /* It's about the time when the non-CM appointed timer goes off */
> + if (rtc_wkalarm_save.enabled) {
> + unsigned long now;
> + struct rtc_time tmp;
> +
> + rtc_read_time(rtc_dev, &tmp);
> + rtc_tm_to_time(&tmp, &now);
> +
> + if (rtc_wkalarm_save_ &&
> + now + CM_RTC_SMALL >= rtc_wkalarm_save_)
> + return false;
> + }
> + return ret;
> +}
> +EXPORT_SYMBOL_GPL(cm_suspend_again);
> +
> +/**
> + * setup_charger_manager - initialize charger_global_desc data
> + * @gd: pointer to instance of charger_global_desc
> + */
> +int setup_charger_manager(struct charger_global_desc *gd)
> +{
> + if (!gd)
> + return -EINVAL;
> +
> + if (rtc_dev)
> + rtc_class_close(rtc_dev);
> + rtc_dev = NULL;
> + g_desc = NULL;
> +
> + if (!gd->is_rtc_only_wakeup_reason) {
> + pr_err("The callback is_wktimer_only_wkreason is not given.\n");
> + return -EINVAL;
> + }
> +
> + if (gd->rtc) {
> + rtc_dev = rtc_class_open(gd->rtc);
> + if (IS_ERR_OR_NULL(rtc_dev)) {
> + rtc_dev = NULL;
> + /* Retry at probe. RTC may be not registered yet */
> + }
> + } else {
> + pr_warn("No wktimer is given for charger manager."
> + "In-suspend monitoring won't work.\n");
> + }
> +
> + g_desc = gd;
> + return 0;
> +}
> +EXPORT_SYMBOL_GPL(setup_charger_manager);
> +
> +static int charger_manager_probe(struct platform_device *pdev)
> +{
> + struct charger_desc *desc = dev_get_platdata(&pdev->dev);
> + struct charger_manager *cm;
> + int ret = 0, i;
> +
> + if (g_desc && !rtc_dev && g_desc->rtc) {
> + rtc_dev = rtc_class_open(g_desc->rtc);
> + if (IS_ERR_OR_NULL(rtc_dev)) {
> + rtc_dev = NULL;
> + dev_err(&pdev->dev, "Cannot get RTC %s.\n",
> + g_desc->rtc);
> + ret = -ENODEV;
> + goto err_alloc;
> + }
> + }
> +
> + if (!desc) {
> + dev_err(&pdev->dev, "No platform data (desc) found.\n");
> + ret = -ENODEV;
> + goto err_alloc;
> + }
> +
> + cm = kzalloc(sizeof(struct charger_manager), GFP_KERNEL);
> + if (!cm) {
> + dev_err(&pdev->dev, "Cannot allocate memory.\n");
> + ret = -ENOMEM;
> + goto err_alloc;
> + }
> +
> + /* Basic Values. Unspecified are Null or 0 */
> + cm->dev = &pdev->dev;
> + cm->desc = desc;
> + cm->last_temp_mC = INT_MIN; /* denotes "unmeasured, yet" */
> +
> + if (!desc->charger_regulators || desc->num_charger_regulators < 1) {
> + ret = -EINVAL;
> + dev_err(&pdev->dev, "charger_regulators undefined.\n");
> + goto err_no_charger;
> + }
> +
> + ret = regulator_bulk_get(&pdev->dev, desc->num_charger_regulators,
> + desc->charger_regulators);
> + if (ret) {
> + dev_err(&pdev->dev, "Cannot get charger regulators.\n");
> + goto err_no_charger;
> + }
> +
> + if (!desc->psy_charger_stat || !desc->psy_charger_stat[0]) {
> + dev_err(&pdev->dev, "No power supply defined.\n");
> + ret = -EINVAL;
> + goto err_no_charger_stat;
> + }
> +
> + for (i = 0; desc->psy_charger_stat[i]; i++)
> + /* Counting index only */ ;
> +
> + cm->charger_stat = kzalloc(sizeof(struct power_supply *) * (i + 1),
> + GFP_KERNEL);
> + if (!cm->charger_stat) {
> + ret = -ENOMEM;
> + goto err_no_charger_stat;
> + }
> +
> + for (i = 0; desc->psy_charger_stat[i]; i++) {
> + cm->charger_stat[i] = power_supply_get_by_name(
> + desc->psy_charger_stat[i]);
> + if (!cm->charger_stat[i]) {
> + dev_err(&pdev->dev, "Cannot find power supply "
> + "\"%s\"\n",
> + desc->psy_charger_stat[i]);
> + ret = -ENODEV;
> + goto err_chg_stat;
> + }
> + }
> +
> + cm->fuel_gauge = power_supply_get_by_name(desc->psy_fuel_gauge);
> + if (!cm->fuel_gauge) {
> + dev_err(&pdev->dev, "Cannot find power supply \"%s\"\n",
> + desc->psy_fuel_gauge);
> + ret = -ENODEV;
> + goto err_chg_stat;
> + }
> +
> + if (desc->polling_interval_ms == 0 ||
> + msecs_to_jiffies(desc->polling_interval_ms) <= CM_JIFFIES_SMALL) {
> + dev_err(&pdev->dev, "polling_interval_ms is too small\n");
> + ret = -EINVAL;
> + goto err_chg_stat;
> + }
> +
> + if (!desc->is_temperature_error) {
> + dev_err(&pdev->dev, "there is no is_temperature_error\n");
> + ret = -EINVAL;
> + goto err_chg_stat;
> + }
> +
> + platform_set_drvdata(pdev, cm);
> +
> + ret = try_charger_enable(cm, true);
> + if (ret) {
> + dev_err(&pdev->dev, "Cannot enable charger regulators\n");
> + goto err_chg_stat;
> + }
> +
> + /* Add to the list */
> + mutex_lock(&cm_list_mtx);
> + list_add(&cm->entry, &cm_list);
> + mutex_unlock(&cm_list_mtx);
> +
> + return 0;
> +
> +err_chg_stat:
> + kfree(cm->charger_stat);
> +err_no_charger_stat:
> + if (desc->charger_regulators)
> + regulator_bulk_free(desc->num_charger_regulators,
> + desc->charger_regulators);
> +err_no_charger:
> + kfree(cm);
> +err_alloc:
> + return ret;
> +}
> +
> +static int __devexit charger_manager_remove(struct platform_device *pdev)
> +{
> + struct charger_manager *cm = platform_get_drvdata(pdev);
> + struct charger_desc *desc = cm->desc;
> +
> + /* Remove from the list */
> + mutex_lock(&cm_list_mtx);
> + list_del(&cm->entry);
> + mutex_unlock(&cm_list_mtx);
> +
> + kfree(cm->charger_stat);
> + if (desc->charger_regulators)
> + regulator_bulk_free(desc->num_charger_regulators,
> + desc->charger_regulators);
> + kfree(cm);
> + return 0;
> +}
> +
> +const struct platform_device_id charger_manager_id[] = {
> + { "charger-manager", 0 },
> + { },
> +};
> +
> +static int cm_suspend_prepare(struct device *dev)
> +{
> + struct platform_device *pdev = container_of(dev, struct platform_device,
> + dev);
> + struct charger_manager *cm = platform_get_drvdata(pdev);
> +
> + if (!cm_suspended) {
> + if (rtc_dev) {
> + struct rtc_time tmp;
> + unsigned long now;
> +
> + rtc_read_alarm(rtc_dev, &rtc_wkalarm_save);
> + rtc_read_time(rtc_dev, &tmp);
> +
> + if (rtc_wkalarm_save.enabled) {
> + rtc_tm_to_time(&rtc_wkalarm_save.time,
> + &rtc_wkalarm_save_);
> + rtc_tm_to_time(&tmp, &now);
> + if (now > rtc_wkalarm_save_)
> + rtc_wkalarm_save_ = 0;
> + } else {
> + rtc_wkalarm_save_ = 0;
> + }
> + }
> + cm_suspended = true;
> + }
Donggeun,
I think that this patch is good for saving energy for charging while
suspend. I think that this driver also works under android. I have
some concerns on rtc wakeup.
As we know, RTC event doesn't link to timer events in kernel. But
Android did. Android transfered timer event to rtc event while it
suspend. So it will occupy one rtc device.
Whatever the device is phone or tablet, it may cost another rtc device
if it wants to implement power-up alarm feature. If we want to support
the charger manager feature, we have to add another rtc device. I
think that it's very hard to find any current platform with three rtc
devices. How about implement the similar mechanism like android?
Charger manager can register wakeup event into alarm mechanism like
android. And we only need two rtc devices.
Loop more guys: list-arm-kernel & linaro.
Thanks
Haojian
> +
> + cm->status_save_ext_pwr_inserted = is_ext_pwr_online(cm);
> + cm->status_save_batt = is_batt_present(cm);
> +
> + if (!cm_rtc_set) {
> + cm_suspend_duration_ms = 0;
> + cm_rtc_set = cm_setup_timer();
> + }
> +
> + return 0;
> +}
> +
> +static void cm_suspend_complete(struct device *dev)
> +{
> + struct platform_device *pdev = container_of(dev, struct platform_device,
> + dev);
> + struct charger_manager *cm = platform_get_drvdata(pdev);
> +
> + if (cm_suspended) {
> + if (rtc_dev) {
> + struct rtc_wkalrm tmp;
> +
> + rtc_read_alarm(rtc_dev, &tmp);
> + rtc_wkalarm_save.pending = tmp.pending;
> + rtc_set_alarm(rtc_dev, &rtc_wkalarm_save);
> + }
> + cm_suspended = false;
> + cm_rtc_set = false;
> + }
> +
> + uevent_notify(cm, NULL);
> +}
> +
> +static const struct dev_pm_ops charger_manager_pm = {
> + .prepare = cm_suspend_prepare,
> + .complete = cm_suspend_complete,
> +};
> +
> +static struct platform_driver charger_manager_driver = {
> + .driver = {
> + .name = "charger-manager",
> + .owner = THIS_MODULE,
> + .pm = &charger_manager_pm,
> + },
> + .probe = charger_manager_probe,
> + .remove = __devexit_p(charger_manager_remove),
> + .id_table = charger_manager_id,
> +};
> +
> +static int __init charger_manager_init(void)
> +{
> + return platform_driver_register(&charger_manager_driver);
> +}
> +late_initcall(charger_manager_init);
> +
> +static void __exit charger_manager_cleanup(void)
> +{
> + platform_driver_unregister(&charger_manager_driver);
> +}
> +module_exit(charger_manager_cleanup);
> +
> +MODULE_AUTHOR("MyungJoo Ham <myungjoo.ham(a)samsung.com>");
> +MODULE_DESCRIPTION("Charger Manager");
> +MODULE_LICENSE("GPL");
> +MODULE_ALIAS("charger-manager");
> diff --git a/include/linux/power/charger-manager.h b/include/linux/power/charger-manager.h
> new file mode 100644
> index 0000000..c91e208
> --- /dev/null
> +++ b/include/linux/power/charger-manager.h
> @@ -0,0 +1,131 @@
> +/* linux/include/linux/power/charger-manager.h
> + *
> + * Copyright (C) 2011 Samsung Electronics Co., Ltd.
> + * MyungJoo.Ham <myungjoo.ham(a)samsung.com>
> + *
> + * Charger Manager.
> + * This framework enables to control and multiple chargers and to
> + * monitor charging even in the context of suspend-to-RAM with
> + * an interface combining the chargers.
> + *
> + * 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.
> +**/
> +
> +#ifndef _CHARGER_MANAGER_H
> +#define _CHARGER_MANAGER_H
> +
> +#include <linux/power_supply.h>
> +
> +enum data_source {
> + CM_FUEL_GAUGE,
> + CM_CHARGER_STAT,
> +};
> +
> +enum polling_modes {
> + CM_POLL_DISABLE = 0,
> + CM_POLL_ALWAYS,
> + CM_POLL_EXTERNAL_POWER_ONLY,
> + CM_POLL_CHARGING_ONLY,
> +};
> +
> +/**
> + * struct charger_global_desc
> + * @rtc: the name of RTC used to wake up the system from suspend.
> + * @is_rtc_only_wakeup_reason:
> + * If the system is woken up by waekup-sources other than the RTC or
> + * callbacks, Charger Manager should recognize with
> + * is_rtc_only_wakeup_reason() returning false.
> + * If the RTC given to CM is the only wakeup reason,
> + * is_rtc_only_wakeup_reason should return true.
> + */
> +struct charger_global_desc {
> + char *rtc;
> +
> + bool (*is_rtc_only_wakeup_reason)(void);
> +};
> +
> +/**
> + * struct charger_desc
> + * @polling_mode:
> + * Determine which polling mode will be used
> + * @polling_interval_ms: interval in millisecond at which
> + * charger manager will monitor battery health
> + * @battery_present:
> + * Specify where information for existance of battery can be obtained
> + * @psy_charger_stat: the names of power-supply for chargers
> + * @num_charger_regulator: the number of entries in charger_regulators
> + * @charger_regulators: array of regulator_bulk_data for chargers
> + * @psy_fuel_gauge: the name of power-supply for fuel gauge
> + * @is_temperature_error:
> + * Determine whether the status is overheat or cold or normal.
> + * return_value > 0: overheat
> + * return_value == 0: normal
> + * return_value < 0: cold
> + */
> +struct charger_desc {
> + enum polling_modes polling_mode;
> + unsigned int polling_interval_ms;
> +
> + enum data_source battery_present;
> +
> + char **psy_charger_stat;
> +
> + int num_charger_regulators;
> + struct regulator_bulk_data *charger_regulators;
> +
> + char *psy_fuel_gauge;
> +
> + int (*is_temperature_error)(int *mC);
> +};
> +
> +#define PSY_NAME_MAX 30
> +
> +/**
> + * struct charger_manager
> + * @entry: entry for list
> + * @dev: device pointer
> + * @desc: instance of charger_desc
> + * @fuel_gauge: power_supply for fuel gauge
> + * @charger_stat: array of power_supply for chargers
> + * @charger_enabled: the state of charger
> + * @emergency_stop:
> + * When setting true, stop charging
> + * @last_temp_mC: the measured temperature in milli-Celsius
> + * @status_save_ext_pwr_inserted:
> + * saved status of external power before entering suspend-to-RAM
> + * @status_save_batt:
> + * saved status of battery before entering suspend-to-RAM
> + */
> +struct charger_manager {
> + struct list_head entry;
> + struct device *dev;
> + struct charger_desc *desc;
> +
> + struct power_supply *fuel_gauge;
> + struct power_supply **charger_stat;
> +
> + bool charger_enabled;
> +
> + int emergency_stop;
> + int last_temp_mC;
> +
> + bool status_save_ext_pwr_inserted;
> + bool status_save_batt;
> +};
> +
> +#ifdef CONFIG_CHARGER_MANAGER
> +extern int setup_charger_manager(struct charger_global_desc *gd);
> +extern bool cm_suspend_again(void);
> +#else
> +static void __maybe_unused setup_charger_manager(struct charger_global_desc *gd)
> +{ }
> +
> +static bool __maybe_unused cm_suspend_again(void)
> +{
> + return false;
> +}
> +#endif
> +
> +#endif /* _CHARGER_MANAGER_H */
> --
> 1.7.4.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo(a)vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
[ I already asked this question here:
http://ask.linaro.org/questions/361/hdmi-output-on-android-build but it
was suggested on irc that I ask on the mailing list as well. ]
I'm trying to get my pandaboard running the Android 4.0 build of Linaro
(https://android-build.linaro.org/builds/~linaro-android/panda/)
outputting to an HDMI capture device (specifically a a blackmagic
decklink extreme 3d card), but am not having any luck so far. From what
I can gather from the schematics (http://www.pandaboard.org/node/222/)
and the documentation, I want to hook up to the first HDMI port for this
and then switch the primary display to that:
http://elinux.org/Panda_How_to_kernel_3_0_rel#Switching_primary_display_to_…
However, this isn't giving me any output. I can tell that Android is up
and running (I can connect to it via adb or even take a screenshot via
ddms). I know the device for capture that I'm using is kind of finnicky
(it demands 1920x1024@60hz or nothing, see
https://wiki.mozilla.org/Project_Eideticker/DeckLink_Primer), so I
tweaked the following settings (after temporarily turning off overlay0,
as suggested):
echo "tv" > /sys/devices/platform/omapdss/overlay0/manager echo
"1920,1024" > /sys/devices/platform/omapdss/overlay0/output_size
Is there anything I'm missing? Are there any other platforms (Android
2.3? Ubuntu?) where HDMI output to this kind of device is known to work?
I have gotten this capture set up working using an LG G2X phone
(http://masalalabs.ca/2011/11/measuring-what-the-user-sees/), so I know
it should be possible in principle. Any help at all appreciated.
Will
Hi everyone,
I am creating a thin Linux environment for a SoC based on the Samsung
EXYNOS4. I have already tried the ALIP image (alip-n-tar-20111024-0.tar.gz),
and seemingly, all of the pre-compiled binaries seem to work including X11
(executes but needs further reconfiguring for my platform).
Now I want to rebuild the same X11 source that came along with this ALIP
image.
Can anyone point me where I can download this ported X11 source tree?
TIA,
Vic
Hi everyone,
Last week I did an initial drop of the end to end audio test we have been
discussing.
The idea is fairly simple, play a sine wave and test the audio stack by
sampling/testing the sine back in via loopback cable. The app is called
testfreq and is driven by a script called e2eaudiotest. It opens and
configures the audio device, takes a sample and then does a discrete
fourier transformation to find the frequency using the fftw3 library. The
test script driver uses speaker-test to play a sine wave at A 440, which
for now is the test frequency. It's still basic at this point, but it does
work on my system. There is a lot of additional things I'd like to do,
initial stack configuration, passing in the device, passing the test
frequency, doing more auto detection, clean up the code, etc, but I wanted
to start getting feedback. Any and all would be appreciated.
Have a look, and if you have a loopback cable, give it a spin:
http://git.linaro.org/gitweb?p=people/kurt-r-taylor/e2eaudiotest.git
You can also read more about it and check my progress here:
https://blueprints.launchpad.net/linaro-multimedia-project/+spec/linaro-mmw…
Enjoy!
--
Kurt Taylor (irc krtaylor)
Linaro Multimedia
Linaro.org <http://www.linaro.org/>* **│ *Open source software for ARM SoCs
Follow *Linaro: *Facebook <http://www.facebook.com/pages/Linaro> |
Twitter<http://twitter.com/#%21/linaroorg>|
Blog <http://www.linaro.org/linaro-blog/>
