Re: [PATCH v2] thermal: ti-soc-thermal: dra7: Implement Workaround for Errata i814 - Bandgap Temperature read Dtemp can be corrupted

2015-04-07 Thread Keerthy
Hi Eduardo, Thanks for the review comments. On Wednesday 08 April 2015 01:35 AM, Eduardo Valentin wrote: Hello K, On Tue, Apr 07, 2015 at 08:50:04AM +0530, Keerthy wrote: Hi Eduardo, On Friday 27 March 2015 02:15 PM, Keerthy wrote: Bandgap Temperature read Dtemp can be corrupted DESCRIPTIO

Re: [PATCH] mm/migrate: Mark unmap_and_move() "noinline" to avoid ICE in gcc 4.7.3

2015-04-07 Thread Kevin Hilman
Andrew Morton writes: > On Tue, 07 Apr 2015 16:27:44 -0700 Kevin Hilman wrote: > >> >> > It should all be there today? >> >> >> >> Nope. >> > >> > huh, I swear I did an mmotm yesterday. >> >> Well, based on the timestamp of the mmotm dir on ozlabs, it appears you >> did. That's why I was co

Re: [PATCH] mm/migrate: Mark unmap_and_move() "noinline" to avoid ICE in gcc 4.7.3

2015-04-07 Thread Andrew Morton
On Tue, 07 Apr 2015 16:27:44 -0700 Kevin Hilman wrote: > >> > It should all be there today? > >> > >> Nope. > > > > huh, I swear I did an mmotm yesterday. > > Well, based on the timestamp of the mmotm dir on ozlabs, it appears you > did. That's why I was confused why the gcc-473 patches from

Re: [PATCH] mm/migrate: Mark unmap_and_move() "noinline" to avoid ICE in gcc 4.7.3

2015-04-07 Thread Kevin Hilman
Andrew Morton writes: > On Tue, 07 Apr 2015 15:41:32 -0700 Kevin Hilman wrote: > >> Andrew Morton writes: >> >> > On Tue, 7 Apr 2015 10:57:52 -0700 Kevin Hilman wrote: >> > >> >> > The diff below[2] on top of yours compiles fine here and at least covers >> >> > the compilers I *know* to trigg

Re: [PATCH 06/14] clkdev: add clkdev_create() helper

