Message ID | 20120704033018.GA1862@S2100-06.ap.freescale.net |
---|---|
State | New |
Headers | show |
On Wednesday 04 July 2012, Shawn Guo wrote: > > Dammit. I forgot testing imx_v4_v5_defconfig again with the last FIQ > change. Thanks, Fabio. > > Arnd, > > I just updated the branch imx/sparse-irq with this fix applied. Can > you please pull it? In the train at the moment without internet access, so I just applied the patch and added your S-o-b, on top of the next/irq branch. Arnd
On Wed, Jul 04, 2012 at 11:30:22AM +0800, Shawn Guo wrote: > Dammit. I forgot testing imx_v4_v5_defconfig again with the last FIQ > change. Thanks, Fabio. So, this begs the question - which configs have you tested this series with?
On Wed, Jul 04, 2012 at 10:53:02AM +0100, Russell King - ARM Linux wrote: > On Wed, Jul 04, 2012 at 11:30:22AM +0800, Shawn Guo wrote: > > Dammit. I forgot testing imx_v4_v5_defconfig again with the last FIQ > > change. Thanks, Fabio. > > So, this begs the question - which configs have you tested this series with? For the patch changing FIQ_START to variable, I tested the following defconfigs. rpc_defconfig s3c2410_defconfig imx_v6_v7_defconfig As avic.c is built with imx_v6_v7_defconfig as well, I thought it's been covered there. But it turns out that CONFIG_FIQ is not enabled by imx_v6_v7_defconfig in any way, while it's enabled for imx_v4_v5_defconfig because there are a couple of imx_v4_v5 audio drivers selecting the symbol. sound/soc/fsl/Kconfig: config SND_SOC_IMX_PCM_FIQ tristate select FIQ select SND_SOC_IMX_PCM