From mboxrd@z Thu Jan  1 00:00:00 1970
From: Hannes Schmelzer <hannes@schmelzer.or.at>
Date: Thu, 23 Mar 2017 15:16:11 +0100
Subject: [U-Boot] [PATCH v4] drivers/net/phy: add fixed-phy / fixed-link
	support
In-Reply-To: <CAH9NwWeEpGtaSak3f9n91bGf1mzJy-z879F34-k3V3BLrQHpaw@mail.gmail.com>
References: <1490127593-22756-1-git-send-email-oe5hpm@oevsv.at>
 <CAH9NwWeEpGtaSak3f9n91bGf1mzJy-z879F34-k3V3BLrQHpaw@mail.gmail.com>
Message-ID: <7888351a-1803-adbe-0445-7751c1cc9e4b@schmelzer.or.at>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de



On 23.03.2017 15:03, Christian Gmeiner wrote:
> Hi
Hi Christian,
>
>> +int fixedphy_probe(struct phy_device *phydev)
>> +{
>> +       struct fixed_link *priv;
>> +       int ofnode = phydev->addr;
>> +       u32 val;
>> +
>> +       /* check for mandatory properties within fixed-link node */
>> +       val = fdt_getprop_u32_default_node(gd->fdt_blob,
>> +                                          ofnode, 0, "speed", 0);
>> +       if (val != SPEED_10 && val != SPEED_1000 && val != SPEED_1000) {
>
> Still wrong - SPEED_100 is missing.
>
>            if (val != SPEED_10 && val != SPEED_100 && val != SPEED_1000) {
What a mess ... looked 100 times to it ...
Sending now a v5. Many thanks.

cheers,
Hannes