2015-04-07 Thread Stephen Boyd
On 04/07/15 05:43, Russell King - ARM Linux wrote: > On Mon, Apr 06, 2015 at 01:19:33PM -0700, Stephen Boyd wrote: >> On 04/03/15 10:12, Russell King wrote: >>> @@ -316,6 +329,29 @@ clkdev_alloc(struct clk *clk, const char *con_id, >>> const char *dev_fmt, ...) >>> } >>> EXPORT_SYMBOL(clkdev_all

Re: [PATCH] mm/migrate: Mark unmap_and_move() "noinline" to avoid ICE in gcc 4.7.3

2015-04-07 Thread Andrew Morton
On Tue, 07 Apr 2015 15:41:32 -0700 Kevin Hilman wrote: > Andrew Morton writes: > > > On Tue, 7 Apr 2015 10:57:52 -0700 Kevin Hilman wrote: > > > >> > The diff below[2] on top of yours compiles fine here and at least covers > >> > the compilers I *know* to trigger the ICE. > >> > >> I see my f

Re: [PATCH] mm/migrate: Mark unmap_and_move() "noinline" to avoid ICE in gcc 4.7.3

2015-04-07 Thread Kevin Hilman
Andrew Morton writes: > On Tue, 7 Apr 2015 10:57:52 -0700 Kevin Hilman wrote: > >> > The diff below[2] on top of yours compiles fine here and at least covers >> > the compilers I *know* to trigger the ICE. >> >> I see my fix in your mmots since last Thurs (4/2), but it's not in >> mmotm (last u

Re: [PATCH] mm/migrate: Mark unmap_and_move() "noinline" to avoid ICE in gcc 4.7.3

2015-04-07 Thread Andrew Morton
On Tue, 7 Apr 2015 10:57:52 -0700 Kevin Hilman wrote: > > The diff below[2] on top of yours compiles fine here and at least covers > > the compilers I *know* to trigger the ICE. > > I see my fix in your mmots since last Thurs (4/2), but it's not in > mmotm (last updated today) so today's linux-n

Re: [PATCH v2] thermal: ti-soc-thermal: dra7: Implement Workaround for Errata i814 - Bandgap Temperature read Dtemp can be corrupted

2015-04-07 Thread Eduardo Valentin
Hello K, On Tue, Apr 07, 2015 at 08:50:04AM +0530, Keerthy wrote: > Hi Eduardo, > > On Friday 27 March 2015 02:15 PM, Keerthy wrote: > >Bandgap Temperature read Dtemp can be corrupted > > > >DESCRIPTION > > Read accesses to registers listed below can be corrupted due to > > incorrect res

Re: [PATCHv2] ti-soc-thermal: implement omap3 support

2015-04-07 Thread Eduardo Valentin
Hi Pavel, On Thu, Apr 02, 2015 at 04:49:07PM +0200, Pavel Machek wrote: > > > This adds support for OMAP3 chips to ti-soc-thermal. As requested by > TI people, it is marked unreliable and warning is printed. > > Signed-off-by: Pavel Machek > > --- > > Patch is against thermal linus tree, ple

Re: [PATCHv2] ti-soc-thermal: implement omap3 support

2015-04-07 Thread Eduardo Valentin
On Thu, Apr 02, 2015 at 04:49:07PM +0200, Pavel Machek wrote: > > > This adds support for OMAP3 chips to ti-soc-thermal. As requested by > TI people, it is marked unreliable and warning is printed. > > Signed-off-by: Pavel Machek > > --- > > Patch is against thermal linus tree, please apply s

Re: [PATCH] mm/migrate: Mark unmap_and_move() "noinline" to avoid ICE in gcc 4.7.3

2015-04-07 Thread Kevin Hilman
Hi Andrew, On Wed, Apr 1, 2015 at 2:54 PM, Kevin Hilman wrote: > Andrew Morton writes: > >> On Wed, 01 Apr 2015 10:47:49 +0100 Marc Zyngier wrote: >> >>> > -static int unmap_and_move(new_page_t get_new_page, free_page_t >>> > put_new_page, >>> > - unsigned long private, struct

Re: [GIT PULL] ARM: OMAP2+: OMAP DRA7xx display-related hwmod data changes for v4.1

2015-04-07 Thread Paul Walmsley
On Tue, 7 Apr 2015, Tomi Valkeinen wrote: > Hi, > > No, don't pull this, it doesn't work. > > Paul, the hwmod patch cannot be picked from the DRA7 DSS series. We have > the problem with the DES HDCP clock, without that clock we will see > WARNINGs when booting if the "set DSS submodule parent hw

Re: ARM errata 430973 on multi platform kernels

2015-04-07 Thread Tony Lindgren
* Tony Lindgren [150407 08:27]: > * Russell King - ARM Linux [150407 06:58]: > > > > Well, one thing we can do is to tweak the proc-v7*.S such that we detect > > Cortex-A8 separately, and only execute the BTB flush for CA8 processors > > if the errata is enabled. Something like this (untested):

Re: ARM errata 430973 on multi platform kernels

2015-04-07 Thread Tony Lindgren
* Russell King - ARM Linux [150407 06:58]: > On Mon, Apr 06, 2015 at 10:42:45AM -0700, Tony Lindgren wrote: > > * Ivaylo Dimitrov [150406 10:15]: > > > On 6.04.2015 18:40, Tony Lindgren wrote: > > > > > > > >Oops sorry, wrong numbers for errata above.. s/458693/430973/, here's > > > >a better ve

Re: ARM errata 430973 on multi platform kernels

2015-04-07 Thread Tony Lindgren
* Matthijs van Duin [150406 20:50]: > On 7 April 2015 at 04:23, Tony Lindgren wrote: > > Oops, sorry user error.. I was trying to clear IBE as a banked register > > like L2 enable bit and of course it did not get cleared.. Clearing it > > with a smc call really clears it. And in that case my test

Re: [PATCH 07/14] media: omap3isp: remove unused clkdev

2015-04-07 Thread Laurent Pinchart
Hi Russell, On Tuesday 07 April 2015 13:45:36 Russell King - ARM Linux wrote: > On Tue, Apr 07, 2015 at 12:42:52PM +0300, Laurent Pinchart wrote: > > On Sunday 05 April 2015 15:20:34 Russell King - ARM Linux wrote: > > > On Sat, Apr 04, 2015 at 12:44:35AM +0300, Laurent Pinchart wrote: > > > > Hi

Re: [PATCH 10/14] ARM: orion: use clkdev_create()

2015-04-07 Thread Gregory CLEMENT
Hi Russell, On 07/04/2015 16:01, Russell King - ARM Linux wrote: > On Tue, Apr 07, 2015 at 03:20:05PM +0200, Gregory CLEMENT wrote: >> Hi Andrew, Russell, >> >> On 04/04/2015 02:17, Andrew Lunn wrote: >>> On Fri, Apr 03, 2015 at 06:13:13PM +0100, Russell King wrote: clkdev_create() is a short

Re: [PATCH 10/14] ARM: orion: use clkdev_create()

2015-04-07 Thread Russell King - ARM Linux
On Tue, Apr 07, 2015 at 03:20:05PM +0200, Gregory CLEMENT wrote: > Hi Andrew, Russell, > > On 04/04/2015 02:17, Andrew Lunn wrote: > > On Fri, Apr 03, 2015 at 06:13:13PM +0100, Russell King wrote: > >> clkdev_create() is a shorter way to write clkdev_alloc() followed by > >> clkdev_add(). Use thi

Re: ARM errata 430973 on multi platform kernels

2015-04-07 Thread Russell King - ARM Linux
On Mon, Apr 06, 2015 at 08:14:30PM +0200, Matthijs van Duin wrote: > * Ivaylo Dimitrov [150406 10:15]: > > Why custom function, if IBE bit is zero, BTB invalidate instruction is a > > NOP. Do you think that "mcr p15, 0, r2, c7, c5, 6" executed as a NOP will > > put so much overhead, that it deserv

Re: ARM errata 430973 on multi platform kernels

2015-04-07 Thread Russell King - ARM Linux
On Mon, Apr 06, 2015 at 10:42:45AM -0700, Tony Lindgren wrote: > * Ivaylo Dimitrov [150406 10:15]: > > On 6.04.2015 18:40, Tony Lindgren wrote: > > > > > >Oops sorry, wrong numbers for errata above.. s/458693/430973/, here's > > >a better version: > > > > > >1. For cortex-a8 revisions affected by

Re: [PATCH 10/14] ARM: orion: use clkdev_create()

2015-04-07 Thread Gregory CLEMENT
Hi Andrew, Russell, On 04/04/2015 02:17, Andrew Lunn wrote: > On Fri, Apr 03, 2015 at 06:13:13PM +0100, Russell King wrote: >> clkdev_create() is a shorter way to write clkdev_alloc() followed by >> clkdev_add(). Use this instead. >> >> Signed-off-by: Russell King > > Acked-by: Andrew Lunn Th

Re: [PATCH 07/14] media: omap3isp: remove unused clkdev

2015-04-07 Thread Russell King - ARM Linux
On Tue, Apr 07, 2015 at 12:42:52PM +0300, Laurent Pinchart wrote: > Hello Russell, > > On Sunday 05 April 2015 15:20:34 Russell King - ARM Linux wrote: > > On Sat, Apr 04, 2015 at 12:44:35AM +0300, Laurent Pinchart wrote: > > > Hi Russell, > > > > > > Thank you for the patch; > > > > > > On Frid

Re: [PATCH 06/14] clkdev: add clkdev_create() helper

2015-04-07 Thread Russell King - ARM Linux
On Mon, Apr 06, 2015 at 01:19:33PM -0700, Stephen Boyd wrote: > On 04/03/15 10:12, Russell King wrote: > > @@ -316,6 +329,29 @@ clkdev_alloc(struct clk *clk, const char *con_id, > > const char *dev_fmt, ...) > > } > > EXPORT_SYMBOL(clkdev_alloc); > > > > +/** > > + * clkdev_create - allocate a

Re: [PATCH 03/14] clkdev: get rid of redundant clk_add_alias() prototype in linux/clk.h

2015-04-07 Thread Russell King - ARM Linux
On Mon, Apr 06, 2015 at 12:04:21PM -0700, Stephen Boyd wrote: > On 04/04/15 05:43, Robert Jarzmik wrote: > > Russell King writes: > > > >> clk_add_alias() is provided by clkdev, and is not part of the clk API. > >> Howver, it is prototyped in two locations: linux/clkdev.h and > >> linux/clk.h. Th

Re: [PATCHv2 03/10] ARM: DRA7: hwmod: set DSS submodule parent hwmods

2015-04-07 Thread Tomi Valkeinen
On 03/04/15 22:03, Paul Walmsley wrote: > On Tue, 24 Mar 2015, Tomi Valkeinen wrote: > >> Set DSS core hwmod as the parent for all the DSS submodules. >> >> Signed-off-by: Tomi Valkeinen > > Thanks, queued. Will send a pull request for this to Tony today, but it > may not make it for v4.1.

Re: [GIT PULL] ARM: OMAP2+: OMAP DRA7xx display-related hwmod data changes for v4.1

2015-04-07 Thread Tomi Valkeinen
Hi, No, don't pull this, it doesn't work. Paul, the hwmod patch cannot be picked from the DRA7 DSS series. We have the problem with the DES HDCP clock, without that clock we will see WARNINGs when booting if the "set DSS submodule parent hwmods" is applied. We do see some errors when booting eve

Re: [PATCH 03/14] clkdev: get rid of redundant clk_add_alias() prototype in linux/clk.h

2015-04-07 Thread Robert Jarzmik
Stephen Boyd writes: > On 04/04/15 05:43, Robert Jarzmik wrote: >> Russell King writes: >> >>> clk_add_alias() is provided by clkdev, and is not part of the clk API. >>> Howver, it is prototyped in two locations: linux/clkdev.h and >>> linux/clk.h. This is a mess. Get rid of the redundant and

Re: [PATCH 07/14] media: omap3isp: remove unused clkdev

2015-04-07 Thread Laurent Pinchart
Hello Russell, On Sunday 05 April 2015 15:20:34 Russell King - ARM Linux wrote: > On Sat, Apr 04, 2015 at 12:44:35AM +0300, Laurent Pinchart wrote: > > Hi Russell, > > > > Thank you for the patch; > > > > On Friday 03 April 2015 18:12:58 Russell King wrote: > > > No merged platform supplies xclk

Re: [rtc-linux] [PATCH] rtc: OMAP: Add external 32k clock feature

2015-04-07 Thread Igor Grinberg
Hi, On 04/07/15 06:29, Keerthy wrote: > Hi Andrew, > > Apologies for replying late. > > On Wednesday 25 March 2015 04:29 AM, Andrew Morton wrote: >> On Tue, 3 Mar 2015 15:12:02 +0530 Keerthy wrote: >> >>> Add external 32k clock feature. The internal clock will be gated during >>> suspend. >>>