Re: [PATCH] OMAP CPUIDLE: CPU Idle latency measurement

2010-08-27 Thread Kevin Hilman
Amit Kucheria amit.kuche...@linaro.org writes: On 10 Aug 28, vishwanath.sripa...@linaro.org wrote: From: Vishwanath BS vishwanath.sripa...@linaro.org This patch has instrumentation code for measuring latencies for various CPUIdle C states for OMAP. Idea here is to capture the timestamp at

Re: [PATCH] OMAP CPUIDLE: CPU Idle latency measurement

2010-08-27 Thread Kevin Hilman
vishwanath.sripa...@linaro.org writes: From: Vishwanath BS vishwanath.sripa...@linaro.org This patch has instrumentation code for measuring latencies for various CPUIdle C states for OMAP. Idea here is to capture the timestamp at various phases of CPU Idle and then compute the sw latency

Re: [PATCH] OMAP CPUIDLE: CPU Idle latency measurement

2010-09-02 Thread Kevin Hilman
Amit Kucheria amit.kuche...@linaro.org writes: On 10 Aug 27, Kevin Hilman wrote: vishwanath.sripa...@linaro.org writes: From: Vishwanath BS vishwanath.sripa...@linaro.org This patch has instrumentation code for measuring latencies for various CPUIdle C states for OMAP. Idea here

Re: PM support for OMAP

2010-09-03 Thread Kevin Hilman
On Wed, 2010-09-01 at 18:52 +0300, Ameya Palande wrote: Hi Amit, On Wed, 2010-09-01 at 17:38 +0200, ext Amit Kucheria wrote: On Wed, Sep 1, 2010 at 6:21 PM, Ameya Palande ameya.pala...@nokia.com wrote: Hi, AFAIU pm branch on linux-omap tree contains required patches, but they

Re: [PATCH 2/2] OMAP3 PM: sleep code clean up

2010-09-24 Thread Kevin Hilman
Vishwanath BS vishwanath...@ti.com writes: This patch has done some clean up of omap3 sleep code. Basically all possible hardcodings are removed and code is Reorganized into more logical buckets for better readability and instrumentation. Tested on ZOOM3. Again, please describe more about

Re: export kernel clock information to user space

2010-10-15 Thread Kevin Hilman
Amit Kucheria amit.kuche...@linaro.org writes: Adding linaro-dev to cc. Kernel consolidation WG might have comments. On Tue, Oct 12, 2010 at 9:04 AM, Yong Shen yong.s...@linaro.org wrote: Hi Amit and Jeremy, This is not a patch review. But patch may better present my idea. Basically, I

Re: OMAP4 MPU DVFS patches

2011-04-06 Thread Kevin Hilman
Nicolas Pitre nicolas.pi...@linaro.org writes: On Wed, 6 Apr 2011, Kevin Hilman wrote: Amit Kucheria amit.kuche...@linaro.org writes: On Sat, Apr 2, 2011 at 6:21 AM, Nicolas Pitre nicolas.pi...@linaro.org wrote: On Fri, 1 Apr 2011, Vishwanath Sripathy wrote: Hi Nicolas, Pls

Re: OMAP4 MPU DVFS patches

2011-04-07 Thread Kevin Hilman
Amit Kucheria amit.kuche...@linaro.org writes: On Wed, Apr 6, 2011 at 10:33 PM, Kevin Hilman khil...@ti.com wrote: Nicolas Pitre nicolas.pi...@linaro.org writes: On Wed, 6 Apr 2011, Kevin Hilman wrote: Amit Kucheria amit.kuche...@linaro.org writes: On Sat, Apr 2, 2011 at 6:21 AM, Nicolas

Re: [PATCH v3 0/4] OMAP serial device tree support

