From owner-freebsd-arm@freebsd.org Fri Nov 6 14:52:45 2015 Return-Path: Delivered-To: freebsd-arm@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id E7FC8A28D5F for ; Fri, 6 Nov 2015 14:52:44 +0000 (UTC) (envelope-from zbb@semihalf.com) Received: from mail-lf0-x22c.google.com (mail-lf0-x22c.google.com [IPv6:2a00:1450:4010:c07::22c]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 89D661E93 for ; Fri, 6 Nov 2015 14:52:44 +0000 (UTC) (envelope-from zbb@semihalf.com) Received: by lfgh9 with SMTP id h9so72998194lfg.1 for ; Fri, 06 Nov 2015 06:52:42 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=semihalf_com.20150623.gappssmtp.com; s=20150623; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; bh=zjLHwvS8s1K5Y5k2XglxaB//dOYXHo82gKs+LX1wXAY=; b=rLzLgcvLUI8qCcSMMZLWzWCEAUx/tG0eqICxmMTF3yVH5Q+toskPkq4fv4/6FMQkFs bP5nfC+LtcYXC5pwWlalhnzErl+R+5gZCqzvDl0FmXXANNL1Ut/gmfJSS520hSC770Fs M4qWxFFYMGF+oLL+rYwD2wb8BNdfBrK/9TOuf0d3lNHJUU7REDFlCyFIjunoWBc4bLOS +u8vF488OGy28HotsRmNLcmB4dYqY5NfnIEnBCF0yEiRXKr0t8wTCQ2oRmkfHEke0HWV MEF3PTJfjbIxdS2ggLT8OonrOMN2J2PcPHGY0sZz7IYQtETUvCN+5nQvIjAcNh3mFnKA eGWQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc:content-type; bh=zjLHwvS8s1K5Y5k2XglxaB//dOYXHo82gKs+LX1wXAY=; b=NMrf4j9uzg+uVaktftg0plKGd/VD7sYHTjQCl5t6yWM2SjKRqPaNm4AOdBlJU/A+qE vB6xDdihEDvousv4Kmj3fyi0GTAn3kf4b/0icgyirt+H9XHtS+ucf6S3SQz3K8QjlBJ7 8T+h78qMhKVTpu3e/MxJFpY71WDCoNbAYKVzebavMaMH4KDMQcpTXB4XSmZskRxaF2FP K1ihvtexDdPV2VairNKR/cbfR66Vm73DTy3CKOswvMvBZ/MBvxgQE/0wyQYk1UUjTVhd OIskmv3N4PSB7HSumbR4q6FDSCn/o6S23iaDSbblAUt2yje88XvkjtkplzW1xhq0XMEy sibQ== X-Gm-Message-State: ALoCoQkRtOEn1JePEm69VKMaV91avyDk2RqjlShs77m4L7/XIS2CX35hWdfVoMmQg+f2tNX5U0j7 X-Received: by 10.25.77.17 with SMTP id a17mr4205799lfb.19.1446821562354; Fri, 06 Nov 2015 06:52:42 -0800 (PST) MIME-Version: 1.0 Received: by 10.25.91.20 with HTTP; Fri, 6 Nov 2015 06:52:22 -0800 (PST) In-Reply-To: <20151106144511.3309d8b2@bender> References: <20151106094714.5e8632c6@bender.Home> <20151106113641.02e0f826@bender> <20151106144511.3309d8b2@bender> From: Zbigniew Bodek Date: Fri, 6 Nov 2015 15:52:22 +0100 Message-ID: Subject: Re: HEADS UP: Cavium ThunderX support in the tree To: Andrew Turner Cc: "freebsd-arm@freebsd.org" Content-Type: text/plain; charset=UTF-8 X-BeenThere: freebsd-arm@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: "Porting FreeBSD to ARM processors." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 06 Nov 2015 14:52:45 -0000 2015-11-06 15:45 GMT+01:00 Andrew Turner : > On Fri, 6 Nov 2015 13:10:10 +0100 > Zbigniew Bodek wrote: > >> 2015-11-06 12:36 GMT+01:00 Andrew Turner : >> > On Fri, 6 Nov 2015 12:23:11 +0100 >> > Zbigniew Bodek wrote: >> >> Secondly, EFI that runs on the board needs to pass the correct DTB >> >> to the kernel (it has to have PCIB, BGX, MDIO, PHYs nodes and they >> >> need to be in the correct order [MDIO needs to attach before >> >> BGX]). >> > If this is the case the code is broken. It should just work with >> > the vendor dtb and GENERIC. >> >> Thanks for quick evaluation but please see comment below. >> >> > >> > We already have to handle this in other parts of the code by having >> > the dependent drivers on an earlier pass than later drivers. >> >> MDIO is a platform device whereas BGX, NICPF (physical function) and >> NICVF (virtual function) are devices on PCI. >> Therefore if the PCIB is placed prior to MDIO in DTB all devices on >> this PCI bridge will be attached before MDIO is attached. >> As far as I know there is no straightforward way to create a logical >> relation between device on PCI and platform devices enumerated using >> DTB. >> That is why we asked Cavium to change the order in their DTBs. >> Of course if you know how to workaround that I would be happy to apply >> this change to "unbreak" the existing code. > > No need, I've fixed it in r290448. > > Andrew OK. Thanks! Best regards zbb