Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rpmsg/rpmsg_ping: fix CI check issues #11675

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Conversation

yf13
Copy link
Contributor

@yf13 yf13 commented Feb 8, 2024

Summary

rpmsg_ping.c is blocking CI checks for RPTUN related patches like #11673. this patch simply fix the compilation issue of rpmsg_ping.c:

  • line 191: overflow conversion from "long unsigned int" to "unsigned int"
  • line 226: "sendlen" used uninitialized

Impact

Ths is to unblock RPTUN/RPMSG related pull requests

Testing

build check

rpmsg_ping.c is blocking CI checks for RPTUN related patches like apache#11673.
this patch simply fix the compilation issue of rpmsg_ping.c:

- line 191: overflow conversion from "long unsigned int" to "unsigned int"
- line 226: "sendlen" used uninitialized

Signed-off-by: Yanfeng Liu <yfliu2008@qq.com>
@yf13 yf13 changed the title rpmsg/rpmsg_ping: tenative fix for build issue rpmsg/rpmsg_ping: fix CI check issues Feb 8, 2024
@acassis acassis merged commit b258680 into apache:master Feb 8, 2024
27 checks passed
@@ -188,11 +188,11 @@ static void rpmsg_ping_logout_rate(uint64_t len, clock_t avg)
int rpmsg_ping(FAR struct rpmsg_endpoint *ept,
FAR const struct rpmsg_ping_s *ping)
{
clock_t min = ULONG_MAX;
clock_t min = UINT_MAX;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yf13 it's better to change to CLOCK_MAX

@yf13 yf13 deleted the fix-rpmsg-ping-ci branch February 13, 2024 04:56
@jerpelea jerpelea added this to To-Add in Release Notes - 12.5.0 Mar 26, 2024
@jerpelea jerpelea moved this from To-Add to Not Applicable in Release Notes - 12.5.0 Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants