On 08/01/2025 12:19, Gokul Sriram P (QUIC) wrote:
> On 07/01/2025 13:56, Gokul Sriram P wrote:
>>>>> +examples:
>>>>> + - |
>>>>> + #include <dt-bindings/interrupt-controller/arm-gic.h>
>>>>> + #include <dt-bindings/clock/qcom,ipq5332-gcc.h>
>>>>> + remoteproc@d100000 {
>>>>> + compatible = "qcom,ipq5332-wcss-sec-pil";
>>>>> + reg = <0xd100000 0x4040>;
>>>>> + firmware-name = "ath12k/IPQ5332/hw1.0/q6_fw0.mdt";
>>>> Nit: .mbn
>>>>
>>>
>>> Hi Dmitry,
>>>
>>> Its .mdt format only in our case.
>>>
>> Then probably you need to fix your format.
>
> Hi Dmitry/ Krzysztof,
> We use split firmware image where .mbn image is split into x segments with
> .b00 to .bxx extension. The mbn header along with signing metadata will be
> part of the .mdt. The secure authenticatior (TrustZone) will expect metadata
> as part of .mdt and will authenticate the .bxx segments.
NAK, you got feedback which you refuse to implement.
>
> Certain msm platforms already support .mdt format as in
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/remoteproc/qcom,sm6115-pas.yaml?h=v6.13-rc6.
That's an overlook, not support. Don't use arguments "someone sneaked
it, so I can as well".
Best regards,
Krzysztof