2011-12-14 Thread Kevin Hilman
Greg, Alan, Rajendra Nayak rna...@ti.com writes: v3 is rebased on top of the latest serial runtime patches[1] and boot tested with/without DT on OMAP4 SDP and OMAP4 Panda boards. With your ack on the drivers/tty/* stuff, I can queue this via the OMAP tree on top of the runtime PM conversion

Re: [PATCH] omap: use usleep_range() instead of mdelay()/udelay()

2012-01-03 Thread Kevin Hilman
Dmitry Antipov dmitry.anti...@linaro.org writes: From e4db974edb5c46360465462518a88b83f1bdedf6 Mon Sep 17 00:00:00 2001 From: Dmitry Antipov dmitry.anti...@linaro.org Date: Wed, 21 Dec 2011 10:57:08 +0400 Subject: [PATCH] omap: use usleep_range() instead of mdelay()/udelay() Please include a

Re: [PATCH V6 4/7] cpufreq: add clk-reg cpufreq driver

2012-01-11 Thread Kevin Hilman
Richard Zhao richard.z...@linaro.org writes: The driver get cpu operation point table from device tree cpu0 node, Since we already have an existing OPP infrastructure in the kernel, seems like this driver should get OPPs by asking the OPP layer. This approach assumes the OPP layer is static

Re: [PATCH V6 4/7] cpufreq: add clk-reg cpufreq driver

2012-01-12 Thread Kevin Hilman
Richard Zhao richard.z...@freescale.com writes: On Wed, Jan 11, 2012 at 03:22:34PM -0800, Kevin Hilman wrote: Richard Zhao richard.z...@linaro.org writes: The driver get cpu operation point table from device tree cpu0 node, Since we already have an existing OPP infrastructure

Re: [PATCH 0/2] change lpj in arm smp common code

2012-02-29 Thread Kevin Hilman
Richard Zhao richard.z...@linaro.org writes: The two patches were originally in [PATCH V6 0/7] add a generic cpufreq driver. I seperated them and hope they can go to upstream earlier. Richard Zhao (2): ARM: add cpufreq transiton notifier to adjust loops_per_jiffy for smp cpufreq:

Re: [PATCH v6 1/9] cpuidle: Add common time keeping and irq enabling

2012-02-29 Thread Kevin Hilman
Robert Lee rob@linaro.org writes: Make necessary changes to implement time keeping and irq enabling in the core cpuidle code. This will allow the removal of these functionalities from various platform cpuidle implementations whose timekeeping and irq enabling follows the form in this

Re: [PATCH v7 0/9] Consolidate cpuidle functionality

2012-03-06 Thread Kevin Hilman
(). Reviewed-by: Kevin Hilman khil...@ti.com ___ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev

Re: [PATCH v8 0/8] Consolidate cpuidle functionality

2012-03-21 Thread Kevin Hilman
Arnd Bergmann arnd.bergm...@linaro.org writes: On Tuesday 20 March 2012, Robert Lee wrote: This patch series moves various functionality duplicated in platform cpuidle drivers to the core cpuidle driver. Also, the platform irq disabling was removed as it appears that all calls into

Re: [PATCH v8 0/8] Consolidate cpuidle functionality

2012-03-21 Thread Kevin Hilman
-by: Robert Leerob@linaro.org (imx51, imx6q) Note that there's a space missing between the name and email in these tags (and for Deepthi's below also.) That seems to exist in all the patches. Reviewed-by: Kevin Hilman khil...@ti.com For my Reviewed-by, it only applies to the core code

Re: [PATCH v8 0/8] Consolidate cpuidle functionality

2012-03-21 Thread Kevin Hilman
Arnd Bergmann arnd.bergm...@linaro.org writes: On Tuesday 20 March 2012, Kevin Hilman wrote: Maybe it's time that drivers/cpuidle gets a maintainer. With lots of discussions of scheduler changes that affect load estimation, I suspect we're all going to have a bit of CPUidle work to do

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-21 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 03/21/2012 02:43 PM, Jean Pihet wrote: On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel, On Wednesday 21 March 2012 02:57 PM, Daniel Lezcano wrote: This patchset is a proposition to improve a bit

Re: [RFC][PATCH 0/7] OMAP4 cpuidle cleanup

2012-03-22 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 03/21/2012 10:54 PM, Kevin Hilman wrote: Daniel Lezcanodaniel.lezc...@linaro.org writes: On 03/21/2012 02:43 PM, Jean Pihet wrote: On Wed, Mar 21, 2012 at 11:07 AM, Santosh Shilimkar santosh.shilim...@ti.com wrote: Daniel

Re: [PATCH 02/17][V2] ARM: OMAP4: cpuidle - Declare the states with the driver declaration

2012-04-09 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: The cpuidle API allows to declare statically the states in the driver structure. Let's use it. We do no longer need the fill_cstate function called at runtime and by the way adding more instructions at boot time. Signed-off-by: Daniel Lezcano

Re: [PATCH 04/17][V2] ARM: OMAP4: cpuidle - fix static omap4_idle_data declaration

2012-04-09 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: Signed-off-by: Daniel Lezcano daniel.lezc...@linaro.org Reviewed-by: Jean Pihet j-pi...@ti.com Reviewed-by: Santosh Shilimkar santosh.shilim...@ti.com Missing changelog. Kevin --- arch/arm/mach-omap2/cpuidle44xx.c |2 +- 1 files

Re: [PATCH 07/17][V2] ARM: OMAP4: cpuidle - remove omap4_idle_data initialization at boot time

2012-04-09 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: We initialized it at compile time, no need to do that at boot time. Signed-off-by: Daniel Lezcano daniel.lezc...@linaro.org Reviewed-by: Jean Pihet j-pi...@ti.com Reviewed-by: Santosh Shilimkar santosh.shilim...@ti.com ---

Re: [PATCH 06/17][V2] ARM: OMAP4: cpuidle - use the omap4_idle_data variable directly

2012-04-09 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: We are storing the 'omap4_idle_data' in the private data field of the cpuidle device. As we are using this variable only in this file, that does not really make sense. Let's use the global variable directly instead dereferencing pointers in an

Re: [PATCH 11/17][V2] ARM: OMAP3: cpuidle - remove cpuidle_params_table

2012-04-09 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: We do not longer need the ''cpuidle_params_table' array as we defined the states in the driver and we checked they are all valid. We also remove the structure definition as it is no longer used. Signed-off-by: Daniel Lezcano

Re: [PATCH 10/17][V2] ARM: OMAP3: cpuidle - remove the 'valid' field

2012-04-09 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: With the previous changes all the states are valid, except the last state which can be handled by decreasing the number of states. Signed-off-by: Daniel Lezcano daniel.lezc...@linaro.org Reviewed-by: Jean Pihet j-pi...@ti.com ---

Re: [PATCH 00/17][V2] ARM: OMAP3/4 : cpuidle34xx and cpuidle44xx cleanups

2012-04-09 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: This patchset makes some cleanup on these cpuidle drivers and consolidate the code across both architecture. Thanks for this really nice cleanup. I have some comments on specific patches, but here's some general comments: Some minor comments:

Re: [PATCH 02/17][V2] ARM: OMAP4: cpuidle - Declare the states with the driver declaration

2012-04-23 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 04/19/2012 03:58 PM, Daniel Lezcano wrote: On 04/10/2012 12:37 AM, Kevin Hilman wrote: Daniel Lezcanodaniel.lezc...@linaro.org writes: The cpuidle API allows to declare statically the states in the driver structure. Let's use it. We do

Re: [PATCH 02/17][V2] ARM: OMAP4: cpuidle - Declare the states with the driver declaration

2012-04-23 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 04/23/2012 07:08 PM, Kevin Hilman wrote: Daniel Lezcanodaniel.lezc...@linaro.org writes: On 04/19/2012 03:58 PM, Daniel Lezcano wrote: On 04/10/2012 12:37 AM, Kevin Hilman wrote: Daniel Lezcanodaniel.lezc...@linaro.org writes

Re: [PATCH 00/18][V3] ARM: OMAP3/4 : cpuidle34xx and cpuidle44xx cleanups

2012-05-03 Thread Kevin Hilman
Hi Daniel, Daniel Lezcano daniel.lezc...@linaro.org writes: This patchset makes some cleanup on these cpuidle drivers and consolidate the code across both architecture. I think I said it before, but it's worth repeating: Very nice cleanup! Thanks for your persistence. I've now been through

Re: [PATCH 1/2] ARM: OMAP3: cpuidle - check the powerdomain lookup

2012-05-04 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: At init time, check the powerdomains lookup is successful otherwise exit the cpuidle driver init function with -ENODEV like what is done for the omap3 cpuidle driver. Signed-off-by: Daniel Lezcano daniel.lezc...@linaro.org Reviewed-by: Jean

Re: [PATCH 2/2] ARM: OMAP3/4: consolidate cpuidle Makefile

2012-05-08 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: Define a CPU_IDLE section in the makefile, declare the functions in the header files conforming to the kernel coding rules and remove the 'define's in the C files. CONFIG_PM is enabled when CPU_IDLE is enabled because the cpuidle drivers use

Re: [PATCH 2/2] ARM: OMAP3/4: consolidate cpuidle Makefile

2012-05-08 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 05/04/2012 07:18 PM, Daniel Lezcano wrote: Define a CPU_IDLE section in the makefile, declare the functions in the header files conforming to the kernel coding rules and remove the 'define's in the C files. CONFIG_PM is enabled when

Re: [RESEND PATCH 1/2] ARM: OMAP2+: nand: Make board_onenand_init() visible to board code

2012-05-15 Thread Kevin Hilman
Tony Lindgren t...@atomide.com writes: * Javier Martinez Canillas jav...@dowhile0.org [120427 02:33]: On Wed, Apr 25, 2012 at 9:59 AM, Enric Balletbò i Serra eballe...@gmail.com wrote: Tony, as this is a fix ,may be included ? Acked-by: Enric Balletbo i Serra eballe...@gmail.com

Re: [RESEND PATCH 1/2] ARM: OMAP2+: nand: Make board_onenand_init() visible to board code

2012-05-15 Thread Kevin Hilman
. Applies to your 'board' branch. Kevin From f4f2c35de0e67e3b8185059ffd78be67f7096d8a Mon Sep 17 00:00:00 2001 From: Kevin Hilman khil...@ti.com Date: Tue, 15 May 2012 13:07:20 -0700 Subject: [PATCH] ARM: OMAP2+: nand: fix build error when CONFIG_MTD_ONENAND_OMAP2=n MIME-Version: 1.0 Content-Type

Re: [PATCH] ARM: OMAP3/4: consolidate cpuidle Makefile

2012-05-30 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 05/14/2012 06:42 AM, Rajendra Nayak wrote: On Thursday 10 May 2012 03:32 PM, Daniel Lezcano wrote: The current Makefile compiles the cpuidle34xx.c and cpuidle44xx.c files even if the cpuidle option is not set in the kernel. This patch

Re: [PATCH] ARM: OMAP3/4: consolidate cpuidle Makefile

2012-05-30 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 05/30/2012 08:07 PM, Kevin Hilman wrote: Daniel Lezcano daniel.lezc...@linaro.org writes: On 05/14/2012 06:42 AM, Rajendra Nayak wrote: On Thursday 10 May 2012 03:32 PM, Daniel Lezcano wrote: The current Makefile compiles

Re: [linux-pm] cpuidle future and improvements

2012-07-11 Thread Kevin Hilman
Daniel Lezcano daniel.lezc...@linaro.org writes: On 06/18/2012 08:15 PM, Colin Cross wrote: On Mon, Jun 18, 2012 at 1:40 AM, Daniel Lezcano daniel.lezc...@linaro.org wrote: I propose to host a cpuidle-next tree where all these modifications will be and where people can send patches against,

[RFC/PATCH 3/5] virt CPU accounting: Kconfig: drop 64-bit requirment

2013-02-20 Thread Kevin Hilman
The 64-bit requirement can be removed after the conversion of kernel_cpustat accessors to the portable atomic64 accessors. Signed-off-by: Kevin Hilman khil...@linaro.org --- init/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/init/Kconfig b/init/Kconfig index 3abb019

[RFC/PATCH 0/5] context_tracking: prerequisites for ARM support

2013-02-20 Thread Kevin Hilman
simpler wrappers in kernel_stat.h would make this a bit cleaner? Kevin Kevin Hilman (5): context tracking: conditionalize guest support based on CONFIG_KVM kernel_cpustat: convert to atomic 64-bit accessors virt CPU accounting: Kconfig: drop 64-bit requirment cputime: use do_div

[RFC/PATCH 2/5] kernel_cpustat: convert to atomic 64-bit accessors

2013-02-20 Thread Kevin Hilman
Use the atomic64_* accessors for all the kernel_cpustat fields to ensure atomic access on non-64 bit platforms. Thanks to Mats Liljegren for CGROUP_CPUACCT related fixes. Cc: Mats Liljegren mats.liljeg...@enea.com Signed-off-by: Kevin Hilman khil...@linaro.org --- fs/proc/stat.c

[RFC/PATCH 4/5] cputime: use do_div() for nsec resolution conversion helpers

2013-02-20 Thread Kevin Hilman
For the nsec resolution conversions to be useful on non 64-bit architectures, do_div() needs to be used for the 64-bit divisions. Signed-off-by: Kevin Hilman khil...@linaro.org --- include/asm-generic/cputime_nsecs.h | 51 +++-- 1 file changed, 37 insertions

Re: [RFC/PATCH 4/5] cputime: use do_div() for nsec resolution conversion helpers

2013-02-21 Thread Kevin Hilman
Namhyung Kim namhy...@kernel.org writes: Hi Kevin, On Wed, Feb 20, 2013 at 11:41:41AM -0800, Kevin Hilman wrote: For the nsec resolution conversions to be useful on non 64-bit architectures, do_div() needs to be used for the 64-bit divisions. Signed-off-by: Kevin Hilman khil...@linaro.org

Re: [RFC/PATCH 2/5] kernel_cpustat: convert to atomic 64-bit accessors

2013-02-21 Thread Kevin Hilman
Kevin Hilman khil...@linaro.org writes: Use the atomic64_* accessors for all the kernel_cpustat fields to ensure atomic access on non-64 bit platforms. Thanks to Mats Liljegren for CGROUP_CPUACCT related fixes. Cc: Mats Liljegren mats.liljeg...@enea.com Signed-off-by: Kevin Hilman khil

Re: [RFC/PATCH 1/5] context tracking: conditionalize guest support based on CONFIG_KVM

2013-02-26 Thread Kevin Hilman
Frederic Weisbecker fweis...@gmail.com writes: On Wed, Feb 20, 2013 at 11:41:38AM -0800, Kevin Hilman wrote: So that it can build on !KVM systems too. Signed-off-by: Kevin Hilman khil...@linaro.org --- kernel/context_tracking.c | 4 1 file changed, 4 insertions(+) diff --git

Re: [RFC/PATCH 1/5] context tracking: conditionalize guest support based on CONFIG_KVM

2013-02-27 Thread Kevin Hilman
Namhyung Kim namhy...@kernel.org writes: 2013-02-26 (화), 11:29 -0800, Kevin Hilman: Frederic Weisbecker fweis...@gmail.com writes: On Wed, Feb 20, 2013 at 11:41:38AM -0800, Kevin Hilman wrote: So that it can build on !KVM systems too. Signed-off-by: Kevin Hilman khil...@linaro.org

Re: automated build coverage

2013-05-14 Thread Kevin Hilman
Nicolas Dechesne nicolas.deche...@linaro.org writes: On Thu, May 9, 2013 at 7:56 PM, Fathi Boudra fathi.bou...@linaro.orgwrote: Hi all, I just wanted to forward this thread from LAKML to linaro-dev: http://article.gmane.org/gmane.linux.ports.tegra/10683 Seems there is lots desire

Re: automated build coverage

2013-05-14 Thread Kevin Hilman
Fathi Boudra fathi.bou...@linaro.org writes: [...] Is there a way for us (linaro folks) to see more of the Jenkins setup for these jobs (including the scripts.) There appears to be some useful add-ons being used. Read-only access to the detailed configuration of the jenkins jobs would be

Re: automated build coverage

2013-05-15 Thread Kevin Hilman
Fathi Boudra fathi.bou...@linaro.org writes: On 14 May 2013 23:49, Kevin Hilman khil...@linaro.org wrote: Fathi Boudra fathi.bou...@linaro.org writes: [...] Is there a way for us (linaro folks) to see more of the Jenkins setup for these jobs (including the scripts.) There appears

Re: automated build coverage

2013-05-17 Thread Kevin Hilman
Fathi Boudra fathi.bou...@linaro.org writes: [...] Is there a way for us (linaro folks) to see more of the Jenkins setup for these jobs (including the scripts.) There appears to be some useful add-ons being used. Read-only access to the detailed configuration of the jenkins jobs would be

Re: automated build coverage

2013-05-20 Thread Kevin Hilman
Fathi Boudra fathi.bou...@linaro.org writes: On 18 May 2013 04:06, Kevin Hilman khil...@linaro.org wrote: Fathi Boudra fathi.bou...@linaro.org writes: [...] Is there a way for us (linaro folks) to see more of the Jenkins setup for these jobs (including the scripts.) There appears

Re: samsung chromebook 2: booting without ctrl-U

2015-11-19 Thread Kevin Hilman
Steve Muckle writes: > Does anyone have instructions on modding the chromebook 2 Peach Pi such > that it is not necessary to hit ctrl-U when booting custom kernels? I > found some instructions (which may be incomplete) for the SS Chromebook > "snow" suggesting it may be