Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751047Ab3HSQa0 (ORCPT ); Mon, 19 Aug 2013 12:30:26 -0400 Received: from smtp.codeaurora.org ([198.145.11.231]:42405 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750810Ab3HSQaY (ORCPT ); Mon, 19 Aug 2013 12:30:24 -0400 Message-ID: <5212481D.6040207@codeaurora.org> Date: Mon, 19 Aug 2013 09:30:21 -0700 From: Stephen Boyd User-Agent: Mozilla/5.0 (X11; Linux i686 on x86_64; rv:17.0) Gecko/20130801 Thunderbird/17.0.8 MIME-Version: 1.0 To: "Ivan T. Ivanov" CC: Stephen Warren , balbi@ti.com, rob.herring@calxeda.com, pawel.moll@arm.com, mark.rutland@arm.com, ian.campbell@citrix.com, rob@landley.net, gregkh@linuxfoundation.org, grant.likely@linaro.org, idos@codeaurora.org, mgautam@codeaurora.org, devicetree@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-usb@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-msm@vger.kernel.org Subject: Re: [PATCH v3 1/3] usb: dwc3: msm: Add device tree binding information References: <1376485183-2664-1-git-send-email-iivanov@mm-sol.com> <1376485183-2664-2-git-send-email-iivanov@mm-sol.com> <520EAB3B.9030607@wwwdotorg.org> <1376915267.26268.16.camel@iivanov-dev.int.mm-sol.com> In-Reply-To: <1376915267.26268.16.camel@iivanov-dev.int.mm-sol.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2055 Lines: 48 On 08/19/13 05:27, Ivan T. Ivanov wrote: > Hi, > > On Fri, 2013-08-16 at 16:44 -0600, Stephen Warren wrote: >> On 08/14/2013 06:59 AM, Ivan T. Ivanov wrote: >>> From: "Ivan T. Ivanov" >>> >>> MSM USB3.0 core wrapper consist of USB3.0 IP from Synopsys >>> (SNPS) and HS, SS PHY's control and configuration registers. >>> >>> It could operate in device mode (SS, HS, FS) and host >>> mode (SS, HS, FS, LS). >>> diff --git a/Documentation/devicetree/bindings/usb/msm-ssusb.txt b/Documentation/devicetree/bindings/usb/msm-ssusb.txt >>> +- clock-names : >> ... >>> + "sleep_a_clk" : Sleep clock, used when USB3 core goes into low >> ... >>> + "ref_clk" : Reference clock - used in host mode. >> ... >>> + "core_clk" : Master/Core clock, have to be >= 125 MHz for SS >> ... >>> + "iface_clk" : System bus AXI clock >>> + "sleep_clk" : Sleep clock, used when USB3 core goes into low >> ... >>> + "utmi_clk" : Generated by HS-PHY. Used to clock the low power >> I think it makes sense to remove "_clk" from all those names, unless the >> HW documentation really talks about a clock named e.g. iface_clk yet >> some other clock names in the documentation don't have the "_clk" >> suffix, e.g. the "xo I didn't quote. > From limited information that I have, I could not say how clock inputs > are named from the controller perspective, but I agree that "_clk" > suffix looks redundant. In downstream trees we've tried to standardize the names on core_clk, iface_clk, bus_clk, etc. Historically the hardware designers have used the names from the clock controller instead of coming up with standard names of their own when they put the clock inputs in their data sheets (if they do at all). -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- 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/