Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 May 2015 15:20:02 +0000
From:      "emaste (Ed Maste)" <phabric-noreply@FreeBSD.org>
To:        freebsd-arm@freebsd.org
Subject:   [Differential] [Commented On] D2340: Support for Alpine platform from Annapurna Labs
Message-ID:  <f02e68e358342e2897e7a4c7493bcc08@localhost.localdomain>
In-Reply-To: <differential-rev-PHID-DREV-gclxe3zq7xamd5hpjagv-req@FreeBSD.org>
References:  <differential-rev-PHID-DREV-gclxe3zq7xamd5hpjagv-req@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
emaste added inline comments.

INLINE COMMENTS
  sys/arm/annapurna/alpine/hal/al_hal_iofic.h:1 For reference, the appropriate top-level vendor directory: https://svnweb.freebsd.org/base/vendor-sys/
  
  You'd import a verbatim copy of an upstream release, then merge it to sys/contrib and apply any changes necessary.  Let me know if you'll need help with the process.
  sys/arm/annapurna/alpine/hal/al_hal_iofic.h:1 True, but it's a lot more awkward to retrofit the vendor import if there is a new update in the future. If nothing else we should at least reference the hal drop in the commit message - e.g. something like
  ```
  The files in the hal/ subdirectory are an unmodified copy of 
  <annapurna-alpine-hal-20150317.txz> with sha1 hash
  f1d2d2f924e986ac86fdf7b36c94bcdf32beec15.
  ```

REVISION DETAIL
  https://reviews.freebsd.org/D2340

EMAIL PREFERENCES
  https://reviews.freebsd.org/settings/panel/emailpreferences/

To: jpa-semihalf.com, ian, andrew, imp
Cc: emaste, meloun-miracle-cz, onwahe-gmail-com, freebsd-arm



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?f02e68e358342e2897e7a4c7493bcc08>