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

labgrid-exporter: add RKUSBLoader match to template #209

Merged
merged 1 commit into from
Nov 13, 2024

Conversation

a3f
Copy link
Member

@a3f a3f commented Nov 12, 2024

RKUSBLoader is the resource associated with Rockchip SoCs that fall through to BootROM mode, when no other bootable media is found.

Add it to the resource list, so the Labgrid exporter running on the LXA TAC can announce suitable resources when they exist.

Note that this on its own is not sufficient: RKUSBDriver, which will operate on RKUSBLoader expects rkdeveloptool to be installed. This is available in the Debian repositories, but we don't have a bitbake recipe for that yet.

What we do have is rk-usb-loader, which is the RK35xx-loader bundled with barebox. This supports only the SoCs supported by barebox, has different arguments than rkdeveloptool and is not supported by Labgrid at all. The fact that the default tool expected by RKUSBDriver is named rk-usb-loader is a quite unfortunate accident...

RKUSBLoader is the resource associated with Rockchip SoCs that fall
through to BootROM mode, when no other bootable media is found.

Add it to the resource list, so the Labgrid exporter running on the LXA
TAC can announce suitable resources when they exist.

Note that this on its own is not sufficient: RKUSBDriver, which will
operate on RKUSBLoader expects rkdeveloptool to be installed.
This is available in the Debian repositories, but we don't have a
bitbake recipe for that yet.

What we do have is rk-usb-loader, which is the RK35xx-loader bundled
with barebox. This supports only the SoCs supported by barebox, has
different arguments than rkdeveloptool and is not supported by Labgrid
at all. The fact that the default tool expected by RKUSBDriver is
named rk-usb-loader is a quite unfortunate accident...

Signed-off-by: Ahmad Fatoum <a.fatoum@pengutronix.de>
Copy link
Member

@hnez hnez left a comment

Choose a reason for hiding this comment

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

Looks good to me.

For the record: we do have rkdeveloptool support in the form of a pull request now #213

@hnez hnez merged commit aa6490f into linux-automation:scarthgap Nov 13, 2024
4 checks passed
@a3f a3f deleted the rkusbloader branch November 13, 2024 07:29
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