Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752867AbaA3MZV (ORCPT ); Thu, 30 Jan 2014 07:25:21 -0500 Received: from comal.ext.ti.com ([198.47.26.152]:57058 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751799AbaA3MZT (ORCPT ); Thu, 30 Jan 2014 07:25:19 -0500 Message-ID: <52EA449E.8080200@ti.com> Date: Thu, 30 Jan 2014 17:55:02 +0530 From: Kishon Vijay Abraham I User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 MIME-Version: 1.0 To: Pratyush Anand CC: Mohit KUMAR DCG , "arnd@arndb.de" , spear-devel , "linux-kernel@vger.kernel.org" , Bjorn Helgaas Subject: Re: [PATCH V3 4/8] phy: Initialize phy core with subsys_initcall References: <6f2dbd4fcb99a070c07b39545d98eca58101daa0.1391077731.git.mohit.kumar@st.com> <52EA3AE9.3030000@ti.com> <20140130115212.GA1171@pratyush-vbox> <20140130121514.GA1857@pratyush-vbox> In-Reply-To: <20140130121514.GA1857@pratyush-vbox> 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 Thursday 30 January 2014 05:45 PM, Pratyush Anand wrote: > On Thu, Jan 30, 2014 at 07:52:12PM +0800, Pratyush ANAND wrote: >> On Thu, Jan 30, 2014 at 07:43:37PM +0800, Kishon Vijay Abraham I wrote: >>> Hi, >>> >>> On Thursday 30 January 2014 04:18 PM, Mohit Kumar wrote: >>>> From: Pratyush Anand >>>> >>>> PCIe RC drivers are initialized with subsys_initcall. Few PCIe drivers >>>> like SPEAr13xx needs phy drivers to be initialized. >>> >>> Instead change PCIe RC drivers to module init. Phy drivers should be loaded >>> very early otherwise. (Hint: drivers/Makefile). > > From hint, you mean that if makefile has pci entry (and hence RC > driver entry) before card drivers entry, then it insures that rc > driver's probe is called before card driver's probe? That's right but here I was referring to PHY and PCI. Thanks Kishon > I think, yes. > And if yes, then what you say is acceptable :) > > Regards > Pratyush >> >> I think PCIe RC driver can not be made module init. Bjorn can comment >> better. >> >> All PCIe card drivers are initialized with module init. RC driver must >> have been initialized before any card driver initialization. >> Currently, card drivers does not have deferred probe concept, so I am >> not sure if keeping RC driver as module init will work always. >> >> By the way, is there any side effect of loading phy driver very early? >> >> Regards >> Pratyush >>> >>> Thanks >>> Kishon >>>> >>>> Therefore initialize phy core driver with subsys_initcall to avoid >>>> calling of phy_get before phy_class is created. >>>> >>>> Signed-off-by: Pratyush Anand >>>> Cc: Mohit Kumar >>>> Cc: Kishon Vijay Abraham I >>>> Cc: Arnd Bergmann >>>> Cc: spear-devel@list.st.com >>>> Cc: linux-kernel@vger.kernel.org >>>> --- >>>> drivers/phy/phy-core.c | 2 +- >>>> 1 files changed, 1 insertions(+), 1 deletions(-) >>>> >>>> diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c >>>> index 03cf8fb..fa73101 100644 >>>> --- a/drivers/phy/phy-core.c >>>> +++ b/drivers/phy/phy-core.c >>>> @@ -685,7 +685,7 @@ static int __init phy_core_init(void) >>>> >>>> return 0; >>>> } >>>> -module_init(phy_core_init); >>>> +subsys_initcall(phy_core_init); >>>> >>>> static void __exit phy_core_exit(void) >>>> { >>>> >>> -- 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/