From: Felipe Balbi <balbi@ti.com>
To: Peter Chen <hzpeterchen@gmail.com>
Cc: balbi@ti.com, Peter Chen <peter.chen@freescale.com>,
"ABRAHAM, KISHON VIJAY" <kishon@ti.com>,
Heikki Krogerus <heikki.krogerus@linux.intel.com>,
Greg KH <gregkh@suse.de>,
linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
Sascha Hauer <s.hauer@pengutronix.de>
Subject: Re: [PATCHv6 01/19] usb: otg: Rename otg_transceiver to usb_phy
Date: Wed, 9 Nov 2011 08:36:39 +0200 [thread overview]
Message-ID: <20111109063638.GF23337@legolas.emea.dhcp.ti.com> (raw)
In-Reply-To: <CAL411-qpW4dNUMTwh4gP_M+i_UOzc7q84gH+qJWeAYaSCgBCJg@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1718 bytes --]
On Wed, Nov 09, 2011 at 02:15:43PM +0800, Peter Chen wrote:
> > > > > Includes fixes to IMX code from Sascha Hauer.
> > > >
> > > > I tend to defer with your opinion of renaming otg_transceiver to
> > > > usb_phy. According to me otg_transceiver should program hardware
> > > > mechanisms associated to VBUS, ID lines, etc.. and phy is responsible
> > > > for transmitting data over differential data lines (with its own
> > > > programming of phy_init, phy_shutdown, setting phy clocks etc..). So
> > > > in my opinion otg_transceiver and usb_phy should be two different and
> > > > separate entities.
> > > >
> > > I am a little puzzled, are there two separate analog usb parts at OMap
> > > 's usb part? What the transceiver do? And what the phy do?
> >
> > Kinda... there's one which is mostly digital inside the SoC handling
> > data lines and also comunicating VBUS/ID levels to the link but the
> > actual VBUS/ID comparators are outside of the SoC, inside the PMIC.
> >
> > It's a bit of a pain, but I understand why they did it that way. It's
> > mostly for power management, although the same is likely to be
> > achievable by keeping everything in the PMIC.
> >
> So, in that way, the users must choose specific PMICs which have USB
> interface if
> they want to use OMAP
OMAP only works with its own matching PMIC. It's not like you can go to
a shelf an buy a PMIC for OMAP. OMAP's too complex for that. That's why
TI also provides the PMIC for every OMAP Silicon.
What can be done, though, and has happened before, is that some
companies will completely ditch the internal transceiver and the pieces
inside the PMIC and use another ULPI-based transceiver.
--
balbi
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
next prev parent reply other threads:[~2011-11-09 6:36 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-28 12:35 [PATCHv6 00/19] First round in OTG rework Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 01/19] usb: otg: Rename otg_transceiver to usb_phy Heikki Krogerus
2011-11-08 5:57 ` ABRAHAM, KISHON VIJAY
2011-11-08 12:06 ` Heikki Krogerus
2011-11-08 14:58 ` ABRAHAM, KISHON VIJAY
2011-11-08 15:04 ` Felipe Balbi
2011-11-09 8:52 ` Heikki Krogerus
2011-11-09 10:44 ` Felipe Balbi
2011-11-09 1:22 ` Peter Chen
2011-11-09 6:06 ` Felipe Balbi
2011-11-09 6:15 ` Peter Chen
2011-11-09 6:36 ` Felipe Balbi [this message]
2011-10-28 12:35 ` [PATCHv6 02/19] usb: otg: Rename usb_otg and usb_xceiv " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 03/19] usb: otg: Separate otg members from usb_phy Heikki Krogerus
2011-11-07 11:27 ` ABRAHAM, KISHON VIJAY
2011-11-08 12:58 ` Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 04/19] usb: otg: ab8500: Start using struct usb_otg Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 05/19] usb: otg: fsl: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 06/19] usb: otg: gpio_vbus: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 07/19] usb: otg: isp1301_omap: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 08/19] usb: otg: msm: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 09/19] usb: otg: langwell: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 10/19] usb: otg: nop: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 11/19] usb: otg: twl4030: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 12/19] usb: otg: twl6030: " Heikki Krogerus
2011-11-08 14:58 ` ABRAHAM, KISHON VIJAY
2011-10-28 12:35 ` [PATCHv6 13/19] usb: otg: ulpi: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 14/19] arm: imx: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 15/19] usb: musb: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 16/19] power_supply: Convert all users to new usb_phy Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 17/19] usb: " Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 18/19] usb: otg: Remove OTG specific members from usb_phy Heikki Krogerus
2011-10-28 12:35 ` [PATCHv6 19/19] usb: otg: Convert all users to pass struct usb_otg for OTG functions Heikki Krogerus
2011-11-04 13:26 ` [PATCHv6 00/19] First round in OTG rework Heikki Krogerus
2011-11-04 13:51 ` Greg KH
2011-11-04 14:40 ` Heikki Krogerus
2011-11-04 15:35 ` Felipe Balbi
2011-11-04 15:45 ` Greg KH
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=20111109063638.GF23337@legolas.emea.dhcp.ti.com \
--to=balbi@ti.com \
--cc=gregkh@suse.de \
--cc=heikki.krogerus@linux.intel.com \
--cc=hzpeterchen@gmail.com \
--cc=kishon@ti.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=peter.chen@freescale.com \
--cc=s.hauer@pengutronix.de \
/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
all inboxes | Powered by JetHome®