Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-15 Thread Víctor M . Jáquez L .
On Tue, Feb 14, 2012 at 08:56:11PM -0600, Ramirez Luna, Omar wrote: On Tue, Feb 14, 2012 at 10:23 AM, Felipe Contreras felipe.contre...@gmail.com wrote: When that case is applicable, we should first modify the loader code or prepare the baseimages to be common so we can get rid of specific

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-15 Thread Felipe Contreras
2012/2/15 Víctor M. Jáquez L. vjaq...@igalia.com: On Tue, Feb 14, 2012 at 08:56:11PM -0600, Ramirez Luna, Omar wrote: On Tue, Feb 14, 2012 at 10:23 AM, Felipe Contreras felipe.contre...@gmail.com wrote: When that case is applicable, we should first modify the loader code or prepare the

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-15 Thread Víctor M . Jáquez L .
On Wed, Feb 15, 2012 at 01:41:19PM +0200, Felipe Contreras wrote: 2012/2/15 Víctor M. Jáquez L. vjaq...@igalia.com: On Tue, Feb 14, 2012 at 08:56:11PM -0600, Ramirez Luna, Omar wrote: On Tue, Feb 14, 2012 at 10:23 AM, Felipe Contreras felipe.contre...@gmail.com wrote: When that case is

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-15 Thread Omar Ramirez Luna
2012/2/15 Víctor M. Jáquez L. vjaq...@igalia.com: On Tue, Feb 14, 2012 at 08:56:11PM -0600, Ramirez Luna, Omar wrote: On Tue, Feb 14, 2012 at 10:23 AM, Felipe Contreras felipe.contre...@gmail.com wrote: When that case is applicable, we should first modify the loader code or prepare the

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-14 Thread Felipe Contreras
On Tue, Feb 14, 2012 at 3:06 AM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Feb 11, 2012 3:03 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Sat, Feb 11, 2012 at 9:19 PM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Feb 10, 2012 12:44 PM, Felipe Contreras

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-14 Thread Ramirez Luna, Omar
On Tue, Feb 14, 2012 at 10:23 AM, Felipe Contreras felipe.contre...@gmail.com wrote: When that case is applicable, we should first modify the loader code or prepare the baseimages to be common so we can get rid of specific loaders and just dump them into memory. I'd say the less workarounds,

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-13 Thread Ramirez Luna, Omar
On Feb 11, 2012 3:03 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Sat, Feb 11, 2012 at 9:19 PM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Feb 10, 2012 12:44 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Fri, Feb 10, 2012 at 10:35 PM, Dan Carpenter

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-13 Thread Ramirez Luna, Omar
On Thu, Feb 9, 2012 at 9:18 PM, Greg KH gre...@linuxfoundation.org wrote: On Thu, Feb 09, 2012 at 04:45:00PM -0800, Ramirez Luna, Omar wrote: Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Again, I'm totally confused as to _WHY_ this needs to be y.  

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-11 Thread Felipe Contreras
On Sat, Feb 11, 2012 at 9:19 PM, Ramirez Luna, Omar omar.rami...@ti.com wrote: On Feb 10, 2012 12:44 PM, Felipe Contreras felipe.contre...@gmail.com wrote: On Fri, Feb 10, 2012 at 10:35 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Fri, Feb 10, 2012 at 09:42:32PM +0200, Felipe

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Justin P. Mattock
On Thu, 9 Feb 2012 21:18:49 -0800 Greg KH gre...@linuxfoundation.org wrote: On Thu, Feb 09, 2012 at 04:45:00PM -0800, Ramirez Luna, Omar wrote: Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Again, I'm totally confused as to _WHY_ this needs to

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Felipe Contreras
On Fri, Feb 10, 2012 at 5:35 PM, Justin P. Mattock justinmatt...@gmail.com wrote: On Thu, 9 Feb 2012 21:18:49 -0800 Greg KH gre...@linuxfoundation.org wrote: On Thu, Feb 09, 2012 at 04:45:00PM -0800, Ramirez Luna, Omar wrote: Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Justin P. Mattock
On Fri, 10 Feb 2012 18:05:59 +0200 Felipe Contreras felipe.contre...@gmail.com wrote: On Fri, Feb 10, 2012 at 5:35 PM, Justin P. Mattock justinmatt...@gmail.com wrote: On Thu, 9 Feb 2012 21:18:49 -0800 Greg KH gre...@linuxfoundation.org wrote: On Thu, Feb 09, 2012 at 04:45:00PM -0800,

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Felipe Contreras
On Fri, Feb 10, 2012 at 7:18 AM, Greg KH gre...@linuxfoundation.org wrote: On Thu, Feb 09, 2012 at 04:45:00PM -0800, Ramirez Luna, Omar wrote: Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Again, I'm totally confused as to _WHY_ this needs to be y.  

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Greg KH
On Fri, Feb 10, 2012 at 06:05:59PM +0200, Felipe Contreras wrote: From 5c7ad6c00d051d574007cdbecdf14bf3d0cb Mon Sep 17 00:00:00 2001 From: Justin P. Mattock justinmatt...@gmail.com Date: Fri, 10 Feb 2012 07:19:45 -0800 Subject: [PATCH] Add dependency TIDSBRIDGE_WDT3 to TIDSBRIDGE.

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Felipe Contreras
On Fri, Feb 10, 2012 at 6:16 PM, Greg KH gre...@linuxfoundation.org wrote: On Fri, Feb 10, 2012 at 06:05:59PM +0200, Felipe Contreras wrote: From 5c7ad6c00d051d574007cdbecdf14bf3d0cb Mon Sep 17 00:00:00 2001 From: Justin P. Mattock justinmatt...@gmail.com Date: Fri, 10 Feb 2012 07:19:45

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Greg KH
On Fri, Feb 10, 2012 at 06:16:19PM +0200, Felipe Contreras wrote: On Fri, Feb 10, 2012 at 7:18 AM, Greg KH gre...@linuxfoundation.org wrote: On Thu, Feb 09, 2012 at 04:45:00PM -0800, Ramirez Luna, Omar wrote: Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Dan Carpenter
On Fri, Feb 10, 2012 at 01:30:48AM +0200, Felipe Contreras wrote: It's not an oops, it's a warning, and again, it depends on the firmware being used. We don't have control over that, and we have no way to detect if this feature is there. It's up to the user. Perhaps just remove the warning

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Felipe Contreras
On Fri, Feb 10, 2012 at 8:00 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Fri, Feb 10, 2012 at 01:30:48AM +0200, Felipe Contreras wrote: It's not an oops, it's a warning, and again, it depends on the firmware being used. We don't have control over that, and we have no way to detect if

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Dan Carpenter
On Fri, Feb 10, 2012 at 09:42:32PM +0200, Felipe Contreras wrote: On Fri, Feb 10, 2012 at 8:00 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Fri, Feb 10, 2012 at 01:30:48AM +0200, Felipe Contreras wrote: It's not an oops, it's a warning, and again, it depends on the firmware being

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-10 Thread Felipe Contreras
On Fri, Feb 10, 2012 at 10:35 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Fri, Feb 10, 2012 at 09:42:32PM +0200, Felipe Contreras wrote: On Fri, Feb 10, 2012 at 8:00 PM, Dan Carpenter dan.carpen...@oracle.com wrote: On Fri, Feb 10, 2012 at 01:30:48AM +0200, Felipe Contreras wrote:

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-09 Thread Greg KH
On Wed, Feb 01, 2012 at 09:26:16AM +0200, Felipe Contreras wrote: On Wed, Feb 1, 2012 at 5:44 AM, Greg KH g...@kroah.com wrote: On Wed, Feb 01, 2012 at 03:37:55AM +0200, Felipe Contreras wrote: On Wed, Feb 1, 2012 at 3:12 AM, Greg KH gre...@suse.de wrote: On Wed, Feb 01, 2012 at 03:01:32AM

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-09 Thread Felipe Contreras
On Thu, Feb 9, 2012 at 7:35 PM, Greg KH gre...@linuxfoundation.org wrote: On Wed, Feb 01, 2012 at 09:26:16AM +0200, Felipe Contreras wrote: On Wed, Feb 1, 2012 at 5:44 AM, Greg KH g...@kroah.com wrote: On Wed, Feb 01, 2012 at 03:37:55AM +0200, Felipe Contreras wrote: On Wed, Feb 1, 2012 at

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-09 Thread Felipe Contreras
On Thu, Feb 9, 2012 at 8:41 PM, Felipe Contreras felipe.contre...@gmail.com wrote: I already gave you a good reason, and this came from Linus Torvalds. http://article.gmane.org/gmane.linux.kernel/995419 Notice how he agreed that setting defaults was a good idea. I can't point to the mails of

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-09 Thread Greg KH
On Thu, Feb 09, 2012 at 08:41:55PM +0200, Felipe Contreras wrote: On Thu, Feb 9, 2012 at 7:35 PM, Greg KH gre...@linuxfoundation.org wrote: On Wed, Feb 01, 2012 at 09:26:16AM +0200, Felipe Contreras wrote: On Wed, Feb 1, 2012 at 5:44 AM, Greg KH g...@kroah.com wrote: On Wed, Feb 01, 2012

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-09 Thread Felipe Contreras
On Thu, Feb 9, 2012 at 8:59 PM, Greg KH gre...@linuxfoundation.org wrote: On Thu, Feb 09, 2012 at 08:41:55PM +0200, Felipe Contreras wrote: On Thu, Feb 9, 2012 at 7:35 PM, Greg KH gre...@linuxfoundation.org wrote: On Wed, Feb 01, 2012 at 09:26:16AM +0200, Felipe Contreras wrote: On Wed, Feb

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-09 Thread Ramirez Luna, Omar
Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Again, I'm totally confused as to _WHY_ this needs to be y.  What is causing this oops without it?  If an oops is happening, then shouldn't this be a strict dependancy?  Why allow it to be disabled at all if

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-09 Thread Greg KH
On Thu, Feb 09, 2012 at 04:45:00PM -0800, Ramirez Luna, Omar wrote: Hi, On Thu, Feb 9, 2012 at 3:30 PM, Felipe Contreras felipe.contre...@gmail.com wrote: Again, I'm totally confused as to _WHY_ this needs to be y.  What is causing this oops without it?  If an oops is happening, then

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-01 Thread Felipe Contreras
On Wed, Feb 1, 2012 at 5:22 AM, Justin P. Mattock justinmatt...@gmail.com wrote: not sure if I see this warning or not on my machine, but is there a fix for the warning? I would rather see that, than hide it with setting: default y (if that's what its doing!) No, there's no fix. IIRC Omar said

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-02-01 Thread Ramirez Luna, Omar
On Wed, Feb 1, 2012 at 2:11 AM, Felipe Contreras felipe.contre...@gmail.com wrote: On Wed, Feb 1, 2012 at 5:22 AM, Justin P. Mattock justinmatt...@gmail.com wrote: not sure if I see this warning or not on my machine, but is there a fix for the warning? I would rather see that, than hide it

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-01-31 Thread Greg KH
On Wed, Feb 01, 2012 at 03:01:32AM +0200, Felipe Contreras wrote: From: Felipe Contreras felipe.contre...@nokia.com The public images have it enabled, it's safer, and we get rid of this warning: WARNING: at arch/arm/mach-omap2/omap_l3_smx.c:162 omap3_l3_app_irq+0x114/0x15c() In-band

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-01-31 Thread Felipe Contreras
On Wed, Feb 1, 2012 at 3:12 AM, Greg KH gre...@suse.de wrote: On Wed, Feb 01, 2012 at 03:01:32AM +0200, Felipe Contreras wrote: From: Felipe Contreras felipe.contre...@nokia.com The public images have it enabled, it's safer, and we get rid of this warning: WARNING: at

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-01-31 Thread Greg KH
On Wed, Feb 01, 2012 at 03:37:55AM +0200, Felipe Contreras wrote: On Wed, Feb 1, 2012 at 3:12 AM, Greg KH gre...@suse.de wrote: On Wed, Feb 01, 2012 at 03:01:32AM +0200, Felipe Contreras wrote: From: Felipe Contreras felipe.contre...@nokia.com The public images have it enabled, it's

Re: [PATCH] staging: tidspbridge: enable watchdog by default

2012-01-31 Thread Felipe Contreras
On Wed, Feb 1, 2012 at 5:44 AM, Greg KH g...@kroah.com wrote: On Wed, Feb 01, 2012 at 03:37:55AM +0200, Felipe Contreras wrote: On Wed, Feb 1, 2012 at 3:12 AM, Greg KH gre...@suse.de wrote: On Wed, Feb 01, 2012 at 03:01:32AM +0200, Felipe Contreras wrote: From: Felipe Contreras