[illumos-Developer] Webrev: New ARECA arcmsr driver
Garrett D'Amore
garrett at damore.org
Thu Mar 24 11:21:29 PDT 2011
On Thu, 2011-03-24 at 14:20 -0400, Albert Lee wrote:
> On Thu, Mar 24, 2011 at 1:45 PM, Garrett D'Amore <garrett at damore.org> wrote:
> > On Thu, 2011-03-24 at 13:41 -0400, Dan McDonald wrote:
> >> On Thu, Mar 24, 2011 at 09:57:55AM -0700, Garrett D'Amore wrote:
> >> > All of these changes are small, and hopefully low risk changes.
> >> >
> >> > The new webrev is here:
> >> >
> >> > http://mexico.purplecow.org/gdamore/webrev/arcmsr-delta/
> >>
>
> Last nit:
> 1005 + arcmsr_cdb->Context = (uintptr_t)arcmsr_cdb;
>
> Use the new PtrToNum?
>
> -Albert
Nah. I hate that macro, actually. :-)
- Garrett
More information about the Developer
mailing list