Skip to content

Conversation

@stephanosio
Copy link
Member

@stephanosio stephanosio commented Mar 24, 2022

This commit pulls in the patches required to support the emulation of
the I2C devices (e.g. LSM303DLHC magnetometer) on the ARM targets.

Signed-off-by: Stephanos Ioannidis root@stephanos.io

QEMU PR: zephyrproject-rtos/qemu#4

@stephanosio stephanosio added ci-linux-x86_64 Run CI for x86-64 Linux host (for PR) ci-linux-aarch64 Run CI for AArch64 Linux host (for PR) area: QEMU Issues related to QEMU labels Mar 24, 2022
@stephanosio stephanosio added this to the 0.14.1 milestone Mar 24, 2022
@stephanosio stephanosio added the DNM DO NOT MERGE label Mar 24, 2022
@stephanosio
Copy link
Member Author

DNM until the QEMU PR is merged and the commit SHA is updated.

@stephanosio
Copy link
Member Author

@microbuilder I have added this as per #421 (comment). Please test it and let me know if it works properly.

p.s. this time, I have requested the AArch64 Linux build as well :)

@microbuilder
Copy link
Member

@stephanosio Thanks. Put some usage comments here, but I'll of course test once this is built: zephyrproject-rtos/qemu#4 (comment)

release-notes.md Outdated

- qemu:
* Updated to QEMU 6.2 release.
* Added LSM303DLHC magnetometer support.
Copy link
Member

@microbuilder microbuilder Mar 24, 2022

Choose a reason for hiding this comment

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

This PR will actually pull in a number of optional I2C sensors if I2C_DEVICES is enabled during QEMU builds, although LSM303DLHC is the main use case since Zephyr has support for that device out of the box, and was what I was hoping to enable in CI for testing I2C.

That is also the only device I've actively tested with Zephyr.

Copy link
Member

Choose a reason for hiding this comment

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

Specifically:

RTC:

  • config DS1338
  • config M41T80

Sensors:

  • config TMP105
  • config TMP421
  • config DPS310
  • config EMC141X
  • config ADM1272
  • config MAX34451
  • config LSM303DLHC_MAG

Copy link
Member Author

Choose a reason for hiding this comment

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

This PR will actually pull in a number of optional I2C sensors if I2C_DEVICES is enabled during QEMU builds, although LSM303DLHC is the main use case since Zephyr has support for that device out of the box, and was what I was hoping to enable in CI for testing I2C.

ok, I will try to update the commit message and the release notes to reflect that.

@stephanosio stephanosio changed the title qemu: Pull in LSM303DLHC magnetometer support qemu: Pull in I2C device emulation support for ARM targets Mar 24, 2022
@stephanosio
Copy link
Member Author

FYI @MaureenHelm @yperess @erwango

Test build will be available here.

@stephanosio stephanosio reopened this Mar 24, 2022
@stephanosio
Copy link
Member Author

Test build can be downloaded here:
https://github.com/zephyrproject-rtos/sdk-ng/actions/runs/2036127434

@microbuilder
Copy link
Member

Test build can be downloaded here: https://github.com/zephyrproject-rtos/sdk-ng/actions/runs/2036127434

Looks good to me. Thanks for the AArch64 build to make testing easier.

This commit pulls in the patches required to support the emulation of
the I2C devices (e.g. LSM303DLHC magnetometer) on the ARM targets.

Signed-off-by: Stephanos Ioannidis <root@stephanos.io>
@stephanosio
Copy link
Member Author

QEMU PR merged and updated commit SHA.

@stephanosio stephanosio removed the DNM DO NOT MERGE label Mar 25, 2022
@stephanosio stephanosio reopened this Mar 25, 2022
@stephanosio stephanosio merged commit 0115907 into zephyrproject-rtos:main Mar 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: QEMU Issues related to QEMU ci-linux-aarch64 Run CI for AArch64 Linux host (for PR) ci-linux-x86_64 Run CI for x86-64 Linux host (for PR)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants