Re: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile

2015-01-19 Thread Felipe Balbi
On Sun, Jan 18, 2015 at 09:52:14AM +, Lee Jones wrote: On Fri, 26 Dec 2014, 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

Re: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile

2015-01-18 Thread Lee Jones
On Fri, 26 Dec 2014, 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

Re: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile

2015-01-16 Thread Felipe Balbi
Hi, On Wed, Jan 14, 2015 at 09:07:17AM -0800, Tony Lindgren 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.

Re: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile

2015-01-14 Thread Tony Lindgren
* Felipe Balbi ba...@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

Re: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile

2015-01-12 Thread Felipe Balbi
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

Re: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile

2015-01-08 Thread Felipe Balbi
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

Re: [PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile

2015-01-06 Thread Felipe Balbi
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

[PATCH 1/5] mfd: tps65218: make INT[12] and STATUS registers volatile

2014-12-26 Thread Felipe Balbi
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