Re: [PATCH] linux-firmware: Add qmss accumulator pdsp firmware for keystone SoCs

[Murali Karicheri <m-karicheri2@ti.com>]

On 09/18/2015 11:37 AM, santosh shilimkar wrote:
> (Bouncing the thread on Murali's request)
> 9/15/2015 11:54 AM, Murali Karicheri wrote:
>> This patch adds firmware for Keystone QMSS Accumulator PDSP. This is
>> required
>> to support Accumulator queues. Accumulator queues are one of the queue
>> types
>> supported in drivers/soc/ti/knav_qmss_acc.c. This queue can be part of a
>> channel that supports one queue or multiple queue per channel and are
>> managed
>> by the Accumulator PDSP. For more details on hardware, please refer
>> http://www.ti.com/lit/ug/sprugr9h/sprugr9h.pdf and DT documentation
>> below in
>> linux kernel source tree
>>
>> Documentation/devicetree/bindings/soc/ti/keystone-navigator-qmss.txt
>>
>> Signed-off-by: Murali Karicheri <m-karicheri2@ti.com>
>> ---
>>   LICENCE.ti-keystone                               |  61
>> ++++++++++++++++++++++
>>   WHENCE                                            |   5 ++
>>   ti-keystone/ks2_qmss_pdsp_acc48_k2_le_1_0_0_9.bin | Bin 0 -> 1764 bytes
>>   3 files changed, 66 insertions(+)
>>   create mode 100644 LICENCE.ti-keystone
>>   create mode 100644 ti-keystone/ks2_qmss_pdsp_acc48_k2_le_1_0_0_9.bin
>>
>> diff --git a/LICENCE.ti-keystone b/LICENCE.ti-keystone
>> new file mode 100644
>> index 0000000..62cc3b3
>> --- /dev/null
>> +++ b/LICENCE.ti-keystone
>> @@ -0,0 +1,61 @@
>> +Copyright (c) 2015 Texas Instruments Incorporated
>> +
>> +All rights reserved not granted herein.
>> +
>> +Limited License.
>> +
>> +Texas Instruments Incorporated grants a world-wide, royalty-free,
>> non-exclusive
>> +license under copyrights and patents it now or hereafter owns or
>> controls to
>> +make, have made, use, import, offer to sell and sell ("Utilize") this
>> software
>> +subject to the terms herein. With respect to the foregoing patent
>> license, such
>> +license is granted solely to the extent that any such patent is
>> necessary to
>> +Utilize the software alone. The patent license shall not apply to any
>> +combinations which include this software, other than combinations
>> with devices
>> +manufactured by or for TI (“TI Devices”). No hardware patent is licensed
>> +hereunder.
>> +
>> +Redistributions must preserve existing copyright notices and
>> reproduce this
>> +license (including the above copyright notice and the disclaimer and
>> +(if applicable) source code license limitations below) in the
>> documentation
>> +and/or other materials provided with the distribution
>> +
>> +Redistribution and use in binary form, without modification, are
>> permitted
>> +provided that the following conditions are met:
>> +
>> +    * No reverse engineering, decompilation, or disassembly of this
>> +      software is permitted with respect to any software provided in
>> binary
>> +      form.
>> +
>> +    * any redistribution and use are licensed by TI for use only with TI
>> +      Devices.
>> +
>> +    * Nothing shall obligate TI to provide you with source code for the
>> +      software licensed and provided to you in object code.
>> +
>> +If software source code is provided to you, modification and
>> redistribution of
>> +the source code are permitted provided that the following conditions
>> are met:
>> +
>> +    * any redistribution and use of the source code, including any
>> +      resulting derivative works, are licensed by TI for use only
>> with TI
>> +      Devices.
>> +
>> +    * any redistribution and use of any object code compiled from the
>> +      source code and any resulting derivative works, are licensed by TI
>> +      for use only with TI Devices.
>> +
>> +Neither the name of Texas Instruments Incorporated nor the names of its
>> +suppliers may be used to endorse or promote products derived from this
>> +software without specific prior written permission.
>> +
>> +DISCLAIMER.
>> +
>> +THIS SOFTWARE IS PROVIDED BY TI AND TI’S LICENSORS "AS IS" AND ANY
>> EXPRESS OR
>> +IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
>> WARRANTIES OF
>> +MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
>> IN NO
>> +EVENT SHALL TI AND TI’S LICENSORS BE LIABLE FOR ANY DIRECT, INDIRECT,
>> +INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
>> BUT NOT
>> +LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
>> DATA, OR
>> +PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
>> +LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
>> NEGLIGENCE
>> +OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE,
>> EVEN IF
>> +ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
>> diff --git a/WHENCE b/WHENCE
>> index c1d5982..fbbb1e4 100644
>> --- a/WHENCE
>> +++ b/WHENCE
>> @@ -2947,3 +2947,8 @@ File: hfi1_sbus.fw
>>
>>   Licence: Redistributable. See LICENCE.hfi1_firmware for details
>>
>> +Driver: knav_qmss_queue - TI Keystone 2 QMSS driver
>> +
>> +File: ti-keystone/ks2_qmss_pdsp_acc48_k2_le_1_0_0_9.bin
>> +
>> +Licence: See LICENCE.ti-keystone for details.
>> diff --git a/ti-keystone/ks2_qmss_pdsp_acc48_k2_le_1_0_0_9.bin
>> b/ti-keystone/ks2_qmss_pdsp_acc48_k2_le_1_0_0_9.bin
>> new file mode 100644
>> index
>> 0000000000000000000000000000000000000000..0232b1d0b128087895dbcfe1ca24fa73563e5ce6
>>
>> GIT binary patch
>> literal 1764
>> zcmZWpL2nyX5T4n6UhfjCSqD{3gd2PD;zTBD1A<Bx67&G(VhL6ps@&)Sx0Vd02u>VW
>> zhg2lEwvx0|MMAzIm4D%uAk~VzN&pp3(Y!<;HBuX=p$RG|Vcy!Y+Hx4ZnfK<+d^7Kx
>> z=YS4AP67g0E&x^FdEbp6ql)0w*F*cnL3P@NgDA@w%iApHTsWCo7|!~x06x=hc}UN>
>> zZ}xPJhTWN-uG3L>sizAX>iX-qya)|;{aH6C*7avSN><mOb(0Kt4SaWBHeDE&zRda#
>> zOaS97x~GA7l@USYQ?_yeXy2WLitFP|_6_%t&s?x7F2YUaa~$&dR`NhkCJzphtRrSJ
>> zrOAxuLMn3wva_w|_<JRPeE967sk@+`ci<|VO_|Y1%A^&V;XM)VC1_^G0S^Lm<KKk2
>> zv0<C0VF`mQ&5$68@HEjPW#{5w<ntcXv^0L&Z~?=+rOJ3lVSk^h9M9<Hcr2fbr~P=W
>> z&>Ya3O=K~f%gvAdCi_O8z;ipq6AAZ5Lp-0s9mX@tc(^ZRmKsZY6$^R`3R)7&WlNZh
>> z(d=WKW{iDY)8qa3SN-AlpF4d2zR#EA!oGWVsc+AO_8*5f^WZ$^9ItRMJy_zpYYh)B
>> z@%lp}jc;)ez=#hPsWeEsvK{!2`)i0u${?=i6$ieD+s4Mrs&7yAb9-?+JGZKpUqr0Y
>> zyUUash*Y^2qIr6uvKlEwZ&&4@U9HyBu-54L9CP3%*mvdD7;e?QR6UxgZ*4)f-uL-v
>> znI1XNhE_Chw6G9a<vy)qS+_4=ZmWK^uzbXWwrD{<7QuGA1np<|JuL)7o=r){E#*Qi
>> z&7AxN7R<E2B8h8t8`F&Souu)Bmox;TVVs*Zesth9urnE@H$#1Wy~BJVFo-M`piYNz
>> z2O?Mwm|3qg2f!1M@Ju*4B7(pYb;6jcH?!!$gHsnVh_Qa91Ut-M3@&Pc$@bL}{Kd8w
>> zT+{=>_BSPX$hIC>T1~b@K|>*>r3G+bt_(p?t0klP{239fJD7$Mt(V7Cd5WB8Ct=s?
>> zKYOTR6*;zwg#qvVYU#arCdbpCipO!nc-W0MPk(Z(hgNt`^zPYGlI$#U%^?N{HXzji
>> zu-DgjpX|%-?toUHxXyE>k*Y!1wHiq{<vaK)oH+Pukud?Ynx)pf-@G$qZ*K0jpsbs6
>> znfto8r}$t>y6(<T{{C#I2x9=M=lq}CxR%t)<xwgYF+n)t8>lcaM@PnRB<H1a6N%i&
>> zh-KusA2+W3Hv0EJ7vTCM6gjyfi`J+t79(Pp|De5HE@zMbhcJiFz7FOg!RLvbik-uW
>> z6i!)Uia9%VIDa9hk8}Q`%r!Ny$9d&>Wxf^q98_FFnBsp7nR670q=-zCvLezIl_S)l
>> ztB3t-G}ivdPy9UO=N(Cpc!pxV8MM?4D*enL?LRed;s2$$eCVV8S9@)(p>kMa>2WR&
>> z$2&tqJMhVDym_8HAl>unh=Vs^b#RmU0{2+9TSqpvWO92u{zf)yiLVKnV?GrZ&cMBU
>> zJMiJGgWp2`jCt-e&mF9VoUp{D$MePP2M@O6op}5qycFgv0f|;}Q+N5FmXMma#808W
>> IilOlS2SY<;Z2$lO
>>
>> literal 0
>> HcmV?d00001
>>
>
>

Thanks Santosh!

Dear firmware maintainer,

Could you please review this patch and comment if it requires any 
change? If looks good, let me know when this will be queued for merge to 
the mainline linux-firmware.git repo.

Thanks

-- 
Murali Karicheri
Linux Kernel, Keystone