<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux/drivers/net/phy/phy_device.c, branch v2.6.19</title>
<subtitle>Linux kernel source tree</subtitle>
<id>https://git.amat.us/linux/atom/drivers/net/phy/phy_device.c?h=v2.6.19</id>
<link rel='self' href='https://git.amat.us/linux/atom/drivers/net/phy/phy_device.c?h=v2.6.19'/>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/'/>
<updated>2006-10-01T11:27:46Z</updated>
<entry>
<title>[netdrvr] phy: Fix bugs in error handling</title>
<updated>2006-10-01T11:27:46Z</updated>
<author>
<name>Jeff Garzik</name>
<email>jeff@garzik.org</email>
</author>
<published>2006-10-01T11:27:46Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=b7a00ecd557859c4037b6465fdd6c9a49b1fa649'/>
<id>urn:sha1:b7a00ecd557859c4037b6465fdd6c9a49b1fa649</id>
<content type='text'>
The recent __must_check stuff flagged some error handling bugs.

phy/fixed.c:
* handle device_bind_driver() failure

phy/phy_device.c:
* handle device_bind_driver() failure
* release rwsem upon failure

Signed-off-by: Jeff Garzik &lt;jeff@garzik.org&gt;
</content>
</entry>
<entry>
<title>[PATCH] Signedness issue in drivers/net/phy/phy_device.c</title>
<updated>2006-09-26T00:01:19Z</updated>
<author>
<name>Eric Sesterhenn</name>
<email>snakebyte@gmx.de</email>
</author>
<published>2006-09-25T23:39:22Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=84c22d7901f793bd267b5f79270080964b252826'/>
<id>urn:sha1:84c22d7901f793bd267b5f79270080964b252826</id>
<content type='text'>
While checking gcc 4.1 -Wextra warnings, I stumbled across the following
two warnings:

drivers/net/phy/phy_device.c:528: warning: comparison of unsigned expression &lt; 0 is always false
drivers/net/phy/phy_device.c:546: warning: comparison of unsigned expression &lt; 0 is always false

Since phy_read() returns an integer and can return negative values, it seems
to me the best way to get proper error handling working again is to make val
an int.  Currently it is an u32, so the &lt; 0 check always fails.

Signed-off-by: Eric Sesterhenn &lt;snakebyte@gmx.de&gt;
Cc: Jeff Garzik &lt;jeff@garzik.org&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Jeff Garzik &lt;jeff@garzik.org&gt;
</content>
</entry>
<entry>
<title>[PATCH] PAL: Support of the fixed PHY</title>
<updated>2006-08-19T21:44:31Z</updated>
<author>
<name>Vitaly Bordug</name>
<email>vbordug@ru.mvista.com</email>
</author>
<published>2006-08-15T06:00:29Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=11b0bacd717c285c94dbb56505a28434b34f0639'/>
<id>urn:sha1:11b0bacd717c285c94dbb56505a28434b34f0639</id>
<content type='text'>
This makes it possible for HW PHY-less boards to utilize PAL goodies.  Generic
routines to connect to fixed PHY are provided, as well as ability to specify
software callback that fills up link, speed, etc.  information into PHY
descriptor (the latter feature not tested so far).

Signed-off-by: Vitaly Bordug &lt;vbordug@ru.mvista.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@osdl.org&gt;
Signed-off-by: Jeff Garzik &lt;jeff@garzik.org&gt;
</content>
</entry>
<entry>
<title>Remove obsolete #include &lt;linux/config.h&gt;</title>
<updated>2006-06-30T17:25:36Z</updated>
<author>
<name>Jörn Engel</name>
<email>joern@wohnheim.fh-wedel.de</email>
</author>
<published>2006-06-30T17:25:36Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=6ab3d5624e172c553004ecc862bfeac16d9d68b7'/>
<id>urn:sha1:6ab3d5624e172c553004ecc862bfeac16d9d68b7</id>
<content type='text'>
Signed-off-by: Jörn Engel &lt;joern@wohnheim.fh-wedel.de&gt;
Signed-off-by: Adrian Bunk &lt;bunk@stusta.de&gt;
</content>
</entry>
<entry>
<title>[PATCH] missing license for libphy.ko</title>
<updated>2005-12-24T15:05:51Z</updated>
<author>
<name>Olaf Hering</name>
<email>olh@suse.de</email>
</author>
<published>2005-12-13T23:33:49Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=afcceaa3c7b7fbf7e2b95c5f0dd43547b03292d2'/>
<id>urn:sha1:afcceaa3c7b7fbf7e2b95c5f0dd43547b03292d2</id>
<content type='text'>
Andy,

