From owner-svn-src-all@freebsd.org Fri Apr 8 16:31:50 2016 Return-Path: Delivered-To: svn-src-all@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 B5A20B092BC; Fri, 8 Apr 2016 16:31:50 +0000 (UTC) (envelope-from sgalabov@gmail.com) Received: from mail-wm0-x243.google.com (mail-wm0-x243.google.com [IPv6:2a00:1450:400c:c09::243]) (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 4886C146A; Fri, 8 Apr 2016 16:31:50 +0000 (UTC) (envelope-from sgalabov@gmail.com) Received: by mail-wm0-x243.google.com with SMTP id y144so5563428wmd.0; Fri, 08 Apr 2016 09:31:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=JmVbKyqa59/y6kCH17DK0N2CsCRr/EnUALd1uzrTvRg=; b=Spp7asnzW5ksoVs1SlPmjQ/qOHC2yyrmYxn752DMyy8nsDyXhFQxwIk/HcmJ8rfbTV YG+keOgOAjZZCDLJx5ia13bH5+OPp0PYFM5RM1uMXgGaNllVkSB1FsjDN2K4Ja8wm5Fo LkGF5q6l1RiZxZ7vXPQLvMv6GDsXYBMgfiPnr0XyMEURxf5TlzIubcFbGevfHQpZ82ns kUUiDHitrHGtFVs/Z5jOmdc/LmmNEc5m+77dRjESKRdiPypQuE57V/NXZenFs9iKZyYN wCgflkzJ8l1tfVD+hk0hbZvXs/CCFT/19jM4j3L67yf2QmXO+hy4MIjEalYxJaWEI61N ILXA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=JmVbKyqa59/y6kCH17DK0N2CsCRr/EnUALd1uzrTvRg=; b=Agtd0kRw+plaFaxGODgs6/lL/i0Dx44hWZe7CzQRwnmNJ89hmOSr7uwspUeUUvvudm IX7123sI4iWgmmCLW32tyC2y2kSbYrD9isW8n82XUBWnDTcG8lrlfjbiK3gZlLXR6vs8 +ubgMAJsLZoD7WGIlHnZ/oRNTmtgGxO0iVQN3EQ8t2y6f87c9W4IS6ggNSr7n9J2DOqM QoB/Em/ITAjx5mFjsHnjmj3FTtE22VCOq7R2y7Xwm3TA/IYGonBgVCvnnKQ3EIV3xNJM /qy1NlEbmVnJqoTINLxB0DIh/boBZwO7tLdSHFX/qTTHtQp4paKmPb41+L6MoESN5wd+ GjKg== X-Gm-Message-State: AOPr4FVy4NI5+szcjnP+5bdSDKyu7ZkPWKPoVUyj6YIMpIKkKBLClNTjJ5K/dhiFwAIzFw== X-Received: by 10.194.60.134 with SMTP id h6mr1474465wjr.115.1460133108006; Fri, 08 Apr 2016 09:31:48 -0700 (PDT) Received: from [10.189.188.175] (85-118-68-185.mtel.net. [85.118.68.185]) by smtp.gmail.com with ESMTPSA id i5sm14144587wja.23.2016.04.08.09.31.46 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 08 Apr 2016 09:31:47 -0700 (PDT) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (1.0) Subject: Re: svn commit: r297717 - head/sys/mips/mediatek From: Stanislav Galabov X-Mailer: iPhone Mail (12B440) In-Reply-To: <2014592.cEATiW0yl7@ralph.baldwin.cx> Date: Fri, 8 Apr 2016 19:31:45 +0300 Cc: Stanislav Galabov , "src-committers@freebsd.org" , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" Content-Transfer-Encoding: quoted-printable Message-Id: <0F1CDE8A-543D-4F3E-A595-95A107B271B9@gmail.com> References: <201604081526.u38FQnvC062652@repo.freebsd.org> <2014592.cEATiW0yl7@ralph.baldwin.cx> To: John Baldwin X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 08 Apr 2016 16:31:50 -0000 > On Apr 8, 2016, at 19:11, John Baldwin wrote: >=20 >> On Friday, April 08, 2016 03:26:49 PM Stanislav Galabov wrote: >> Author: sgalabov >> Date: Fri Apr 8 15:26:49 2016 >> New Revision: 297717 >> URL: https://svnweb.freebsd.org/changeset/base/297717 >>=20 >> Log: >> Introduce Mediatek/Ralink PCIe support >>=20 >> This revision introduces PCIe support for the relevant Mediatek/Ralink >> SoCs. >> Currently the PCIe support is not converted to INTRNG, this may be a >> task for the future. >>=20 >> Approved by: adrian (mentor) >> Sponsored by: Smartcom - Bulgaria AD >> Differential Revision: https://reviews.freebsd.org/D5886 >=20 > I would _really_ like to get away from having N different Foo-PCI bridge > drivers that all do an initial walk of the PCI hierarchy configuring > BARs and PCI-PCI bridge I/O windows. With NEW_PCIB, the PCI-PCI bridge > driver is capable of figuring out windows on its own, and the PCI bus is > also capable of allocating resource for BARs. Leveraging that means > removing N copies of this approach and having one place to manage BARs, et= c. > Can you please try removing all of your I/O window and BAR setup and seein= g > if the PCI bus works ok? If not, we should fix it so that it does, but > fixing the PCI bus means we only have to fix one place, not N. >=20 > (And yes, I want to axe the code from the various arm places that do this > as well and want to avoid adding even more copies of it.) >=20 > --=20 > John Baldwin No problem, I will try to look into this next week. Best wishes, Stanislav=