[Differential] [Updated] D2340: Support for Alpine platform from Annapurna Labs
jpa-semihalf.com (Jakub Palider)
phabric-noreply at FreeBSD.org
Tue Apr 28 07:10:38 UTC 2015
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
More information about the freebsd-arm
mailing list