All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tony Lindgren <tony@atomide.com>
To: Felipe Balbi <balbi@ti.com>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Samuel Ortiz <sameo@linux.intel.com>,
	Lee Jones <lee.jones@linaro.org>,
	Linux OMAP Mailing List <linux-omap@vger.kernel.org>,
	linux-input@vger.kernel.org, devicetree@vger.kernel.org,
	Linux ARM Kernel Mailing List
	<linux-arm-kernel@lists.infradead.org>,
	stable@vger.kernel.org, Keerthy <j-keerthy@ti.com>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile
Date: Wed, 14 Jan 2015 09:07:17 -0800	[thread overview]
Message-ID: <20150114170716.GL2419@atomide.com> (raw)
In-Reply-To: <20150112164654.GB6525@saruman>

* Felipe Balbi <balbi@ti.com> [150112 08:50]:
> Hi,
> 
> On Thu, Jan 08, 2015 at 10:25:12AM -0600, Felipe Balbi wrote:
> > On Tue, Jan 06, 2015 at 11:37:34AM -0600, Felipe Balbi wrote:
> > > On Fri, Dec 26, 2014 at 01:28:20PM -0600, Felipe Balbi wrote:
> > > > STATUS register can be modified by the HW, so we
> > > > should bypass cache because of that.
> > > > 
> > > > In the case of INT[12] registers, they are the ones
> > > > that actually clear the IRQ source at the time they
> > > > are read. If we rely on the cache for them, we will
> > > > never be able to clear the interrupt, which will cause
> > > > our IRQ line to be disabled due to IRQ throttling.
> > > > 
> > > > Fixes: 44b4dc6 mfd: tps65218: Add driver for the TPS65218 PMIC
> > > > Cc: <stable@vger.kernel.org> # v3.15+
> > > > Cc: Keerthy <j-keerthy@ti.com>
> > > > Cc: Lee Jones <lee.jones@linaro.org>
> > > > Signed-off-by: Felipe Balbi <balbi@ti.com>
> > > 
> > > ping
> > 
> > another ping. Without this and the following patch TPS65218 power button
> > driver which was already applied by Dmitry, won't work.
> 
> Anybody ? -rc4 is out and tps65218 is still broken because nobody has
> acted on these two patches. All other patches which are meant for 3.20
> merge window are applied and because of these pending, those patches
> won't work.

Lee, are you planning to pick these two drivers/mfd patches?

Regards,

Tony

WARNING: multiple messages have this Message-ID (diff)
From: tony@atomide.com (Tony Lindgren)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile
Date: Wed, 14 Jan 2015 09:07:17 -0800	[thread overview]
Message-ID: <20150114170716.GL2419@atomide.com> (raw)
In-Reply-To: <20150112164654.GB6525@saruman>

* Felipe Balbi <balbi@ti.com> [150112 08:50]:
> Hi,
> 
> On Thu, Jan 08, 2015 at 10:25:12AM -0600, Felipe Balbi wrote:
> > On Tue, Jan 06, 2015 at 11:37:34AM -0600, Felipe Balbi wrote:
> > > On Fri, Dec 26, 2014 at 01:28:20PM -0600, Felipe Balbi wrote:
> > > > STATUS register can be modified by the HW, so we
> > > > should bypass cache because of that.
> > > > 
> > > > In the case of INT[12] registers, they are the ones
> > > > that actually clear the IRQ source at the time they
> > > > are read. If we rely on the cache for them, we will
> > > > never be able to clear the interrupt, which will cause
> > > > our IRQ line to be disabled due to IRQ throttling.
> > > > 
> > > > Fixes: 44b4dc6 mfd: tps65218: Add driver for the TPS65218 PMIC
> > > > Cc: <stable@vger.kernel.org> # v3.15+
> > > > Cc: Keerthy <j-keerthy@ti.com>
> > > > Cc: Lee Jones <lee.jones@linaro.org>
> > > > Signed-off-by: Felipe Balbi <balbi@ti.com>
> > > 
> > > ping
> > 
> > another ping. Without this and the following patch TPS65218 power button
> > driver which was already applied by Dmitry, won't work.
> 
> Anybody ? -rc4 is out and tps65218 is still broken because nobody has
> acted on these two patches. All other patches which are meant for 3.20
> merge window are applied and because of these pending, those patches
> won't work.

Lee, are you planning to pick these two drivers/mfd patches?

Regards,

Tony

  reply	other threads:[~2015-01-14 17:11 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-26 19:28 [PATCH 0/5] AM437x SK: Add power-button support Felipe Balbi
2014-12-26 19:28 ` Felipe Balbi
2014-12-26 19:28 ` [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile Felipe Balbi
2014-12-26 19:28   ` Felipe Balbi
     [not found]   ` <1419622104-25812-2-git-send-email-balbi-l0cyMroinI0@public.gmane.org>
2015-01-06 17:37     ` Felipe Balbi
2015-01-06 17:37       ` Felipe Balbi
2015-01-08 16:25       ` Felipe Balbi
2015-01-08 16:25         ` Felipe Balbi
2015-01-12 16:46         ` Felipe Balbi
2015-01-12 16:46           ` Felipe Balbi
2015-01-14 17:07           ` Tony Lindgren [this message]
2015-01-14 17:07             ` Tony Lindgren
2015-01-16 22:57             ` Felipe Balbi
2015-01-16 22:57               ` Felipe Balbi
2015-01-18  9:52   ` Lee Jones
2015-01-18  9:52     ` Lee Jones
2015-01-19 14:41     ` Felipe Balbi
2015-01-19 14:41       ` Felipe Balbi
2014-12-26 19:28 ` [PATCH 2/5] mfd: tps65218: make INT1 our status_base register Felipe Balbi
2014-12-26 19:28   ` Felipe Balbi
2015-01-06 17:37   ` Felipe Balbi
2015-01-06 17:37     ` Felipe Balbi
2015-01-18  9:52   ` Lee Jones
2015-01-18  9:52     ` Lee Jones
2014-12-26 19:28 ` [PATCH 3/5] input: misc: add tps65218 power button driver Felipe Balbi
2014-12-26 19:28   ` Felipe Balbi
2014-12-26 19:33   ` Felipe Balbi
2014-12-26 19:33     ` Felipe Balbi
2014-12-26 23:41   ` Dmitry Torokhov
2014-12-26 23:41     ` Dmitry Torokhov
2014-12-27  3:36     ` [PATCH v2 " Felipe Balbi
2014-12-27  3:36       ` Felipe Balbi
2014-12-28  4:50       ` Dmitry Torokhov
2014-12-28  4:50         ` Dmitry Torokhov
2014-12-26 19:28 ` [PATCH 4/5] arm: boot: dts: am437x-sk: add power button binding Felipe Balbi
2014-12-26 19:28   ` Felipe Balbi
2015-01-08  0:09   ` Tony Lindgren
2015-01-08  0:09     ` Tony Lindgren
2014-12-26 19:28 ` [PATCH 5/5] arm: omap2plus_defconfig: enable TPS65218 power button Felipe Balbi
2014-12-26 19:28   ` Felipe Balbi
2015-01-08  0:55   ` Tony Lindgren
2015-01-08  0:55     ` Tony Lindgren

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=20150114170716.GL2419@atomide.com \
    --to=tony@atomide.com \
    --cc=akpm@linux-foundation.org \
    --cc=balbi@ti.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dmitry.torokhov@gmail.com \
    --cc=j-keerthy@ti.com \
    --cc=lee.jones@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=sameo@linux.intel.com \
    --cc=stable@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.