Linaro / meta-qcom

OpenEmbedded/Yocto Project BSP layer for Qualcomm based platforms
MIT License
60 stars 69 forks source link

linux-yocto: enable various remotprocs for qcm6490 boards #588

Closed adhudase closed 1 week ago

adhudase commented 3 months ago

Add support for the ADSP, CDSP and WPSS remoteprocs found on SC7280 and enable various applicable remoteproc nodes for qcm6490-idp and qcs6490-rb3gen2.

This has dependency on PR#587.

lumag commented 2 months ago

@adhudase could you please update / rebase this PR?

adhudase commented 2 months ago

Sure, fixing the rebase. I'll update this PR once upstream patches are updated by Komal.

lumag commented 2 months ago

@adhudase But there is one already, it was picked up by Bjorn: https://lore.kernel.org/linux-arm-msm/171373856754.1196479.8348340567259271476.b4-ty@kernel.org/

lumag commented 2 months ago

@adhudase I mean, there is an updated series by Komal, it has been picked up by Bjorn.

adhudase commented 2 months ago

Hi @lumag, there was internal discussion around your comments regarding separate firmware folders for qcs6490 vs qcm6490 and modem vs modem-nm, which is not yet concluded. I will wait for that to conclude and then update this PR accordingly.

Thanks

lumag commented 2 months ago

@adhudase the patches are already picked up for -next. So if there are any changes, you have to base it on top of the accepted patches.

lumag commented 1 month ago

@adhudase any updates? A cadence of several weeks is really depressing

lumag commented 3 weeks ago

@adhudase any updates

adhudase commented 2 weeks ago

Hi @lumag, sorry for delay in response; somehow missed your message. I still don't see zap shader in no-login release and modem is not expected to be shipped in no-login. So if you agree can we go ahead with firmware recipe without these? Thanks

lumag commented 2 weeks ago

@adhudase The main question was about firmware names in DT, not about them being present in the public archive. Please update to the latest series picked up by Bjorn and that's it.

adhudase commented 2 weeks ago

Seems mainline went ahead with mdt for modem on qcs6490;missed mbn. I've modified that to mbn here. qcom/qcs6490/modem.mdt -> qcom/qcs6490/modem.mbn

lumag commented 2 weeks ago

@adhudase thanks! please mark it as 'ready', I'll merge it. Also please send a patch to linux-arm-msm, changing .mdt to .mbn. Thank you!

adhudase commented 1 week ago

Hi @lumag, can we merge this PR?

lumag commented 1 week ago

For some reason, I didn't get the notification that it's marked as ready. Merged now.