Re: [U-Boot] [PATCH v3 3/4] net: phy: Force master mode A20-OLinuXino-Lime2

2016-03-25 Thread Michael Haas
On 03/26/2016 12:46 AM, Iain Paton wrote: > On 24/03/16 06:46, Michael Haas wrote: >> Force master mode on the A20-OLinuXino-Lime2. This change is required >> to get a reliable link at gigabit speeds. >> >> Signed-off-by: Michael Haas > Acked-by: Iain Paton > >

[U-Boot] [PATCH v5 1/3] net: phy: Optionally force master mode for RTL PHY

2016-03-25 Thread Michael Haas
: hdego...@redhat.com CC: i...@hellion.org.uk CC: joe.hershber...@ni.com Signed-off-by: Michael Haas Tested-by: Karsten Merker --- Changes in v5: - Improve formatting of Kconfig help text. No content change. Change suggested by Karsten Merker. Changes in v4: - Squashed previously separate

[U-Boot] [PATCH v5 2/3] sunxi: A20-Olimex-SOM-EVB: Force 8211CL to master

2016-03-25 Thread Michael Haas
Force master mode for 1000BASE-T operation on the A20-Olimex-SOM-EVB. Karsten Merker reports that this change is necessary to get a reliable link at gigabit speeds. Signed-off-by: Michael Haas --- Changes in v5: - Fix order of defconfig entry (suggested by Karsten Marker) Series-changes: 4

[U-Boot] [PATCH v5 3/3] sunxi: A20-OLinuXino-Lime2: Force 8211CL to master

2016-03-25 Thread Michael Haas
Force master mode on the A20-OLinuXino-Lime2. This change is required to get a reliable link at gigabit speeds. Signed-off-by: Michael Haas --- Changes in v5: - Fix order of defconfig entry (suggested by Karsten Marker) Changes in v4: - Changed commit summary according to Chen-Yu Tsai&#

[U-Boot] [PATCH v5 0/3] net: phy: Force master mode for RTL8211C on some boards

2016-03-25 Thread Michael Haas
def instead (thanks to Karsten Merker) - Introduced constants for register bits Changes in v2: - Removed accidental inclusion of Karsten's patch in my first submission of this series. - Fix a typo in the code: 6 -> & Michael Haas (3): net: phy: Optionally force master mode for RTL PHY

Re: [U-Boot] Linux hangs due to commit v2015.10-15-g02cc27c on loading i2c-mv64xx

2016-03-25 Thread Michael Haas
On 03/20/2016 08:15 PM, Hans de Goede wrote: > > I'm running Debian Jessie with Linux 4.3.0-0.bpo.1-armmp-lpae on my > > a20-olinuxino-lime2. > > I have noticed that my board hangs with my recent u-boot versions > when I > > load the i2c module. > > > > git-bisect narrowed the problem down to the f

[U-Boot] [PATCH v4 1/3] net: phy: Optionally force master mode for RTL PHY

