From owner-freebsd-arm@FreeBSD.ORG Tue Apr 28 07:10:38 2015 Return-Path: Delivered-To: freebsd-arm@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 78FA91C8 for ; Tue, 28 Apr 2015 07:10:38 +0000 (UTC) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [IPv6:2001:4f8:3:ffe0:406a:0:50:2]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 410F91FD5 for ; Tue, 28 Apr 2015 07:10:38 +0000 (UTC) Received: from phabric-backend.isc.freebsd.org (phabric-backend.isc.freebsd.org [127.0.1.5]) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9) with ESMTP id t3S7Ac0l041629 for ; Tue, 28 Apr 2015 07:10:38 GMT (envelope-from daemon-user@phabric-backend.isc.freebsd.org) Received: (from daemon-user@localhost) by phabric-backend.isc.freebsd.org (8.14.9/8.14.9/Submit) id t3S7AcIY041628; Tue, 28 Apr 2015 07:10:38 GMT (envelope-from daemon-user) Date: Tue, 28 Apr 2015 07:10:38 +0000 To: freebsd-arm@freebsd.org From: "jpa-semihalf.com (Jakub Palider)" Subject: [Differential] [Updated] D2340: Support for Alpine platform from Annapurna Labs Message-ID: <689f8a64e3aefbf3c116977ae72cdcad@localhost.localdomain> X-Priority: 3 Thread-Topic: D2340: Support for Alpine platform from Annapurna Labs X-Herald-Rules: none X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-To: X-Phabricator-Cc: X-Phabricator-Cc: X-Phabricator-Cc: In-Reply-To: References: Thread-Index: MzFmODEzYTliMzhlZDlhOTEyM2NlYTg2YmJiIFU/Mm4= Precedence: bulk X-Phabricator-Sent-This-Message: Yes X-Mail-Transport-Agent: MetaMTA X-Auto-Response-Suppress: All X-Phabricator-Mail-Tags: , MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="utf-8" X-BeenThere: freebsd-arm@freebsd.org X-Mailman-Version: 2.1.20 List-Id: "Porting FreeBSD to ARM processors." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 28 Apr 2015 07:10:38 -0000 jpa-semihalf.com marked 28 inline comments as done. INLINE COMMENTS sys/arm/annapurna/alpine/alpine_machdep.c:144 Frankly speaking, this is how it looks on other platforms, and this parts uses existing code base. sys/arm/annapurna/alpine/alpine_machdep.c:169 I am not sure if most, but some of them, definitely yes. sys/arm/annapurna/alpine/alpine_machdep_mp.c:143 In general that would be the way to go, but other errors are handled with same panic message anyway, so I would rather not duplicate code unless necessary. sys/arm/annapurna/alpine/alpine_machdep_mp.c:172 Same as above. sys/arm/annapurna/alpine/alpine_pci.c:1476 I believe it has the same meaning as one found in ofw_pcib_pci_route_interrupt() sys/arm/annapurna/alpine/hal/al_hal_iofic.h:1 Yes, the HAL will change. It was put together with platform code to make some context but if you think it makes more sense to do that separately, please, let me know and I will go that way. sys/arm/annapurna/alpine/std.alpine:12 Yes, kernel.bin is built. sys/arm/annapurna/alpine/std.alpine:15 I think it does not relate to Alpine. sys/arm/conf/ALPINE:110 I think so. sys/boot/fdt/dts/arm/annapurna-alpine.dts:2 FreeBSD and Linux dts are not compatible, so the differences are limited to the minimum. REVISION DETAIL https://reviews.freebsd.org/D2340 EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: jpa-semihalf.com, ian, imp, andrew Cc: meloun-miracle-cz, onwahe-gmail-com, freebsd-arm