Skip to content

boards/make: use shared makefiles for serial port selection#8484

Closed
haukepetersen wants to merge 4 commits intoRIOT-OS:masterfrom
haukepetersen:opt_make_sharedserialmakefiles
Closed

boards/make: use shared makefiles for serial port selection#8484
haukepetersen wants to merge 4 commits intoRIOT-OS:masterfrom
haukepetersen:opt_make_sharedserialmakefiles

Conversation

@haukepetersen
Copy link
Copy Markdown
Contributor

Contribution description

This PR introduces two shared convenience makefiles for the most common serial configurations. Though I adapted the most obvious boards, there are others that can potentially also benefit from the new makefiles. I suggest to adapt other boards on a 'when we get to it' basis.

Issues/PRs references

#8475 gave me the idea for this...

@haukepetersen haukepetersen added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation Area: build system Area: Build system CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: boards Area: Board ports labels Jan 30, 2018
@jnohlgard
Copy link
Copy Markdown
Member

In my opinion this is only moving the problem of wrong TTY selection from one place to another. I would prefer using #7695 to provide a centralized USB UART configuration instead of these static configurations.

@haukepetersen
Copy link
Copy Markdown
Contributor Author

Oh, I did not know about #7695. That is indeed the better approach, I like it!

So closing this PR in favor of #7695.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: boards Area: Board ports Area: build system Area: Build system CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants