Add a test case for k_queue_unique_append() API.
Signed-off-by: Lixin Guo <lixinx.guo@intel.com>

MISRA C:2012 Rule 10.2 (Expressions of essentially character type
shall not be used inappropriately in addition and subtraction
operations.)

Use the compliant form char - integer instead of the non-compliant
one integer - char.
Signed-off-by: Abramo Bagnara <abramo.bagnara@bugseng.com>

MISRA C:2012 Rule 10.5 (The value of an expression should not be cast
to an inappropriate essential type.)

Avoid integer-to-Boolean and Boolean-to-integer type casts.
Signed-off-by: Abramo Bagnara <abramo.bagnara@bugseng.com>

Do not load empty files through the ELF parser and raise exception when
magic number of ELF is not matched.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Make sure we have the latest actions are available and that the PR is
rebased on top of latest branch HEAD.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

We now use GH actions on this branch, so remove buildkite related files.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

MISRA C:2012 Rule 14.4 (The controlling expression of an if statement
and the controlling expression of an iteration-statement shall have
essentially Boolean type.)

Use `do { ... } while (false)' instead of `do { ... } while (0)'.
Use comparisons with zero instead of implicitly testing integers.
Use comparisons with NULL instead of implicitly testing pointers.
Use comparisons with NUL instead of implicitly testing plain chars.
Use `bool' instead of `int' to represent Boolean values.
Use `while (true)' instead of `while (1)' to express infinite loops.
Signed-off-by: Abramo Bagnara <abramo.bagnara@bugseng.com>

Fix cron for twister action and fix target branch.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

This action replaces current buildkite workflow and uses github actions
to build and run tests in the zephyr tree using twister. The main
differences to current builtkite workflow:

- the action handles all 3 events: pull requests, push and schedule

- the action determines size of matrix (number of build hosts) based on
  the change with a minimum of 1 builder. If more tests are built/run
  due to changes to boards or tests/samples, the matrix size is
  increased. This will avoid timeouts when running over capacity due to
  board/test changes.

- We use ccache and store cache files on amazon S3 for more flexibility

- Results are collected per build host and merged in the final step and
  failures are posted into github action check runs.

- It runs on more powerful instances that can handle more load.
  Currently we have 10 build hosts per run (that can increase depending
  on number of tests run) and can deliver results within 1 hour.

- the action can deal with non code changes and will not allocate more
  than required to deal with changes to documentation and other files
  that do not require running twister

The goal long-term is better integrate this workflow with other actions
and not run unncessarily if other workflows have failed, for example, if
commit message is bogus, we should stop at that check, to avoid wasting
resources given that the commit message will have to be fixed anyways
which would later trigger another run on the same code.

Currently there is 1 open issue with this action related to a github
workflow bug where the final results are not posted to the same workflow
and might appear under other workflows. Github is working on this bug.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Add branch name to the ccache key to avoid cache contamination from old
branches.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Remove actions not relevant to a branch.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Many tests and CI activties are being missed by excluding tests
mistakingly when running twister.
This is visibile when you change one or more tests in kernel/ for
example, twister does not run those tests that have changed at all and
marking the PR as tested and ready to be merged.

Temporary fix for #40235.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Otherwise reporting is skipped and failures are not recorded.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Use ccache to speed up builds.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Retry west when update fails and use update.narrow configuration.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Add missing ")".
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Avoid rebasing and instead use the commit range. This avoids issues with
trees having intermediate rebase data after a reboot (due to
cancellation).
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Run code coverage reporting on main zephyr repo only.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

To avoid conflicts in reporting.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Use released version instead of master.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Namespace job names and retry west update if something goes wrong the
first time.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Rename to make action file name obvious referring to bluetooth, rather
than the tool used.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Misc fixes including:
- unique job names
- Change description to mention Bluetooth
- Retry west update
- Use latest unit test publication action
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

No need to run on weekends, nothing much happens, so lets save some
resources.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Fix a minor typo in action and always set variable controlling reports.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Configure git for rebase by setting user name, email.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

- use zephyr runner
- reduce number of builders and adapt matrix to be platform based
- check for changed files and optimize run accordingly, should reduce
  build times depending on what has changed
