Skip to content

boards/arm: fix div64_const not compatible for pre-process macro#18253

Closed
wangzhi16 wants to merge 1 commit intoapache:masterfrom
wangzhi16:div64
Closed

boards/arm: fix div64_const not compatible for pre-process macro#18253
wangzhi16 wants to merge 1 commit intoapache:masterfrom
wangzhi16:div64

Conversation

@wangzhi16
Copy link
Contributor

Summary

from board/cxd56_automount.c:34:
board/cxd56_automount.c:94:19: error: braced-group within expression allowed only inside a function
94 | .ddelay = MSEC2TICK(CONFIG_CXD56_SDCARD_AUTOMOUNT_DDELAY),
| ^~~~~~~~~
board/cxd56_automount.c:95:19: error: braced-group within expression allowed only inside a function
95 | .udelay = MSEC2TICK(CONFIG_CXD56_SDCARD_AUTOMOUNT_UDELAY),
| ^~~~~~~~~

Compile-time assignment uses regular division, and runtime assignment uses high-performance division

Impact

None

Testing

ostest PASS

from board/cxd56_automount.c:34:
 board/cxd56_automount.c:94:19: error: braced-group within expression allowed only inside a function
 94 | .ddelay = MSEC2TICK(CONFIG_CXD56_SDCARD_AUTOMOUNT_DDELAY),
 | ^~~~~~~~~
 board/cxd56_automount.c:95:19: error: braced-group within expression allowed only inside a function
 95 | .udelay = MSEC2TICK(CONFIG_CXD56_SDCARD_AUTOMOUNT_UDELAY),
 | ^~~~~~~~~

Compile-time assignment uses regular division, and runtime assignment uses high-performance division

Signed-off-by: wangzhi16 <wangzhi16@xiaomi.com>
@github-actions github-actions bot added the Size: S The size of the change in this PR is small label Jan 29, 2026
Copy link
Contributor

@linguini1 linguini1 left a comment

Choose a reason for hiding this comment

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

Please make sure the PR adheres to the contributing guidelines.

Testing should include logs where possible and should be an actual test of the change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Size: S The size of the change in this PR is small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants