All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Olimex-A20-EVB ethernet stops working with latest U-boot, works with tag v2016.01
Date: Mon, 14 Mar 2016 08:46:18 +0100	[thread overview]
Message-ID: <56E66C4A.2030105@redhat.com> (raw)
In-Reply-To: <20160313111439.GA29892@excalibur.cnev.de>

Hi,

On 13-03-16 12:14, Karsten Merker wrote:
> On Wed, Mar 09, 2016 at 02:39:11PM -0700, Simon Glass wrote:
>> On 9 March 2016 at 14:04, Karsten Merker <merker@debian.org> wrote:
>>> [Olimex A20-SOM-EVB: ethernet non-functional with current u-boot
>>>                       git head, but no problems with v2016.01]
> [...]
>>> I am not the original poster, but I have bisected the issue down
>>> to the following commit:
>>>
>>> commit c32a6fd07b1839e4a45729587ebc8e1c55601a4d
>>> Author: Simon Glass <sjg@chromium.org>
>>> Date:   Sun Jan 17 14:51:56 2016 -0700
>>>
>>>      net: Don't call board/cpu_eth_init() with driver model
>>>
>>>      We should avoid weak functions with driver model. Existing boards that use
>>>      driver model don't need them, so let's kill them off.
>>>
>>>      Signed-off-by: Simon Glass <sjg@chromium.org>
>>>      Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
>>>      Acked-by: Joe Hershberger <joe.hershberger@ni.com>
>>
>> Is this board-specific init, or sunxi-specific?
>
> Hello Simon,
>
> I am unsure whether I understand your question correctly. There
> is only one cpu_eth_init() function for sunxi-based devices, but
> its behaviour depends on several configuration options. Some of
> those depend on the SoC type (and are thereby indirectly
> board-specific) and some of those (e.g. the GPIO pin controlling
> the regulator that powers the PHY) are directly board-specific.
>
>> Can you just call the init on start-up, instead of from the
>> Ethernet driver?
>
>  From a look at the source in arch/arm/cpu/armv7/sunxi/board.c and
> board/sunxi/gmac.c I guess that should be possible, but I don't
> know enough about u-boot's internals to really assess that.
>
> Hans, could you take a look at the issue?

Yes I've had this thread on my radar / todo the whole time already,
not sure when I'll get around to it. I'll make sure to get this
fixed soon though, as we'll have a new u-boot release soonish.

Regards,

Hans

  parent reply	other threads:[~2016-03-14  7:46 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-24  8:12 [U-Boot] Olimex-A20-EVB ethernet stops working with latest U-boot, works with tag v2016.01 Ing. Damiano Bolla
2016-02-25  1:11 ` Peng Fan
2016-02-25  7:31   ` Ing. Damiano Bolla
2016-03-08 10:54     ` Hans de Goede
     [not found]       ` <20160309210423.GB1855@excalibur.cnev.de>
2016-03-09 21:39         ` Simon Glass
     [not found]           ` <20160313111439.GA29892@excalibur.cnev.de>
2016-03-14  7:46             ` Hans de Goede [this message]
2016-03-14 13:45               ` Ing. Damiano Bolla
2016-03-02  6:02   ` Ing. Damiano Bolla

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=56E66C4A.2030105@redhat.com \
    --to=hdegoede@redhat.com \
    --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 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.