Hi,

On 3/2/2015 4:39 AM, Paul Walmsley wrote:
> Hi
> 
> On Fri, 27 Feb 2015, Vignesh R wrote:
> 
>> From: "Poddar, Sourav" <sourav.pod...@ti.com>
>>
>> This patch adds hwmod data for hdq/1w driver on AM43xx.
>>
>> Signed-off-by: Sourav Poddar <sourav.pod...@ti.com>
>> [vigne...@ti.com: Ported patch to v4.0-rc1]
>> Signed-off-by: Vignesh R <vigne...@ti.com>
>> ---
>>  arch/arm/mach-omap2/omap_hwmod_43xx_data.c | 36 
>> ++++++++++++++++++++++++++++++
>>  arch/arm/mach-omap2/prcm43xx.h             |  1 +
>>  2 files changed, 37 insertions(+)
>>
>> diff --git a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c 
>> b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
>> index 8eb85925e444..d4f1df28475c 100644
>> --- a/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
>> +++ b/arch/arm/mach-omap2/omap_hwmod_43xx_data.c
>> @@ -20,6 +20,7 @@
>>  #include "omap_hwmod_33xx_43xx_common_data.h"
>>  #include "prcm43xx.h"
>>  #include "omap_hwmod_common_data.h"
>> +#include "hdq1w.h"
>>  
>>  
>>  /* IP blocks */
>> @@ -516,6 +517,33 @@ static struct omap_hwmod am43xx_dss_rfbi_hwmod = {
>>      .parent_hwmod   = &am43xx_dss_core_hwmod,
>>  };
>>  
>> +/* HDQ1W */
>> +static struct omap_hwmod_class_sysconfig am43xx_hdq1w_sysc = {
>> +    .rev_offs       = 0x0000,
>> +    .sysc_offs      = 0x0014,
>> +    .syss_offs      = 0x0018,
>> +    .sysc_flags     = (SYSC_HAS_SOFTRESET),
> 
> This is missing the SYSC_HAS_AUTOIDLE bit.  Is this intentional?

Sorry.. my bad.. I didn't observe this while porting the patch.
I will fix this and repost.

Regards
Vignesh

> 
>> +    .sysc_fields    = &omap_hwmod_sysc_type1,
>> +};
>> +
>> +static struct omap_hwmod_class am43xx_hdq1w_hwmod_class = {
>> +    .name   = "hdq1w",
>> +    .sysc   = &am43xx_hdq1w_sysc,
>> +    .reset  = &omap_hdq1w_reset,
>> +};
>> +
>> +static struct omap_hwmod am43xx_hdq1w_hwmod = {
>> +    .name           = "hdq1w",
>> +    .class          = &am43xx_hdq1w_hwmod_class,
>> +    .clkdm_name     = "l4ls_clkdm",
>> +    .prcm = {
>> +            .omap4 = {
>> +                    .clkctrl_offs = AM43XX_CM_PER_HDQ1W_CLKCTRL_OFFSET,
>> +                    .modulemode   = MODULEMODE_SWCTRL,
>> +            },
>> +    },
>> +};
>> +
>>  /* Interfaces */
>>  static struct omap_hwmod_ocp_if am43xx_l3_main__l4_hs = {
>>      .master         = &am33xx_l3_main_hwmod,
>> @@ -790,6 +818,13 @@ static struct omap_hwmod_ocp_if am43xx_l4_ls__dss_rfbi 
>> = {
>>      .user           = OCP_USER_MPU | OCP_USER_SDMA,
>>  };
>>  
>> +static struct omap_hwmod_ocp_if am43xx_l4_ls__hdq1w = {
>> +    .master         = &am33xx_l4_ls_hwmod,
>> +    .slave          = &am43xx_hdq1w_hwmod,
>> +    .clk            = "l4ls_gclk",
>> +    .user           = OCP_USER_MPU | OCP_USER_SDMA,
>> +};
>> +
>>  static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] __initdata = {
>>      &am33xx_l4_wkup__synctimer,
>>      &am43xx_l4_ls__timer8,
>> @@ -889,6 +924,7 @@ static struct omap_hwmod_ocp_if *am43xx_hwmod_ocp_ifs[] 
>> __initdata = {
>>      &am43xx_l4_ls__dss,
>>      &am43xx_l4_ls__dss_dispc,
>>      &am43xx_l4_ls__dss_rfbi,
>> +    &am43xx_l4_ls__hdq1w,
>>      NULL,
>>  };
>>  
>> diff --git a/arch/arm/mach-omap2/prcm43xx.h b/arch/arm/mach-omap2/prcm43xx.h
>> index ad7b3e9977f8..48df3b55057e 100644
>> --- a/arch/arm/mach-omap2/prcm43xx.h
>> +++ b/arch/arm/mach-omap2/prcm43xx.h
>> @@ -143,5 +143,6 @@
>>  #define AM43XX_CM_PER_USB_OTG_SS1_CLKCTRL_OFFSET        0x0268
>>  #define AM43XX_CM_PER_USBPHYOCP2SCP1_CLKCTRL_OFFSET 0x05C0
>>  #define AM43XX_CM_PER_DSS_CLKCTRL_OFFSET            0x0a20
>> +#define AM43XX_CM_PER_HDQ1W_CLKCTRL_OFFSET          0x04a0
>>  
>>  #endif
>> -- 
>> 1.9.1
>>
> 
> 
> - Paul
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to