From: david@lechnology.com (David Lechner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/5] da8xx USB clocks
Date: Tue, 15 Mar 2016 17:35:44 -0500 [thread overview]
Message-ID: <1458081344-2449-1-git-send-email-david@lechnology.com> (raw)
I've implemented some proper clocks for the USB PHY clocks on da8xx. For those
not familiar with the architecture, the SoC has one USB 1.1 OHCI port and one
USB 2.O OTG port. The USB 1.1 PHY clock can optionally be supplied by the PLL
in the USB 2.0 PHY.
I have seen some comments in the past that these clock details don't belong in
the USB drivers and I agree with that. So, I have moved the handling of the
clocks out of the USB drivers to the mach code with the rest of the SoC clocks.
This code has been tested on LEGO MINDSTORMS EV3 (AM1808/da850 family). Here is
an output of the davinci clock debug to give you a better idea of how clocks
are related.
root at ev3dev:~# cat /sys/kernel/debug/davinci_clocks
ref_clk users=23 24000000 Hz
pll0 users=20 pll 300000000 Hz
pll0_aux_clk users= 4 pll 24000000 Hz
...
usb20_phy users= 2 24000000 Hz
usb11_phy users= 1 24000000 Hz
...
usb_ref_clk users= 0 48000000 Hz
usb20_phy and usb11_phy can optionally be children of usb_ref_clk instead.
I'm planning on adding device tree bindings for the ohci driver, but I need to
get some things sorted out with the regulator subsystem first. I see that Petr
has been working on device tree support for the musb driver. This should take
care of some of the concerns related to his changes too, for example, the
ti,usb2-phy-refclock-hz device tree property is no longer needed because it
is now taken care of in the clock code. I've actually included one of Petr's
patchs here since one of my patches depends on it.
I'm also working on device tree bindings for davinci clocks, but it will take
me a while to get there. But that should not hold up the device tree bindings
for da8xx ohci and musb.
David Lechner (4):
ARM: davinci: da8xx: add usb phy clocks
ARM: davinci: da8xx: add cfgchip2 to resources
usb: ohci-da8xx: Remove clock code that references mach
usb: musb-da8xx: remove board-specific clock handling
Petr Kulhavy (1):
ARM: davinci: defined missing CFGCHIP2_REFFREQ_* macros for MUSB PHY
arch/arm/mach-davinci/board-da830-evm.c | 12 ---
arch/arm/mach-davinci/board-omapl138-hawk.c | 7 --
arch/arm/mach-davinci/da830.c | 128 ++++++++++++++++++++++++++--
arch/arm/mach-davinci/da850.c | 113 ++++++++++++++++++++++++
arch/arm/mach-davinci/usb.c | 5 ++
drivers/usb/host/ohci-da8xx.c | 81 +++++++++---------
drivers/usb/musb/da8xx.c | 93 ++++++++------------
include/linux/platform_data/usb-davinci.h | 6 ++
8 files changed, 318 insertions(+), 127 deletions(-)
--
1.9.1
next reply other threads:[~2016-03-15 22:35 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-15 22:35 David Lechner [this message]
2016-03-16 12:36 ` [PATCH 0/5] da8xx USB clocks Sergei Shtylyov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1458081344-2449-1-git-send-email-david@lechnology.com \
--to=david@lechnology.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).