From: horms@verge.net.au (Simon Horman)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] Renesas ARM Based SoC Drivers Updates for v4.7
Date: Wed, 20 Apr 2016 08:48:55 +1000 [thread overview]
Message-ID: <20160419224852.GA9826@verge.net.au> (raw)
In-Reply-To: <CAMuHMdWtZg2A62vr9hDua+wvMnqTQVPeTQO7a=GZGoBzusz5Tg@mail.gmail.com>
On Tue, Apr 19, 2016 at 09:25:20AM +0200, Geert Uytterhoeven wrote:
> Hi Simon,
>
> On Mon, Apr 18, 2016 at 5:55 AM, Simon Horman <horms@verge.net.au> wrote:
> > On Fri, Apr 15, 2016 at 08:35:48AM +0200, Geert Uytterhoeven wrote:
> >> On Fri, Apr 15, 2016 at 2:33 AM, Simon Horman <horms@verge.net.au> wrote:
> >> > On Thu, Apr 14, 2016 at 10:19:04AM +0200, Geert Uytterhoeven wrote:
> >> >> On Wed, Apr 13, 2016 at 10:23 PM, Olof Johansson <olof@lixom.net> wrote:
> >> >> > On Wed, Apr 13, 2016 at 09:57:39PM +0200, Geert Uytterhoeven wrote:
> >> >> >> On Wed, Apr 13, 2016 at 9:43 PM, Olof Johansson <olof@lixom.net> wrote:
> >> >> >> > On Mon, Apr 04, 2016 at 10:22:56AM +0900, Simon Horman wrote:
> >> >> >> >> Geert Uytterhoeven (2):
> >> >> >> >> ARM: shmobile: timer: Fix preset_lpj leading to too short delays
> >> >> >> >> ARM: shmobile: timer: Drop support for Cortex A8
> >>
> >> >> Simon: I'm a bit surprised to see that "ARM: shmobile: timer: Fix preset_lpj
> >> >> leading to too short delays" got postponed to v4.7, while commit 406663ed61c8
> >> >> ("ARM: shmobile: defconfig: Do not enable CONFIG_CPU_BPREDICT_DISABLE") is
> >> >> already in v4.6-rc1. The latter makes bad behavior due to too short delays
> >> >> more likely to happen.
> >> >>
> >> >> Sorry for not noticing before.
> >> >
> >> > Likewise, sorry for the mix-up.
> >> >
> >> > I can think of two ways to resolve this, likely there are others:
> >> > 1. Submit "ARM: shmobile: timer: Fix preset_lpj leading to too short delays"
> >> > or conversely
> >> > 2. Submit a reversion of 406663ed61c8 ("ARM: shmobile: defconfig: Do not
> >> > enable CONFIG_CPU_BPREDICT_DISABLE") as a fix for v4.6.
> >> >
> >> > Or perhaps it is a problem that can just be ironed out in v4.7.
> >> > Let me know what you think.
> >>
> >> I prefer option 1, i.e. fixing the bug instead of papering over it in
> >> the defconfig.
> >> Users may not use the defconfig, and have CONFIG_CPU_BPREDICT_DISABLE
> >> enabled anyway.
> >
> > Thanks, that reasoning sounds fine to me.
> >
> > I plan to re-queue "ARM: shmobile: timer: Fix preset_lpj leading to too
> > short delays" in fixes-for-v4.6 and I plan to push that in a revised
> > renesas-next tag later today. I would appreciate you double checking that
> > all is in order.
>
> Thanks, that commit looks fine!
Great, thanks for checking.
prev parent reply other threads:[~2016-04-19 22:48 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-04 1:22 [GIT PULL] Renesas ARM Based SoC Drivers Updates for v4.7 Simon Horman
2016-04-04 1:22 ` [PATCH 1/2] ARM: shmobile: timer: Fix preset_lpj leading to too short delays Simon Horman
2016-04-04 1:22 ` [PATCH 2/2] ARM: shmobile: timer: Drop support for Cortex A8 Simon Horman
2016-04-04 1:22 ` Simon Horman
2016-04-13 19:41 ` Olof Johansson
2016-04-13 19:50 ` Geert Uytterhoeven
2016-04-13 19:43 ` [GIT PULL] Renesas ARM Based SoC Drivers Updates for v4.7 Olof Johansson
2016-04-13 19:57 ` Geert Uytterhoeven
2016-04-13 20:23 ` Olof Johansson
2016-04-14 8:19 ` Geert Uytterhoeven
2016-04-15 0:33 ` Simon Horman
2016-04-15 6:35 ` Geert Uytterhoeven
2016-04-18 3:55 ` Simon Horman
2016-04-19 7:25 ` Geert Uytterhoeven
2016-04-19 22:48 ` Simon Horman [this message]
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=20160419224852.GA9826@verge.net.au \
--to=horms@verge.net.au \
--cc=linux-arm-kernel@lists.infradead.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.