From: Tero Kristo <t-kristo@ti.com>
To: Paul Walmsley <paul@pwsan.com>
Cc: linux-omap@vger.kernel.org, khilman@linaro.org, tony@atomide.com,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 00/11] ARM: OMAP2+: CM cleanup series
Date: Sat, 12 Oct 2013 13:18:47 +0300 [thread overview]
Message-ID: <52592207.4070508@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1310120139280.27397@utopia.booyaka.com>
On 10/12/2013 05:19 AM, Paul Walmsley wrote:
> Hi
>
> On Fri, 11 Oct 2013, Tero Kristo wrote:
>
>> A small cleanup set for CM. This basically gets rid of the omap2_cm_*
>> register accesses from the random code locations, and gathers these
>> under cm2xxx.c and cm3xxx.c. This is done in preparation for creating
>> a separate CM driver. The set also contains a couple of PRM cleanups
>> which I decided to take care of at the same time due to overlapping
>> code (PRCM interrupt handler + DSP reset.)
>>
>> TODO: cleanup dsp bridge code (arch/arm/mach-omap2/dsp.c.)
>>
>> Tested on OMAP3 beagle + 3.12-rc3.
>
> Looks like at least one patch has a dependency on your DT clk patches:
>
> Importing patch "clk-ti-add-dt-alias-clock" ...
> error: drivers/clk/ti/Makefile: does not exist in index
> error: include/linux/clk/ti.h: does not exist in index
> stg import: Diff does not apply cleanly
Hmm what? No it doesn't have any dependency. At most, there could be
merge conflicts between the two, as this is completely separated set.
-Tero
>
> Tried compiling the rest, but the build broke:
>
> arch/arm/mach-omap2/mcbsp.c: In function 'omap3_enable_st_clock':
> arch/arm/mach-omap2/mcbsp.c:46:3: error: implicit declaration of function 'omap2_clk_deny_idle'
> arch/arm/mach-omap2/mcbsp.c:48:3: error: implicit declaration of function 'omap2_clk_allow_idle'
> make[1]: *** [arch/arm/mach-omap2/mcbsp.o] Error 1
> make: *** [arch/arm/mach-omap2] Error 2
> make: *** Waiting for unfinished jobs....
>
>
> - Paul
>
WARNING: multiple messages have this Message-ID (diff)
From: t-kristo@ti.com (Tero Kristo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 00/11] ARM: OMAP2+: CM cleanup series
Date: Sat, 12 Oct 2013 13:18:47 +0300 [thread overview]
Message-ID: <52592207.4070508@ti.com> (raw)
In-Reply-To: <alpine.DEB.2.02.1310120139280.27397@utopia.booyaka.com>
On 10/12/2013 05:19 AM, Paul Walmsley wrote:
> Hi
>
> On Fri, 11 Oct 2013, Tero Kristo wrote:
>
>> A small cleanup set for CM. This basically gets rid of the omap2_cm_*
>> register accesses from the random code locations, and gathers these
>> under cm2xxx.c and cm3xxx.c. This is done in preparation for creating
>> a separate CM driver. The set also contains a couple of PRM cleanups
>> which I decided to take care of at the same time due to overlapping
>> code (PRCM interrupt handler + DSP reset.)
>>
>> TODO: cleanup dsp bridge code (arch/arm/mach-omap2/dsp.c.)
>>
>> Tested on OMAP3 beagle + 3.12-rc3.
>
> Looks like at least one patch has a dependency on your DT clk patches:
>
> Importing patch "clk-ti-add-dt-alias-clock" ...
> error: drivers/clk/ti/Makefile: does not exist in index
> error: include/linux/clk/ti.h: does not exist in index
> stg import: Diff does not apply cleanly
Hmm what? No it doesn't have any dependency. At most, there could be
merge conflicts between the two, as this is completely separated set.
-Tero
>
> Tried compiling the rest, but the build broke:
>
> arch/arm/mach-omap2/mcbsp.c: In function 'omap3_enable_st_clock':
> arch/arm/mach-omap2/mcbsp.c:46:3: error: implicit declaration of function 'omap2_clk_deny_idle'
> arch/arm/mach-omap2/mcbsp.c:48:3: error: implicit declaration of function 'omap2_clk_allow_idle'
> make[1]: *** [arch/arm/mach-omap2/mcbsp.o] Error 1
> make: *** [arch/arm/mach-omap2] Error 2
> make: *** Waiting for unfinished jobs....
>
>
> - Paul
>
next prev parent reply other threads:[~2013-10-12 10:19 UTC|newest]
Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-11 16:15 [PATCH 00/11] ARM: OMAP2+: CM cleanup series Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-11 16:15 ` [PATCH 01/11] ARM: OMAP2: CM/PM: remove direct register accesses outside CM code Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:12 ` Paul Walmsley
2013-10-19 17:12 ` Paul Walmsley
2013-10-11 16:15 ` [PATCH 02/11] ARM: OMAP3: clock: add API to enable/disable autoidle for a single clock Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:16 ` Paul Walmsley
2013-10-19 17:16 ` Paul Walmsley
2013-10-21 12:26 ` Mike Turquette
2013-10-21 12:26 ` Mike Turquette
2013-10-21 12:53 ` Paul Walmsley
2013-10-21 12:53 ` Paul Walmsley
2013-10-22 1:14 ` Paul Walmsley
2013-10-22 1:14 ` Paul Walmsley
2013-10-21 12:54 ` Rajendra Nayak
2013-10-21 12:54 ` Rajendra Nayak
2013-10-11 16:15 ` [PATCH 03/11] ARM: OMAP3: McBSP: do not access CM register directly Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:17 ` Paul Walmsley
2013-10-19 17:17 ` Paul Walmsley
2013-10-11 16:15 ` [PATCH 04/11] ARM: OMAP3: CM/control: move CM scratchpad save to CM driver Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:17 ` Paul Walmsley
2013-10-19 17:17 ` Paul Walmsley
2013-10-11 16:15 ` [PATCH 05/11] ARM: OMAP3: CM/PM: add API for accessing module clock enable registers Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:40 ` Paul Walmsley
2013-10-19 17:40 ` Paul Walmsley
2013-10-11 16:15 ` [PATCH 06/11] ARM: OMAP3: CM/PM: add API for forcing IVA2 clk enable/disable Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:19 ` Paul Walmsley
2013-10-19 17:19 ` Paul Walmsley
2013-10-19 17:30 ` Paul Walmsley
2013-10-19 17:30 ` Paul Walmsley
2013-10-19 17:42 ` Paul Walmsley
2013-10-19 17:42 ` Paul Walmsley
2013-10-11 16:15 ` [PATCH 07/11] ARM: OMAP3: CM/PM: add new API for checking IVA2 idle status Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:41 ` Paul Walmsley
2013-10-19 17:41 ` Paul Walmsley
2013-10-11 16:15 ` [PATCH 08/11] ARM: OMAP3: control: add API for setting IVA bootmode Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:18 ` Paul Walmsley
2013-10-19 17:18 ` Paul Walmsley
2013-10-11 16:15 ` [PATCH 09/11] ARM: OMAP3: PRM: move iva2 force idle functionality to PRM driver Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:36 ` Paul Walmsley
2013-10-19 17:36 ` Paul Walmsley
2013-10-11 16:15 ` [PATCH 10/11] ARM: OMAP3: PRM: move PRCM interrupt handler helper " Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-11 16:15 ` [PATCH 11/11] ARM: OMAP2+: CM: move direct register write macros to internal header Tero Kristo
2013-10-11 16:15 ` Tero Kristo
2013-10-19 17:36 ` Paul Walmsley
2013-10-19 17:36 ` Paul Walmsley
2013-10-12 0:09 ` [PATCH 00/11] ARM: OMAP2+: CM cleanup series Tony Lindgren
2013-10-12 0:09 ` Tony Lindgren
2013-10-12 0:18 ` Tony Lindgren
2013-10-12 0:18 ` Tony Lindgren
2013-10-12 2:19 ` Paul Walmsley
2013-10-12 2:19 ` Paul Walmsley
2013-10-12 10:18 ` Tero Kristo [this message]
2013-10-12 10:18 ` Tero Kristo
2013-10-12 21:45 ` Paul Walmsley
2013-10-12 21:45 ` Paul Walmsley
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=52592207.4070508@ti.com \
--to=t-kristo@ti.com \
--cc=khilman@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.com \
--cc=tony@atomide.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.