Hi,
On 5/22/2017 3:03 PM, Dwivedi, Avaneesh Kumar (avani) wrote:
>
>
> On 5/20/2017 8:25 AM, Sricharan R wrote:
>> Hi Bjorn/Avaneesh,
>>
>> On 5/16/2017 11:32 PM, Avaneesh Kumar Dwivedi wrote:
>>> This patch refactor code to first load all firmware blobs
>>> and then update modem proc to authenticate and boot fw.
>>> Also make a trivial change in a error log.
>>>
>>> Signed-off-by: Avaneesh Kumar Dwivedi <akdwived@xxxxxxxxxxxxxx>
>>> ---
>>> drivers/remoteproc/qcom_q6v5_pil.c | 25 ++++++++++++-------------
>>> 1 file changed, 12 insertions(+), 13 deletions(-)
>>>
>>> diff --git a/drivers/remoteproc/qcom_q6v5_pil.c b/drivers/remoteproc/qcom_q6v5_pil.c
>>> index 8fd697a..2626954 100644
>>> --- a/drivers/remoteproc/qcom_q6v5_pil.c
>>> +++ b/drivers/remoteproc/qcom_q6v5_pil.c
>>> @@ -466,9 +466,10 @@ static int q6v5_mpss_init_image(struct q6v5 *qproc, const struct firmware *fw)
>>> ret = q6v5_rmb_mba_wait(qproc, RMB_MBA_META_DATA_AUTH_SUCCESS, 1000);
>>> if (ret == -ETIMEDOUT)
>>> - dev_err(qproc->dev, "MPSS header authentication timed out\n");
>>> + dev_err(qproc->dev, "metadata authentication timed out\n");
>>> else if (ret < 0)
>>> - dev_err(qproc->dev, "MPSS header authentication failed: %d\n", ret);
>>> + dev_err(qproc->dev,
>>> + "metadata authentication failed: %d\n", ret);
>>> dma_free_attrs(qproc->dev, fw->size, ptr, phys, dma_attrs);
>>> @@ -503,7 +504,7 @@ static int q6v5_mpss_load(struct q6v5 *qproc)
>>> bool relocate = false;
>>> char seg_name[10];
>>> ssize_t offset;
>>> - size_t size;
>>> + size_t size = 0;
>>> void *ptr;
>>> int ret;
>>> int i;
>>> @@ -541,7 +542,9 @@ static int q6v5_mpss_load(struct q6v5 *qproc)
>>> }
>>> mpss_reloc = relocate ? min_addr : qproc->mpss_phys;
>>> -
>>> + /* Load firmware completely before letting mss to start
>>> + * authentication and then boot firmware
>>> + */
>>> for (i = 0; i < ehdr->e_phnum; i++) {
>>> phdr = &phdrs[i];
>>> @@ -574,17 +577,13 @@ static int q6v5_mpss_load(struct q6v5 *qproc)
>>> memset(ptr + phdr->p_filesz, 0,
>>> phdr->p_memsz - phdr->p_filesz);
>>> }
>>> -
>>> - size = readl(qproc->rmb_base + RMB_PMI_CODE_LENGTH_REG);
>>> - if (!size) {
>>> - boot_addr = relocate ? qproc->mpss_phys : min_addr;
>>> - writel(boot_addr, qproc->rmb_base + RMB_PMI_CODE_START_REG);
>>> - writel(RMB_CMD_LOAD_READY, qproc->rmb_base + RMB_MBA_COMMAND_REG);
>>> - }
>>> -
>>> size += phdr->p_memsz;
>>> - writel(size, qproc->rmb_base + RMB_PMI_CODE_LENGTH_REG);
>>> }
>> So while moving this down, can we use qcom_mdt_load instead for the mpss
>> image loading part above ?
> qcom_mdt_load() can not be used to load segments for mpss, as MPSS blobs are self authenticated.
> while qcom_mdt_load() is used in cases where authentication of loaded blobs is done by trustzone.
> for that qcom_mdt_load() does extra steps to send pas_id to trustzone and mem_setup() etc.
Right, so my intention of asking this was if the code which does the calculation and
loads the segments in qcom_mdt_load can somehow be abstracted out, so that future
self authenticating rproc (even mpss in this case) can use them to load mdt ?
>>> + /* Transfer ownership of modem region with modem fw */
>>> + boot_addr = relocate ? qproc->mpss_phys : min_addr;
>>> + writel(boot_addr, qproc->rmb_base + RMB_PMI_CODE_START_REG);
>>> + writel(RMB_CMD_LOAD_READY, qproc->rmb_base + RMB_MBA_COMMAND_REG);
>>> + writel(size, qproc->rmb_base + RMB_PMI_CODE_LENGTH_REG);
>> For ipq8074 [1], wcnss core has an Q6V5 version of the ip for which the
>> initialization/boot sequence is pretty much the same as that has been added
>> for msm8996 in this series. So wanted to understand if its better to
>> use this remoteproc itself by keeping the Q6 and mpss parts separately (or)
>> add a new remoteproc ?
> Bjorn can better answer this query, but i believe this remoteproc can be extended to load
> mpss part by adding private initialization for the IP.
ya, the mpss part can be separated out, so that this can be a Q6 + MPSS (or) Q6 + WCNSS
remoteproc. Was asking this to get the right way for adding the Q6 + WCNSS remoteproc,
as the Q6 part is same what you have added for msm8996.
Regards,
Sricharan
>>
>> [1] https://patchwork.kernel.org/patch/9711725/
>>
>> Regards,
>> Sricharan
>>
>>> ret = q6v5_rmb_mba_wait(qproc, RMB_MBA_AUTH_COMPLETE, 10000);
>>> if (ret == -ETIMEDOUT)
>>>
>
--
"QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
