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

meta-lxatac-software: bcu: add recipe for nxp board control utilities #93

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

hnez
Copy link
Member

@hnez hnez commented Dec 20, 2023

The bcu command is used to communicate with board control hardware on NXP development boards to:

  • reset / power cycle the board into a specified boot mode
  • monitor the power consumption
  • eeprom read / write
  • temperature measurements

Sadly the board I tried to use this with is not actually supported, so I could only check that it builds and shows the help message.
It also discovered the FTDI device on the board, so that seems to work as well.

@hnez hnez requested a review from jluebbe December 20, 2023 14:09
@jluebbe
Copy link
Member

jluebbe commented Dec 20, 2023

There is a open PR (labgrid-project/labgrid#1273) to support BCU via labgrid.

@jluebbe
Copy link
Member

jluebbe commented Jan 11, 2024

What is the status of the libcurl patch? Could it be upstreamed?

@hnez
Copy link
Member Author

hnez commented Jan 17, 2024

What is the status of the libcurl patch? Could it be upstreamed?

I had a quick look at it and it looks to me like that would require changes to the build system for macOS, linux and windows, making testing the changes a chore.
Since we do not have a real usecase for bcu yet and only wanted to check it out for fun, I don't think it is worth it to put in the effort.

I'd say we close this PR for now and have a second look once an actual usecase arises.

@jluebbe
Copy link
Member

jluebbe commented Jan 17, 2024

What is the status of the libcurl patch? Could it be upstreamed?

I had a quick look at it and it looks to me like that would require changes to the build system for macOS, linux and windows, making testing the changes a chore. Since we do not have a real usecase for bcu yet and only wanted to check it out for fun, I don't think it is worth it to put in the effort.

I'd say we close this PR for now and have a second look once an actual usecase arises.

I'd also be OK with at least reporting this as an issue to upstream and then pointing to that issue in a patch status header. Perhaps that way, it's solved upstream by the time we get around to updating it.

The bcu command is used to communicate with board control hardware on
NXP development boards to:

  - reset / power cycle the board into a specified boot mode
  - monitor the power consumption
  - eeprom read / write
  - temperature measurements

Signed-off-by: Leonard Göhrs <l.goehrs@pengutronix.de>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Upstream-Status: Inappropriate [upstream ticket https://github.com/nxp-imx/bcu/issues/17]
Copy link
Member Author

Choose a reason for hiding this comment

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

I've opened an issue in the bcu repository nxp-imx/bcu#17 and added an Upstream-Status line to the patch.

@hnez hnez assigned jluebbe and unassigned hnez Jan 26, 2024
@hnez
Copy link
Member Author

hnez commented Feb 2, 2024

Hi @jluebbe,

now that we have an Upstream-Status entry, do we want this recipe in meta-lxatac?

@hnez hnez merged commit 19a3e44 into linux-automation:nanbield Feb 8, 2024
2 checks passed
@hnez hnez deleted the bcu branch February 8, 2024 12:19
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