- If no source has changed, skip twister completely.
Signed-off-by: Anas Nashif <anas.nashif@intel.com>

Add a code coverage collection action that triggers based on a schedule
on the main branch and posts results to

 https://app.codecov.io/gh/zephyrproject-rtos/zephyr

Signed-off-by: Anas Nashif <anas.nashif@intel.com>

MISRA C:2012 Rule 9.3 (Arrays shall not be partially initialized.)

Systematically use `{0}' to specify full 0 initialization
(not `{}', not `{0U}').
Signed-off-by: Abramo Bagnara <abramo.bagnara@bugseng.com>

MISRA C:2012 Rule 4.1 (Octal and hexadecimal escape sequences shall be
terminated.)

Use string literal concatenation to properly terminate hexadecimal
escape sequences.
Signed-off-by: Abramo Bagnara <abramo.bagnara@bugseng.com>

MISRA C:2012 Rule 8.2 (Function types shall be in prototype form with
named parameters.)

Added missing parameter names.
Signed-off-by: Abramo Bagnara <abramo.bagnara@bugseng.com>

MISRA C:2012 Rule 21.13 (Any value passed to a function in <ctype.h>
shall be representable as an unsigned char or be the value EOF).

Functions in <ctype.h> have undefined behavior if they are called with
any other value. Callers affected by this change are not prepared to
handle EOF anyway. The addition of these casts avoids the issue
and does not result in any performance penalty.
Signed-off-by: Abramo Bagnara <abramo.bagnara@bugseng.com>

MISRA C:2012 Rule 7.2 (A `u' or `U' suffix shall be applied to all
integer constants that are represented in an unsigned type)

Added missing `U' suffixes in constants that are involved in the
analyzed build, plus a few more not to introduce inconsistencies
with respect to nearby constants that are either unused in the
build (but implicitly unsigned) or are used and are immediately
converted to unsigned.
Signed-off-by: Abramo Bagnara <abramo.bagnara@bugseng.com>

This commit is fixing the error on clock control for the AHB3
in the stm32_clock_control_off function.
Signed-off-by: Francois Ramu <francois.ramu@st.com>

When user only use pb-gatt provisioning, which unable to
send out connectable advertising, due to adv thread not started.
Signed-off-by: Lingao Meng <menglingao@xiaomi.com>

After commit eeb15aa3

 ("timer: hpet: enable 64 bit mode for
better usages") was applied, main thread stack usage on qemu_x86
platform increased from 984 to 1040 bytes.

Default stack size, which is 1024, is no longer enough. Change that to
1536 to make sure this sample runs correctly on qemu_x86.
Signed-off-by: Marcin Niestroj <m.niestroj@emb.dev>

The OpenThread stack uses uint32_t to calculate expiry time for
alarms, while comparing to zephyr's uint64_t uptime.

This commit fixes broken milliseconds alarms after ~49.7 days of
uptime.

Fixes #39704
Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>

There was a brief (but seen in practice on real apps on real
hardware!) race with the switch-based z_swap() implementation.  The
thread return value was being initialized to -EAGAIN after the
enclosing lock had been released.  But that lock is supposed to be
atomic with the thread suspend.

This opened a window for another racing thread to come by and "wake
up" our pending thread (which is fine on its own), set its return
value (e.g. to 0 for success) and then have that value clobbered by
the thread continuing to suspend itself outside the lock.

Melodramatic aside: I continue to hate this
arch_thread_return_value_set() API; it needs to die.  At best it's a
mild optimization on a handful of architectures (e.g. x86 implements
it by writing to the EAX register save slot in the context block).
Asynchronous APIs are almost always worse than synchronous ones, and
in this case it's an async operation that races against literal
context switch code that can't use traditional locking strategies.

Fixes #39575
Signed-off-by: Andy Ross <andrew.j.ross@intel.com>

There were some leftovers in logging after attempt to use
logging as tracing backend. Removing all references since it
lead to test compilation failures.
Signed-off-by: Krzysztof Chruscinski <krzysztof.chruscinski@nordicsemi.no>

The commit updates mcumgr revision to include backport of
   345caab img_mgmt: fix callback parameter values
           (backport-upstream-137-to-v2.7-branch)
Signed-off-by: Dominik Ermel <dominik.ermel@nordicsemi.no>