Re: [PATCH] qla1280: Reduce can_queue to 32

2016-04-22 Thread Laurence Oberman
artin K . Petersen" , "James E . J . Bottomley" , linux-s...@vger.kernel.org, linux-kernel@vger.kernel.org, "Laura Abbott" , "Michael Reed" , sta...@vger.kernel.org Sent: Friday, April 22, 2016 10:01:48 AM Subject: Re: [PATCH] qla1280: Reduce can_queue to 32 On Fri,

Re: [PATCH] qla1280: Reduce can_queue to 32

2016-04-22 Thread Johannes Thumshirn
On Fri, Apr 22, 2016 at 08:16:44AM -0400, Laurence Oberman wrote: > The change looks fine, I see its hard-coded to 32 in qla1280_set_defaults() > Would it be better to create a #define like other drivers and use that in > both. > Also did the below patch resolve this for the bug reporter. Yes, th

Re: [PATCH] qla1280: Reduce can_queue to 32

2016-04-22 Thread Laurence Oberman
The change looks fine, I see its hard-coded to 32 in qla1280_set_defaults() Would it be better to create a #define like other drivers and use that in both. Also did the below patch resolve this for the bug reporter. I ask because if I check 4.3 it was also set to the same value of 0xf and tha