ChanServ changed the topic of #linux-msm to:
srinik has joined #linux-msm
srinik has quit [Ping timeout: 480 seconds]
srinik has joined #linux-msm
srinik has quit [Remote host closed the connection]
f_ has quit [Remote host closed the connection]
f_ has joined #linux-msm
f_ has quit [Max SendQ exceeded]
f_ has joined #linux-msm
deathmist1 has joined #linux-msm
deathmist has quit [Ping timeout: 480 seconds]
deathmist has joined #linux-msm
deathmist1 has quit [Ping timeout: 480 seconds]
<Adrian[m]> Hey there, since https://lore.kernel.org/all/20240628114501.14310-7-srinivas.kandagatla@linaro.org/, attaching to root or sensor PD requires a secure device node. Virtually all QCOM SoC devicetrees define ADSP, CDSP and SDSP as insecure, meaning that it's impossible to attach to root/sensor PD on the ADSP/SDSP. How are we supposed to do that now? Mark the ADSP/SDSP as secure? If so, why was that not done alongside the change upstream?
<z3ntu> Adrian: On sm7225 and qcm6490 the sensors with hexagonrpcd attach to the non-secure version. What do you need the secure one for?
<Adrian[m]> z3ntu: You cannot attach to the non-secure version on 6.10 or newer (see the patch)
<z3ntu> Maybe I messed up testing 6.10 but I think sensors still worked on 6.10?
<z3ntu> (for me on sm7225 and qcm6490)
<Adrian[m]> They don't on SDM845 and SM7150, either reverting the change or marking ADSP/SDSP as secure (and using /dev/fastrpc-adsp-secure et al) makes them work again
<Adrian[m]> * They don't on SDM845 and SM7150, either reverting the change or marking ADSP/SDSP as secure (and using /dev/fastrpc-adsp-secure et al) makes them work again
<Adrian[m]> I'm pretty sure it should be broken on SM7225 and QCM6490 as well
<z3ntu> :(
<flamingradian[m]> sm6350 doesn't have qcom,non-secure-domain so I guess it was always marked secure
<Adrian[m]> There's a patch on LKML to mark them as insecure
<z3ntu> yes...
<z3ntu> that was the correct thing "back then"
marc|gonzalez has joined #linux-msm
<marc|gonzalez> oy!
<calebccff> bamse: konradybcio: lumag: would it fly with you if we move https://gitlab.com/sdm845-mainline/qbootctl, https://gitlab.com/sdm670-mainline/hexagonrpc, and maybe https://gitlab.com/flamingradian/81voltd (FOSS QMI IMS impl) to github.com/linux-msm? I spoke with flamingradian[m] and he's open to moving hexagonrpc and 81voltd there
<calebccff> these are all broadly useful across Qualcomm platforms, so having them somewhere more upstream affiliated would be quite nice
<bamse> calebccff: no concerns from my side
<calebccff> thanks, moved over qbootctl. I'll wait for some more feedback bfeore the others