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

build: run compliance checks #58

Closed
wants to merge 14 commits into from

Conversation

cfriedt
Copy link
Owner

@cfriedt cfriedt commented Dec 29, 2020

This change ensures that code changes satisfy a number of
criteria before being considered for submission.

The compliance checks are similar to those in Zephyr.

Fixes #57

cfriedt and others added 13 commits December 28, 2020 21:42
* Added Documentation and instructions in reST format
* Deprecated older material and moved to doc/old.md
* Added CI via GitHub Actions
* Added zephyr-aligned-allocators.patch since that is not yet merged upstream
* Added zephyr-dynamic-pthread-stacks.patch since that is not yet merged upstream
* Added zephyr-gpio-emul.patch since that is not yet merged upstream
* Added zephyr-gpio-get-direction.patch
* Added zephyr-sys-byteorder-posix.patch

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
This change adds cfriedt/manifesto as a submodule of
greybus-for-zephyr at the path scripts/manifesto/.

It removes a lot of unecessary code that was used for
assembling the manifest at runtime.

Rather than using C code and consuming microcontroller RAM
and resources at runtime, we can now link in the Greybus
Manifest into ROM as a static const unsigned char[].

Fixes #30

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
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>
This change unifies logging under all of Greybus and brings
out control of the log level through Kconfig.

It might not be best to use a single log level for all Greybus
components (i.e. eventually they could each have their own
independent Kconfig setting) but for now it's better to tie them
all into a single Kconfig variable than to hard-code them.

Fixes #36

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
Update devicetree macros / eliminate warnings.

It also required that we define a label in each and
every greybus node in devicetree.

Fixes #39

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
Kind of messing combinding git submodules and west, but
I don't really want to add a manifest for zephyr in
the manifesto repository.

Also fix another typo in the readme about applying patches.

Fixes #37

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
The only remaining warning that existed when building Greybus for Zephyr (in #34) was fixed by cfriedt/manifesto#2.

Fixes #42

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
This change adds builds for the cc1352r_sensortag for both the
net and uart sample applications in addition to builds for the
cc1352r1_launchxl for both the net and uart sample applications.

Fixes #5
Fixes #44

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
This change allows the gpio test to work in Qemu, which is
a necessary but separable step to get TLS working because
`native_posix` / Linux does not support Zephyr's TLS
extensions for the standard sockets API.

Fixes #48

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
Fixes #12
Changes as per discussion in  https://github.com/cfriedt/zephyr/pull/5

Currently priorities are assigned in the following manner:

CONFIG_GREYBUS_SERVICE_PRIORITY N-5
CONFIG_GREYBUS_STRING_PRIORITY N-4
CONFIG_GREYBUS_INTERFACE_PRIORITY N-3
CONFIG_GREYBUS_BUNDLE_PRIORITY N-2
CONFIG_GREYBUS_CPORT_PRIORITY N-1

where N = CONFIG_APPLICATION_INIT_PRIORITY

Tested GPIO, I2C and SPI on CC1352R SensorTag

Signed-off-by: Vaishnav M A <vaishnav@beagleboard.org>
This chnage adds TLS support to Greybus as well as a default
method of storing certificates (building them into the
application).

Fixes #25
Fixes #28

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
This change removes the gpio-emul patch because it has been
merged upstream in zephyrproject/zephyr#26484.

Fixes #6

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
This change removes the aligned allocators patch because it
has been merged upstream in zephyrproject/zephyr#30363.

Fixes #9

Signed-off-by: Christopher Friedt <chrisfriedt@gmail.com>
@cfriedt cfriedt self-assigned this Dec 29, 2020
@cfriedt cfriedt force-pushed the issue/57/build-run-compliance-checks branch 12 times, most recently from 23b70c8 to d392ee8 Compare December 29, 2020 04:37
This change ensures that code changes satisfy a number of
criteria before being considered for submission.

The compliance checks are similar to those in Zephyr.

Fixes #57

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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

build: run compliance checks
2 participants