From owner-freebsd-ppc@FreeBSD.ORG Fri Oct 25 16:18:38 2013 Return-Path: Delivered-To: freebsd-ppc@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id BAE80703 for ; Fri, 25 Oct 2013 16:18:38 +0000 (UTC) (envelope-from nwhitehorn@freebsd.org) Received: from smtpauth3.wiscmail.wisc.edu (wmauth3.doit.wisc.edu [144.92.197.226]) (using TLSv1 with cipher RC4-MD5 (128/128 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 8BB732155 for ; Fri, 25 Oct 2013 16:18:38 +0000 (UTC) MIME-version: 1.0 Content-transfer-encoding: 7BIT Content-type: text/plain; CHARSET=US-ASCII; format=flowed Received: from avs-daemon.smtpauth3.wiscmail.wisc.edu by smtpauth3.wiscmail.wisc.edu (Oracle Communications Messaging Server 7u4-27.01(7.0.4.27.0) 64bit (built Aug 30 2012)) id <0MV800K00F9PDG00@smtpauth3.wiscmail.wisc.edu> for freebsd-ppc@freebsd.org; Fri, 25 Oct 2013 11:18:37 -0500 (CDT) X-Spam-PmxInfo: Server=avs-3, Version=6.0.3.2322014, Antispam-Engine: 2.7.2.2107409, Antispam-Data: 2013.10.25.161214, SenderIP=0.0.0.0 X-Spam-Report: AuthenticatedSender=yes, SenderIP=0.0.0.0 Received: from comporellon.tachypleus.net (adsl-76-208-69-44.dsl.mdsnwi.sbcglobal.net [76.208.69.44]) by smtpauth3.wiscmail.wisc.edu (Oracle Communications Messaging Server 7u4-27.01(7.0.4.27.0) 64bit (built Aug 30 2012)) with ESMTPSA id <0MV800JJ4FZ0PL00@smtpauth3.wiscmail.wisc.edu>; Fri, 25 Oct 2013 11:18:37 -0500 (CDT) Message-id: <526A99DC.9040406@freebsd.org> Date: Fri, 25 Oct 2013 11:18:36 -0500 From: Nathan Whitehorn User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:24.0) Gecko/20100101 Thunderbird/24.0 To: John Clark Subject: Re: Updates to MPC85XX PCI driver References: <52669D8A.8020302@freebsd.org> In-reply-to: Cc: FreeBSD PowerPC ML X-BeenThere: freebsd-ppc@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Porting FreeBSD to the PowerPC List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Oct 2013 16:18:38 -0000 On 10/22/13 11:27, John Clark wrote: > On Oct 22, 2013, at 8:45 AM, Nathan Whitehorn wrote: > >> I've been going through our tree trying to remove code duplication between AIM and Book-E, as people attentive to the commit logs might have seen. At the same time, I'm also trying to deduplicate the FDT and OFW code, since they are implementing many of the same interfaces and specifications. As part of that, I've adapted the MPC85XX PCI driver to use the generic ofw_pci code, which gives a quite substantial reduction in the driver and also substantially improves its robustness to variations on the device tree setup: >> http://people.freebsd.org/~nwhitehorn/pci_mpc85xx.diff >> >> I can currently only test this patch in QEMU, where it works fine, and there should be no functional changes at all, but tests on real hardware would be appreciated. If I don't hear anything, I'll commit this at the end of the week. >> -Nathan > I had quite a bit of problem with PCI and the P1020WLAN eval board, which is based on the MPC85XX processor 'class'. Unfortunately right at the moment I've had to return to updated my OpenWRT linux environment, so perhaps next week I can 'test' what you have on my hardware. > > John Clark. > > > > I've gone ahead and committed this patch, but have my finger squarely on the revert button in case there is any fallout in more complex situations than I was able to test. Please tell me if so as soon as possible. -Nathan