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

Garrett D'Amore garrett at damore.org
Thu Feb 17 23:13:18 PST 2011


  Ok, I've updated the webrev at the same URL:

http://mexico.purplecow.org/gdamore/webrev/qlc2322/

Feedback graciously sought.

     - Garrett

On 02/17/11 08:44 AM, Garrett D'Amore wrote:
>  I'll be resending.  It turns out that both ,132 and ,117 are not 
> needed, as they are subsystem ids.  Which means that we can have a 
> single list for both sparc and x86.
>
> Furthermore, when I was perusing the code, I found that there were 
> specific excludes for certain devices or parts on SPARC, on the basis 
> of a "marketing request" to not support them on SPARC.  (In particular 
> the less expensive devices were excluded.)  This was an arbitrary 
> business decision, and not based on technical merit, so I've gone 
> ahead and removed the special code to prevent those devices from 
> attaching on SPARC.
>
> A new webrev will be coming shortly.
>
>     - Garrett
>
> On 02/17/11 08:37 AM, Richard Lowe wrote:
>> Driver action PCI-IDs become unsorted with your changes (the ,117 id).
>> You'll need to fix this to appease the format check, I expect.
>>
>> Why is the ,117 id not listed on sparc? (and there's no code change 
>> for it?)
>>
>> -- Rich
>>
>> _______________________________________________
>> 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






More information about the Developer mailing list