From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759643Ab1LPOzR (ORCPT ); Fri, 16 Dec 2011 09:55:17 -0500 Received: from devils.ext.ti.com ([198.47.26.153]:33796 "EHLO devils.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751962Ab1LPOzN (ORCPT ); Fri, 16 Dec 2011 09:55:13 -0500 Message-ID: <4EEB5BBC.7050807@ti.com> Date: Fri, 16 Dec 2011 15:54:52 +0100 From: "Cousson, Benoit" Organization: Texas Instruments User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:8.0) Gecko/20111105 Thunderbird/8.0 MIME-Version: 1.0 To: Paul Walmsley CC: Ming Lei , Mauro Carvalho Chehab , Tony Lindgren , Sylwester Nawrocki , Alan Cox , , , , Subject: Re: [RFC PATCH v2 1/8] omap4: introduce fdif(face detect module) hwmod References: <1323871214-25435-1-git-send-email-ming.lei@canonical.com> <1323871214-25435-2-git-send-email-ming.lei@canonical.com> In-Reply-To: Content-Type: text/plain; charset="ISO-8859-15"; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Paul, On 12/16/2011 6:53 AM, Paul Walmsley wrote: > Hi Benoît > > On Wed, 14 Dec 2011, Ming Lei wrote: > >> Signed-off-by: Ming Lei Acked-by: Benoit Cousson >> --- >> arch/arm/mach-omap2/omap_hwmod_44xx_data.c | 81 ++++++++++++++++++++++++++++ >> 1 files changed, 81 insertions(+), 0 deletions(-) > > any comments on this patch? I'd like to queue it if it looks good to you. It looks good to me. The only minor comment is about fdif location in the list that should be sorted and thus cannot be after wd_timer2. Regards, Benoit > > - Paul > >> >> diff --git a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >> index 6cf21ee..30db754 100644 >> --- a/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >> +++ b/arch/arm/mach-omap2/omap_hwmod_44xx_data.c >> @@ -53,6 +53,7 @@ static struct omap_hwmod omap44xx_dmm_hwmod; >> static struct omap_hwmod omap44xx_dsp_hwmod; >> static struct omap_hwmod omap44xx_dss_hwmod; >> static struct omap_hwmod omap44xx_emif_fw_hwmod; >> +static struct omap_hwmod omap44xx_fdif_hwmod; >> static struct omap_hwmod omap44xx_hsi_hwmod; >> static struct omap_hwmod omap44xx_ipu_hwmod; >> static struct omap_hwmod omap44xx_iss_hwmod; >> @@ -354,6 +355,14 @@ static struct omap_hwmod_ocp_if omap44xx_dma_system__l3_main_2 = { >> .user = OCP_USER_MPU | OCP_USER_SDMA, >> }; >> >> +/* fdif -> l3_main_2 */ >> +static struct omap_hwmod_ocp_if omap44xx_fdif__l3_main_2 = { >> + .master =&omap44xx_fdif_hwmod, >> + .slave =&omap44xx_l3_main_2_hwmod, >> + .clk = "l3_div_ck", >> + .user = OCP_USER_MPU | OCP_USER_SDMA, >> +}; >> + >> /* hsi -> l3_main_2 */ >> static struct omap_hwmod_ocp_if omap44xx_hsi__l3_main_2 = { >> .master =&omap44xx_hsi_hwmod, >> @@ -5444,6 +5453,75 @@ static struct omap_hwmod omap44xx_wd_timer3_hwmod = { >> .slaves_cnt = ARRAY_SIZE(omap44xx_wd_timer3_slaves), >> }; >> >> +/* 'fdif' class */ >> +static struct omap_hwmod_class_sysconfig omap44xx_fdif_sysc = { >> + .rev_offs = 0x0000, >> + .sysc_offs = 0x0010, >> + .sysc_flags = (SYSC_HAS_MIDLEMODE | SYSC_HAS_RESET_STATUS | >> + SYSC_HAS_SIDLEMODE | SYSC_HAS_SOFTRESET), >> + .idlemodes = (SIDLE_FORCE | SIDLE_NO | SIDLE_SMART | >> + MSTANDBY_FORCE | MSTANDBY_NO | >> + MSTANDBY_SMART), >> + .sysc_fields =&omap_hwmod_sysc_type2, >> +}; >> + >> +static struct omap_hwmod_class omap44xx_fdif_hwmod_class = { >> + .name = "fdif", >> + .sysc =&omap44xx_fdif_sysc, >> +}; >> + >> +/*fdif*/ >> +static struct omap_hwmod_addr_space omap44xx_fdif_addrs[] = { >> + { >> + .pa_start = 0x4a10a000, >> + .pa_end = 0x4a10afff, >> + .flags = ADDR_TYPE_RT >> + }, >> + { } >> +}; >> + >> +/* l4_cfg -> fdif */ >> +static struct omap_hwmod_ocp_if omap44xx_l4_cfg__fdif = { >> + .master =&omap44xx_l4_cfg_hwmod, >> + .slave =&omap44xx_fdif_hwmod, >> + .clk = "l4_div_ck", >> + .addr = omap44xx_fdif_addrs, >> + .user = OCP_USER_MPU | OCP_USER_SDMA, >> +}; >> + >> +/* fdif slave ports */ >> +static struct omap_hwmod_ocp_if *omap44xx_fdif_slaves[] = { >> + &omap44xx_l4_cfg__fdif, >> +}; >> +static struct omap_hwmod_irq_info omap44xx_fdif_irqs[] = { >> + { .irq = 69 + OMAP44XX_IRQ_GIC_START }, >> + { .irq = -1 } >> +}; >> + >> +/* fdif master ports */ >> +static struct omap_hwmod_ocp_if *omap44xx_fdif_masters[] = { >> + &omap44xx_fdif__l3_main_2, >> +}; >> + >> +static struct omap_hwmod omap44xx_fdif_hwmod = { >> + .name = "fdif", >> + .class =&omap44xx_fdif_hwmod_class, >> + .clkdm_name = "iss_clkdm", >> + .mpu_irqs = omap44xx_fdif_irqs, >> + .main_clk = "fdif_fck", >> + .prcm = { >> + .omap4 = { >> + .clkctrl_offs = OMAP4_CM_CAM_FDIF_CLKCTRL_OFFSET, >> + .context_offs = OMAP4_RM_CAM_FDIF_CONTEXT_OFFSET, >> + .modulemode = MODULEMODE_SWCTRL, >> + }, >> + }, >> + .slaves = omap44xx_fdif_slaves, >> + .slaves_cnt = ARRAY_SIZE(omap44xx_fdif_slaves), >> + .masters = omap44xx_fdif_masters, >> + .masters_cnt = ARRAY_SIZE(omap44xx_fdif_masters), >> +}; >> + >> static __initdata struct omap_hwmod *omap44xx_hwmods[] = { >> >> /* dmm class */ >> @@ -5593,6 +5671,9 @@ static __initdata struct omap_hwmod *omap44xx_hwmods[] = { >> &omap44xx_wd_timer2_hwmod, >> &omap44xx_wd_timer3_hwmod, >> >> + /* fdif class */ >> + &omap44xx_fdif_hwmod, >> + >> NULL, >> }; >> >> -- >> 1.7.5.4 >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-omap" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> > > > - Paul