From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@deeprootsystems.com (Kevin Hilman) Date: Tue, 07 Dec 2010 12:59:51 -0800 Subject: OMAP2+: PM/serial: hold console semaphore while OMAP UARTs are disabled In-Reply-To: <20101125005923.GE31536@atomide.com> (Tony Lindgren's message of "Wed, 24 Nov 2010 16:59:23 -0800") References: <87oc9e5j4c.fsf@deeprootsystems.com> <20101125005923.GE31536@atomide.com> Message-ID: <87sjy9l2ns.fsf@deeprootsystems.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Tony Lindgren writes: > * Kevin Hilman [101124 16:32]: >> Paul Walmsley writes: > > > >> Acked-by: Kevin Hilman >> >> Very nice. I've been exploring various solutions to this problem as >> well, but this one is much cleaner. Also, I hadn't discovered the 'try' >> version of the console semaphore, so was running into recursive locking. >> >> Anyways, tested on omap35xx: omap3evm (uart1/core console) and beagle >> (uart3/per console) and omap34xx/n900 (uart3/per console) using both >> retention-idle and off-idle. > > Thanks, queuing this as fix for the -rc cycle. FYI.. just found a regression caused by this patch. It prevented suspend from ever happening since the console semaphore is already held during suspend. Just posted a fix for this: Date: Tue, 7 Dec 2010 12:24:13 -0800 Subject: [PATCH] OMAP2+: PM/serial: fix console semaphore acquire during suspend Kevin