From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Ujfalusi Subject: Re: [PATCH] ARM: OMAP: DRA7: hwmod: Add data for McASP3 Date: Wed, 11 Nov 2015 21:25:53 +0200 Message-ID: <56439641.6070901@ti.com> References: <1447261631-26362-1-git-send-email-balbi@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <1447261631-26362-1-git-send-email-balbi@ti.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Felipe Balbi , Paul Walmsley Cc: Tony Lindgren , Linux OMAP Mailing List , Linux ARM Kernel Mailing List List-Id: linux-omap@vger.kernel.org Felipe, On 11/11/2015 07:07 PM, Felipe Balbi wrote: > From: Peter Ujfalusi > = > McASP3 is used by default on DRA7x based boards for audio. While this patch works, it is not the correct one(s) to apply. https://lkml.org/lkml/2015/10/30/91 is the series which should have been applied to 4.4. Paul acked the series, but for some reason it missed the initial ARM pull f= or 4.4 -- = P=E9ter > = > This fixes following error with Beagle x15: > = > [ 0.402190] platform 48468000.mcasp: Cannot lookup hwmod 'mcasp3' > = > which is followed by a splat: > = > [ 11.207663] Unhandled fault: imprecise external abort (0x1406) at 0x00= 000000 > [ 11.207670] ------------[ cut here ]------------ > [ 11.207683] WARNING: CPU: 0 PID: 272 at /home/balbi/workspace/linux/dr= ivers/bus/omap_l3_noc.c:147 l3_interrupt_handler+0x220/0x34c() > [ 11.207688] 44000000.ocp:L3 Custom Error: MASTER MPU TARGET L4_PER2_P3= (Read): Data Access in User mode during Functional access > [ 11.207742] Modules linked in: snd_soc_simple_card(+) omapdss leds_gpi= o led_class gpio_fan extcon_usb_gpio cpufreq_dt omap_wdt phy_omap_usb2 snd_= soc_davinci_mcasp snd_soc_edma snd_soc_omap snd_soc_tlv320aic3x rtc_omap sn= d_soc_core ti_soc_thermal dwc3_omap snd_pcm_dmaengine snd_pcm extcon_palmas= snd_timer extcon snd rtc_palmas rtc_ds1307 tmp102 palmas_pwrbutton thermal= _sys soundcore hwmon autofs4 > [ 11.207748] CPU: 0 PID: 272 Comm: systemd-udevd Not tainted 4.3.0-1148= 2-g3d26f1f5fc82-dirty #27 > [ 11.207751] Hardware name: Generic DRA74X (Flattened Device Tree) > [ 11.207765] [] (unwind_backtrace) from [] (show_st= ack+0x10/0x14) > [ 11.207772] [] (show_stack) from [] (dump_stack+0x= 84/0x9c) > [ 11.207783] [] (dump_stack) from [] (warn_slowpath= _common+0x78/0xb4) > [ 11.207790] [] (warn_slowpath_common) from [] (war= n_slowpath_fmt+0x30/0x40) > [ 11.207798] [] (warn_slowpath_fmt) from [] (l3_int= errupt_handler+0x220/0x34c) > [ 11.207805] [] (l3_interrupt_handler) from [] (han= dle_irq_event_percpu+0x44/0x1f0) > [ 11.207811] [] (handle_irq_event_percpu) from [] (= handle_irq_event+0x38/0x5c) > [ 11.207818] [] (handle_irq_event) from [] (handle_= fasteoi_irq+0xcc/0x1b0) > [ 11.207824] [] (handle_fasteoi_irq) from [] (gener= ic_handle_irq+0x20/0x34) > [ 11.207829] [] (generic_handle_irq) from [] (__han= dle_domain_irq+0x64/0xe0) > [ 11.207835] [] (__handle_domain_irq) from [] (gic_= handle_irq+0x40/0x8c) > [ 11.207842] [] (gic_handle_irq) from [] (__irq_usr= +0x54/0x80) > [ 11.207846] Exception stack(0xed143fb0 to 0xed143ff8) > [ 11.207851] 3fa0: 7f69b370 7f69b36= 8 00000370 00000000 > [ 11.207857] 3fc0: b6ddb4e0 7f69b370 00000016 7f69b308 be8db278 be8db2a= 0 7f68b024 00000001 > [ 11.207863] 3fe0: b6ddb4e0 be8db180 b6d46a47 b6d46a50 60000030 ffffffff > [ 11.207866] ---[ end trace 7d8de48d1bc8fbac ]--- > = > Signed-off-by: Peter Ujfalusi > Signed-off-by: Felipe Balbi > --- > arch/arm/mach-omap2/omap_hwmod_7xx_data.c | 41 +++++++++++++++++++++++++= ++++++ > 1 file changed, 41 insertions(+) > = > diff --git a/arch/arm/mach-omap2/omap_hwmod_7xx_data.c b/arch/arm/mach-om= ap2/omap_hwmod_7xx_data.c > index 51d1ecb384bd..b730c582a1f7 100644 > --- a/arch/arm/mach-omap2/omap_hwmod_7xx_data.c > +++ b/arch/arm/mach-omap2/omap_hwmod_7xx_data.c > @@ -1298,6 +1298,38 @@ static struct omap_hwmod dra7xx_mcspi4_hwmod =3D { > }; > = > /* > + * 'mcasp' class > + * > + */ > +static struct omap_hwmod_class_sysconfig dra7xx_mcasp_sysc =3D { > + .sysc_offs =3D 0x0004, > + .sysc_flags =3D SYSC_HAS_SIDLEMODE, > + .idlemodes =3D (SIDLE_FORCE | SIDLE_NO | SIDLE_SMART), > + .sysc_fields =3D &omap_hwmod_sysc_type3, > +}; > + > +static struct omap_hwmod_class dra7xx_mcasp_hwmod_class =3D { > + .name =3D "mcasp", > + .sysc =3D &dra7xx_mcasp_sysc, > +}; > + > +/* mcasp3 */ > +static struct omap_hwmod dra7xx_mcasp3_hwmod =3D { > + .name =3D "mcasp3", > + .class =3D &dra7xx_mcasp_hwmod_class, > + .clkdm_name =3D "l4per2_clkdm", > + .main_clk =3D "mcasp3_ahclkx_mux", > + .flags =3D HWMOD_SWSUP_SIDLE, > + .prcm =3D { > + .omap4 =3D { > + .clkctrl_offs =3D DRA7XX_CM_L4PER2_MCASP3_CLKCTRL_OFFSET, > + .context_offs =3D DRA7XX_RM_L4PER2_MCASP3_CONTEXT_OFFSET, > + .modulemode =3D MODULEMODE_SWCTRL, > + }, > + }, > +}; > + > +/* > * 'mmc' class > * > */ > @@ -2566,6 +2598,14 @@ static struct omap_hwmod_ocp_if dra7xx_l3_main_1__= hdmi =3D { > .user =3D OCP_USER_MPU | OCP_USER_SDMA, > }; > = > +/* l4_per2 -> mcasp3 */ > +static struct omap_hwmod_ocp_if dra7xx_l4_per2__mcasp3 =3D { > + .master =3D &dra7xx_l4_per2_hwmod, > + .slave =3D &dra7xx_mcasp3_hwmod, > + .clk =3D "l3_iclk_div", > + .user =3D OCP_USER_MPU | OCP_USER_SDMA, > +}; > + > /* l4_per1 -> elm */ > static struct omap_hwmod_ocp_if dra7xx_l4_per1__elm =3D { > .master =3D &dra7xx_l4_per1_hwmod, > @@ -3308,6 +3348,7 @@ static struct omap_hwmod_ocp_if *dra7xx_hwmod_ocp_i= fs[] __initdata =3D { > &dra7xx_l4_wkup__dcan1, > &dra7xx_l4_per2__dcan2, > &dra7xx_l4_per2__cpgmac0, > + &dra7xx_l4_per2__mcasp3, > &dra7xx_gmac__mdio, > &dra7xx_l4_cfg__dma_system, > &dra7xx_l3_main_1__dss, > = From mboxrd@z Thu Jan 1 00:00:00 1970 From: peter.ujfalusi@ti.com (Peter Ujfalusi) Date: Wed, 11 Nov 2015 21:25:53 +0200 Subject: [PATCH] ARM: OMAP: DRA7: hwmod: Add data for McASP3 In-Reply-To: <1447261631-26362-1-git-send-email-balbi@ti.com> References: <1447261631-26362-1-git-send-email-balbi@ti.com> Message-ID: <56439641.6070901@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Felipe, On 11/11/2015 07:07 PM, Felipe Balbi wrote: > From: Peter Ujfalusi > > McASP3 is used by default on DRA7x based boards for audio. While this patch works, it is not the correct one(s) to apply. https://lkml.org/lkml/2015/10/30/91 is the series which should have been applied to 4.4. Paul acked the series, but for some reason it missed the initial ARM pull for 4.4 -- P?ter > > This fixes following error with Beagle x15: > > [ 0.402190] platform 48468000.mcasp: Cannot lookup hwmod 'mcasp3' > > which is followed by a splat: > > [ 11.207663] Unhandled fault: imprecise external abort (0x1406) at 0x00000000 > [ 11.207670] ------------[ cut here ]------------ > [ 11.207683] WARNING: CPU: 0 PID: 272 at /home/balbi/workspace/linux/drivers/bus/omap_l3_noc.c:147 l3_interrupt_handler+0x220/0x34c() > [ 11.207688] 44000000.ocp:L3 Custom Error: MASTER MPU TARGET L4_PER2_P3 (Read): Data Access in User mode during Functional access > [ 11.207742] Modules linked in: snd_soc_simple_card(+) omapdss leds_gpio led_class gpio_fan extcon_usb_gpio cpufreq_dt omap_wdt phy_omap_usb2 snd_soc_davinci_mcasp snd_soc_edma snd_soc_omap snd_soc_tlv320aic3x rtc_omap snd_soc_core ti_soc_thermal dwc3_omap snd_pcm_dmaengine snd_pcm extcon_palmas snd_timer extcon snd rtc_palmas rtc_ds1307 tmp102 palmas_pwrbutton thermal_sys soundcore hwmon autofs4 > [ 11.207748] CPU: 0 PID: 272 Comm: systemd-udevd Not tainted 4.3.0-11482-g3d26f1f5fc82-dirty #27 > [ 11.207751] Hardware name: Generic DRA74X (Flattened Device Tree) > [ 11.207765] [] (unwind_backtrace) from [] (show_stack+0x10/0x14) > [ 11.207772] [] (show_stack) from [] (dump_stack+0x84/0x9c) > [ 11.207783] [] (dump_stack) from [] (warn_slowpath_common+0x78/0xb4) > [ 11.207790] [] (warn_slowpath_common) from [] (warn_slowpath_fmt+0x30/0x40) > [ 11.207798] [] (warn_slowpath_fmt) from [] (l3_interrupt_handler+0x220/0x34c) > [ 11.207805] [] (l3_interrupt_handler) from [] (handle_irq_event_percpu+0x44/0x1f0) > [ 11.207811] [] (handle_irq_event_percpu) from [] (handle_irq_event+0x38/0x5c) > [ 11.207818] [] (handle_irq_event) from [] (handle_fasteoi_irq+0xcc/0x1b0) > [ 11.207824] [] (handle_fasteoi_irq) from [] (generic_handle_irq+0x20/0x34) > [ 11.207829] [] (generic_handle_irq) from [] (__handle_domain_irq+0x64/0xe0) > [ 11.207835] [] (__handle_domain_irq) from [] (gic_handle_irq+0x40/0x8c) > [ 11.207842] [] (gic_handle_irq) from [] (__irq_usr+0x54/0x80) > [ 11.207846] Exception stack(0xed143fb0 to 0xed143ff8) > [ 11.207851] 3fa0: 7f69b370 7f69b368 00000370 00000000 > [ 11.207857] 3fc0: b6ddb4e0 7f69b370 00000016 7f69b308 be8db278 be8db2a0 7f68b024 00000001 > [ 11.207863] 3fe0: b6ddb4e0 be8db180 b6d46a47 b6d46a50 60000030 ffffffff > [ 11.207866] ---[ end trace 7d8de48d1bc8fbac ]--- > > Signed-off-by: Peter Ujfalusi > Signed-off-by: Felipe Balbi > --- > arch/arm/mach-omap2/omap_hwmod_7xx_data.c | 41 +++++++++++++++++++++++++++++++ > 1 file changed, 41 insertions(+) > > diff --git a/arch/arm/mach-omap2/omap_hwmod_7xx_data.c b/arch/arm/mach-omap2/omap_hwmod_7xx_data.c > index 51d1ecb384bd..b730c582a1f7 100644 > --- a/arch/arm/mach-omap2/omap_hwmod_7xx_data.c > +++ b/arch/arm/mach-omap2/omap_hwmod_7xx_data.c > @@ -1298,6 +1298,38 @@ static struct omap_hwmod dra7xx_mcspi4_hwmod = { > }; > > /* > + * 'mcasp' class > + * > + */ > +static struct omap_hwmod_class_sysconfig dra7xx_mcasp_sysc = { > + .sysc_offs = 0x0004, > + .sysc_flags = SYSC_HAS_SIDLEMODE, > + .idlemodes = (SIDLE_FORCE | SIDLE_NO | SIDLE_SMART), > + .sysc_fields = &omap_hwmod_sysc_type3, > +}; > + > +static struct omap_hwmod_class dra7xx_mcasp_hwmod_class = { > + .name = "mcasp", > + .sysc = &dra7xx_mcasp_sysc, > +}; > + > +/* mcasp3 */ > +static struct omap_hwmod dra7xx_mcasp3_hwmod = { > + .name = "mcasp3", > + .class = &dra7xx_mcasp_hwmod_class, > + .clkdm_name = "l4per2_clkdm", > + .main_clk = "mcasp3_ahclkx_mux", > + .flags = HWMOD_SWSUP_SIDLE, > + .prcm = { > + .omap4 = { > + .clkctrl_offs = DRA7XX_CM_L4PER2_MCASP3_CLKCTRL_OFFSET, > + .context_offs = DRA7XX_RM_L4PER2_MCASP3_CONTEXT_OFFSET, > + .modulemode = MODULEMODE_SWCTRL, > + }, > + }, > +}; > + > +/* > * 'mmc' class > * > */ > @@ -2566,6 +2598,14 @@ static struct omap_hwmod_ocp_if dra7xx_l3_main_1__hdmi = { > .user = OCP_USER_MPU | OCP_USER_SDMA, > }; > > +/* l4_per2 -> mcasp3 */ > +static struct omap_hwmod_ocp_if dra7xx_l4_per2__mcasp3 = { > + .master = &dra7xx_l4_per2_hwmod, > + .slave = &dra7xx_mcasp3_hwmod, > + .clk = "l3_iclk_div", > + .user = OCP_USER_MPU | OCP_USER_SDMA, > +}; > + > /* l4_per1 -> elm */ > static struct omap_hwmod_ocp_if dra7xx_l4_per1__elm = { > .master = &dra7xx_l4_per1_hwmod, > @@ -3308,6 +3348,7 @@ static struct omap_hwmod_ocp_if *dra7xx_hwmod_ocp_ifs[] __initdata = { > &dra7xx_l4_wkup__dcan1, > &dra7xx_l4_per2__dcan2, > &dra7xx_l4_per2__cpgmac0, > + &dra7xx_l4_per2__mcasp3, > &dra7xx_gmac__mdio, > &dra7xx_l4_cfg__dma_system, > &dra7xx_l3_main_1__dss, >