2016-03-25 Thread Michael Haas
: hdego...@redhat.com CC: i...@hellion.org.uk CC: joe.hershber...@ni.com Signed-off-by: Michael Haas Tested-by: Karsten Merker --- Changes in v4: - Squashed previously separate commit introducing KCONFIG variable into commit containing main code change (Hans de Goede's suggestion) - Ch

[U-Boot] [PATCH v4 2/3] sunxi: A20-Olimex-SOM-EVB: Force 8211CL to master

2016-03-25 Thread Michael Haas
Force master mode for 1000BASE-T operation on the A20-Olimex-SOM-EVB. Karsten Merker reports that this change is necessary to get a reliable link at gigabit speeds. Signed-off-by: Michael Haas --- Changes in v4: - Changed commit summary according to Chen-Yu Tsai's suggestion, modifi

[U-Boot] [PATCH v4 0/3] net: phy: Force master mode for RTL8211C on some boards

2016-03-25 Thread Michael Haas
tion of RTL8211CL and use #ifdef instead (thanks to Karsten Merker) - Introduced constants for register bits Changes in v2: - Removed accidental inclusion of Karsten's patch in my first submission of this series. - Fix a typo in the code: 6 -> & Michael Haas (3): net: phy: O

[U-Boot] [PATCH v4 3/3] sunxi: A20-OLinuXino-Lime2: Force 8211CL to master

2016-03-25 Thread Michael Haas
Force master mode on the A20-OLinuXino-Lime2. This change is required to get a reliable link at gigabit speeds. Signed-off-by: Michael Haas --- Changes in v4: - Changed commit summary according to Chen-Yu Tsai's suggestion, modified to fit the 70 character limit Changes in v3: None Ch

[U-Boot] [PATCH v3 1/4] net: phy: Optionally force master mode for RTL PHY

2016-03-23 Thread Michael Haas
...@hellion.org.uk CC: joe.hershber...@ni.com Signed-off-by: Michael Haas --- Changes in v3: - Remove incorrect detection of RTL8211CL and use #ifdef instead (thanks to Karsten Merker) - Introduced constants for register bits Changes in v2: - Removed accidental inclusion of Karsten's patch in my

[U-Boot] [PATCH v3 4/4] net: phy: Add RTL8211X_PHY_FORCE_MASTER to Kconfig

2016-03-23 Thread Michael Haas
This change mainly serves as a way to document the define. Signed-off-by: Michael Haas --- Changes in v3: None Changes in v2: None drivers/net/Kconfig | 17 + 1 file changed, 17 insertions(+) diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig index 2a229b8..b102bc6 100644

[U-Boot] [PATCH v3 2/4] net: phy: Force master mode for A20-Olimex-SOM-EVB

2016-03-23 Thread Michael Haas
Force master mode for 1000BASE-T operation on the A20-Olimex-SOM-EVB. Karsten Merker reports that this change is necessary to get a reliable link at gigabit speeds. Signed-off-by: Michael Haas --- Changes in v3: None Changes in v2: None configs/A20-Olimex-SOM-EVB_defconfig | 1 + 1 file

[U-Boot] [PATCH v3 3/4] net: phy: Force master mode A20-OLinuXino-Lime2

2016-03-23 Thread Michael Haas
Force master mode on the A20-OLinuXino-Lime2. This change is required to get a reliable link at gigabit speeds. Signed-off-by: Michael Haas --- Changes in v3: None Changes in v2: None configs/A20-OLinuXino-Lime2_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/A20

[U-Boot] [PATCH v3 0/4] net: phy: Force master mode for RTL8211C on some boards

2016-03-23 Thread Michael Haas
n v3: - Remove incorrect detection of RTL8211CL and use #ifdef instead (thanks to Karsten Merker) - Introduced constants for register bits Changes in v2: - Removed accidental inclusion of Karsten's patch in my first submission of this series. - Fix a typo in the code: 6 -> & M

Re: [U-Boot] [PATCH v2] sunxi: Fix gmac not working due to cpu_eth_init no longer being called

2016-03-23 Thread Michael Haas
ccm_reg *const ccm = > @@ -79,16 +79,4 @@ int sunxi_gmac_initialize(bd_t *bis) > for (pin = SUNXI_GPA(26); pin <= SUNXI_GPA(27); pin++) > sunxi_gpio_set_cfgpin(pin, SUN6I_GPA_GMAC); > #endif > - > -#ifdef CONFIG_DM_ETH > - return 0; > -#else > -#

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-22 Thread Michael Haas
Am 22. März 2016 09:57:16 MEZ, schrieb Hans de Goede : >Hi, > >On 22-03-16 09:46, Peter Korsgaard wrote: >>> "Hans" == Hans de Goede writes: >> >> Hi, >> >> > Oh good catch, so I believe that we should submit a patch to the >u-boot >> > realtek phy driver which: >> >> > 1) Adds a seperat

[U-Boot] [RFC PATCH] net: phy: Force master mode for RTL8211C

