Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966240AbaFRLWg (ORCPT ); Wed, 18 Jun 2014 07:22:36 -0400 Received: from arroyo.ext.ti.com ([192.94.94.40]:35491 "EHLO arroyo.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965421AbaFRLWe (ORCPT ); Wed, 18 Jun 2014 07:22:34 -0400 Message-ID: <53A17667.5050903@ti.com> Date: Wed, 18 Jun 2014 14:22:15 +0300 From: Roger Quadros User-Agent: Mozilla/5.0 (X11; Linux i686; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 MIME-Version: 1.0 To: Rajendra Nayak CC: , "Menon, Nishanth" , , , , , , , , , Nikhil Devshatwar , Benoit Cousson , Paul Walmsley Subject: Re: [PATCH v4 3/4] ARM: DRA7: hwmod: Add ocp2scp3 and sata hwmods References: <1398274237-24510-1-git-send-email-rogerq@ti.com> <1398274559-24777-1-git-send-email-rogerq@ti.com> <53A14784.1030008@ti.com> <53A169FC.70907@ti.com> In-Reply-To: <53A169FC.70907@ti.com> Content-Type: text/plain; charset="ISO-8859-1" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 06/18/2014 01:29 PM, Rajendra Nayak wrote: > On Wednesday 18 June 2014 01:32 PM, Roger Quadros wrote: >> On 04/23/2014 08:35 PM, Roger Quadros wrote: >>> From: Nikhil Devshatwar >>> >>> Add hwmods for ocp2scp3 and sata modules. > > From what I see this is actually adding the ocp2scp3 data and fixing up some > of the sata data which is already added and erroneous. > > It would help if this is split up and the changelog explains whats fixed up for > sata and why. Like for instance I see the opt clock data being removed but no > mention of why. OK. I will split this up into 2 patches, one to add ocp2scp3 data and the other to fix sata hwmod. cheers, -roger > >>> >>> [Roger Q] Clean up. >>> >>> CC: Benoit Cousson >>> CC: Paul Walmsley >>> Signed-off-by: Balaji T K >>> Signed-off-by: Nikhil Devshatwar >>> Signed-off-by: Roger Quadros >> >> Tested-by: Roger Quadros >> against 3.16-rc1, no dependency patches needed. >> >> Nishant/Rajendra, >> >> Could you please review this? We need this for 3.16 SATA support and >> Paul won't take this in without your reviewed-by tag. Thanks. >> >> cheers, >> -roger >> >>> --- >>> arch/arm/mach-omap2/omap_hwmod_7xx_data.c | 31 ++++++++++++++++++++++++++----- >>> 1 file changed, 26 insertions(+), 5 deletions(-) >>> >>> diff --git a/arch/arm/mach-omap2/omap_hwmod_7xx_data.c b/arch/arm/mach-omap2/omap_hwmod_7xx_data.c >>> index 810c205..b02a4ab 100644 >>> --- a/arch/arm/mach-omap2/omap_hwmod_7xx_data.c >>> +++ b/arch/arm/mach-omap2/omap_hwmod_7xx_data.c >>> @@ -1215,6 +1215,30 @@ static struct omap_hwmod dra7xx_ocp2scp1_hwmod = { >>> }, >>> }; >>> >>> +/* ocp2scp3 */ >>> +static struct omap_hwmod dra7xx_ocp2scp3_hwmod; >>> + >>> +/* l4_cfg -> ocp2scp3 */ >>> +static struct omap_hwmod_ocp_if dra7xx_l4_cfg__ocp2scp3 = { >>> + .master = &dra7xx_l4_cfg_hwmod, >>> + .slave = &dra7xx_ocp2scp3_hwmod, >>> + .clk = "l4_root_clk_div", >>> + .user = OCP_USER_MPU | OCP_USER_SDMA, >>> +}; > > All hwmod data files are organized in a certain way with all the hwmod/IP block info on top > and Interface structs at the bottom. Can we follow the same for ocp2scp3? It helps with better > readability. > >>> + >>> +static struct omap_hwmod dra7xx_ocp2scp3_hwmod = { >>> + .name = "ocp2scp3", >>> + .class = &dra7xx_ocp2scp_hwmod_class, >>> + .clkdm_name = "l3init_clkdm", > > No main_clk? > > regards, > Rajendra > >>> + .prcm = { >>> + .omap4 = { >>> + .clkctrl_offs = DRA7XX_CM_L3INIT_OCP2SCP3_CLKCTRL_OFFSET, >>> + .context_offs = DRA7XX_RM_L3INIT_OCP2SCP3_CONTEXT_OFFSET, >>> + .modulemode = MODULEMODE_HWCTRL, >>> + }, >>> + }, >>> +}; >>> + >>> /* >>> * 'qspi' class >>> * >>> @@ -1268,9 +1292,6 @@ static struct omap_hwmod_class dra7xx_sata_hwmod_class = { >>> }; >>> >>> /* sata */ >>> -static struct omap_hwmod_opt_clk sata_opt_clks[] = { >>> - { .role = "ref_clk", .clk = "sata_ref_clk" }, >>> -}; >>> >>> static struct omap_hwmod dra7xx_sata_hwmod = { >>> .name = "sata", >>> @@ -1278,6 +1299,7 @@ static struct omap_hwmod dra7xx_sata_hwmod = { >>> .clkdm_name = "l3init_clkdm", >>> .flags = HWMOD_SWSUP_SIDLE | HWMOD_SWSUP_MSTANDBY, >>> .main_clk = "func_48m_fclk", >>> + .mpu_rt_idx = 1, >>> .prcm = { >>> .omap4 = { >>> .clkctrl_offs = DRA7XX_CM_L3INIT_SATA_CLKCTRL_OFFSET, >>> @@ -1285,8 +1307,6 @@ static struct omap_hwmod dra7xx_sata_hwmod = { >>> .modulemode = MODULEMODE_SWCTRL, >>> }, >>> }, >>> - .opt_clks = sata_opt_clks, >>> - .opt_clks_cnt = ARRAY_SIZE(sata_opt_clks), >>> }; >>> >>> /* >>> @@ -2682,6 +2702,7 @@ static struct omap_hwmod_ocp_if *dra7xx_hwmod_ocp_ifs[] __initdata = { >>> &dra7xx_l4_per1__mmc4, >>> &dra7xx_l4_cfg__mpu, >>> &dra7xx_l4_cfg__ocp2scp1, >>> + &dra7xx_l4_cfg__ocp2scp3, >>> &dra7xx_l3_main_1__qspi, >>> &dra7xx_l4_cfg__sata, >>> &dra7xx_l4_cfg__smartreflex_core, >>> >> > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/