[Freedreno] [PATCH] qcom: Add firmware files for Adreno A200

Fri Sep 6 15:09:37 UTC 2019

Hi All,

I'll start by saying that I'm no expert on this subject!  My
understanding with this Adreno 2xx firmware though is that these two
firmware files are generally equivalent to what was distributed
previously in an include file in the following two files for the NXP
i.MX51 and i.MX53 which utilize the Adreno 200:

https://source.codeaurora.org/external/imx/linux-imx/tree/drivers/mxc/amd-gpu/common/pm4_microcode.inl?h=imx_2.6.35_1.1.0_caf&id=61ade304d0eddc3f44fb2ee0b2b7ccf3ffcf85d5
https://source.codeaurora.org/external/imx/linux-imx/tree/drivers/mxc/amd-gpu/common/pfp_microcode_nrt.inl?h=imx_2.6.35_1.1.0_caf&id=61ade304d0eddc3f44fb2ee0b2b7ccf3ffcf85d5

I'm not suggesting we don't get someone from qcom to provide a
Signed-off-by.  I'm only pointing out that it looks like this firmware
was previously made available and that it did have a license that
perhaps we can continue to use for this firmware that's also for the
Adreno 200?

Regards,

Chris

On Wed, Sep 4, 2019 at 8:35 AM Josh Boyer <jwboyer at kernel.org> wrote:
>
> On Wed, Sep 4, 2019 at 8:09 AM Nicolas Dechesne
> <nicolas.dechesne at linaro.org> wrote:
> >
> > hi Fabio,
> >
> > On Wed, Sep 4, 2019 at 1:27 PM Fabio Estevam <festevam at gmail.com> wrote:
> > >
> > > Add firmware files for Adreno A200.
> > >
> > > These firmware files are needed for running the adreno kernel
> > > driver on i.MX51 and i.MX53 SoCs.
> > >
> > > Signed-off-by: Fabio Estevam <festevam at gmail.com>
> > > ---
> > >  WHENCE             |   2 ++
> > >  qcom/yamato_pfp.fw | Bin 0 -> 1156 bytes
> > >  qcom/yamato_pm4.fw | Bin 0 -> 9220 bytes
> > >  3 files changed, 2 insertions(+)
> > >  create mode 100644 qcom/yamato_pfp.fw
> > >  create mode 100644 qcom/yamato_pm4.fw
> > >
> > > diff --git a/WHENCE b/WHENCE
> > > index a8ec628..5e46d33 100644
> > > --- a/WHENCE
> > > +++ b/WHENCE
> > > @@ -4418,6 +4418,8 @@ File: qcom/a530_zap.b00
> > >  File: qcom/a530_zap.b01
> > >  File: qcom/a530_zap.b02
> > >  File: qcom/a530_zap.mdt
> > > +File: qcom/yamato_pfp.fw
> > > +File: qcom/yamato_pm4.fw
> >
> > Where did you get these files from in the first place? On which
> > devices do you expect these firmware files to work? We were recently
> > discussing how to structure /lib/firmware/qcom more carefully. And the
> > general consensus that we reached with Rob C. and Bjorn is that if a
> > firmware is expected to work on a specific device (maybe it is signed)
> > it should be placed in /lib/firmware/qcom/<device> and if it's
> > unsigned and can be used on several devices based on the same SoC (dev
> > boards, ..) then it should be /lib/firmware/qcom/<SOC>.
> >
> > >
> > >  Licence: Redistributable. See LICENSE.qcom and qcom/NOTICE.txt for details
> >
> > Are they really distributed under these license terms? That's a
> > specific license that we came up with with Qualcomm a couple of years
> > ago based on their Dragonboard offers.
>
> So everyone understands, I'm not pulling this for now.  If there are
> questions on provenance of the firmware and the license it's under,
> that's a big red flag for me.  Ideally we'd see someone from qcom add
> a Signed-off-by or otherwise acknowledge these are meant to be
> redistributable.
>
> josh
>
> > > diff --git a/qcom/yamato_pfp.fw b/qcom/yamato_pfp.fw
> > > new file mode 100644
> > > index 0000000000000000000000000000000000000000..61a576eb997ce2ee0d32b135adc5d8c789d8e118
> > > GIT binary patch
> > > literal 1156
> > > zcmaKr%WG3X7{!10-ZY7A3K1$6=_76G1N!h4AK=1^>7q~<E%;a!K{qZG{{WkEu`Vig
> > > znc9aU-RVwT2qFk+Q_ at myY|Wy$6#N(5N<A}4Y7An+ at Xef=?>pywGZ*xWlo-D at qsc1s
> > > zPiBZtq;Lo<?)jnc0FONz87grhu?2;tYGjUXtrI+jc!`*Bk|n%g5tAFK<$08Cey`T`
> > > zvx)jy!_iTK&JzB^96f&7=Jy2U|K?k+gt(ON1sa9FwHhv?DgHb|R9rnZ-6s0-6jM&z
> > > z<oUHc%}p<2ua40-x6P;G0z=6yT7xAz^XG{8D<toSI0F%~&MtQ~T=}{_nNy9j>b=By
> > > z8`b=UTDGpZL%olqwSLyuMi<gPtom*~LmSJCbZV|Tu6+Mc?V^V9)FoZr!s at x6yK8%C
> > > z_Iu;H3)M*(Z_;Vjwjatfch;}iOVyu=r at M4c9_##P#7|+_;(b9#$`3g#E&H)G(&EW#
> > > zxHmG)zn>;t$l(^GuMl!ic#FAX%}uRm#-DUZ1hd+EFuQ1Z%pKHqw^N~=sumk-JviU7
> > > z&1%0qeGi$+F4LlD>Nu(BfzF~^Ik at AK4f|)~+w9LSVPYeflWINSJtW>%jmAE-X`N`o
> > > z($Z#o#}~_Q`saDO4AgL=$zW+=$D6RMuoF#~Id3&ZPDx9}ei!DQJAO!1<Cne}H@=}a
> > > zw84=e&!8{|-Sa>8&3oD$%MmN&SySBjIZa%1lT+oXSj=m{xNPnN27?{z{g3~>S}*2o
> > > z+!Sjl6<_79MgKjLeBh|wK|hyBla<>kPV$Ih9y6j|j?+bkAujSpCsw$^Yp(K*QPvsb
> > > aBNKdPk}Yn^%`G{<%{x^na)+4oG>_jjN)4X?
> > >
> > > literal 0
> > > HcmV?d00001
> > >
> > > diff --git a/qcom/yamato_pm4.fw b/qcom/yamato_pm4.fw
> > > new file mode 100644
> > > index 0000000000000000000000000000000000000000..b45ea20ba5773ef6da25e21abd29914ea9162c07
> > > GIT binary patch
> > > literal 9220
> > > zcmeHMU2I%O6+Y{%UFT-AU2k1ujB&@i>-9SF&jM9cEJ3bFYHh4aq;GvR>H~i&R3H_K
> > > z5Otf75}?qOh&)jG$6sG~=>rdaC at Ulc0tr>TAbx8k3J4 at 36+}TQTE1^)&b at QTyG{^A
> > > zykxAs_uMnzIdkUBnKM5Y5qZ}cBT_bfkhUc9lzsoTBr<i%SQ?MgR4=3PnMxVY!9Zkj
> > > zs{_iHp8Q-%V)C+gYfkUTJ!4MRB%QwUAG&=-Ns*Cy0@<xx_8|NPv}H at h_3^buk=3od
> > > ztwX(3!185BGQ8wIk&&{-$_-A4Y>aKF61?{+{P@??#kOKQ^LA6hMQS3=y7wpO))1Zl
> > > z4D7E$WD54PTsoAGR|((X_sMs~^(&N}M1 at Hf`yPPT-t-!tH{ho_iBb9%&={{2FL at Jw
> > > zjmWl>;L+7Fe;YiQwvT@&c#NSKe;0VuBltHD;qQPK_6wgf=KKEN*i1c)VcKc+RGqKt
> > > z6S<#>I26u{UJx^$9%38CWC1_pb;|!h=lJWSGxTBThvaqD|Gqxw-6&rn&pfEP$i=py
> > > z3uOR<G$!%6YpO0jL8qp!`JpV0t at q1UeIBSy8&tc}EXHthk5<g4{d+-M{d2jVcBp68
> > > zC_0{}$w#$jKJDM9dKPj$%b}k6T+djZZ{_<Y&#Qmzowgd05;$j5ehlzyBn~scq}9H&
> > > zo6Us=Au^7isp}k{l$OseH-X<8%d?M2fBGzV at blB^XMAGr8@}AUp1IA^=kZ*cx!2}!
> > > zGRFOSL3(Y*@Wp8(&FUGs2$JKK at q0O+IU(btdy1$1%AXAQ^?)yi_%1lEoISc?3D|!r
> > > zwEyLv<FoSYshpNslUe~j{8}^2C#)W&6Z|iQZB6(*Oc at ckPFZr&^zECEX-9J&EzO1g
> > > z-($)Xj^k3*jVZ@^sqQdn>#Vj#|FryAe2bWw+z;VcG>#k8qNMQE_8EB&B*!A-b1Xcc
> > > zW0CRcbI)g6%C}?T`5X)7gB10 at 9hYrq4{ScCHWMRZZp601^6Vp(C-4S-=p6s}cp3YU
> > > zooxQ1L;XzFugmtkw%^Z|KkNAqpiw_x<aXF!ocopUsnYpCkH{Oy=2$enEa_PGN8pXJ
> > > zpQ~IenC~2r9|P~oj%;#JP+s)&9U+jlWyh;%XYx~v7I{_*`;GFSJ)int0d2=FlKt6m
> > > z*|AvG#zQ3gtKqVhSa!&op8R&WY!b^dUq!x6em`7xB9@&PBCD9lSCh(DU&+C-k<f%m
> > > z{F+G+w%9M#{MgB<AZB#mbrqejQmHM0Y!KTw+>uHalUIvv>KPKZ*BX-=dsGwo%I;&y
> > > zr#-KcVEO;q!(q0C_DsV6N5L7EXR7C8{{FK`=%2&Ts_We-oou)k=J_zU`!m!4<KCEa
> > > zIs8W88x0pTXiGW`$ETi7G`||?3&Uy37*=E97HJnT<z)g|`}h-<kE-);bZcDLFRQZ$
> > > zWww;1u!pHAUv?Yq+_sfsyOp<42}-G!=c_~QeBC+h^~8QFW$otsWU6Y<Ia%h4#R*4~
> > > z-)X!JPgi39XdgLeS;9W at TgMLkuZQuV{6C?AkY5gL!~SP3MLlGnP|TGVzWp1OrGA``
> > > z=1w|HLI087uC0yE%X?<K*S8h!G2&djBX_Xlwxs at zSH<N7*z<X{hn0HP7$C6poX8k!
> > > z`L&*TyH$7lG&w`s`L-Hhp!4BSr}}|rwCc|Pz?>h3c{4Cgzb7%EFPa9P?tItr?Lzi5
> > > zDm!>Lbo%{34Y=maqQ%qxr5P7*wl99Zu=1PpnGIi5J?z)=w(FPGqx&N0(LL=^VAKz=
> > > z;d3fWo!ZAy{$Y{}WIsJjmU at 1e+xDnpLfb^Tvc7qNP+O)}jeXO%8}D>uVHrW*h}OHx
> > > z{aHNh&sSDfvOAD$zr?jZjMZ*&{wAMPoxa_}z&E1(|2f4#|D#lhGUey>4CtnhG4f;C
> > > zhcj6hxBQGTpN%kVBF0<-hB?6;cY5sk#nTf3pL-<5Oa?UjrF<^hN*@Vmj#u37<;*rW
> > > zAKe~i5lAil=_i3fTzyT)-PZI>%C~K++US#>!cTqmH>glNF{|6}vA%WtoTC0`px+hO
> > > zclI~7^I2!Z<Erx(V9fSnN;%l}W##i*_uKSUtmiqaXK-SDGt2pXJwH<PL$|H)#WJh>
> > > zFb2?8AqUE;vzY=%dWJ8%Uy9p(LHp?Q+fBf!uXwkkG-Jcm8!2A4oUOUJ!$XqJQ;uVH
> > > ze}ZMX>Ca7r!Qe{JudpwWNb%FM|BOa|HmZvXz}uh8&{)IkpC!iUAI6F1SPDH-Gkn1?
> > > z@|_RitL*H?s&l6^e&Kwixt$o4z=%}(@Ei0zq~nXv!h9D$uXzut<HbCgpZysF8`SH|
> > > zUh-=yv1RYrA3*N{zP{k%0kkX^@s(eAd}ThBUxKFowFQHxySG^m?o at eaNSzdk+nwj&
> > > z+OpSi?e5A!?fx!k_vHiHB}{W5e3j)!HWl0O7l0W*if8`UI|r^KYjXxq`uwHA`esjF
> > > z0iI*6c&?|4#}X9auK=ITDX9f_JMWFoYwUYyC+EBBk7H)s&nfUPXbb=@6FL8yVyRYh
> > > zF>lv9e at 8?;v43uKV`SDhU97&kSiHynfxlM??PpF?`>DU{*9+z!<<qyyhwR0$FH!t2
> > > z0ZRWoQpBXNl{JVNZ`<GhL)$oZYC~un$5Huc_pxHT2DMpfk^<%nMU1lz{A|86#tQqf
> > > z5qBd|-_awhKb!CT)_ZvB)H}~TBs<LaKI_qL7V`+L%;%)z at m+-vt>5ZyeBB+zX;}x~
> > > zj&*^r7`Bq%!$dkOw?cMETb57B?o3ZMf#H6y>-T;(PW(L>`R!mI7 at d6*%DR&F`5rr3
> > > zycde>NlpOI9MxU0?sV<8d(s<jXL3?)^Er?gC{wM};@Mu;^R3Rd(`o789xt-F@^<Lp
> > > zcrE$+I at 0Yqw~r;gypYo?ZAWw8zT(eT at p($ubckkipZKLlmn at XBbo^Zt$D)(*;fszT
> > > z0ObJ8q_KztG5-EMwoPL=K6fWQ)ye<M<h15AHrOd`@a4;1ayR_Oel2_Z87F$*>wM_W
> > > z2o#8Xp4>@UTT=Us&ShoIMT+ at AhTrM)|B}jA3TMOWhco`JWrzQ#2=g8*E8frC2d=dZ
> > > zS+=Z3eZ0@>oa(0XG16FIR^RHzF`Ub+Qk(4_(#IZ^c1tb}N%Ic>QvgHu8=99G1I!zr
> > > zi1<XSPVQ0l|Dn`>Y}@ICo at a`B48;n0CbW%pYR_Qsz)QLg*1$h at e3#Xfy{tO<UGeb{
> > > zwne_$27Z>)_P6yQp|(tQjeWDNRo at o-eywM&%T#WQ%Pp2X;EeVak3TYLEEIMY_QUx9
> > > zqvg9te~$lq0_I8W`>?x(r!pIM1KI23bq`$kK<I&9X$(uVn=FnRMSqtmKwd1}Ep>5w
> > > za^l+s*`HWUYwZ4^I_qL&rhs>CJukJ=9PD7(D%kV9tV_ at 8{7C6>+4dM`=gpQ+`X5sE
> > > l^kJ#ukL4Y57ixAa{T)yr9DB?*aBv?-_5Eu6ah%T`=HI~S at q7RP
> > >
> > > literal 0
> > > HcmV?d00001
> > >
> > > --
> > > 2.17.1
> > >