2016-03-22 Thread Michael Haas
two. The NetBSD driver uses the same approach. CC: fra...@gmail.com CC: mer...@debian.org CC: hdego...@redhat.com CC: i...@hellion.org.uk CC: joe.hershber...@ni.com Signed-off-by: Michael Haas --- drivers/net/phy/realtek.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers

[U-Boot] [RFC PATCH 0/1] net: phy: Force master mode for RTL8211C

2016-03-22 Thread Michael Haas
1B/C PHY ID fix'. Series-Version: 2 Series-changes: 2 - Removed accidental inclusion of Karsten's patch in my first submission of this series. - Fix a typo in the code: 6 -> & Michael Michael Haas (1): net: phy: Force master mode for RTL8211C drivers/net/phy/realtek.c |

Re: [U-Boot] [RFC PATCH] net: phy: Force master mode for RTL8211C

2016-03-22 Thread Michael Haas
On 03/22/2016 08:19 AM, Michael Haas wrote: > The RTL8211C found on the A20-OlinuXino-Lime2 does not word in slave > mode. This patch disables master/slave mode autonegotiation and forces > master mode. > > The RTL8211C identifies itself as RTL8211B via its UID. This patch uses

[U-Boot] [RFC PATCH 0/1] net: phy: Force master mode for RTL8211C

2016-03-22 Thread Michael Haas
This patch is an RFC based on recent discussions. It's only lightly tested. I have yet to verify that the registers are being set correctly. I'm sending it out now because I have to leave for today. Michael Michael Haas (1): net: phy: Force master mode for RTL8211C drive

[U-Boot] [RFC PATCH] net: phy: Force master mode for RTL8211C

2016-03-22 Thread Michael Haas
two. The NetBSD driver uses the same approach. CC: fra...@gmail.com CC: mer...@debian.org CC: hdego...@redhat.com CC: i...@hellion.org.uk CC: joe.hershber...@ni.com Signed-off-by: Michael Haas --- drivers/net/phy/realtek.c | 16 +++- 1 file changed, 15 insertions(+), 1 deletion

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-21 Thread Michael Haas
On 03/21/2016 11:00 PM, Hans de Goede wrote: > Hi, > > On 21-03-16 22:47, Michael Haas wrote: >>> Hmm, I just realized / read that RTL8211B and RTL8211C have the same >> phyid, maybe there is some other reg which we can use to >> differentiate between them ? >>

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-21 Thread Michael Haas
>Hmm, I just realized / read that RTL8211B and RTL8211C have the same phyid, maybe there is some other reg which we can use to differentiate between them ? >Otherwise we are back to having a #ifdef for the fix ... >Regards, Hans The BSD people seem to key off a revision field, which has value

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-21 Thread Michael Haas
On 03/21/2016 04:47 PM, Karsten Merker wrote: Are any other sunxi boards impacted by the same problem that you know ? >>> No, I don't know of any other boards, but I have not looked very hard :) > At least the Olimex A20-SOM-EVB (which uses the same RTL8211CL > PHY) has the same issue and forc

[U-Boot] Linux hangs due to commit v2015.10-15-g02cc27c on loading i2c-mv64xx

2016-03-20 Thread Michael Haas
Hello all, I'm running Debian Jessie with Linux 4.3.0-0.bpo.1-armmp-lpae on my a20-olinuxino-lime2. I have noticed that my board hangs with my recent u-boot versions when I load the i2c module. git-bisect narrowed the problem down to the following commit: 02cc27c74f9b884b538bcd1b93342a4c05b5d608

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-20 Thread Michael Haas
On 03/20/2016 02:45 PM, Hans de Goede wrote: > Hi, > > On 19-03-16 14:40, Michael Haas wrote: >> On 03/19/2016 10:32 AM, Hans de Goede wrote: >>> Hi, >>> >>> On 19-03-16 09:39, François-David Collin wrote: >>>> Hi, >>>> >&g

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-20 Thread Michael Haas
On 03/18/2016 08:06 PM, Michael Haas wrote: > On 03/18/2016 08:02 PM, Karsten Merker wrote: >> The installer supports a number of systems that can only load images >> of limited size, therefore the installer includes only an >> installation-related subset of all the kernel mo

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-20 Thread Michael Haas
On 03/17/2016 11:52 AM, Hans de Goede wrote: > > 3) The phy settings, unfortunately I do not know of > a way to dump these, so lets just forget about these > and focus on the other 2. > You can dump these with 'mii read' in u-boot. I sat down with the rtl8211eg data sheet and decided to look at 'P

