public inbox for linux-omap@vger.kernel.org
 help / color / mirror / Atom feed
From: "Girish S G" <girishsg@ti.com>
To: "'Woodruff, Richard'" <r-woodruff2@ti.com>,
	'Tero Kristo' <tero.kristo@nokia.com>,
	linux-omap@vger.kernel.org
Cc: 'Jouni Hogander' <jouni.hogander@nokia.com>
Subject: RE: [PATCH 12/17] OMAP2/3: Do not enable AUTOIDLE in interrupt controller
Date: Fri, 16 Oct 2009 12:47:34 -0500	[thread overview]
Message-ID: <004f01ca4e88$be2a6a50$3d1018ac@am.dhcp.ti.com> (raw)
In-Reply-To: <13B9B4C6EF24D648824FF11BE8967162039B2A021C@dlee02.ent.ti.com>



> -----Original Message-----
> From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-owner@vger.kernel.org] On Behalf Of
> Woodruff, Richard
> Sent: Friday, October 16, 2009 9:40 AM
> To: Tero Kristo; linux-omap@vger.kernel.org
> Cc: Jouni Hogander
> Subject: RE: [PATCH 12/17] OMAP2/3: Do not enable AUTOIDLE in interrupt controller
> 
> 
> > From: linux-omap-owner@vger.kernel.org [mailto:linux-omap-
> > owner@vger.kernel.org] On Behalf Of Tero Kristo
> > Sent: Friday, October 16, 2009 5:49 AM
> 
> > OMAP interrupt controller goes to unknown state when there is right
> > combination of l3,l4 sleep/wake-up transitions, l4 autoidle in
> > interrupt controller and some interrupt. When this happens, interrupts
> > are not delivered to ARM anymore and ARM will remain in WFI (wait for
> > interrupt) until interrupt controller is forced to wake-up
> > (i.e. lauterbach). Disable AUTOIDLE in INTC for now.
> 
> Optimal work around enables and disables this around WFI.

On one of the custom board the power measured didn't show any major impact, with just one time disabling of INTC-AUTOIDL. However,
optimizing this WA looks good though. I did give it a try,
Disabling/enabling INTC autoidle around WFI on custom board, works well but didn't get chance to
measure the numbers.

