From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mugunthan V N Subject: Re: [PATCH 3/7] ARM: OMAP3+: hwmod: Add AM33XX HWMOD data for davinci_mdio module Date: Tue, 6 Nov 2012 16:54:53 +0530 Message-ID: <5098F385.6020009@ti.com> References: <1352189545-26303-1-git-send-email-mugunthanvnm@ti.com> <1352189545-26303-4-git-send-email-mugunthanvnm@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: netdev-owner@vger.kernel.org To: "Bedia, Vaibhav" Cc: "netdev@vger.kernel.org" , Richard Cochran , Paul Walmsley , "devicetree-discuss@lists.ozlabs.org" , "Hiremath, Vaibhav" , "linux-omap@vger.kernel.org" , "davem@davemloft.net" , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org On 11/6/2012 3:39 PM, Bedia, Vaibhav wrote: > On Tue, Nov 06, 2012 at 13:42:21, N, Mugunthan V wrote: > [...] >> +struct omap_hwmod_addr_space am33xx_mdio_addr_space[] = { >> + { >> + .pa_start = 0x4A101000, >> + .pa_end = 0x4A101000 + SZ_256 - 1, >> + .flags = ADDR_MAP_ON_INIT, > Based on the recent discussions and looking the hwmod code, > I guess ADDR_MAP_ON_INIT does not make sense here. Since you > are just creating a parent-child relationship here, maybe no > flag is needed? > >> + }, >> + { } >> +}; >> + >> +struct omap_hwmod_ocp_if am33xx_cpgmac0__mdio = { >> + .master = &am33xx_cpgmac0_hwmod, >> + .slave = &am33xx_mdio_hwmod, >> + .addr = am33xx_mdio_addr_space, >> + .user = OCP_USER_MPU, > Is this flag necessary? Shouldn't you just skip the > user field since there's nothing for the hwmod code > to do here? > > Will remove the unnecessary flags and submit the patch. Regards Mugunthan V N