* [PATCH] doc: board: j721e_evm: Add DM firmware steps
@ 2022-10-07 20:37 Andrew Davis
2022-10-07 23:47 ` Heinrich Schuchardt
0 siblings, 1 reply; 5+ messages in thread
From: Andrew Davis @ 2022-10-07 20:37 UTC (permalink / raw)
To: Simon Glass, Tom Rini, u-boot; +Cc: Andrew Davis
J721e needs DM firmware when using updated SYSFW. Add steps to fetch,
build, and deploy the same.
Signed-off-by: Andrew Davis <afd@ti.com>
---
doc/board/ti/j721e_evm.rst | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git a/doc/board/ti/j721e_evm.rst b/doc/board/ti/j721e_evm.rst
index 44dc316afd..a7d7229722 100644
--- a/doc/board/ti/j721e_evm.rst
+++ b/doc/board/ti/j721e_evm.rst
@@ -142,7 +142,11 @@ Sources:
Tree: https://github.com/OP-TEE/optee_os.git
Branch: master
-4. U-Boot:
+4. DM:
+ Tree: git://git.ti.com/processor-firmware/ti-linux-firmware.git
+ Branch: ti-linux-firmware
+
+5. U-Boot:
Tree: https://source.denx.de/u-boot/u-boot
Branch: master
@@ -152,7 +156,7 @@ Build procedure:
.. code-block:: text
- $ make CROSS_COMPILE=arm-linux-gnueabihf-
+ $ make CROSS_COMPILE=arm-linux-gnueabihf- SOC=j721e
2. ATF:
@@ -172,15 +176,15 @@ Build procedure:
.. code-block:: text
- $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig O=/tmp/r5
- $ make CROSS_COMPILE=arm-linux-gnueabihf- O=/tmp/r5
+ $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig O=build/r5
+ $ make CROSS_COMPILE=arm-linux-gnueabihf- O=build/r5
* 4.2 A72:
.. code-block:: text
- $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig O=/tmp/a72
- $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<path to ATF dir>/build/k3/generic/release/bl31.bin TEE=<path to OPTEE OS dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=<path to DM firmware image> O=/tmp/a72
+ $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig O=build/a72
+ $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<ATF dir>/build/k3/generic/release/bl31.bin TEE=<OPTEE OS dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=DM=<DM firmware>/ti-dm/j721e/ipc_echo_testb_mcu1_0_release_strip.xer5f O=build/a72
Target Images
--------------
--
2.37.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] doc: board: j721e_evm: Add DM firmware steps
2022-10-07 20:37 [PATCH] doc: board: j721e_evm: Add DM firmware steps Andrew Davis
@ 2022-10-07 23:47 ` Heinrich Schuchardt
2022-11-01 14:06 ` Andrew Davis
0 siblings, 1 reply; 5+ messages in thread
From: Heinrich Schuchardt @ 2022-10-07 23:47 UTC (permalink / raw)
To: Andrew Davis; +Cc: Simon Glass, Tom Rini, u-boot
On 10/7/22 22:37, Andrew Davis wrote:
> J721e needs DM firmware when using updated SYSFW. Add steps to fetch,
> build, and deploy the same.
>
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
> doc/board/ti/j721e_evm.rst | 16 ++++++++++------
> 1 file changed, 10 insertions(+), 6 deletions(-)
>
> diff --git a/doc/board/ti/j721e_evm.rst b/doc/board/ti/j721e_evm.rst
> index 44dc316afd..a7d7229722 100644
> --- a/doc/board/ti/j721e_evm.rst
> +++ b/doc/board/ti/j721e_evm.rst
> @@ -142,7 +142,11 @@ Sources:
> Tree: https://github.com/OP-TEE/optee_os.git
> Branch: master
>
> -4. U-Boot:
> +4. DM:
> + Tree: git://git.ti.com/processor-firmware/ti-linux-firmware.git
> + Branch: ti-linux-firmware
> +
> +5. U-Boot:
> Tree: https://source.denx.de/u-boot/u-boot
> Branch: master
>
> @@ -152,7 +156,7 @@ Build procedure:
>
> .. code-block:: text
Thanks for updating the documentation.
The code-blocks look like bash and not text. Please, correct this
information to allow syntax highlighting.
>
> - $ make CROSS_COMPILE=arm-linux-gnueabihf-
> + $ make CROSS_COMPILE=arm-linux-gnueabihf- SOC=j721e
Please, remove the leading $-signs which are not helpful when copying to
the console.
Please indent code blocks by 4 spaces. Otherwise Sphinx cannot recognize
them.
Best regards
Heinrich
>
> 2. ATF:
>
> @@ -172,15 +176,15 @@ Build procedure:
>
> .. code-block:: text
>
> - $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig O=/tmp/r5
> - $ make CROSS_COMPILE=arm-linux-gnueabihf- O=/tmp/r5
> + $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig O=build/r5
> + $ make CROSS_COMPILE=arm-linux-gnueabihf- O=build/r5
>
> * 4.2 A72:
>
> .. code-block:: text
>
> - $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig O=/tmp/a72
> - $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<path to ATF dir>/build/k3/generic/release/bl31.bin TEE=<path to OPTEE OS dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=<path to DM firmware image> O=/tmp/a72
> + $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig O=build/a72
> + $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<ATF dir>/build/k3/generic/release/bl31.bin TEE=<OPTEE OS dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=DM=<DM firmware>/ti-dm/j721e/ipc_echo_testb_mcu1_0_release_strip.xer5f O=build/a72
>
> Target Images
> --------------
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] doc: board: j721e_evm: Add DM firmware steps
2022-10-07 23:47 ` Heinrich Schuchardt
@ 2022-11-01 14:06 ` Andrew Davis
2022-11-06 8:49 ` Heinrich Schuchardt
0 siblings, 1 reply; 5+ messages in thread
From: Andrew Davis @ 2022-11-01 14:06 UTC (permalink / raw)
To: Heinrich Schuchardt; +Cc: Simon Glass, Tom Rini, u-boot
On 10/7/22 6:47 PM, Heinrich Schuchardt wrote:
> On 10/7/22 22:37, Andrew Davis wrote:
>> J721e needs DM firmware when using updated SYSFW. Add steps to fetch,
>> build, and deploy the same.
>>
>> Signed-off-by: Andrew Davis <afd@ti.com>
>> ---
>> doc/board/ti/j721e_evm.rst | 16 ++++++++++------
>> 1 file changed, 10 insertions(+), 6 deletions(-)
>>
>> diff --git a/doc/board/ti/j721e_evm.rst b/doc/board/ti/j721e_evm.rst
>> index 44dc316afd..a7d7229722 100644
>> --- a/doc/board/ti/j721e_evm.rst
>> +++ b/doc/board/ti/j721e_evm.rst
>> @@ -142,7 +142,11 @@ Sources:
>> Tree: https://github.com/OP-TEE/optee_os.git
>> Branch: master
>>
>> -4. U-Boot:
>> +4. DM:
>> + Tree: git://git.ti.com/processor-firmware/ti-linux-firmware.git
>> + Branch: ti-linux-firmware
>> +
>> +5. U-Boot:
>> Tree: https://source.denx.de/u-boot/u-boot
>> Branch: master
>>
>> @@ -152,7 +156,7 @@ Build procedure:
>>
>> .. code-block:: text
>
> Thanks for updating the documentation.
>
> The code-blocks look like bash and not text. Please, correct this
> information to allow syntax highlighting.
>
>>
>> - $ make CROSS_COMPILE=arm-linux-gnueabihf-
>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- SOC=j721e
>
> Please, remove the leading $-signs which are not helpful when copying to
> the console.
>
> Please indent code blocks by 4 spaces. Otherwise Sphinx cannot recognize
> them.
>
Will fix all the code blocks then in a single patch, add it to this as
a follow up in a series.
Thanks,
Andrew
> Best regards
>
> Heinrich
>
>>
>> 2. ATF:
>>
>> @@ -172,15 +176,15 @@ Build procedure:
>>
>> .. code-block:: text
>>
>> - $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig O=/tmp/r5
>> - $ make CROSS_COMPILE=arm-linux-gnueabihf- O=/tmp/r5
>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig O=build/r5
>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- O=build/r5
>>
>> * 4.2 A72:
>>
>> .. code-block:: text
>>
>> - $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig O=/tmp/a72
>> - $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<path to ATF dir>/build/k3/generic/release/bl31.bin TEE=<path to OPTEE OS dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=<path to DM firmware image> O=/tmp/a72
>> + $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig O=build/a72
>> + $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<ATF dir>/build/k3/generic/release/bl31.bin TEE=<OPTEE OS dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=DM=<DM firmware>/ti-dm/j721e/ipc_echo_testb_mcu1_0_release_strip.xer5f O=build/a72
>>
>> Target Images
>> --------------
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] doc: board: j721e_evm: Add DM firmware steps
2022-11-01 14:06 ` Andrew Davis
@ 2022-11-06 8:49 ` Heinrich Schuchardt
2022-11-09 17:27 ` Andrew Davis
0 siblings, 1 reply; 5+ messages in thread
From: Heinrich Schuchardt @ 2022-11-06 8:49 UTC (permalink / raw)
To: Andrew Davis; +Cc: Simon Glass, Tom Rini, u-boot
On 11/1/22 15:06, Andrew Davis wrote:
> On 10/7/22 6:47 PM, Heinrich Schuchardt wrote:
>> On 10/7/22 22:37, Andrew Davis wrote:
>>> J721e needs DM firmware when using updated SYSFW. Add steps to fetch,
>>> build, and deploy the same.
>>>
>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>> ---
>>> doc/board/ti/j721e_evm.rst | 16 ++++++++++------
>>> 1 file changed, 10 insertions(+), 6 deletions(-)
>>>
>>> diff --git a/doc/board/ti/j721e_evm.rst b/doc/board/ti/j721e_evm.rst
>>> index 44dc316afd..a7d7229722 100644
>>> --- a/doc/board/ti/j721e_evm.rst
>>> +++ b/doc/board/ti/j721e_evm.rst
>>> @@ -142,7 +142,11 @@ Sources:
>>> Tree: https://github.com/OP-TEE/optee_os.git
>>> Branch: master
>>>
>>> -4. U-Boot:
>>> +4. DM:
DM in the context of U-Boot is used as abbreviation for driver model.
Please, provide a meaningful title.
Best regards
Heinrich
>>> + Tree: git://git.ti.com/processor-firmware/ti-linux-firmware.git
>>> + Branch: ti-linux-firmware
>>> +
>>> +5. U-Boot:
>>> Tree: https://source.denx.de/u-boot/u-boot
>>> Branch: master
>>>
>>> @@ -152,7 +156,7 @@ Build procedure:
>>>
>>> .. code-block:: text
>>
>> Thanks for updating the documentation.
>>
>> The code-blocks look like bash and not text. Please, correct this
>> information to allow syntax highlighting.
>>
>>>
>>> - $ make CROSS_COMPILE=arm-linux-gnueabihf-
>>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- SOC=j721e
>>
>> Please, remove the leading $-signs which are not helpful when copying to
>> the console.
>>
>> Please indent code blocks by 4 spaces. Otherwise Sphinx cannot recognize
>> them.
>>
>
>
> Will fix all the code blocks then in a single patch, add it to this as
> a follow up in a series.
>
> Thanks,
> Andrew
>
>
>> Best regards
>>
>> Heinrich
>>
>>>
>>> 2. ATF:
>>>
>>> @@ -172,15 +176,15 @@ Build procedure:
>>>
>>> .. code-block:: text
>>>
>>> - $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig
>>> O=/tmp/r5
>>> - $ make CROSS_COMPILE=arm-linux-gnueabihf- O=/tmp/r5
>>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig
>>> O=build/r5
>>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- O=build/r5
>>>
>>> * 4.2 A72:
>>>
>>> .. code-block:: text
>>>
>>> - $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig
>>> O=/tmp/a72
>>> - $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<path to ATF
>>> dir>/build/k3/generic/release/bl31.bin TEE=<path to OPTEE OS
>>> dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=<path to DM firmware
>>> image> O=/tmp/a72
>>> + $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig
>>> O=build/a72
>>> + $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<ATF
>>> dir>/build/k3/generic/release/bl31.bin TEE=<OPTEE OS
>>> dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=DM=<DM
>>> firmware>/ti-dm/j721e/ipc_echo_testb_mcu1_0_release_strip.xer5f
>>> O=build/a72
>>>
>>> Target Images
>>> --------------
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] doc: board: j721e_evm: Add DM firmware steps
2022-11-06 8:49 ` Heinrich Schuchardt
@ 2022-11-09 17:27 ` Andrew Davis
0 siblings, 0 replies; 5+ messages in thread
From: Andrew Davis @ 2022-11-09 17:27 UTC (permalink / raw)
To: Heinrich Schuchardt; +Cc: Simon Glass, Tom Rini, u-boot
On 11/6/22 2:49 AM, Heinrich Schuchardt wrote:
> On 11/1/22 15:06, Andrew Davis wrote:
>> On 10/7/22 6:47 PM, Heinrich Schuchardt wrote:
>>> On 10/7/22 22:37, Andrew Davis wrote:
>>>> J721e needs DM firmware when using updated SYSFW. Add steps to fetch,
>>>> build, and deploy the same.
>>>>
>>>> Signed-off-by: Andrew Davis <afd@ti.com>
>>>> ---
>>>> doc/board/ti/j721e_evm.rst | 16 ++++++++++------
>>>> 1 file changed, 10 insertions(+), 6 deletions(-)
>>>>
>>>> diff --git a/doc/board/ti/j721e_evm.rst b/doc/board/ti/j721e_evm.rst
>>>> index 44dc316afd..a7d7229722 100644
>>>> --- a/doc/board/ti/j721e_evm.rst
>>>> +++ b/doc/board/ti/j721e_evm.rst
>>>> @@ -142,7 +142,11 @@ Sources:
>>>> Tree: https://github.com/OP-TEE/optee_os.git
>>>> Branch: master
>>>>
>>>> -4. U-Boot:
>>>> +4. DM:
>
> DM in the context of U-Boot is used as abbreviation for driver model.
>
> Please, provide a meaningful title.
>
Will rename to "DM Firmware" for v3.
Thanks,
Andrew
> Best regards
>
> Heinrich
>
>>>> + Tree: git://git.ti.com/processor-firmware/ti-linux-firmware.git
>>>> + Branch: ti-linux-firmware
>>>> +
>>>> +5. U-Boot:
>>>> Tree: https://source.denx.de/u-boot/u-boot
>>>> Branch: master
>>>>
>>>> @@ -152,7 +156,7 @@ Build procedure:
>>>>
>>>> .. code-block:: text
>>>
>>> Thanks for updating the documentation.
>>>
>>> The code-blocks look like bash and not text. Please, correct this
>>> information to allow syntax highlighting.
>>>
>>>>
>>>> - $ make CROSS_COMPILE=arm-linux-gnueabihf-
>>>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- SOC=j721e
>>>
>>> Please, remove the leading $-signs which are not helpful when copying to
>>> the console.
>>>
>>> Please indent code blocks by 4 spaces. Otherwise Sphinx cannot recognize
>>> them.
>>>
>>
>>
>> Will fix all the code blocks then in a single patch, add it to this as
>> a follow up in a series.
>>
>> Thanks,
>> Andrew
>>
>>
>>> Best regards
>>>
>>> Heinrich
>>>
>>>>
>>>> 2. ATF:
>>>>
>>>> @@ -172,15 +176,15 @@ Build procedure:
>>>>
>>>> .. code-block:: text
>>>>
>>>> - $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig
>>>> O=/tmp/r5
>>>> - $ make CROSS_COMPILE=arm-linux-gnueabihf- O=/tmp/r5
>>>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- j721e_evm_r5_defconfig
>>>> O=build/r5
>>>> + $ make CROSS_COMPILE=arm-linux-gnueabihf- O=build/r5
>>>>
>>>> * 4.2 A72:
>>>>
>>>> .. code-block:: text
>>>>
>>>> - $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig
>>>> O=/tmp/a72
>>>> - $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<path to ATF
>>>> dir>/build/k3/generic/release/bl31.bin TEE=<path to OPTEE OS
>>>> dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=<path to DM firmware
>>>> image> O=/tmp/a72
>>>> + $ make CROSS_COMPILE=aarch64-linux-gnu- j721e_evm_a72_defconfig
>>>> O=build/a72
>>>> + $ make CROSS_COMPILE=aarch64-linux-gnu- ATF=<ATF
>>>> dir>/build/k3/generic/release/bl31.bin TEE=<OPTEE OS
>>>> dir>/out/arm-plat-k3/core/tee-pager_v2.bin DM=DM=<DM
>>>> firmware>/ti-dm/j721e/ipc_echo_testb_mcu1_0_release_strip.xer5f
>>>> O=build/a72
>>>>
>>>> Target Images
>>>> --------------
>>>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-11-09 17:28 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-07 20:37 [PATCH] doc: board: j721e_evm: Add DM firmware steps Andrew Davis
2022-10-07 23:47 ` Heinrich Schuchardt
2022-11-01 14:06 ` Andrew Davis
2022-11-06 8:49 ` Heinrich Schuchardt
2022-11-09 17:27 ` Andrew Davis
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.