Skip to content

fix: add e54c use rkdevtool download spi.img tutorial#828

Merged
jack-ma merged 3 commits into
radxa-docs:mainfrom
Milir-Radxa:e54c
Jun 23, 2025
Merged

fix: add e54c use rkdevtool download spi.img tutorial#828
jack-ma merged 3 commits into
radxa-docs:mainfrom
Milir-Radxa:e54c

Conversation

@Milir-Radxa

Copy link
Copy Markdown
Contributor
Description of changes

@Milir-Radxa Milir-Radxa requested review from a team, akgnah and xzuoqi as code owners June 20, 2025 03:20

@windsurf-bot windsurf-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 To request another review, post a new comment with "/windsurf-review".

autoreconf -i
./configure
make -j $(nproc)
cp rkdeveloptool /opt/local/sbin/

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The MacOS installation instructions copy rkdeveloptool to '/opt/local/sbin/', but this directory might not exist by default on many MacOS systems. Consider using a more standard location like '/usr/local/bin/' or adding instructions to create the directory and add it to PATH if needed.

:::

<div style={{textAlign: 'center'}}>
<img src="/en/img/rock4/4d/rkddevtool.webp" style={{width: '50%', maxWidth: '700'}} />

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

There appears to be a typo in the image filename. The path '/en/img/rock4/4d/rkddevtool.webp' contains a double 'd' in 'rkddevtool' which should likely be 'rkdevtool'.

@jack-ma jack-ma merged commit 9a10cc7 into radxa-docs:main Jun 23, 2025
5 of 6 checks passed
@Milir-Radxa Milir-Radxa deleted the e54c branch June 24, 2025 01:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants