From: Sam Ravnborg <sam@ravnborg.org>
To: Barebox List <barebox@lists.infradead.org>
Cc: Sam Ravnborg <sam@ravnborg.org>
Subject: [PATCH v3 0/3] Add Davicom phy + reset-gpios
Date: Mon, 20 Aug 2018 16:52:15 +0200 [thread overview]
Message-ID: <20180820145215.GA22058@ravnborg.org> (raw)
Changes in v3:
- Consider the v3 a new implmentation - almost nothing was
left from v2. Thus also invalidate any former review - sorry.
- Dropped the need for the mdio {} node, as this node is
only used when there is dedicated HW support for the mdio.
- Added so PHY child nodes to ethernet nodes are registered,
and as part of registration the PHY nodes are reset if
the reset-gpios property is present.
- Further dropped that the parsed info is stored in the bus,
as we only do reset once, thus there is no need to
save the info from the DT
- Futher dropped helper function to reset. They was not needed
- Named the gpio with the name of the PHY node
in the DT. This makes it unique and easier to recognize.
Changes in v2:
- Added patch to enable Davicom PHY on at91sam9263ek - evaluation kit
- Fix so we do reset before comunicating with the PHY
- Rename to mdio_reset()
- Reference correct binding file in commit log (mdio.txt)
- Tested on at91sam9263ek
The at91sam9263ek kit do not require the reset like my
proprietary board, so no DT changes required
Intro:
The following patches was necessary to get networking
operational on my proprietary target.
The target is at91sam9263 based with a Davicom PHY.
The Davicom PHY is a straight copy form the Linux
kernel with the interrupt routine removed and
minor adjustments to the rest.
The davicom PHY would not work until it had seen a reset
cycle - which I think may be an artifact of the board design.
To fix the reset issue I have implemented support for the
reset-gpios binding (see net/phy.txt bindings).
A minimal implmentation was done, just enough to get
my target running.
I could have implemented something in macb -
but I preferred the more generic solution.
Also included are a patch that for the at91sam9263ek
evaluation board. The patch adds several extra tools
that are usefull for testing, and enable the Davicom PHY.
Sam
Sam Ravnborg (3):
phylib: add Davicom PHY support
phylib: add support for reset-gpios
at91sam9263ek: add PHY, miitool etc. to config
arch/arm/configs/at91sam9263ek_defconfig | 12 ++
drivers/net/phy/Kconfig | 5 +
drivers/net/phy/Makefile | 1
drivers/net/phy/davicom.c | 140 +++++++++++++++++++++++++++++++
drivers/net/phy/mdio_bus.c | 82 ++++++++++++++++++
5 files changed, 239 insertions(+), 1 deletion(-)
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next reply other threads:[~2018-08-20 14:52 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-20 14:52 Sam Ravnborg [this message]
2018-08-20 14:56 ` [PATCH v3 1/3] phylib: add Davicom PHY support Sam Ravnborg
2018-08-20 14:56 ` [PATCH v3 2/3] phylib: add support for reset-gpios Sam Ravnborg
2018-08-21 7:50 ` Sascha Hauer
2018-08-21 19:46 ` Sam Ravnborg
2018-08-22 13:14 ` Ahmad Fatoum
2018-08-22 18:26 ` Sam Ravnborg
2018-08-20 14:56 ` [PATCH v3 3/3] at91sam9263ek: add PHY, miitool etc. to config Sam Ravnborg
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=20180820145215.GA22058@ravnborg.org \
--to=sam@ravnborg.org \
--cc=barebox@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.