Re: [PATCH 1/3] ARM: OMAP: 32k counter: resolve sparse warnings

2012-12-25 Thread Santosh Shilimkar
On Monday 24 December 2012 06:54 AM, Paul Walmsley wrote: Commit 1fe97c8f6a1de67a5f56e029a818903d5bed8017 (ARM: OMAP: Make OMAP clocksource source selection using kernel param) results in a new warning from sparse: arch/arm/plat-omap/counter_32k.c:86:12: warning: symbol

Re: [PATCH 00/15] OMAP SHAM AES Crypto Updates

2012-12-25 Thread Mark A. Greer
On Sun, Dec 23, 2012 at 05:03:45PM +0100, Peter Korsgaard wrote: Mark == Mark A Greer mgr...@animalcreek.com writes: Hi, Huh, does am335x have a sham module? I don't see any mention of that in spruh73g.pdf Mark Yes. It has sham, aes, and rng modules. Ok, interesting. Mark

Re: [PATCH 00/15] OMAP SHAM AES Crypto Updates

2012-12-25 Thread Mark A. Greer
On Sun, Dec 23, 2012 at 08:40:43AM +, Paul Walmsley wrote: Hi Mark, On Fri, 21 Dec 2012, Mark A. Greer wrote: From: Mark A. Greer mgr...@animalcreek.com [This series supersedes the hwmod related patches sent in the crypto: omap-sham updates and crypto: omap-aes updates

Re: [PATCH 00/15] OMAP SHAM AES Crypto Updates

2012-12-25 Thread Mark A. Greer
On Tue, Dec 25, 2012 at 12:29:37PM -0700, Mark A. Greer wrote: On Sun, Dec 23, 2012 at 08:40:43AM +, Paul Walmsley wrote: Hi Mark, On Fri, 21 Dec 2012, Mark A. Greer wrote: From: Mark A. Greer mgr...@animalcreek.com [This series supersedes the hwmod related patches sent in

Re: [PATCH 2/3] ARM: OMAP AM33xx: hwmod data: resolve sparse warnings

2012-12-25 Thread Mugunthan V N
On 12/24/2012 6:55 AM, Paul Walmsley wrote: Commit 70384a6af0914d5dcec034643941e29d3e3e69f7 (ARM: OMAP3+: hwmod: Add AM33XX HWMOD data for davinci_mdio module) adds two new sparse warnings: arch/arm/mach-omap2/omap_hwmod_33xx_data.c:2518:30: warning: symbol 'am33xx_mdio_addr_space' was not

Re: [PATCH v2 0/2] cpts fixes for v3.8-rc2

2012-12-25 Thread Mugunthan V N
On 12/24/2012 12:49 PM, Richard Cochran wrote: Changed in v2: Use clk_prepare_enable instead of clk_prepare + clk_enable. The new cpts driver has two small issues, but it otherwise seems to be working in -rc1. Thanks, Richard Richard Cochran (2): cpts: fix build error by removing useless

[PATCH] arm: omap: Remove apollon board support

2012-12-25 Thread Kyungmin Park
From: Kyungmin Park kyungmin.p...@samsung.com As apollon board doesn't used anymore, remove it. Signed-off-by: Kyungmin Park kyungmin.p...@samsung.com --- diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig index 41b581f..d4e4f95 100644 --- a/arch/arm/mach-omap2/Kconfig +++

RE: [PATCH 02/12] ARM: OMAP2+: PM: introduce power domains functional states

2012-12-25 Thread Bedia, Vaibhav
Hi Paul, A minor comment below. On Sun, Dec 09, 2012 at 23:23:01, Paul Walmsley wrote: [...] + + pr_debug(powerdomain: convert pwrst (%0x,%0x) to fpwrst %0x\n, + pwrst, logic, *fpwrst); + This function alone does not print the powerdomain name. Can you add that in the final

RE: [PATCH 09/12] ARM: OMAP2+: powerdomain: skip register reads for powerdomains known to be on

2012-12-25 Thread Bedia, Vaibhav
Hi Paul, On Mon, Dec 10, 2012 at 01:33:28, Paul Walmsley wrote: There's no need to determine the current power state for powerdomains that must be on while the kernel is running. We mark these powerdomains with a new flag, PWRDM_ACTIVE_WITH_KERNEL. Any powerdomain marked with that flag is

RE: [PATCH 09/12] ARM: OMAP2+: powerdomain: skip register reads for powerdomains known to be on

2012-12-25 Thread Bedia, Vaibhav
On Wed, Dec 26, 2012 at 11:51:46, Bedia, Vaibhav wrote: Hi Paul, On Mon, Dec 10, 2012 at 01:33:28, Paul Walmsley wrote: There's no need to determine the current power state for powerdomains that must be on while the kernel is running. We mark these powerdomains with a new flag,

RE: [PATCH 09/10] ARM: OMAP2+: clockdomain: convert existing atomic usecounts into spinlock-protected shorts/ints

2012-12-25 Thread Bedia, Vaibhav
Hi Paul, On Sun, Dec 09, 2012 at 06:53:43, Paul Walmsley wrote: The atomic usecounts seem to be confusing, and are no longer needed since the operations that they are attached to really should take place under lock. Replace the atomic counters with simple integers, protected by the enclosing