All of lore.kernel.org
 help / color / mirror / Atom feed
From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: Ingo Molnar <mingo@kernel.org>
Cc: Thomas Gleixner <tglx@linutronix.de>,
	Axel Lin <axel.lin@ingics.com>, Dinh Nguyen <dinguyen@altera.com>,
	Ezequiel Garcia <ezequiel.garcia@free-electrons.com>,
	Marc Zyngier <marc.zyngier@arm.com>,
	Thierry Reding <thierry.reding@gmail.com>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [GIT PULL] clocksource/clockevents fixes for 3.13
Date: Tue, 10 Dec 2013 13:59:19 +0100	[thread overview]
Message-ID: <52A71027.4000902@linaro.org> (raw)
In-Reply-To: <20131210122051.GD30001@gmail.com>

On 12/10/2013 01:20 PM, Ingo Molnar wrote:
>
> * Daniel Lezcano <daniel.lezcano@linaro.org> wrote:
>
>>
>> Hi Thomas,
>>
>> here are the fixes for 3.13 based on timers/urgent
>
> Please generally also post the patches to make it easier to review
> them.

Hi Ingo,

what do you mean by "also post the patches" ? The patches were posted in 
the mailing lists and I picked them up after people reviewed them.

Or shall I give a combined diff when sending the PR ?

>>   * Axel Lin removed the bcm_timer_ids which are no longer used due
>> to the conversion to clksrc-of and added a missing dependency on
>> CLKSRC_MMIO in the Kconfig for the time-efm32.
>
> Is the ID removal a necessary regression fix? If not then it has no
> place in the urgent queue.

No, you are right. It is not a regression fix. I assumed it was an 
acceptable patch for urgent but apparently not. Thanks for clarifying 
this. Let me resend a new PR without this patch.

   -- Daniel


-- 
  <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog


  reply	other threads:[~2013-12-10 12:59 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-05 16:39 [GIT PULL] clocksource/clockevents fixes for 3.13 Daniel Lezcano
2013-12-10 12:20 ` Ingo Molnar
2013-12-10 12:59   ` Daniel Lezcano [this message]
2013-12-10 13:36     ` Ingo Molnar
2013-12-10 14:30       ` Daniel Lezcano

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=52A71027.4000902@linaro.org \
    --to=daniel.lezcano@linaro.org \
    --cc=axel.lin@ingics.com \
    --cc=dinguyen@altera.com \
    --cc=ezequiel.garcia@free-electrons.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marc.zyngier@arm.com \
    --cc=mingo@kernel.org \
    --cc=tglx@linutronix.de \
    --cc=thierry.reding@gmail.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.