There was a Kconfig option defined to limit the number of additional DNS
queries sent for aliases received in CNAME records (to avoid potential
query loops), however it was not implemented. This commit implements the
feature - the resolver will now only send up to
CONFIG_DNS_RESOLVER_ADDITIONAL_QUERIES follow-up queries after
receiving CNAME record with an alias w/o any IP addresses.
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
In case CNAME record is received with no IP addresses, DNS resolver will
attempt to re-send the query for the name indicated by the CNAME record.
The logic for scheduling the new query was inconsistent though:
dispatcher_cb() assumes that it'll reuse the query context with the same
query id, while dns_read() cancelled the query, which shouldn't really
happen, as that would cause an error to be reported to the application
via callback.
Fix that by skipping the query cancel in case of DNS_EAI_AGAIN result.
The query context will be properly reused then, and freed either when
reply for another query arrives, or the query times out.
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Don't report CNAME records via application callback (as it used to be
done). They don't carry any data in the info struct and are only used
internally to redirect DNS queries.
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Disable HW acceleration to fix CI failures caused by -Werror triggered
by prototype mismatches in the crypto accelerator path between TF-M and
Mbed TLS.
Signed-off-by: Sudan Landge <sudan.landge@arm.com>
TF-M documentation states that code sharing on Musca-B1 is only
supported with software crypto and requires crypto hardware acceleration
to be disabled. So, force CRYPTO_HW_ACCELERATOR=OFF for the Musca-B1
non-secure TF-M build to match this requirement.
This also fixes CI failures caused by -Werror triggered by prototype
mismatches in the crypto accelerator path between TF-M and Mbed TLS.
Signed-off-by: Sudan Landge <sudan.landge@arm.com>
Flash layout of musca_b1 and musca_s1 is updated as recommended by
mcuboot. This also fixes the below cmake build failures reported in
Zephyr ci.
```
CMake Error at
zephyrproject/zephyr/cmake/modules/extensions.cmake:3877
(message):
required nodelabel not found: slot0_partition
Call Stack (most recent call first):
zephyrproject/zephyr/modules/trusted-firmware-m/CMakeLists.txt:489
(dt_nodelabel)
```
Note that the flash address of musca_b1 board is fixed from 0x0A07_0000
to 0x0A08_0000 as this was missed after the layout was changed in TF-M
in e1570bd143ef4843a531b57f77f0c2814b075dd9
Signed-off-by: Sudan Landge <sudan.landge@arm.com>
Added ifdef guard (CONFIG_GETOPT_LONG) around the functions in
getopt_shim.c that requires getopt_long implementation.
Signed-off-by: Magne Værnes <magne.varnes@nordicsemi.no>
Tested with the command mentioned in index.rst
Compile testing of the overlay file is done via
the dht_polling sample.
Product photo from https://learn.adafruit.com/assets/89387
Signed-off-by: Jonas Berg <jonas.s.t.berg@gmail.com>
When frame_clk_freq is set to 0, the I2S API specifies that the
stream should transition to NOT_READY state (i.e., unconfigure).
In this case, other config fields like mem_slab and block_size
are not used by the driver.
Skip the mem_slab and block_size validation in the syscall handler
when frame_clk_freq is 0 to match the driver behavior and avoid
rejecting valid unconfigure requests due to uninitialized fields.
Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
Initialize the i2s_config struct to zero before use in the
test_i2s_state_not_ready_neg test case.
When frame_clk_freq is set to 0 to unconfigure a stream, other
fields in the struct were left uninitialized. On platforms with
userspace support, the syscall handler validates block_size against
the mem_slab block size before calling the driver.
Uninitialized garbage values could cause this validation to
fail unexpectedly.
Signed-off-by: Sylvio Alves <sylvio.alves@espressif.com>
Add clock_control_configure() call during initialization to properly
configure the LPSPI clock. The implementation gracefully handles
platforms that don't support clock configuration by checking for
-ENOTSUP and -ENOSYS return codes and continuing with default
settings. Real configuration errors are logged and propagated.
Signed-off-by: Albort Xue <yao.xue@nxp.com>
Convert the DUT class from a traditional class to a dataclass to enable
proper serialization and support for future multi-device testing with
pytest-harness.
Key changes:
- Migrated DUT class to use `dataclass` decorator with proper type hints
- Renamed `baud` property to `serial_baud` for consistency
- Updated hardware map schema to support both `baud` (legacy) and
`serial_baud` fields for backward compatibility
- Updated tests
Signed-off-by: Grzegorz Chwierut <grzegorz.chwierut@nordicsemi.no>
The ina2xx channel get helpers assume that channel descriptors are
valid when the driver exposes a given sensor channel.
Checking for NULL channel pointers after they are already
dereferenced does not provide any real safety and can hide
configuration errors.
Drop the late NULL checks and rely on the driver contract that
supported channels must be properly defined.
No functional change intended.
Signed-off-by: Gaetan Perrot <gaetan.perrot@spacecubics.com>
The INA2xx fetch path assumes valid channel descriptors for all
supported sensor channels.
NULL checks performed after channel data is accessed are ineffective
and misleading, as invalid channel definitions indicate a
configuration error rather than a runtime condition.
Remove the redundant checks and keep the channel handling consistent
with the driver expectations.
No functional change intended.
Signed-off-by: Gaetan Perrot <gaetan.perrot@spacecubics.com>
Select KERNEL_NO_LTO only when LTO is enabled. This ensures proper
handling when kernel code is placed in RAM.
Signed-off-by: Tim Lin <tim2.lin@ite.corp-partner.google.com>
Some SoCs require kernel code to be placed in RAM, which makes
link-time optimization (LTO) unsuitable for these files.
Disabling LTO allows the affected code to be linked as separate
objects and placed in specific memory regions.
Running kernel code from RAM can improve execution performance,
especially for timing-critical routines or context switch paths.
Signed-off-by: Tim Lin <tim2.lin@ite.corp-partner.google.com>
Do not add this folder to the include path when this component is not
enabled. As that creates noise and slows down builds.
Signed-off-by: Alberto Escolar Piedras <alberto.escolar.piedras@nordicsemi.no>
In case DPP reconfiguration test, if the dpp_akm has psk,
it will set WPA_KEY_MGMT_PSK | WPA_KEY_MGMT_PSK_SHA256 |
WPA_KEY_MGMT_FT_PSK to key_mgmt, then wifi status shows
security as "Unknown".
Signed-off-by: Gang Li <gang.li_1@nxp.com>
Change to use DT_HAS_NXP_SIM_UUID_ENABLED/DT_HAS_NXP_RCM_HWINFO_ENABLED
as the dependency condition for HWINFO_MCUX_SIM/HWINFO_MCUX_RCM to
enable the driver when the nxp,sim-uuid/nxp,rcm-hwinfo compatible
is present in the devicetree.
Signed-off-by: Lucien Zhao <lucien.zhao@nxp.com>
Add vendor-specific compatible strings to RCM and SIM nodes:
- "nxp,rcm-hwinfo" for Reset Control Module (hwinfo-reset functionality)
- "nxp,sim-uuid" for System Integration Module (hwinfo-UUID functionality)
Signed-off-by: Lucien Zhao <lucien.zhao@nxp.com>
These DTS uses absolute GPIO numbers for some
pins where relative pin numbers are required.
This causes crash on booth when these pins
are enabled. Follow on PR to #101857.
Signed-off-by: Peter Hoddie <peter@moddable.com>
When removing TOOLCHAIN_DISABLE_CLANG_WARNING, building the
settings.mgmt test with clang fails with:
tests/subsys/mgmt/mcumgr/settings_mgmt/src/main.c:87:22: error:
variable 'test_response_read_data_start' is not needed and will not be
emitted [-Werror,-Wunneeded-internal-declaration]
87 | static const uint8_t test_response_read_data_start[5] = {
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
tests/subsys/mgmt/mcumgr/settings_mgmt/src/main.c:91:22: error:
variable 'test_response_read_data_end' is not needed and will not be
emitted [-Werror,-Wunneeded-internal-declaration]
91 | static const uint8_t test_response_read_data_end[1] = {
Add additional asserts to use these variables in the test.
Signed-off-by: Tom Hughes <tomhughes@chromium.org>
Warning as error was:
.../drivers/display/display_rm67162.c: In function ‘rm67162_write_fb’:
.../drivers/display/display_rm67162.c:383:9: error: ‘wlen’ may be used
uninitialized in this function [-Werror=maybe-uninitialized]
383 | return wlen;
| ^~~~
cc1: all warnings being treated as errors
Signed-off-by: Stephan Linz <linz@li-pro.net>
The original implementation used a single global parameter structure for
GATT operations, preventing concurrent operations on different bearers. By
introducing a per-bearer context pool, each bearer can maintain its own
operation state, enabling simultaneous GATT operations.
Signed-off-by: Evgenii Kosenko <Evgenii.Kosenko@silabs.com>
Modify the description format of all SCMI-related bindings such that the
SCMI/SHMEM acronym is spelled out in between parentheses and not the other
way around. This will make the generated documentation more compact.
Signed-off-by: Laurentiu Mihalcea <laurentiu.mihalcea@nxp.com>
Update board YAML files to use 'pinctrl' instead of 'pin_ctrl'
in the supported features list for consistency with Zephyr
naming conventions.
Affected boards:
- cy8cproto_041tp
- kit_pse84_ai
- kit_pse84_eval
Signed-off-by: Braeden Lane <Braeden.Lane@infineon.com>
The debug code for dumping resolved addresses was modifying the addrinfo
res pointer (effectively setting it to NULL), causing crash later in the
sample. Fix this, by using a temporary pointer copy for iterating over
resolved addresses.
Signed-off-by: Robert Lubos <robert.lubos@nordicsemi.no>
Sort ble radio feature properties alphabetically according to
the coding standard.
Certain features on xg22 and xg28 depend on the specific SoC
selected, move the properties from the generic .dtsi file to
the SoC specific one.
Signed-off-by: Aksel Skauge Mellbye <aksel.mellbye@silabs.com>
Propagate the HAL error otherwise it may return success even
when it fails causing invalid entropy and compromising security.
Signed-off-by: Flavio Ceolin <flavio@hubblenetwork.com>
With -Hccm, the linker automatically moves stuff in RODATA
section into DATA section. Our current kobject related
scripts cannot accommodate this, resulting in space not
being reserved correctly. So for now, disable -Hccm
compiler option if userspace is enabled.
Signed-off-by: Daniel Leung <daniel.leung@intel.com>
As the board now supports TF-M with bl2, spport
from sample.tfm_ipc.no_bl2 is removed and
added in sample.tfm_ipc test.
Signed-off-by: Waqar Tahir <waqar.tahir@nxp.com>
- Updated partitions in non-secure device tree
files as tfm now supports Bl2.
- Flash_base address updated for ns-app as
tfm apps are now using bl2 by default.
Signed-off-by: Waqar Tahir <waqar.tahir@nxp.com>
Add new APIs to set operating mode 11n, 11ac and 11ax for SoftAP.
Currently, the 11n, 11ac and 11ax are enabled by default. Theses APIs
can be used to configure operating modes enable/disable for SoftAP.
Signed-off-by: Hui Bai <hui.bai@nxp.com>
