All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Antoine Ténart" <antoine.tenart@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH/next 4/5] solidrun/macchiatobin: add new board
Date: Tue, 14 Nov 2017 08:41:48 +0100	[thread overview]
Message-ID: <20171114074148.GC25491@kwain> (raw)
In-Reply-To: <20171113233628.64f1e58d@windsurf.home>

Hello Thomas, Sergey,

On Mon, Nov 13, 2017 at 11:36:28PM +0100, Thomas Petazzoni wrote:
> On Mon, 13 Nov 2017 23:06:10 +0300, Sergey Matyukevich wrote:
> 
> > Meanwhile this is not the only problem with mainline BSP. It turned out that
> > network interfaces do not work properly neither on v4.13.11 nor in fresh v4.14.
> > Kernel v4.13.11 doesn't enable them in DTS. Kernel v4.14 does enable them,
> > but I couldn't make them work: Tx looks ok, but no Rx. Meanwhile on kernel from
> > devel-4.14/all branch from https://github.com/MISL-EBU-System-SW/mainline-public
> > management 1Gb interface appears to be working fine.
> 
> The fact that network interfaces don't work on mcbin with the stock
> 4.14 is not expected. I've added in Cc: my colleague Antoine T?nart who
> worked on this topic.
> 
> Antoine, do you know which patches are missing in 4.14 ?

The 4.14 network support for the mcbin is partial but you should be able
to use 2 out of the 4 interfaces in the mode U-Boot configured them.
tl;dr; eth0 in 10G (sfp) and eth2 in 1G.

See https://elinux.org/Marvell_EBU:MacchiatoBin for the network
interfaces description.

- eth0 should work on 10G (the sfp cage).
- eth1 is not working because of a bug which is not cleanly fixed yet.
  The devel-4.14/all branch contains a workaround (which is not suitable
  for the upstream kernel), see 0968cd30a0d4 ("DO NOT UPSTREAM: phy: marvell10g:
  Do not reset PHY autoneg for 10G KR mode"). With this patch it should
  work on 10G (sfp) with the 4.14 kernel.
- eth2 should be working just fine on 1G.
- eth3 is not supported yet, in both the upstream kernel and the
  devel-4.14/all branch.

One feature not supported in the upstream 4.14 kernel is the dynamic
reconfiguration of the interfaces, i.e. the ability to switch from/to
10G to/from 1G on eth0 and eth1 given what's connected. To do this you
would need a few device tree patches from devel-4.14/all (which will be
in 4.15):
- 0f0dfbec373e ("arm64: dts: marvell: extend the cp110 syscon register area length")
- c90dab5d2bf4 ("arm64: dts: marvell: add comphy nodes on cp110 master and slave")
- 26c08c06b827 ("arm64: dts: marvell: mcbin: add comphy references to Ethernet ports")

Also, please note the network PHY drivers are not selected to be
built-in in the arm64 upstream defconfig, so you should probably
select both CONFIG_MARVELL_PHY and CONFIG_MARVELL_10G_PHY. This is done
in the devel-4.14/all arm64 defconfig.

Antoine

-- 
Antoine T?nart, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

  reply	other threads:[~2017-11-14  7:41 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-11-12 20:16 [Buildroot] [PATCH/next 0/5] Add support for MacchiatoBin board Sergey Matyukevich
2017-11-12 20:16 ` [Buildroot] [PATCH/next 1/5] armada-firmware: new package Sergey Matyukevich
2017-11-13  7:30   ` Baruch Siach
2017-11-13 19:47     ` Sergey Matyukevich
2017-11-14 22:07   ` Arnout Vandecappelle
2017-11-15 21:15     ` Sergey Matyukevich
2017-11-15 21:55       ` Arnout Vandecappelle
2017-11-12 20:16 ` [Buildroot] [PATCH/next 2/5] armada-ddr: " Sergey Matyukevich
2017-11-13  8:00   ` Baruch Siach
2017-11-13 20:12     ` Sergey Matyukevich
2017-11-12 20:16 ` [Buildroot] [PATCH/next 3/5] atf: add support for Marvell Armada SoCs Sergey Matyukevich
2017-11-13  8:23   ` Baruch Siach
2017-11-13 20:10     ` Sergey Matyukevich
2017-11-12 20:16 ` [Buildroot] [PATCH/next 4/5] solidrun/macchiatobin: add new board Sergey Matyukevich
2017-11-13 12:27   ` Baruch Siach
2017-11-13 12:51     ` Jan Kundrát
2017-11-13 13:13       ` Baruch Siach
2017-11-13 20:06     ` Sergey Matyukevich
2017-11-13 22:36       ` Thomas Petazzoni
2017-11-14  7:41         ` Antoine Ténart [this message]
2017-11-14  9:20           ` Sergey Matyukevich
2017-11-14  9:42             ` Antoine Ténart
2017-11-14 20:27               ` Sergey Matyukevich
2017-11-14 20:53                 ` Antoine Ténart
2017-11-15 21:20                   ` Sergey Matyukevich
2017-11-13 14:52   ` Baruch Siach
2017-11-13 19:51     ` Sergey Matyukevich
2017-11-14 19:11       ` Sergey Matyukevich
2017-11-12 20:16 ` [Buildroot] [PATCH/next 5/5] Add myself to MacchiatoBin releated packages Sergey Matyukevich
2017-11-14 21:58 ` [Buildroot] [PATCH/next 0/5] Add support for MacchiatoBin board Arnout Vandecappelle
2017-11-15 15:59   ` Baruch Siach

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=20171114074148.GC25491@kwain \
    --to=antoine.tenart@free-electrons.com \
    --cc=buildroot@busybox.net \
    /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.