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.

Sign up for GitHub

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

Jump to bottom

build: eliminate warnings #1

Closed
cfriedt opened this issue Dec 13, 2020 · 0 comments · Fixed by #33
Closed

build: eliminate warnings #1

cfriedt opened this issue Dec 13, 2020 · 0 comments · Fixed by #33
Assignees
@cfriedt
Labels
build enhancement New feature or request
Milestone
Alpha

Comments

@cfriedt
Copy link
Owner

cfriedt commented Dec 13, 2020 •
edited
Loading

Currently tests and samples build with a number of warnings.

We should compile with zero warnings.

A frequent warning has to do with const-ness of things at the device init stage.
@cfriedt cfriedt self-assigned this Dec 13, 2020
@cfriedt cfriedt added this to the Release milestone Dec 13, 2020
@cfriedt cfriedt changed the title Eliminate all Compiler Warnings build: eliminate warnings Dec 13, 2020
@cfriedt cfriedt added build enhancement New feature or request labels Dec 13, 2020
@cfriedt cfriedt mentioned this issue Dec 15, 2020
Merged
cfriedt added a commit that referenced this issue Dec 15, 2020
@cfriedt
build: eliminate warnings
9d5f486 
This change eliminates a number of compiler warnings.

The main gist was that `struct device *` was replaced
with `const struct device *` in a number of places.

Fixes #1

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
cfriedt added a commit that referenced this issue Dec 15, 2020
@cfriedt
build: eliminate warnings
c85b292 
This change eliminates a number of compiler warnings.

The main gist was that `struct device *` was replaced
with `const struct device *` in a number of places.

Fixes #1

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
@cfriedt cfriedt mentioned this issue Dec 16, 2020
Closed
cfriedt added a commit that referenced this issue Dec 29, 2020
@cfriedt
build: eliminate warnings
86f9510 
This change eliminates a number of compiler warnings.

The main gist was that `struct device *` was replaced
with `const struct device *` in a number of places.

Fixes #1

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
cfriedt added a commit that referenced this issue Dec 29, 2020
@cfriedt
build: eliminate warnings
33037a5 
This change eliminates a number of compiler warnings.

The main gist was that `struct device *` was replaced
with `const struct device *` in a number of places.

Fixes #1

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cfriedt cfriedt

Labels
build enhancement New feature or request
Projects
None yet
Milestone
Alpha
Development

Successfully merging a pull request may close this issue.

build: eliminate warnings cfriedt/greybus-for-zephyr
1 participant
@cfriedt