linux-kernel - Re:Re: [PATCH v2 1/2] bus: mhi: host: pci_generic: Update the file path for Foxconn SDX55/SDX72

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]

Message-ID: <6dd00891.8235.190e91ac139.Coremail.slark_xiao@163.com>
Date: Thu, 25 Jul 2024 16:56:03 +0800 (CST)
From: "Slark Xiao" <slark_xiao@....com>
To: "Manivannan Sadhasivam" <manivannan.sadhasivam@...aro.org>
Cc: mhi@...ts.linux.dev, linux-arm-msm@...r.kernel.org, 
	linux-kernel@...r.kernel.org
Subject: Re:Re: [PATCH v2 1/2] bus: mhi: host: pci_generic: Update the file
 path for Foxconn SDX55/SDX72


At 2024-07-25 11:59:54, "Manivannan Sadhasivam" <manivannan.sadhasivam@...aro.org> wrote:
>On Thu, Jul 25, 2024 at 10:29:40AM +0800, Slark Xiao wrote:
>> To separate the images of Foxconn from other vendors, adding a
>> new foxconn subfolder under qcom/<platform> for edl image path.
>> And delete the fw patch since it's useless for Foxconn devices.
>> 
>> Fixes: bf30a75e6e00 ("bus: mhi: host: Add support for Foxconn SDX72 modems")
>> Signed-off-by: Slark Xiao <slark_xiao@....com>
>> ---
>> v2: change the folder path architecture
>> ---
>>  drivers/bus/mhi/host/pci_generic.c | 13 +++++--------
>>  1 file changed, 5 insertions(+), 8 deletions(-)
>> 
>> diff --git a/drivers/bus/mhi/host/pci_generic.c b/drivers/bus/mhi/host/pci_generic.c
>> index 14a11880bcea..f159a9dd53e7 100644
>> --- a/drivers/bus/mhi/host/pci_generic.c
>> +++ b/drivers/bus/mhi/host/pci_generic.c
>> @@ -433,8 +433,7 @@ static const struct mhi_controller_config modem_foxconn_sdx72_config = {
>>  
>>  static const struct mhi_pci_dev_info mhi_foxconn_sdx55_info = {
>>  	.name = "foxconn-sdx55",
>> -	.fw = "qcom/sdx55m/sbl1.mbn",
>> -	.edl = "qcom/sdx55m/edl.mbn",
>> +	.edl = "qcom/sdx55m/foxconn/prog_firehose_sdx55.mbn",
>
>I think you misunderstood what I suggested in earlier revision. You should add
>the Foxconn specific fw only if it is different from the qcom one. Is it really
>different for all these modems? Otherwise, what is the point of adding them?
>
>- Mani

>
Hi Mani,
Yes, all programer files are different with default. We add a sign step for each image
file. That means other vendor's edl image(including Qualcomm) can't be applied for
Foxconn devices. 

>>  	.config = &modem_foxconn_sdx55_config,
>>  	.bar_num = MHI_PCI_DEFAULT_BAR_NUM,
>>  	.dma_data_width = 32,
>> @@ -444,8 +443,7 @@ static const struct mhi_pci_dev_info mhi_foxconn_sdx55_info = {
>>  
>>  static const struct mhi_pci_dev_info mhi_foxconn_t99w175_info = {
>>  	.name = "foxconn-t99w175",
>> -	.fw = "qcom/sdx55m/sbl1.mbn",
>> -	.edl = "qcom/sdx55m/edl.mbn",
>> +	.edl = "qcom/sdx55m/foxconn/prog_firehose_sdx55.mbn",
>>  	.config = &modem_foxconn_sdx55_config,
>>  	.bar_num = MHI_PCI_DEFAULT_BAR_NUM,
>>  	.dma_data_width = 32,
>> @@ -455,8 +453,7 @@ static const struct mhi_pci_dev_info mhi_foxconn_t99w175_info = {
>>  
>>  static const struct mhi_pci_dev_info mhi_foxconn_dw5930e_info = {
>>  	.name = "foxconn-dw5930e",
>> -	.fw = "qcom/sdx55m/sbl1.mbn",
>> -	.edl = "qcom/sdx55m/edl.mbn",
>> +	.edl = "qcom/sdx55m/foxconn/prog_firehose_sdx55.mbn",
>>  	.config = &modem_foxconn_sdx55_config,
>>  	.bar_num = MHI_PCI_DEFAULT_BAR_NUM,
>>  	.dma_data_width = 32,
>> @@ -502,7 +499,7 @@ static const struct mhi_pci_dev_info mhi_foxconn_dw5932e_info = {
>>  
>>  static const struct mhi_pci_dev_info mhi_foxconn_t99w515_info = {
>>  	.name = "foxconn-t99w515",
>> -	.edl = "fox/sdx72m/edl.mbn",
>> +	.edl = "qcom/sdx72m/foxconn/edl.mbn",
>>  	.edl_trigger = true,
>>  	.config = &modem_foxconn_sdx72_config,
>>  	.bar_num = MHI_PCI_DEFAULT_BAR_NUM,
>> @@ -513,7 +510,7 @@ static const struct mhi_pci_dev_info mhi_foxconn_t99w515_info = {
>>  
>>  static const struct mhi_pci_dev_info mhi_foxconn_dw5934e_info = {
>>  	.name = "foxconn-dw5934e",
>> -	.edl = "fox/sdx72m/edl.mbn",
>> +	.edl = "qcom/sdx72m/foxconn/edl.mbn",
>>  	.edl_trigger = true,
>>  	.config = &modem_foxconn_sdx72_config,
>>  	.bar_num = MHI_PCI_DEFAULT_BAR_NUM,
>> -- 
>> 2.25.1
>> 
>
>-- 
>மணிவண்ணன் சதாசிவம்