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

[BUG] nuttx/ioexpander/mcp23x08.h is missing #15592

Closed
1 task done
paolovolpi opened this issue Jan 17, 2025 · 3 comments · Fixed by #15612
Closed
1 task done

[BUG] nuttx/ioexpander/mcp23x08.h is missing #15592

paolovolpi opened this issue Jan 17, 2025 · 3 comments · Fixed by #15612
Labels
Arch: arm Issues related to ARM (32-bit) architecture Area: Specific Peripheral Specific Peripheral issues OS: Linux Issues related to Linux (building system, etc) Type: Bug Something isn't working

Comments

@paolovolpi
Copy link
Contributor

Description / Steps to reproduce the issue

Enable DeviceDrivers/IO Expander Gpio Support/MCP23008
make -j
got the following compilation error:
CC: environ/env_getenv.c In file included from ioexpander/mcp23x08.c:38:
ioexpander/mcp23x08.h:37:10: fatal error: nuttx/ioexpander/mcp23x08.h: No such file or directory
37 | #include <nuttx/ioexpander/mcp23x08.h>

On which OS does this issue occur?

[OS: Linux]

What is the version of your OS?

Ubuntu 24.04

NuttX Version

master

Issue Architecture

[Arch: arm]

Issue Area

[Area: Specific Peripheral]

Verification

  • I have verified before submitting the report.
@paolovolpi paolovolpi added the Type: Bug Something isn't working label Jan 17, 2025
@github-actions github-actions bot added Arch: arm Issues related to ARM (32-bit) architecture Area: Specific Peripheral Specific Peripheral issues OS: Linux Issues related to Linux (building system, etc) labels Jan 17, 2025
@xiaoxiang781216
Copy link
Contributor

xiaoxiang781216 commented Jan 17, 2025

@paolovolpi
Copy link
Contributor Author

But it's not the required header, as an example take a look to mcp23x17, there is an header under drivers/ioexpander and another under include/nuttx/ioexpander.

@acassis
Copy link
Contributor

acassis commented Jan 17, 2025

@paolovolpi good finding, actually it was missing in my PR: 4b67e23

Could you please use mcp23x17.h as base and create the mcp23x08.h? If you need help, please ping me or we can discuss it at Discord: https://discord.gg/YaqhJjHF95

paolovolpi added a commit to paolovolpi/nuttx that referenced this issue Jan 20, 2025
paolovolpi added a commit to paolovolpi/nuttx that referenced this issue Jan 20, 2025
paolovolpi added a commit to paolovolpi/nuttx that referenced this issue Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arch: arm Issues related to ARM (32-bit) architecture Area: Specific Peripheral Specific Peripheral issues OS: Linux Issues related to Linux (building system, etc) Type: Bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants