Only call reloc when reloc_len is non-zero #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
In order to have the linker include the resource table in the baseimage, the resource variable needs to be used in some form. Otherwise the optimization in the linker detects that this is a non-referenced variable and removes it from the baseimage. To alleviate this, a dummy trace was used to make sure the resource table section is present in the firmware binary. The new IpcMemory module inherently adds this trace, and so the trace in DCE image is no longer required and is removed. Signed-off-by: Nicolas Dechesne <[email protected]>
With sysbios-rpmsg >= 1.10+ there is a new resource table mechanism, and we need to update our VirtToPhys function. Fortunately, this API is now implemented and provided by rpmsg. Signed-off-by: Nicolas Dechesne <[email protected]>
Signed-off-by: Nicolas Dechesne <[email protected]>
This is now taken care by rpmsg after moving to 1.20+ Signed-off-by: Nicolas Dechesne <[email protected]>
It isn't enough that we use the correct address.. HDVICP2 needs to as well. Conflicts: ducati/ti/dce/ivahd.c firmware/ducati-m3-core0.xem3
Signed-off-by: Nicolas Dechesne <[email protected]>
The 'package' ti.dce both had the DCE code, as well as the baseimage code (BIOS cfg, main(), ...). With this change we split the DCE code from the baseimage. there is now: - ti.dce - ti.dce.baseimage that will allow a simpler reuse of ti.dce into other base images there is still some clean up that can be done in the base image to move the thread creation, and some config in ti.dce, and this will be done in further commits. This commit only changes the file 'location' without changing any implementation detail. Signed-off-by: Nicolas Dechesne <[email protected]>
Signed-off-by: Nicolas Dechesne <[email protected]>
Configure ti.dce so that it automatically registers a startup function when it's being used, so that the base image that uses DCE does not need to make the call, it's managed automagically. As such, dce_init() will be called when BIOS_start() is called. Signed-off-by: Nicolas Dechesne <[email protected]>
…er alloc Signed-off-by: Nicolas Dechesne <[email protected]>
With this image, we can decode h264 video on OMAP5 with kernel 3.4. (ducati-build commit: d5f49bdcd65412c52372aedea1cee47284f19ec0) Signed-off-by: Vincent Stehlé <[email protected]>
This is needed to be able to share the fd opened by dce_init
XDM_MEMTYPE_BO_OFFSET can be used in VIDDEC3_process outBufs to signal that a buffer descriptor denotes an offset to the bo passed in the first descriptor. Useful for example to decode NV12 into a single bo.
Signed-off-by: Vincent Stehlé <[email protected]>
- Fix SMP define - Enable use of smp platforms with $FOR_SMP environment variable. Signed-off-by: Vincent Stehlé <[email protected]>
Signed-off-by: Vincent Stehlé <[email protected]>
Signed-off-by: Vincent Stehlé <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Vincent Stehlé [email protected]