Open phuerta79 opened 1 year ago
@phuerta79 please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]
Options:
- (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
@microsoft-github-policy-service agree
- (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.
@microsoft-github-policy-service agree company="Microsoft"
@microsoft-github-policy-service agree company="Hubbell Inc"
Hi @phuerta79 - I would like to make this change, but we are discussing this internally. In the meantime, you can use these extensions to pass 64-bit values to threads: https://github.com/azure-rtos/threadx/blob/master/common/inc/tx_api.h#L234-L259 You may need to modify your tx_port.h file, as done here: https://github.com/azure-rtos/threadx/blob/master/ports_smp/cortex_a5x_smp/gnu/inc/tx_port.h#L281
Hi @goldscott , Thanks for your feedback. I will take a look at what yo propose.
These functions have an ULONG type param, so if you want to pass a pointer to something you can cast it and it will fit in the ULONG param in 32 bit machines. But in 64 bit machines the pointer will not fit, so I propose to change this param to ALIGN_TYPE type.
For example, the following code will work in 32 bit machines, but it will not work on 64 bit machines:
With the changes proposed, the code will turn into:
And it will work ok both in 32 and 64 bit systems.