[illumos-Developer] Webrev: New ARECA arcmsr driver

Albert Lee trisk at opensolaris.org
Thu Mar 24 11:20:37 PDT 2011


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



More information about the Developer mailing list