From: Sylwester Nawrocki <s.nawrocki@samsung.com>
To: Kishon Vijay Abraham I <kishon@ti.com>,
Tomasz Stanislawski <t.stanislaws@samsung.com>,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org
Cc: t.figa@samsung.com, kgene.kim@samsung.com,
grant.likely@linaro.org, rob.herring@calxeda.com,
sylvester.nawrocki@gmail.com, kyungmin.park@samsung.com
Subject: Re: [PATCH] phy: Add exynos-phy driver
Date: Wed, 11 Dec 2013 11:16:39 +0100 [thread overview]
Message-ID: <52A83B87.205@samsung.com> (raw)
In-Reply-To: <52A8364B.5020204@ti.com>
On 11/12/13 10:54, Kishon Vijay Abraham I wrote:
> On Wednesday 27 November 2013 06:56 PM, Tomasz Stanislawski wrote:
>> > Hello everyone,
>> > The Samsung SoCs from Exynos family are enhanced with a bunch of switches
>> > dedicated for IP blocks. Those switches are called PHYs in Exynos
>> > specification. They are usually controlled by a single bit in a single
>> > one-word-long register.
>
> So only enabling this switch is enough for the controller or some other actual
> PHY IP is needed along with this switch?
>
> However I'm not sure if the switch should be modelled as PHY as it is not a PHY
> in the real sense.
These are ordinary PHY devices embedded in an SoC. I wouldn't really call
them "switches", as they indeed provide the physical layer functionality
for various interfaces, like USB, HDMI, MIPI CSI/DSI, etc. Their control
interface is often very simple - usually only an enable and a reset
control bit. But that can't change the fact they are real PHY devices,
so let's not call them switches, that's just untrue.
Regards,
Sylwester
next prev parent reply other threads:[~2013-12-11 10:16 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-11-27 13:26 [PATCH] phy: Add exynos-phy driver Tomasz Stanislawski
2013-11-27 13:26 ` Tomasz Stanislawski
[not found] ` <1385558788-10880-1-git-send-email-t.stanislaws-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2013-12-11 9:54 ` Kishon Vijay Abraham I
2013-12-11 10:16 ` Sylwester Nawrocki [this message]
2013-12-13 11:22 ` Kishon Vijay Abraham I
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=52A83B87.205@samsung.com \
--to=s.nawrocki@samsung.com \
--cc=devicetree@vger.kernel.org \
--cc=grant.likely@linaro.org \
--cc=kgene.kim@samsung.com \
--cc=kishon@ti.com \
--cc=kyungmin.park@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=rob.herring@calxeda.com \
--cc=sylvester.nawrocki@gmail.com \
--cc=t.figa@samsung.com \
--cc=t.stanislaws@samsung.com \
/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).