From: Kevin Hilman <khilman@ti.com>
To: Jean Pihet <jean.pihet@newoldbits.com>
Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH/RFT 5/8] ARM: OMAP: clock: remove unused CK_3505 flag
Date: Mon, 30 Apr 2012 10:19:40 -0700 [thread overview]
Message-ID: <87397lw30z.fsf@ti.com> (raw)
In-Reply-To: <CAORVsuVgNKUcsM+GuQfahp0r5U=24qBQhrX1E-DewhNvh0hk7g@mail.gmail.com> (Jean Pihet's message of "Mon, 30 Apr 2012 11:05:41 +0200")
Jean Pihet <jean.pihet@newoldbits.com> writes:
> Hi Kevin,
>
> On Fri, Apr 27, 2012 at 1:29 AM, Kevin Hilman <khilman@ti.com> wrote:
>> This flag is no longer used since clock init all AM35x devices
>> is now the same.
>>
>> Acked-by: Vaibhav Hiremath <hvaibhav@ti.com>
>> Tested-by: Vaibhav Hiremath <hvaibhav@ti.com>
>> Signed-off-by: Kevin Hilman <khilman@ti.com>
>> ---
>> arch/arm/plat-omap/include/plat/clkdev_omap.h | 3 +--
>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/plat-omap/include/plat/clkdev_omap.h b/arch/arm/plat-omap/include/plat/clkdev_omap.h
>> index b299b8d..45afa4d 100644
>> --- a/arch/arm/plat-omap/include/plat/clkdev_omap.h
>> +++ b/arch/arm/plat-omap/include/plat/clkdev_omap.h
>> @@ -34,7 +34,6 @@ struct omap_clk {
>> #define CK_243X (1 << 5) /* 243x, 253x */
>> #define CK_3430ES1 (1 << 6) /* 34xxES1 only */
>> #define CK_3430ES2PLUS (1 << 7) /* 34xxES2, ES3, non-Sitara 35xx only */
>> -#define CK_3505 (1 << 8)
>> #define CK_3517 (1 << 9)
> If this is the only flag remaining, should it better be renamed to CK_35XX?
That happens later in the series.
Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
WARNING: multiple messages have this Message-ID (diff)
From: khilman@ti.com (Kevin Hilman)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH/RFT 5/8] ARM: OMAP: clock: remove unused CK_3505 flag
Date: Mon, 30 Apr 2012 10:19:40 -0700 [thread overview]
Message-ID: <87397lw30z.fsf@ti.com> (raw)
In-Reply-To: <CAORVsuVgNKUcsM+GuQfahp0r5U=24qBQhrX1E-DewhNvh0hk7g@mail.gmail.com> (Jean Pihet's message of "Mon, 30 Apr 2012 11:05:41 +0200")
Jean Pihet <jean.pihet@newoldbits.com> writes:
> Hi Kevin,
>
> On Fri, Apr 27, 2012 at 1:29 AM, Kevin Hilman <khilman@ti.com> wrote:
>> This flag is no longer used since clock init all AM35x devices
>> is now the same.
>>
>> Acked-by: Vaibhav Hiremath <hvaibhav@ti.com>
>> Tested-by: Vaibhav Hiremath <hvaibhav@ti.com>
>> Signed-off-by: Kevin Hilman <khilman@ti.com>
>> ---
>> ?arch/arm/plat-omap/include/plat/clkdev_omap.h | ? ?3 +--
>> ?1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/arch/arm/plat-omap/include/plat/clkdev_omap.h b/arch/arm/plat-omap/include/plat/clkdev_omap.h
>> index b299b8d..45afa4d 100644
>> --- a/arch/arm/plat-omap/include/plat/clkdev_omap.h
>> +++ b/arch/arm/plat-omap/include/plat/clkdev_omap.h
>> @@ -34,7 +34,6 @@ struct omap_clk {
>> ?#define CK_243X ? ? ? ? ? ? ? ?(1 << 5) ? ? ? ?/* 243x, 253x */
>> ?#define CK_3430ES1 ? ? (1 << 6) ? ? ? ?/* 34xxES1 only */
>> ?#define CK_3430ES2PLUS (1 << 7) ? ? ? ?/* 34xxES2, ES3, non-Sitara 35xx only */
>> -#define CK_3505 ? ? ? ? ? ? ? ?(1 << 8)
>> ?#define CK_3517 ? ? ? ? ? ? ? ?(1 << 9)
> If this is the only flag remaining, should it better be renamed to CK_35XX?
That happens later in the series.
Kevin
next prev parent reply other threads:[~2012-04-30 17:19 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-26 23:29 [PATCH/RFT 0/8] ARM: OMAP: remove IP checks from SoC family detection Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 1/8] ARM: OMAP: remove unused cpu_is macros that depend on specific IP checks Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 2/8] ARM: OMAP3: clock data: replace 3503/3517 flag with AM35x flag for UART4 Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 3/8] ARM: OMAP3: clock data: treat all AM35x devices the same Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 4/8] ARM: OMAP: AM35x: remove redunant cpu_is checks for AM3505 Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 5/8] ARM: OMAP: clock: remove unused CK_3505 flag Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-30 9:05 ` Jean Pihet
2012-04-30 9:05 ` Jean Pihet
2012-04-30 17:19 ` Kevin Hilman [this message]
2012-04-30 17:19 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 6/8] ARM: OMAP: remove unused cpu_is_omap3505() Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-30 9:07 ` Jean Pihet
2012-04-30 9:07 ` Jean Pihet
2012-04-30 9:35 ` Koen Kooi
2012-04-30 9:35 ` Koen Kooi
2012-04-30 17:20 ` Kevin Hilman
2012-04-30 17:20 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 7/8] ARM: OMAP: remove unused cpu_is_omap3530() Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-26 23:29 ` [PATCH/RFT 8/8] ARM: OMAP: AM35xx: convert 3517 detection/flags to AM35xx Kevin Hilman
2012-04-26 23:29 ` Kevin Hilman
2012-04-27 22:08 ` [PATCH/RFT 0/8] ARM: OMAP: remove IP checks from SoC family detection Mark A. Greer
2012-04-27 22:08 ` Mark A. Greer
2012-04-27 22:25 ` Kevin Hilman
2012-04-27 22:25 ` Kevin Hilman
2012-04-28 20:20 ` Derald D. Woods
2012-04-29 9:48 ` Hiremath, Vaibhav
2012-04-30 16:30 ` Mark A. Greer
2012-04-30 16:27 ` Mark A. Greer
2012-04-30 16:25 ` Mark A. Greer
2012-04-30 16:25 ` Mark A. Greer
2012-04-30 20:48 ` Kevin Hilman
2012-04-30 20:48 ` Kevin Hilman
2012-04-30 23:47 ` Kevin Hilman
2012-04-30 23:47 ` Kevin Hilman
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=87397lw30z.fsf@ti.com \
--to=khilman@ti.com \
--cc=jean.pihet@newoldbits.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
/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.