After the error code is checked to be zero, it is possible to return 0
explicitly to help with readability. Also, when available, forward the
return code from the failing function instead of a locally chosen error
code like -EIO.
Signed-off-by: Josuah Demangeon <me@josuah.net>
While applying the format, the pixel format and drv_data->fmt were set
immediately, and the resolution was set only if it had a matching
entry on the "caps".
This commit makes sure the requested format matches the caps before
applying the format as well as drv_data->fmt. This does not guards
against partial failure (i.e. only pixelformat set and not
resolution).
Signed-off-by: Josuah Demangeon <me@josuah.net>
The following devices enable TRIGGER in a way that is different from the
template definition, so add these settings to sensos_no_default.conf.
- bosch,bmc150
- bosch,bmm150
- bosch,bmp150
- ti,ina230
- we,wsen-itds
- lm77
- st,lsm9ds0-gyro
- ti,tmag5170
Signed-off-by: TOKITA Hiroshi <tokita.hiroshi@gmail.com>
This commit adds C++20 onwards support for the
PINCTRL_DT_STATE_INIT macro. Since C++20, support was
added for designated initializers but with the restriction that
they're ordered. PINCTRL_DT_STATE_INIT initializes a
pinctrl_state struct but the current initializer list is un-ordered,
this PR resolves that for >= C++20 compat
Signed-off-by: Chandler Keep <chandlersamkeep@gmail.com>
Since fc007eeef5 TLS sockets support
timeout based on CONFIG_NET_SOCKETS_CONNECT_TIMEOUT value. The TLS test
suite reduced this config value to speed up the tests, however it turns
out the 100ms timeout is too low to finalize the handshake on slower
platforms (nrf52840 specifically). Therefore increase the timeout value
to 200ms to prevent test failures on those.
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
All the board.cmake contains more or less the same data. This patch
applies the cosmetics changes to align the structure of all these files.
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
It is a good practice to add a comment after #endif when the condition
is not obvious. However, when the condition is well known, "Don't Repeat
Yourself" rule applies and no comment should be added.
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Most of the Kconfig files in soc/silabs start with a comment describing
the chip. This comment is redundant with the help message in associated
to the chip. Let's don't repeat ourselves.
This patch also ensure the Kconfig help message the full name the chip.
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Most of the Kconfig files in boards/silabs start with a comment
describing the board. This comment is redundant with the content of
board.yaml. Let's don't repeat ourselves.
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Usually commercial names mention the chipset included on the board.
"Thunderboard Sense 2" is the only exception in the Silabs boards.
This information is probably useful, so add it to the description of
this board.
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Yaml files allows the user to find the internal name for their board
using the "name" field.
In addition of the commercial names/references, Silabs boards have
internal board references. These references are not advertised, but can
be found printed on (every) boards.
It makes sense to add this reference to the Yaml file in case a user
would have the idea to grep it.
This patch also fixes some commercial names of very old boards.
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
The entry listed in the docs does not exist. The correct entry
(dt_compat_enabled) is present.
Fixes#79081
Signed-off-by: Florian Grandel <fgrandel@code-for-humans.de>
The security check case statements were in frequency band
setting checks. This is totally wrong and will cause compiler
warnings. Moving the checks to correct function.
Signed-off-by: Jukka Rissanen <jukka.rissanen@nordicsemi.no>
Add flpr and ppr core to the sample.
Since there are only 2 UARTs available on DK, local UART configuration
is using only cpuapp and cpurad.
Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
Add interrupt locking during test to ensure that main thread is not
interrupts which would impact the test.
Add longer sleep time between tests to ensure that all logs are
processed on time and not dropped.
Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
Currently we can assume that flpr and ppr cores are always owned by
cpuapp. In that case, cpuapp (which processes ETR buffer content) has
access to the memory where logging strings from ppr and flpr are
located. Those strings do not need to be appended to the message
which is written to the STM (in case of standalone configuration).
This approach reduced logging time, e.g. logging a message with
one argument takes 1.8us on cpuapp and 6.3us on cpurad.
This setting applies only to the case when cpuapp is the owner
of those co-oprocessors and shall be updated if that becomes
configurable but currently there are no means to detect core
owner.
Signed-off-by: Krzysztof Chruściński <krzysztof.chruscinski@nordicsemi.no>
The architecture description was quite out of date in some parts,
specially those which changed with the introduction of the native
simulator.
Let's update it.
Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
Fix trailing `\\r\\n` (escaped CR/LF) didn't cut off because of rstrip()
removed by #58338, so the CR/LF suffix was never found as the actual line
end was `\\r\\n\n`.
Add ANSI code sequence to `test_handlers` Twister unit test.
Signed-off-by: Dmitrii Golovanov <dmitrii.golovanov@intel.com>
This fixes the following warnings:
> unit address and first address in 'reg' (0xa080000) don't match for
> /mram@a080400
Signed-off-by: Reto Schneider <reto.schneider@husqvarnagroup.com>
Some applications may require the SPI NOR driver to be initialized
earlier. This commit enables the user to change the default
initialization priority.
Signed-off-by: Vidar Berg <vidar.berg@nordicsemi.no>
-add nucleo_u083rc and stm32u083c_dk overlays files
for testbench purpose.
- add platform_exclude property to skip test on boards
with little ram size.
Signed-off-by: Fabrice DJIATSA <fabrice.djiatsa-ext@st.com>
tests: drivers: adc: adc_api: update adc_api test
-enable adc in nucleo_u031r8, stm32u083c_dk and nucleo_u083rc
- add adc as new supported interface in docs
- update yaml files
Signed-off-by: Fabrice DJIATSA <fabrice.djiatsa-ext@st.com>
Because timing of the pause/resume for peripheral/sync is based on
central/broadcaster active clock, the sleep-clock jitter shall not be
added.
Signed-off-by: Morten Priess <mtpr@oticon.com>
- Add next pointer to node_rx_iso_meta for pre-transmission linked
list (does not increase memory usage as union is larger).
- Fix broadcaster not setting PTC with test configuration
- Refactor stopping of sync ISO tickers to separate function
Signed-off-by: Morten Priess <mtpr@oticon.com>
Share pause/resume functionality by moving resume ticker handling to
ull_iso. Rename interface function to ull_conn_iso_resume_ticker_start,
and provide handles instead of instance pointer.
Signed-off-by: Morten Priess <mtpr@oticon.com>
Currently NXP MPU depends on "cpu" command in uboot to kick Zephyr to
Cortex-A Core, added U-Boot version which support "cpu" command in the
board documents.
Refined boot commands.
Signed-off-by: Jiafei Pan <Jiafei.Pan@nxp.com>
Adds a new command that will return information on slots
themselves, listing the slot sizes, without any internal
information about the slot data. Will also return the maximum size
of an image for a set of slots if built using sysbuild.
Signed-off-by: Jamie McCrae <jamie.mccrae@nordicsemi.no>
In order to be able to ifdef based on `CONFIG_RUST`, the config must be
defined in the main repo (because the rust module is optional). Add
this as a placeholder. If the module is not availabe, the symbol will
be present, and always disabled. The module will provide a full
definition for the Kconfig.
Signed-off-by: David Brown <david.brown@linaro.org>
This is a little complicated. We currently just jump directly to rust's
panic, so we aren't likely to actually need these unwind tables. But, they
would be needed if we ever reall paniced. As of the time of this change,
the tables seem to just be 24 bytes of ROM space.
Signed-off-by: David Brown <david.brown@linaro.org>
Although the Cmake rules to build Rust applications keeps the target
directory inside of the build directory, some IDE tools may generate a
target directory while editing the code. Ignore these so they never get
checked in.
Signed-off-by: David Brown <david.brown@linaro.org>
Josuah Demangeon