Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753609AbaDOGJ3 (ORCPT ); Tue, 15 Apr 2014 02:09:29 -0400 Received: from mail-qg0-f43.google.com ([209.85.192.43]:37127 "EHLO mail-qg0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750951AbaDOGJZ (ORCPT ); Tue, 15 Apr 2014 02:09:25 -0400 MIME-Version: 1.0 In-Reply-To: References: <1396967803-28868-1-git-send-email-gautam.vivek@samsung.com> <1396967803-28868-2-git-send-email-gautam.vivek@samsung.com> <534529B2.2020107@samsung.com> <53454C11.7060607@samsung.com> Date: Tue, 15 Apr 2014 11:39:24 +0530 X-Google-Sender-Auth: zos44zB0VRX0KQj8JxCHgcIv0nU Message-ID: Subject: Re: [PATCH V4 1/5] phy: Add new Exynos5 USB 3.0 PHY driver From: Vivek Gautam To: Tomasz Figa Cc: Linux USB Mailing List , "linux-samsung-soc@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , linux-doc@vger.kernel.org, kishon , Greg KH , Felipe Balbi , Kukjin Kim , Kamil Debski , Jingoo Han , Sylwester Nawrocki Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Tomasz, On Thu, Apr 10, 2014 at 5:09 PM, Vivek Gautam wrote: > On Wed, Apr 9, 2014 at 7:03 PM, Tomasz Figa wrote: >> On 09.04.2014 13:49, Vivek Gautam wrote: >>> >>> Hi, >>> >>> >>> On Wed, Apr 9, 2014 at 4:36 PM, Tomasz Figa wrote: >>>> >>>> Hi Vivek, >>>> >>>> Please see my comments inline. >>>> >>>> >>>> On 08.04.2014 16:36, Vivek Gautam wrote: >>>>> >>>>> >>>>> Add a new driver for the USB 3.0 PHY on Exynos5 series of SoCs. >>>>> The new driver uses the generic PHY framework and will interact >>>>> with DWC3 controller present on Exynos5 series of SoCs. >>>>> Thereby, removing old phy-samsung-usb3 driver and related code >>>>> used untill now which was based on usb/phy framework. >>>>> >>>>> Signed-off-by: Vivek Gautam >>>>> --- >>>>> .../devicetree/bindings/phy/samsung-phy.txt | 42 ++ >>>>> drivers/phy/Kconfig | 11 + >>>>> drivers/phy/Makefile | 1 + >>>>> drivers/phy/phy-exynos5-usbdrd.c | 668 >>>>> ++++++++++++++++++++ >>>>> 4 files changed, 722 insertions(+) >>>>> create mode 100644 drivers/phy/phy-exynos5-usbdrd.c >>>> >>>> >>>> >>>> [snip] >>>> >>>> >>>>> + Additional clock required for Exynos5420: >>>>> + - usb30_sclk_100m: Additional special clock used for PHY >>>>> operation >>>>> + depicted as 'sclk_usbphy30' in CMU of >>>>> Exynos5420. >>>> >>>> >>>> >>>> Are you sure this isn't simply a gate for the ref clock, as it can be >>>> found >>>> on another SoC that is not upstream yet? I don't have documentation for >>>> Exynos 5420 so I can't tell, but I'd like to ask you to recheck this. >>> >>> >>>> From what i can see in the manual : >>> >>> sclk_usbphy30 is derived from OSCCLK. >>> It is coming from a MUX (default input line to this is OSCCLK) and >>> then through a DIV >>> there's this gate. >>> >>> {OSCCLK + other sources} --->[MUX] ---> [DIV] --> [GATE for >>> sclk_usbphy30] >>> >>> the {rate of sclk_usbphy30} == OSCCLK >>> >>> However the 'ref' clock that we have been using is the actual oscillator >>> clock. >>> And on SoC Exynos5250, we don't have any such gate (sclk_usbphy30). >>> So should this mean that ref clock and sclk_usbphy30 are still be >>> controlled by >>> two different gates ? >>> >> >> Is there maybe a diagram of PHY input clocks in the datasheet, like for USB >> 2.0 PHY in Exynos4210/4412/5250 datasheets in the chapter about USB2.0 >> Device? Something like: >> >> ____________________________________ >> | | >> | ___________| >> XusbXTI | Phy_fsel[2:0] | _______ | >> _______[X]_______| | __________|_|___|\__|_| >> | | _v___ | _____ ^ | |/ | | >> _____ | | | | | | | | ___ | | >> ___ | | | | | | | | | |_|_| >> |___| | | X 0 |____|_| PLL |__|_|_|CLK|_|_| >> _____ | | | | | | |DIV|_|_| >> |_______[X] | |_____| 12 |_____|480 | |___| | | >> | MHz MHz |Digital| | >> XusbXTO | USB PHY |_______| | >> |____________________________________| >> >> > > Below is the block diagram given for DRD controller. > > ___________________ > | | > | ____________ | > | | PHY | | > | | controller |-----|----------------------------------------------- > | |__________ | | | > | | > | > | USB 3.0 | V > | DRD | > ----------------------- > | Controller | | > | > | | USB30_SCLK_100M | USB 3.0 DRD | > | ---------------- | -----------------------> > | PHY | > | | Link cont. | | | > | > | ------------- | > | | > |___________________| |_____________| > > Does this help ? > > So, USB30_SCLK_100M is the SCLK that we are talking in the driver. I > don't see any reference to XXTI in the USB 3.0 DRD controller chapter > (in both Exynos5250 and 5420) > In addition to this there's one more point to be noticed here. > On Exynos5420 system, the sclk_usbphy300 (which is the sclk_usbphy30 > for USB DRD channel 0), is also the PICO phy clock, i.e. USB 2.0 phy > clock. > So we should add a similar clk_get() for this clock in the > phy-exynos5250-usb2 driver too, to support Exynos5420. Is something clear from the above block diagram ? (although the diagram looks weird - space and tabs problem :-( ) Basically there's the clock USB30_SCLK_100M which is going into the USB 3.0 DRD PHY controller. And this is the only sclk mentioned in the block diagram for USB 3.0 DRD controller in Exynos5420. Same is not there in the block diagram in Exynos5250 UM. -- 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/