Re: [PATCH v3.18] iwlwifi: pcie: support loading FW with extended mem range

[Sasha Levin <sasha.levin@oracle.com>]

On 03/10/2016 12:02 PM, Grumbach, Emmanuel wrote:
> 
> On 03/10/2016 06:31 PM, Greg KH wrote:
>> > On Thu, Mar 10, 2016 at 10:46:22AM +0000, Grumbach, Emmanuel wrote:
>>> >> Hi Greg,
>>> >>
>>>>> >>>>
>>>>> >>>>
>>>>> >>>> On 02/07/2016 11:00 PM, Greg KH wrote:
>>>>>> >>>>> On Sun, Feb 07, 2016 at 10:51:08PM +0200, Emmanuel Grumbach wrote:
>>>>>>> >>>>>> From: Arik Nemtsov <arik@wizery.com>
>>>>>>> >>>>>>
>>>>>>> >>>>>> [commit fe45773b5baa154468416aac1304f6325939f775 upstream]
>>>>>>> >>>>>>
>>>>>>> >>>>>> Toggle the LMPM_CHICK register when writing chunks into the FW's
>>>>>>> >>>>>> extended SRAM. This tells the FW to put the chunk into a different
>>>> >>> memory space.
>>>>>>> >>>>>>
>>>>>>> >>>>>> Signed-off-by: Arik Nemtsov <arikx.nemtsov@intel.com>
>>>>>>> >>>>>> Signed-off-by: Emmanuel Grumbach
>>>> >>> <emmanuel.grumbach@intel.com>
>>>>>>> >>>>>> ---
>>>>>>> >>>>>> backporting this patch to 3.18 fixes
>>>>>>> >>>>>> https://bugzilla.kernel.org/show_bug.cgi?id=109531
>>>>>>> >>>>>> ---
>>>>>>> >>>>>>  drivers/net/wireless/iwlwifi/iwl-prph.h   |  6 ++++++
>>>>>>> >>>>>>  drivers/net/wireless/iwlwifi/pcie/trans.c | 26
>>>>>>> >>>>>> ++++++++++++++++++++++----
>>>>>>> >>>>>>  2 files changed, 28 insertions(+), 4 deletions(-)
>>>>>> >>>>> What stable tree(s) do you want this to be applied to?
>>>>> >>>>
>>>>> >>>> That was fast :) 3.18 as said in the title of the patch. Do you have
>>>>> >>>> any preference on how to indicate that?
>>>> >>>
>>>> >>> Ugh, no, you did it right, I just didn't read the subject: my fault, sorry for the
>>>> >>> noise...
>>> >>
>>> >> Any reason this was not picked up for v3.18.28?
>>> >> Do I miss anything?
>> > 
>> > Why are you asking me about 3.18-stable?
>> > 
>> > confused,
>> > 
> Ouch - somehow, I was convinced that you maintain all the stable
> kernels, at least all those being listed in kernel.org. Sorry...

Sorry about that, my scripts skipped it because it didn't have a cc: stable@ tag. Fixed that and added it to the queue.


Thanks,
Sasha