Skip to content

arch/arm/samv7: fix incorrect Kconfig SAMV7_EMAC0_RMII dependency#17832

Merged
simbit18 merged 1 commit intoapache:masterfrom
michallenc:samv7-kconfig-rmii-fix
Jan 15, 2026
Merged

arch/arm/samv7: fix incorrect Kconfig SAMV7_EMAC0_RMII dependency#17832
simbit18 merged 1 commit intoapache:masterfrom
michallenc:samv7-kconfig-rmii-fix

Conversation

@michallenc
Copy link
Contributor

Summary

SAMV7_EMAC0_RMII should not depend on ARCH_CHIP_SAM4E as this is a completely different chip. This is likely a relict from copying the code base.

Impact

None as ARCH_CHIP_SAM4E can't be set anyway if SAMv7 chip is used.

@github-actions github-actions bot added Arch: arm Issues related to ARM (32-bit) architecture Size: XS The size of the change in this PR is very small labels Jan 12, 2026
@simbit18
Copy link
Contributor

simbit18 commented Jan 12, 2026

@raiden00pl @szafonimateusz-mi problems on the step

# get NTFC test cases

Reading package lists...
Building dependency tree...
Reading state information...
The following additional packages will be installed:
  python3-pip-whl python3-setuptools-whl python3.10-venv
The following NEW packages will be installed:
  python3-pip-whl python3-setuptools-whl python3-venv python3.10-venv
0 upgraded, 4 newly installed, 0 to remove and 30 not upgraded.
Need to get 2479 kB of archives.
After this operation, 2897 kB of additional disk space will be used.
Err:1 http://security.ubuntu.com/ubuntu jammy-security/universe amd64 python3.10-venv amd64 3.10.12-1~22.04.13
  404  Not Found [IP: 91.189.91.81 80]
Get:2 http://archive.ubuntu.com/ubuntu jammy-updates/universe amd64 python3-pip-whl all 22.0.2+dfsg-1ubuntu0.7 [1683 kB]
Get:3 http://archive.ubuntu.com/ubuntu jammy-updates/universe amd64 python3-setuptools-whl all 59.6.0-1.2ubuntu0.22.04.3 [789 kB]
Get:4 http://archive.ubuntu.com/ubuntu jammy-updates/universe amd64 python3-venv amd64 3.10.6-1~22.04.1 [1042 B]
E: Failed to fetch http://security.ubuntu.com/ubuntu/pool/universe/p/python3.10/python3.10-venv_3.10.12-1%7e22.04.13_amd64.deb  404  Not Found [IP: 91.189.91.81 80]
E: Unable to fetch some archives, maybe run apt-get update or try with --fix-missing?
Fetched 2473 kB in 1s (1870 kB/s)

here too
#17831
https://github.com/apache/nuttx/actions/runs/20913722946/job/60086900950?pr=17831#logs

@raiden00pl
Copy link
Member

@simbit18 It seems that github actions doesn't work well with ubuntu mirrors, I'll try another approach to install venv

@jerpelea jerpelea changed the title arch/arm/src/samv7/Kconfig: fix incorrect SAMV7_EMAC0_RMII dependency arch/arm/samv7: fix incorrect Kconfig SAMV7_EMAC0_RMII dependency Jan 12, 2026
@raiden00pl
Copy link
Member

@simbit18 CI fixed here: #17834

@simbit18
Copy link
Contributor

@michallenc please rebase the latest master

SAMV7_EMAC0_RMII should not depend on ARCH_CHIP_SAM4E as this is
a completely different chip. This is likely a relict from copying
the code base.

Signed-off-by: Michal Lenc <michallenc@seznam.cz>
@michallenc michallenc force-pushed the samv7-kconfig-rmii-fix branch from e0368d6 to f1a54a3 Compare January 15, 2026 12:43
@michallenc
Copy link
Contributor Author

Sorry for the delay, rebased

@simbit18 simbit18 merged commit d53cd33 into apache:master Jan 15, 2026
26 checks passed
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 Size: XS The size of the change in this PR is very small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants