lkml.org 
[lkml]   [2018]   [Aug]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3 2/4] venus: firmware: move load firmware in a separate function
Hi Stanimir,

On 2018-07-25 15:05, Stanimir Varbanov wrote:
> Hi Vikash,
>
> On 07/04/2018 10:06 PM, Vikash Garodia wrote:

<snip>

>> +
>> + if (!IS_ENABLED(CONFIG_QCOM_MDT_LOADER))
>> + return -EPROBE_DEFER;
>
> the original venus_boot was checking for || !qcom_scm_is_available()
> why
> is that removed? It was done on purpose.

For booting venus without tz, scm calls are not needed. Does it look
good to
keep the check for "!core->no_tz" case alone ?

if (!core->no_tz && !qcom_scm_is_available())
return -EPROBE_DEFER;

Thanks,
Vikash

\
 
 \ /
  Last update: 2018-08-06 10:26    [W:0.076 / U:0.088 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site