All of lore.kernel.org
 help / color / mirror / Atom feed
From: Govind Singh <govinds@codeaurora.org>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Andy Gross <andy.gross@linaro.org>,
	ath10k@lists.infradead.org, linux-wireless@vger.kernel.org
Subject: Re: [PATCH 06/12] firmware: qcom: scm: Add WLAN VMID for Qualcomm SCM interface
Date: Wed, 09 May 2018 09:20:14 +0530	[thread overview]
Message-ID: <55ee4bf22e6557a37cbd46691816bef5@codeaurora.org> (raw)
In-Reply-To: <20180507204236.GI2259@tuxbook-pro>

On 2018-05-08 02:12, Bjorn Andersson wrote:
> On Sun 25 Mar 22:40 PDT 2018, Govind Singh wrote:
> 
>> Add WLAN related VMID's to support wlan driver to set up
>> the remote's permissions call via TrustZone.
>> 
>> Signed-off-by: Govind Singh <govinds@codeaurora.org>
> 
> Please use ./scripts/get_maintainer.pl for each patch to get a hint
> about whom to send your patches to.
> 
> While I think it's okay that Kalle picks it through his tree Andy 
> should
> give his ack first.
> 
> 
> Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> 
> Regards,
> Bjorn
> 

Thanks Bjorn for the review.
I will take care this point while sending the next version of the qmi 
patch set.

>> ---
>>  include/linux/qcom_scm.h | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>> 
>> diff --git a/include/linux/qcom_scm.h b/include/linux/qcom_scm.h
>> index b401b96..da63d84 100644
>> --- a/include/linux/qcom_scm.h
>> +++ b/include/linux/qcom_scm.h
>> @@ -1,4 +1,4 @@
>> -/* Copyright (c) 2010-2015, The Linux Foundation. All rights 
>> reserved.
>> +/* Copyright (c) 2010-2015, 2018, The Linux Foundation. All rights 
>> reserved.
>>   * Copyright (C) 2015 Linaro Ltd.
>>   *
>>   * This program is free software; you can redistribute it and/or 
>> modify
>> @@ -33,6 +33,8 @@ struct qcom_scm_vmperm {
>> 
>>  #define QCOM_SCM_VMID_HLOS       0x3
>>  #define QCOM_SCM_VMID_MSS_MSA    0xF
>> +#define QCOM_SCM_VMID_WLAN       0x18
>> +#define QCOM_SCM_VMID_WLAN_CE    0x19
>>  #define QCOM_SCM_PERM_READ       0x4
>>  #define QCOM_SCM_PERM_WRITE      0x2
>>  #define QCOM_SCM_PERM_EXEC       0x1
>> --
>> 1.9.1
>> 

WARNING: multiple messages have this Message-ID (diff)
From: Govind Singh <govinds@codeaurora.org>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: Andy Gross <andy.gross@linaro.org>,
	linux-wireless@vger.kernel.org, ath10k@lists.infradead.org
Subject: Re: [PATCH 06/12] firmware: qcom: scm: Add WLAN VMID for Qualcomm SCM interface
Date: Wed, 09 May 2018 09:20:14 +0530	[thread overview]
Message-ID: <55ee4bf22e6557a37cbd46691816bef5@codeaurora.org> (raw)
In-Reply-To: <20180507204236.GI2259@tuxbook-pro>

On 2018-05-08 02:12, Bjorn Andersson wrote:
> On Sun 25 Mar 22:40 PDT 2018, Govind Singh wrote:
> 
>> Add WLAN related VMID's to support wlan driver to set up
>> the remote's permissions call via TrustZone.
>> 
>> Signed-off-by: Govind Singh <govinds@codeaurora.org>
> 
> Please use ./scripts/get_maintainer.pl for each patch to get a hint
> about whom to send your patches to.
> 
> While I think it's okay that Kalle picks it through his tree Andy 
> should
> give his ack first.
> 
> 
> Acked-by: Bjorn Andersson <bjorn.andersson@linaro.org>
> 
> Regards,
> Bjorn
> 

Thanks Bjorn for the review.
I will take care this point while sending the next version of the qmi 
patch set.

>> ---
>>  include/linux/qcom_scm.h | 4 +++-
>>  1 file changed, 3 insertions(+), 1 deletion(-)
>> 
>> diff --git a/include/linux/qcom_scm.h b/include/linux/qcom_scm.h
>> index b401b96..da63d84 100644
>> --- a/include/linux/qcom_scm.h
>> +++ b/include/linux/qcom_scm.h
>> @@ -1,4 +1,4 @@
>> -/* Copyright (c) 2010-2015, The Linux Foundation. All rights 
>> reserved.
>> +/* Copyright (c) 2010-2015, 2018, The Linux Foundation. All rights 
>> reserved.
>>   * Copyright (C) 2015 Linaro Ltd.
>>   *
>>   * This program is free software; you can redistribute it and/or 
>> modify
>> @@ -33,6 +33,8 @@ struct qcom_scm_vmperm {
>> 
>>  #define QCOM_SCM_VMID_HLOS       0x3
>>  #define QCOM_SCM_VMID_MSS_MSA    0xF
>> +#define QCOM_SCM_VMID_WLAN       0x18
>> +#define QCOM_SCM_VMID_WLAN_CE    0x19
>>  #define QCOM_SCM_PERM_READ       0x4
>>  #define QCOM_SCM_PERM_WRITE      0x2
>>  #define QCOM_SCM_PERM_EXEC       0x1
>> --
>> 1.9.1
>> 

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

  reply	other threads:[~2018-05-09  3:50 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-03-26  5:40 [PATCH 06/12] firmware: qcom: scm: Add WLAN VMID for Qualcomm SCM interface Govind Singh
2018-03-26  5:40 ` Govind Singh
2018-05-07 20:42 ` Bjorn Andersson
2018-05-07 20:42   ` Bjorn Andersson
2018-05-09  3:50   ` Govind Singh [this message]
2018-05-09  3:50     ` Govind Singh

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55ee4bf22e6557a37cbd46691816bef5@codeaurora.org \
    --to=govinds@codeaurora.org \
    --cc=andy.gross@linaro.org \
    --cc=ath10k@lists.infradead.org \
    --cc=bjorn.andersson@linaro.org \
    --cc=linux-wireless@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.