libphy has no license tag. Something like the attached (untested!) patch
is needed. Hopefully such a change finds its way into 2.6.15.

filename: /lib/modules/2.6.15-rc5-3-ppc64/kernel/drivers/net/phy/libphy.ko
vermagic:       2.6.15-rc5-3-ppc64 SMP gcc-4.1
depends:
srcversion:     ACC921B5E82701BE1E6F603

 drivers/net/phy/phy_device.c |    4 ++++
 1 files changed, 4 insertions(+)

Signed-off-by: Jeff Garzik &lt;jgarzik@pobox.com&gt;
</content>
</entry>
<entry>
<title>Remove linux/version.h include from drivers/net/phy/* and net/ieee80211/*.</title>
<updated>2005-11-06T04:39:54Z</updated>
<author>
<name>Jeff Garzik</name>
<email>jgarzik@pobox.com</email>
</author>
<published>2005-11-06T04:39:54Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=a10b5aacea01d59152b9d003a14476ee99d394d8'/>
<id>urn:sha1:a10b5aacea01d59152b9d003a14476ee99d394d8</id>
<content type='text'>
Unused, and causes the files to be needlessly rebuilt in some cases.
</content>
</entry>
<entry>
<title>[netdrvr] delete CONFIG_PHYCONTROL</title>
<updated>2005-09-24T02:58:49Z</updated>
<author>
<name>Jeff Garzik</name>
<email>jgarzik@pobox.com</email>
</author>
<published>2005-09-24T02:58:49Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=8cee0cd5bec53b78dd3c43a73b27821a688133e6'/>
<id>urn:sha1:8cee0cd5bec53b78dd3c43a73b27821a688133e6</id>
<content type='text'>
</content>
</entry>
<entry>
<title>[PATCH] PHY Layer fixup</title>
<updated>2005-08-29T00:28:25Z</updated>
<author>
<name>Andy Fleming</name>
<email>afleming@freescale.com</email>
</author>
<published>2005-08-24T23:46:21Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=e13934563db047043ccead26412f552375cea90c'/>
<id>urn:sha1:e13934563db047043ccead26412f552375cea90c</id>
<content type='text'>
This patch adds back the code that was taken out, thus re-enabling:

* The PHY Layer to initialize without crashing
* Drivers to actually connect to PHYs
* The entire PHY Control Layer

This patch is used by the gianfar driver, and other drivers which are in
development.

Signed-off-by: Andy Fleming &lt;afleming@freescale.com&gt;
Signed-off-by: Jeff Garzik &lt;jgarzik@pobox.com&gt;
</content>
</entry>
<entry>
<title>phy subsystem: more cleanups</title>
<updated>2005-08-11T06:47:54Z</updated>
<author>
<name>Jeff Garzik</name>
<email>jgarzik@pobox.com</email>
</author>
<published>2005-08-11T06:47:54Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=2bf69b5fe90b3246ab50064c5a690a363e8c53e2'/>
<id>urn:sha1:2bf69b5fe90b3246ab50064c5a690a363e8c53e2</id>
<content type='text'>
- unexport symbols never used outside of home module
- remove dead code
- remove CONFIG_PHYCONTROL, make it unconditionally enabled
</content>
</entry>
<entry>
<title>Fix numerous minor problems with new phy subsystem.</title>
<updated>2005-08-11T06:07:25Z</updated>
<author>
<name>Jeff Garzik</name>
<email>jgarzik@pobox.com</email>
</author>
<published>2005-08-11T06:07:25Z</published>
<link rel='alternate' type='text/html' href='https://git.amat.us/linux/commit/?id=67c4f3fa25502ce7ed82fb0307e09cf36f1f81da'/>
<id>urn:sha1:67c4f3fa25502ce7ed82fb0307e09cf36f1f81da</id>
<content type='text'>
Includes fixes for problems noted by Adrian Bunk, Andrew Morton,
and one other person lost in the annals of history (and email folders).
</content>
</entry>
</feed>
