Re: Venus probe issues on SM6350 SoC

["Luca Weiss" <luca.weiss@fairphone.com>]

On Tue Apr 1, 2025 at 11:21 AM CEST, Vikash Garodia wrote:
>
> On 4/1/2025 1:12 PM, Luca Weiss wrote:
>> On Tue Apr 1, 2025 at 8:55 AM CEST, Vikash Garodia wrote:
>>>
>>>
>>> On 4/1/2025 11:47 AM, Vikash Garodia wrote:
>>>> Hi Luca,
>>>>
>>>> On 3/28/2025 8:52 PM, Luca Weiss wrote:
>>>>> Hi all, and Vikash and Dikshita,
>>>>>
>>>>> Konrad Dybcio was suggesting I write an email here, maybe someone has a
>>>>> good idea what I can try.
>>>>>
>>>>> I've been working on bringup for SM6350/SM7225 ("lagoon") for the
>>>>> Fairphone 4 smartphone but have been stuck on getting Venus working for
>>>>> a long time (~January 2022). Essentially, whatever I try probe fails
>>>>> with the following error:
>>>>>
>>>>> [   41.939451] qcom-venus aa00000.video-codec: non legacy binding
>>>>> [   42.162105] qcom-venus aa00000.video-codec: wait for cpu and video core idle fail (-110)
>>>>> [   42.167037] qcom-venus aa00000.video-codec: probe with driver qcom-venus failed with error -110
>>>>>
>>>>> I've double checked the videocc driver against downstream, the videocc
>>>>> GDSCs and their flags, compared dts bits, checked basic driver bits but
>>>>> I couldn't find anything to get further than this.
>>>>>
>>>>> For driver / compatible, it doesn't seem to matter if I use sm8250 or
>>>>> sc7280 compatible or actually the struct I created for sm6350.
>>>>>
>>>>> I'll attach the log with some extra debug prints below.
>>>>>
>>>>> My git branch for reference: https://github.com/z3ntu/linux/commits/sm6350-6.14.y-wip-venus/
>>>>>
>>>>> I didn't try the new Iris driver yet, mostly because as far as I can
>>>>> tell, VPU_VERSION_IRIS2_1 is not yet supported there, just IRIS2
>>>>> (sm8250) and IRIS3 (sm8550). But I'm also happy to try something there.
>>>> The good part if IRIS2_1 configuration have firmware support with gen2 HFIs.
>>>> Since you are wiling to give a try, let me or Dikshita share the firmware with
>>>> you in a short while, and with that, you can directly try SM6350 with IRIS
>>>> driver. Given that we have already validated SC7280 with gen2 (downstream
>>>> version of IRIS though), i am quite hopeful that SM6350 can be up with iris.
>>>> FYI, iris is having just the h264 decoder at this point with RFC patches posted
>>>> for VP9 and H265 decoder.
>>> Pls try with below firmware bins on IRIS driver. We can debug from there. Hope
>>> you have the access to download the binary.
>>>
>>> https://git.codelinaro.org/clo/linux-kernel/linux-firmware/-/commit/56035bf6edac7eb4d2867528c598eb646ee68651
>> 
>> Hi Vikash,
>> 
>> Thanks for sharing these binaries!
>> 
>> I've signed them with sectools for my device and tried them first with
>> the venus driver.
>> 
>> There it looks like the firmware doesn't load at all:
>> [   44.798028] qcom-venus aa00000.video-codec: fail to load video firmware
>> [   44.800367] qcom-venus aa00000.video-codec: probe with driver qcom-venus failed with error -22
>> 
>> Then with iris driver and qcom,sm8250-venus compatible while it does
>> seem to complete the probe function (I see a print I've added in
>> iris_probe before the last "return 0;"), the phone crashes.
> The approach to update the SM8250 platform data to bringup looks good. Could you
> share the boot crash stack as well ?

Hi Vikash,

Unfortunately there's no log in dmesg when this happens, the phone just
freezes, I'm not sure right now whether it reboots into ramdump mode then.

But also for ramdump I haven't yet used the tools to debug such a crash,
so that wouldn't help me too much right now.

I can also try later to try and find what part of the code triggers
this, by adding some prints around the files, maybe that reveals a bit
of what's happening there.

Regards
Luca