[U-Boot] Which entry in drivers/net/phy/realtek.c matches my device?

2016-03-20 Thread Michael Haas
Hello all, I'd like to add some device-specific hacks to realtek.c. I'm using the Olimex A20-OlinuXino-Lime2 which uses the RTL8211CL PHY. Which of the various phy_driver structs is responsible for that device? I presume it's keyed off the .uid, but I'm not sure how to find the ID of the specific

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-19 Thread Michael Haas
Hello all, After the board locked up, I am now working exclusively from within u-boot. Occasionally, gbit will work well. At other times, it will appear to work well but the kernel won't boot. And sometimes, downloading a kernel will just time-out. A reset is often enough to get a behavior chan

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-19 Thread Michael Haas
d directly to the Gbit interface of my laptop, >> speed are OK: > > Michael Haas' work to debug this by looking at the phy registers > seems to be the most promising so-far, the clk reg and axp209 > registers all seem to be identical between good and bad setups. > >

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-19 Thread Michael Haas
Hello all, On 03/16/2016 09:36 PM, Karsten Merker wrote: > Michael, if you would like to try testing higher RX delay chain values > on the Lime2, I have attached a small patch that adds an appropriate > Kconfig option. Regards, Karsten I will be testing your patch along with Hans' patch. In fac

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-19 Thread Michael Haas
d directly to the Gbit interface of my laptop, >> speed are OK: > > Michael Haas' work to debug this by looking at the phy registers > seems to be the most promising so-far, the clk reg and axp209 > registers all seem to be identical between good and bad setups. > >

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-19 Thread Michael Haas
d directly to the Gbit interface of my laptop, >> speed are OK: > > Michael Haas' work to debug this by looking at the phy registers > seems to be the most promising so-far, the clk reg and axp209 > registers all seem to be identical between good and bad setups. > >

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-19 Thread Michael Haas
On 03/17/2016 11:52 AM, Hans de Goede wrote: > Hi, > > There are 3 things which would be interesting to save and compare > with a boot which does have the problem: > > 1) The gmc clk register, to do this download mmio_dump from: > > Working GBit: 01c20164: 0c06 Broken GBit: tomorrow :) > > 2

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-18 Thread Michael Haas
On 03/18/2016 08:02 PM, Karsten Merker wrote: > The installer supports a number of systems that can only load images > of limited size, therefore the installer includes only an > installation-related subset of all the kernel modules and i2c-mv64xxx > currently isn't among those. This hasn't been a

Re: [U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-16 Thread Michael Haas
On 03/15/2016 07:09 PM, Karsten Merker wrote: > On Tue, Mar 15, 2016 at 06:41:39AM +0100, Michael Haas wrote: > >> This change is required to get GBIT Ethernet to work >> reliably on my board. Without CONFIG_GMAC_TX_DELAY=4, the connection >> suffers severe packet loss

[U-Boot] [PATCH] Add CONFIG_GMAC_TX_DELAY=4 for OlinuXino Lime2

2016-03-15 Thread Michael Haas
ff-by: Michael Haas Tested-by: Michael Haas Cc: i...@hellion.org.uk Cc: hdego...@redhat.com --- configs/A20-OLinuXino-Lime2_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/configs/A20-OLinuXino-Lime2_defconfig b/configs/A20-OLinuXino-Lime2_defconfig index b5181c6..f75b191 100644