| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Fri, 16 Dec 2011 15:54:52 +0100
From: "Cousson, Benoit" <b-cousson@...com>
To: Paul Walmsley <paul@...an.com>
CC: Ming Lei <ming.lei@...onical.com>,
Mauro Carvalho Chehab <mchehab@...radead.org>,
Tony Lindgren <tony@...mide.com>,
Sylwester Nawrocki <snjw23@...il.com>,
Alan Cox <alan@...rguk.ukuu.org.uk>,
<linux-omap@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>,
<linux-kernel@...r.kernel.org>, <linux-media@...r.kernel.org>
Subject: Re: [RFC PATCH v2 1/8] omap4: introduce fdif(face detect module)
hwmod
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<ming.lei@...onical.com>
Acked-by: Benoit Cousson <b-cousson@...com>
>> ---
>> 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@...r.kernel.org
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>
>
>
> - Paul
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists