Re: [PATCH 2/2] ath10k: remove 10.1 firmware support

[Ben Greear <greearb@candelatech.com>]

On 06/03/2016 09:09 AM, Valo, Kalle wrote:
> Rajkumar Manoharan <rmanohar@codeaurora.org> writes:
>
>> On 2016-06-03 21:18, Ben Greear wrote:
>>> On 06/03/2016 08:33 AM, Rajkumar Manoharan wrote:
>>>> Earlier qca9888 device was brought up using 10.1 firmware and then
>>>> later all firmware fixes and new features are migrated to 10.2/10.2.x
>>>> firmware branch. As all of 10.1 funtionalities are supported in 10.2
>>>> based firmware, removing 10.1 firmware support for qca9888 device.
>>>
>>> Oh please do not do this.  My 10.1 firmware works very nicely,
>>> and out-performs 10.2 in my testing.  Lots of people use my firmware
>>> when they need IBSS support or other features not found in
>>> official firmware, so it is not just me that will have
>>> problems if you remove this support.
>>>
>> Aah.. I thought CT firmware is 10.2 based. Since most of firmware bug
>> fixes and enhancements are integrated into 10.2 based firmware, we
>> thought of get rid of 10.1 firmware to reduce code size. Moreover
>> existing 10.1 official firmware has known issues.
>
> Yeah, we really want to reduce the amount firmware branches to support
> in ath10k.
>
>> Is it possible to upgrade CT firmware to 10.2 WMI/HTT interfaces?
>
> That's a good idea. That shouldn't be too hard, right? I don't think HTT needs
> any changes, but for WMI few commands and ids need to be changed.

I was thinking the 10.2 mgt-over-htt API would be trouble-some to make work on
10.1 firmware.  I have my own way to do mgt-over-htt in 10.1, but API
is different and I would need a small driver patch upstream
to make that work.  I'm not sure what other changes would be needed, but
if it is just conditionally compiling some WMI defines, that should be
doable.

Thanks,
Ben

-- 
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc  http://www.candelatech.com