Skip to content

boards/jiminy-mega256rfr2: remove mentions in applications and board/common#12399

Merged
aabadie merged 2 commits intoRIOT-OS:masterfrom
cladmi:pr/board/jiminy-mega256rfr2/correctly_cleanup
Oct 8, 2019
Merged

boards/jiminy-mega256rfr2: remove mentions in applications and board/common#12399
aabadie merged 2 commits intoRIOT-OS:masterfrom
cladmi:pr/board/jiminy-mega256rfr2/correctly_cleanup

Conversation

@cladmi
Copy link
Contributor

@cladmi cladmi commented Oct 8, 2019

Contribution description

The board was removed in #12182 but specific handling or definitions were still referenced in the repository.

Remove the deprecated board from BOARD_BLACKLIST and BOARD_INSUFFICIENT_MEMORY in applications.

Remove special case for jiminy-mega256rfr2 in boards/common/arduino-atmega

Having the arduino and periph_pwm features is now required for all
boards using 'arduino-atmega' (as it was except for that board).

If this should change in the future, it should be defined either in each
arduino board, in another board common, or per CPU_MODEL.

Testing procedure

The board is now not referenced anymore except in release-notes and lost and found.

git grep jiminy-mega256rfr2
LOSTANDFOUND.md:### boards/jiminy-mega256rfr2 [232aed3e18118624b862d36bfec7cd1c21ca2d26]
release-notes.txt:    + jiminy-mega256rfr2

Issues/PRs references

Found while rebasing the cleanup to not do per board blacklisting of features.

Board was removed in "boards: Remove support for the Jiminy-Mega256RFR2" #12182

@fjmolinas fjmolinas added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation labels Oct 8, 2019
cladmi added 2 commits October 8, 2019 18:37
Remove the deprecated board from BOARD_BLACKLIST and BOARD_INSUFFICIENT_MEMORY.
The board is deprecated, no need to ignore this board anymore here.

Having the `arduino` and `periph_pwm` features is now required for all
boards using 'arduino-atmega' (as it was except for that board).

If this should change in the future, it should be defined either in each
arduino board, in another board common, or per CPU_MODEL.
@aabadie aabadie force-pushed the pr/board/jiminy-mega256rfr2/correctly_cleanup branch from 8da5bd7 to ec93d4c Compare October 8, 2019 16:38
Copy link
Contributor

@aabadie aabadie left a comment

Choose a reason for hiding this comment

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

ACK

Changes make sense. git grep confirms that the remaining occurrences are removed. I allowed myself to quickly fix the conflict with master (introduced by #12343) and force push to this PR.

@aabadie aabadie merged commit 3584f5a into RIOT-OS:master Oct 8, 2019
@cladmi
Copy link
Contributor Author

cladmi commented Oct 9, 2019

Thank you for the review and fixing the conflicts.

I noticed yesterday evening the conflicts in the BOARD_BLACKLIST and INSUFFICIENT_MEMORY but did not have time to do it directly.

@cladmi cladmi deleted the pr/board/jiminy-mega256rfr2/correctly_cleanup branch October 9, 2019 09:12
@kb2ma kb2ma added this to the Release 2019.10 milestone Oct 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants