public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: "Heiko Stübner" <heiko@sntech.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 0/9] From Simon:
Date: Wed, 30 Nov 2016 00:45:13 +0100	[thread overview]
Message-ID: <1690690.ACfjhlv7gu@diego> (raw)
In-Reply-To: <1480266109-7209-1-git-send-email-sjg@chromium.org>

Hi Simon,

Am Sonntag, 27. November 2016, 10:01:40 schrieb Simon Glass:
> This v3 patch is an update on Sjoerd's original v2 series from Feburary.
> I have dealt with the changes requested at the time, and adjusted the way
> that the speed change is handled.
> 
> Tested on firefly-rk3288, rock2.

Tested on a firefly as well. Though I get mixed results in my netboot 
environment. At 100MBit (manually limited) everything seems fine:

----------------------
Speed: 100, full duplex
BOOTP broadcast 1
BOOTP broadcast 2
DHCP client bound to address 192.168.140.58 (269 ms)
Using ethernet at ff290000 device
TFTP from server 192.168.140.1; our IP address is 192.168.140.58
Filename 'hstuebner/firefly.vmlinuz'.
Load address: 0x4000000
Loading: #################################################################
         #################################################################
         #################################################################
         #################################################################
         #################################################################
         #################################################################
         #################################################################
         #########################
         2.9 MiB/s
done
Bytes transferred = 7033483 (6b528b hex)
----------------------

5 out of 5 boots worked fine.


but at 1000MHz I only get:

----------------------
Speed: 1000, full duplex
BOOTP broadcast 1
BOOTP broadcast 2
DHCP client bound to address 192.168.140.57 (270 ms)
Using ethernet at ff290000 device
TFTP from server 192.168.140.1; our IP address is 192.168.140.57
Filename 'hstuebner/firefly.vmlinuz'.
Load address: 0x4000000
Loading: #####T #T #T ##T T #T #T T ####T #T ####
Retry count exceeded; starting again
----------------------

on 5 boots. 1 lonely boot also worked at 1000MBit for some unknown reason.
I'm not sure if just my switch is some special snowflake (TL-SG1024 from TP-
Link) or there is some other voodoo at work here.

The rootfs over nfs seems to work fine on 1000MBit though.


Heiko

> Original cover letter:
> To add support I've taken a slightly different approach then some of the
> other boards with a designware IP block, by creating a new driver to
> take care of the platfrom glue which subclasses the main designware driver
> instead of adding the compatibility string the designware driver
> directly and doing the SoC specific setup in the board files. This seems
> quite a bit more elegant in a device model based world.
> 
> I've only tested this series on a Radxa Rock 2 board, it would be great
> if someone could test this on other boards with the designware IP
> especially for those with the reset GPIO in devicetree (e.g. some of the
> Allwinner boards).
> 
> Compared to the first one round the pinctrl related bits were dropped as
> RK3288 now has a full pinctrl driver. Furthermore the started hook in the
> designware driver was renamed to fix_mac_speed in line with what linux
> uses and moved to the dw_link_adjust function.
> 
> Changes in v3:
> - Add a few new patches
> - Add comments for struct gmac_rk3288_platdata
> - Add new patch to adjust dw_adjust_link() to return an error
> - Add new patch to enable networking on evb-rk3288
> - Add new patch to export the operation functions
> - Add new patch to split the link init into a separate function
> - Adjust binding to use r/tx-delay instead of r/tx_delay
> - Drop the 'net: designware: Add a fix_mac_speed hook' patch
> - Sort includes
> - Use debug() instead of printf() for error
> - Use function calls instead of fix_mac_speed() hook
> - Use new clk interface
> 
> Changes in v2:
> - Adjust to new hook name
> - Fix various coding style nits
> 
> Simon Glass (4):
>   net: designware: Adjust dw_adjust_link() to return an error
>   net: designware: Split the link init into a separate function
>   net: designware: Export the operation functions
>   rockchip: evb-rk3339: Enable DHCP
> 
> Sjoerd Simons (5):
>   net: designware: Export various functions/struct to allow subclassing
>   net: gmac_rk3288: Add RK3288 GMAC driver
>   rockchip: Enable networking support on rock2 and firefly
>   rockchip: Add PXE and DHCP to the default boot targets
>   rockchip: Drop Ethernet from the TODO
> 
>  configs/evb-rk3399_defconfig      |   3 +
>  configs/firefly-rk3288_defconfig  |   4 +
>  configs/rock2_defconfig           |   4 +
>  doc/README.rockchip               |   1 -
>  drivers/net/Kconfig               |   7 ++
>  drivers/net/Makefile              |   1 +
>  drivers/net/designware.c          |  57 ++++++++++----
>  drivers/net/designware.h          |  13 ++++
>  drivers/net/gmac_rk3288.c         | 154
> ++++++++++++++++++++++++++++++++++++++ include/configs/rockchip-common.h | 
>  4 +-
>  10 files changed, 230 insertions(+), 18 deletions(-)
>  create mode 100644 drivers/net/gmac_rk3288.c

  parent reply	other threads:[~2016-11-29 23:45 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-27 17:01 [U-Boot] [PATCH v3 0/9] From Simon: Simon Glass
2016-11-27 17:01 ` [U-Boot] [PATCH v3 1/9] net: designware: Export various functions/struct to allow subclassing Simon Glass
2016-11-29 22:28   ` Joe Hershberger
2016-11-27 17:01 ` [U-Boot] [PATCH v3 2/9] net: designware: Adjust dw_adjust_link() to return an error Simon Glass
2016-11-29 22:10   ` Joe Hershberger
2016-11-27 17:01 ` [U-Boot] [PATCH v3 3/9] net: designware: Split the link init into a separate function Simon Glass
2016-11-29 22:28   ` Joe Hershberger
2016-11-27 17:01 ` [U-Boot] [PATCH v3 4/9] net: designware: Export the operation functions Simon Glass
2016-11-29 22:31   ` Joe Hershberger
2016-11-27 17:01 ` [U-Boot] [PATCH v3 5/9] net: gmac_rk3288: Add RK3288 GMAC driver Simon Glass
2016-11-29 22:34   ` Joe Hershberger
2016-11-27 17:01 ` [U-Boot] [PATCH v3 6/9] rockchip: Enable networking support on rock2 and firefly Simon Glass
2016-11-27 17:01 ` [U-Boot] [PATCH v3 7/9] rockchip: evb-rk3339: Enable DHCP Simon Glass
2016-11-29 10:22   ` Heiko Stübner
2016-11-27 17:01 ` [U-Boot] [PATCH v3 8/9] rockchip: Add PXE and DHCP to the default boot targets Simon Glass
2016-11-27 17:01 ` [U-Boot] [PATCH v3 9/9] rockchip: Drop Ethernet from the TODO Simon Glass
2016-11-29 23:45 ` Heiko Stübner [this message]
2016-11-30  2:47   ` [U-Boot] [PATCH v3 0/9] From Simon: Simon Glass
2016-11-30 11:52     ` Heiko Stübner
2016-11-30 19:50       ` Heiko Stübner
2016-12-01  2:20       ` Simon Glass
2016-12-01 15:30         ` Heiko Stübner
2016-12-26  5:23           ` Simon Glass

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=1690690.ACfjhlv7gu@diego \
    --to=heiko@sntech.de \
    --cc=u-boot@lists.denx.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