linux-kernel - Re: [PATCH] remoteproc: qcom_q6v5_pas: enable sm8550 adsp & cdsp autoboot

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 6 Feb 2023 14:59:03 +0100
From:   Neil Armstrong <neil.armstrong@...aro.org>
To:     Konrad Dybcio <konrad.dybcio@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <andersson@...nel.org>,
        Mathieu Poirier <mathieu.poirier@...aro.org>
Cc:     linux-arm-msm@...r.kernel.org, linux-remoteproc@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] remoteproc: qcom_q6v5_pas: enable sm8550 adsp & cdsp
 autoboot

On 06/02/2023 14:54, Konrad Dybcio wrote:
> 
> 
> On 6.02.2023 14:53, Neil Armstrong wrote:
>> Set the SM8550 adsp & cdsp auto_boot to true to match the behavior
>> of the previous platforms.
>>
>> Fixes: 7eddedc97563 ("remoteproc: qcom_q6v5_pas: add sm8550 adsp, cdsp & mpss compatible & data")
>> Signed-off-by: Neil Armstrong <neil.armstrong@...aro.org>
>> ---
> Reviewed-by: Konrad Dybcio <konrad.dybcio@...aro.org>
> 
> Konrad
>>   drivers/remoteproc/qcom_q6v5_pas.c | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/remoteproc/qcom_q6v5_pas.c b/drivers/remoteproc/qcom_q6v5_pas.c
>> index 4e1d9b08def4..90c5612a3045 100644
>> --- a/drivers/remoteproc/qcom_q6v5_pas.c
>> +++ b/drivers/remoteproc/qcom_q6v5_pas.c
>> @@ -1108,7 +1108,7 @@ static const struct adsp_data sm8550_adsp_resource = {
>>   	.pas_id = 1,
>>   	.dtb_pas_id = 0x24,
>>   	.minidump_id = 5,
>> -	.auto_boot = false,
>> +	.auto_boot = true,
>>   	.proxy_pd_names = (char*[]){
>>   		"lcx",
>>   		"lmx",
>> @@ -1127,7 +1127,7 @@ static const struct adsp_data sm8550_cdsp_resource = {
>>   	.pas_id = 18,
>>   	.dtb_pas_id = 0x25,
>>   	.minidump_id = 7,
>> -	.auto_boot = false,
>> +	.auto_boot = true,
>>   	.proxy_pd_names = (char*[]){
>>   		"cx",
>>   		"mxc",
>>
>> ---
>> base-commit: ea4dabbb4ad7eb52632a2ca0b8f89f0ea7c55dcf
>> change-id: 20230206-topic-sm8550-upstream-fix-remoteproc-autoboot-5c7effcf197c
> P.S not sure if you wanted to send that to lkml

It's part of what b4 sends, there's nothing secret

Neil

>>
>> Best regards,