Regards,
-Girish



  reply	other threads:[~2009-10-16 17:47 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-10-16 10:48 [PATCH 00/17] Misc PM patches Tero Kristo
2009-10-16 10:48 ` [PATCH 01/17] PM: fix suspend control for IVA2 Tero Kristo
2009-10-16 10:48   ` [PATCH 02/17] OMAP3: PM: Dynamic check for CORE target state Tero Kristo
2009-10-16 10:48     ` [PATCH 03/17] PM: Block CORE off when DSS is active Tero Kristo
2009-10-16 10:48       ` [PATCH 04/17] OMAP3: PM: Do not enable IO wake-up on boot Tero Kristo
2009-10-16 10:48         ` [PATCH 05/17] OMAP:PM: Precedence fix Tero Kristo
2009-10-16 10:48           ` [PATCH 06/17] OMAP3: PM: Added next state check for IVA2, USB and PER into idle loop Tero Kristo
2009-10-16 10:49             ` [PATCH 07/17] OMAP3: Disable Smartreflex before pwrdm enters RET Tero Kristo
2009-10-16 10:49               ` [PATCH 08/17] OMAP2/3: GPTIMER: Clear pending interrupts when entering suspend Tero Kristo
2009-10-16 10:49                 ` [PATCH 09/17] OMAP3: PM: Ack pending interrupts before " Tero Kristo
2009-10-16 10:49                   ` [PATCH 10/17] OMAP3: PM: Enable system control module autoidle Tero Kristo
2009-10-16 10:49                     ` [PATCH 11/17] Make USBHOST powerdomain go to sleep after warm reset Tero Kristo
2009-10-16 10:49                       ` [PATCH 12/17] OMAP2/3: Do not enable AUTOIDLE in interrupt controller Tero Kristo
2009-10-16 10:49                         ` [PATCH 13/17] OMAP3: Fixed ARM aux ctrl register save/restore Tero Kristo
2009-10-16 10:49                           ` [PATCH 14/17] OMAP3: PM: Disabled I2C4 repeated start operation mode Tero Kristo
2009-10-16 10:49                             ` [PATCH 15/17] OMAP3: PM: Added support for L2 aux ctrl register save and restore Tero Kristo
2009-10-16 10:49                               ` [PATCH 16/17] OMAP3: PM: Write voltage and clock setup times dynamically in idle loop Tero Kristo
2009-10-16 10:49                                 ` [PATCH 17/17] OMAP3: PM: Force disable OTG autoidle Tero Kristo
2009-10-20 18:44                                   ` Kevin Hilman
2009-10-21 10:21                                     ` Tero.Kristo
2009-10-20 17:47                                 ` [PATCH 16/17] OMAP3: PM: Write voltage and clock setup times dynamically in idle loop Kevin Hilman
2009-10-21  9:55                                   ` Tero.Kristo
2009-10-21 14:15                                     ` Kevin Hilman
2009-10-21 15:37                                       ` Tero.Kristo
2009-10-16 14:39                         ` [PATCH 12/17] OMAP2/3: Do not enable AUTOIDLE in interrupt controller Woodruff, Richard
2009-10-16 17:47                           ` Girish S G [this message]
2009-10-16 18:04                             ` Woodruff, Richard
     [not found]                               ` <005001ca4e8b$ea107c10$3d1018ac@am.dhcp.ti! .com>
2009-10-16 18:10                               ` Girish S G
2009-10-19 10:18                                 ` Tero.Kristo
2009-10-19 17:21                                   ` Woodruff, Richard
2009-10-19 18:31                                   ` Girish S G
2009-10-20 16:01                                   ` Kevin Hilman
2009-10-20 16:15                                     ` Tero.Kristo
2009-10-20 16:32                                       ` Kevin Hilman
2009-10-21  4:37                                         ` Gopinath, Thara
2009-10-21 15:25                                           ` Girish S G
2009-10-20 16:19                                     ` Girish S G
2009-10-20 17:39                       ` [PATCH 11/17] Make USBHOST powerdomain go to sleep after warm reset Kevin Hilman
2009-10-20 17:30                   ` [PATCH 09/17] OMAP3: PM: Ack pending interrupts before entering suspend Kevin Hilman
2009-10-21 10:03                     ` Tero.Kristo
2009-10-21 14:19                       ` Kevin Hilman
2009-10-21 16:01                         ` Tero.Kristo
2009-10-20 17:36                 ` [PATCH 08/17] OMAP2/3: GPTIMER: Clear pending interrupts when " Kevin Hilman
2009-10-21 10:09                   ` Tero.Kristo
2009-10-16 13:16             ` [PATCH 06/17] OMAP3: PM: Added next state check for IVA2, USB and PER into idle loop Sripathy, Vishwanath
2009-10-16 13:35               ` Tero.Kristo
2009-10-16 14:57               ` Cousson, Benoit
2009-10-20 19:04             ` Kevin Hilman
2009-10-20 17:23           ` [PATCH 05/17] OMAP:PM: Precedence fix Kevin Hilman
2009-10-20 17:21         ` [PATCH 04/17] OMAP3: PM: Do not enable IO wake-up on boot Kevin Hilman
2009-10-16 15:27       ` [PATCH 03/17] PM: Block CORE off when DSS is active Cousson, Benoit
2009-10-19  9:37         ` Tero.Kristo
2009-10-20 17:19           ` Kevin Hilman
2009-10-20 16:16     ` [PATCH 02/17] OMAP3: PM: Dynamic check for CORE target state Kevin Hilman
2009-10-16 17:16   ` [PATCH 01/17] PM: fix suspend control for IVA2 Girish S G
2009-10-19  9:22     ` Tero.Kristo
2009-10-19 18:26       ` Girish S G
2009-10-20 16:10       ` Kevin Hilman
2009-10-22 10:24   ` Paul Walmsley
2009-10-22 21:21     ` Woodruff, Richard
2009-10-23  5:22       ` Artem Bityutskiy

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='004f01ca4e88$be2a6a50$3d1018ac@am.dhcp.ti.com' \
    --to=girishsg@ti.com \
    --cc=jouni.hogander@nokia.com \
    --cc=linux-omap@vger.kernel.org \
    --cc=r-woodruff2@ti.com \
    --cc=tero.kristo@nokia.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox