Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934822AbcJZUDY (ORCPT ); Wed, 26 Oct 2016 16:03:24 -0400 Received: from vern.gendns.com ([206.190.152.46]:54611 "EHLO vern.gendns.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933390AbcJZUDX (ORCPT ); Wed, 26 Oct 2016 16:03:23 -0400 Subject: Re: [PATCH] ARM: davinci: register the usb20_phy clock on the DT file To: ahaslam@baylibre.com, nsekhar@ti.com, khilman@kernel.org References: <20161026194916.14546-1-ahaslam@baylibre.com> Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org From: David Lechner Message-ID: <48fa5315-74db-cf33-1f34-86764f0e2c1c@lechnology.com> Date: Wed, 26 Oct 2016 15:03:24 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - vern.gendns.com X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - lechnology.com X-Get-Message-Sender-Via: vern.gendns.com: authenticated_id: davidmain+lechnology.com/only user confirmed/virtual account not confirmed X-Authenticated-Sender: vern.gendns.com: davidmain@lechnology.com X-Source: X-Source-Args: X-Source-Dir: Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2246 Lines: 61 On 10/26/2016 03:01 PM, David Lechner wrote: > On 10/26/2016 02:49 PM, ahaslam@baylibre.com wrote: >> From: Axel Haslam >> >> The usb20_phy clock needs to be registered for the driver to be able >> to get and enable a clock. Currently the usb phy clocks are registered >> form board files, which will not be called during a device tree based >> boot. >> >> To be able to probe correctly usb form a device tree boot, register >> the usb phy clocks form the DT specific init. >> >> Unfourtunatly, davinci does not have proper clock support on device tree >> yet, so by registering the clock form de DT specific file we are >> forced to hardcode the parent clock, and cannot select refclkin as >> parent for any of the phy clocks of the da850 family. >> >> As none of the current da850 based boards currently in mainline use >> refclkin as source. I guess we can live with this limitation until clocks >> are correctly represented through CCF/device tree. >> >> Signed-off-by: Axel Haslam >> --- >> >> This patch depends on David's series for usb clocks: >> >> currently: >> [PATCH v6 0/5] da8xx USB PHY platform devices and clocks >> https://lkml.org/lkml/2016/10/25/867 >> >> arch/arm/mach-davinci/da8xx-dt.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/arch/arm/mach-davinci/da8xx-dt.c >> b/arch/arm/mach-davinci/da8xx-dt.c >> index c9f7e92..7947267 100644 >> --- a/arch/arm/mach-davinci/da8xx-dt.c >> +++ b/arch/arm/mach-davinci/da8xx-dt.c >> @@ -45,6 +45,8 @@ >> >> static void __init da850_init_machine(void) >> { >> + da8xx_register_usb20_phy_clk(false); >> + da8xx_register_usb11_phy_clk(false); >> of_platform_default_populate(NULL, da850_auxdata_lookup, NULL); >> } >> >> > > > You should check for error on return and print a warning if there is an > error. Like this: https://patchwork.kernel.org/patch/9395635/ > > (I actually just had to make this change a few minutes ago to > troubleshoot a problem that I am working on. > da8xx_register_usb11_phy_clk() is failing for some reason that I have > not figured out yet.) And actually, since my patch that provides these functions has not been accepted yet, why don't I just add this change to that series?