From: balbi@ti•com (Felipe Balbi)
To: linux-arm-kernel@lists•infradead.org
Subject: Dropping "depends on SMP" for HAVE_ARM_TWD -- take 2
Date: Tue, 6 Oct 2015 15:01:05 -0500 [thread overview]
Message-ID: <87k2qz22da.fsf@saruman.tx.rr.com> (raw)
In-Reply-To: <20151006193815.GI12338@codeaurora.org>
Stephen Boyd <sboyd@codeaurora•org> writes:
> On 10/06, Mason wrote:
>> On 02/10/2015 20:02, Stephen Boyd wrote:
>>
>> > The warning has been removed in commit 5028090d1da1 (ARM: 8434/1:
>> > Revert "7655/1: smp_twd: make twd_local_timer_of_register() no-op
>> > for nosmp", 2015-09-14) sitting in linux-next. Oddly, that commit
>> > doesn't remove the depends on SMP for the Kconfig. So it seems
>> > that we can apply this patch and everyone is happy? We could also
>> > drop the "if SMP" part of most of the platform selects if anyone
>> > actually cares.
>> >
>> > ---8<----
>> > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
>> > index 369791fb619c..be64d9d604c3 100644
>> > --- a/arch/arm/Kconfig
>> > +++ b/arch/arm/Kconfig
>> > @@ -1389,7 +1389,6 @@ config HAVE_ARM_ARCH_TIMER
>> >
>> > config HAVE_ARM_TWD
>> > bool
>> > - depends on SMP
>> > select CLKSRC_OF if OF
>> > help
>> > This options enables support for the ARM timer and watchdog unit
>>
>> Hello Stephen,
>>
>> Should I make a formal submission for the above patch to be accepted?
>
> No. I believe Felipe is going to update commit 5028090d1da1 to
> remove the depends on.
Already did it. Should be in next shortly.
--
balbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20151006/5823d60b/attachment.sig>
next prev parent reply other threads:[~2015-10-06 20:01 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-02 9:52 Dropping "depends on SMP" for HAVE_ARM_TWD -- take 2 Mason
2015-10-02 13:24 ` Mason
2015-10-02 18:02 ` Stephen Boyd
2015-10-02 18:48 ` Felipe Balbi
2015-10-02 18:55 ` Russell King - ARM Linux
2015-10-02 19:18 ` Felipe Balbi
2015-10-02 18:50 ` Mason
2015-10-02 19:41 ` Stephen Boyd
2015-10-02 23:34 ` Måns Rullgård
2015-10-03 9:32 ` Marc Zyngier
2015-10-03 9:49 ` Mason
2015-10-03 10:12 ` Marc Zyngier
2015-10-05 5:46 ` Sören Brinkmann
2015-10-05 7:38 ` Mason
2015-10-05 9:03 ` Linus Walleij
2015-10-05 11:49 ` Afzal Mohammed
2015-10-05 15:13 ` Sören Brinkmann
2015-10-05 15:25 ` Russell King - ARM Linux
2015-10-06 8:21 ` Mason
2015-10-06 19:38 ` Stephen Boyd
2015-10-06 20:01 ` Felipe Balbi [this message]
2015-10-06 20:17 ` Mason
2015-10-06 21:05 ` Felipe Balbi
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=87k2qz22da.fsf@saruman.tx.rr.com \
--to=balbi@ti$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox