[illumos-Developer] Add support for Qlogic QLE2360 (device id 0x2322)

[Garrett D'Amore]

  Accepted, once I verify that 132 is indeed a subsystem id for an 
already covered device id.

     - Garrett

On 02/17/11 05:28 AM, ken mays wrote:
> Garrett,
>
> Possible change update per Rev. 735?
> Ref: http://mexico.purplecow.org/gdamore/webrev/qlc2322/
>
> - 47 alias=pci1077,132 \
> - 48 alias=pci1077,117 \
>
> ~ Ken Mays
>
>
> --- On Thu, 2/17/11, Cyril Plisko<cyril.plisko at mountall.com>  wrote:
>
>> From: Cyril Plisko<cyril.plisko at mountall.com>
>> Subject: Re: [illumos-Developer] Add support for Qlogic QLE2360 (device id 0x2322)
>> To: "Garrett D'Amore"<garrett at nexenta.com>
>> Cc: "developer at lists.illumos.org"<developer at lists.illumos.org>
>> Date: Thursday, February 17, 2011, 5:14 AM
>> On Thu, Feb 17, 2011 at 10:54 AM,
>> Garrett D'Amore<garrett at nexenta.com>
>> wrote:
>>>   We have added support for this device to our latest
>> internal builds of NexentaStor -- it turns out that we have
>> quite a few of these cards around the lab, and enabling
>> support is beneficial *and* easy.
>>> http://mexico.purplecow.org/gdamore/webrev/qlc2322/
>>>
>>> Note that the work on this webrev was performed by
>> Dmitry Yusopov at Nexenta, although I made some minor
>> changes to make it fit within illumos (e.g. IPS packaging)
>> and I have of course already done a reasonable technical
>> review.
>>> Still, I'd like to offer this up for other feedback.
>>   Unless someone notices something grossly wrong here, I'll
>> probably be integrating this modest change in the next
>> couple of days.
>>
>>
>>
>> usr/src/pkg/manifests/driver-network-qlc.mf
>>
>>        48 +    alias=pci1077,117 \
>>
>> That is subsystem ids, right ?I wonder why did you need
>> these ?
>> I believe that this one
>>
>>         52 +   
>> alias=pci1077,2322 \
>>
>> should provide adequate coverage.
>>
>>
>>
>> --
>> Regards,
>>          Cyril
>>
>> _______________________________________________
>> Developer mailing list
>> Developer at lists.illumos.org
>> http://lists.illumos.org/m/listinfo/developer
>>
>
>
>
> _______________________________________________
> Developer mailing list
> Developer at lists.illumos.org
> http://lists.illumos.org/m/listinfo/developer