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

zephyr: boards: fix nrf53 downstream #22

Open
wants to merge 33 commits into
base: master
Choose a base branch
from

Conversation

AyturkDuzen
Copy link
Member

Pass '-DSNIPPET=bt-ll-sw-split' for building zll downstream.

sjanc and others added 30 commits November 28, 2024 09:31
Added venv and .venv which are common names for
python virtual environments to the gitignore.

Signed-off-by: Emil Gydesen <[email protected]>
Added .vscode to gitignore so that local editor settings
are not pushed to the repo.

Signed-off-by: Emil Gydesen <[email protected]>
The tmp directory is creating during manual testing and
should never be pushed.

Signed-off-by: Emil Gydesen <[email protected]>
Fix following:
autopts/wid/tmap.py:718: SyntaxWarning: invalid escape sequence '\d'
This was due to bug in Qualification Workspace.
The test also requires discovery and was failing due to this.
Re-arranged the list of tests to be in alphabetical order to
make it easier to follow.

Signed-off-by: Emil Gydesen <[email protected]>
Redefine the btp service values from decimal to
hexidecimal, so they are aligned with opcodes, indexes,
etc.

Signed-off-by: Emil Gydesen <[email protected]>
Log BTP headers as hex instead of decimal so it is
easier to compare to the defined values in defs.py

Since the headers are using a namedtuple instead of
a class, it is not straigh forward to just update the
str or repr functions of the class.
Since printing a namedtuple uses the repr of the class,
a new function, repr_hdr, was added to format the header
as a string as hex.

Signed-off-by: Emil Gydesen <[email protected]>
The following tests require 2 CCIDs, and are failing
if the device does not have have 2 CCIDs registered:

CAP/INI/BST/BV-07-C
CAP/INI/BST/BV-10-C

CAP/INI/UST/BV-13-C
CAP/INI/UST/BV-14-C
CAP/INI/UST/BV-27-C
CAP/INI/UST/BV-28-C
CAP/INI/UST/BV-31-C
CAP/INI/UST/BV-36-C
CAP/INI/UST/BV-37-C

CAP/INI/UTB/BV-04-C

Signed-off-by: Emil Gydesen <[email protected]>
Make this consistent with rest of opcodes.
The Request ID 147314 applies to nearly all tests that
uses multiple lower testers, as it affects how IRKs are
handled by PTS.

Signed-off-by: Emil Gydesen <[email protected]>
If a PTS dongle has been corrupted, its stack has to be reinitialized.
The PTS apparently can do this by calling GetPTSBluetoothAddress()
before RunTestCase(). The dongle verification takes less than 100ms
on a real-hardware machine, but for VM it might take longer. Let's
add an option --dongle_init_delay that allows to customize this time.
It happens rarely, but sometimes a socket port does not get released
in time and is not usable for a while.

  File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 653, in run_test
    _run_test(config)
  File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 644, in _run_test
    srv_process, bot_process = _restart_processes(config)
  File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 582, in _restart_processes
    return _start_processes(config, checkout_repos=False)
  File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 557, in _start_processes
    start_remote_autoptsserver(config, checkout_repos)
  File "/home/codecoup/workspace/autopts/auto-pts-cron/tools/cron/common.py", line 474, in start_remote_autoptsserver
    log(client.open_process(config['server_start_cmd'],
  File "/usr/lib/python3.10/xmlrpc/client.py", line 1122, in __call__
    return self.__send(self.__name, args)
    self.sock = self._create_connection(
(...)
  File "/usr/lib/python3.10/socket.py", line 833, in create_connection
    sock.connect(sa)
OSError: [Errno 113] No route to host
When running autoptsclient-zephyr.py --help, traceback was printed
in terminal. This was caused by logging.exception(e) which should not
be called when exiting with status 0.
There is no point in repeating test case if MISSING WID ERROR is returned.
These tests (and possibly more) makes the PTS dongle crash.
Tested with both SDC and ZLL with the Zephyr host, using
sha 67980643d799f76aff93118346a8dabfb472a642.

Signed-off-by: Emil Gydesen <[email protected]>
Modified the hdl_wid_312 handler to call cap_unicast_audio_stop
with release = False, so that it performs a disable (and stop)
without release.

Why PTS wants us to disable first is unclear, as the test spec
states to release, which would be handled by hdl_wid_309.

This affects tests where PTS asks us to stop/disable the
streams (hdl_wid_312) before releasing it (hdl_wid_309),
such as CAP/INI/UST/BV-40-C.

Signed-off-by: Emil Gydesen <[email protected]>
Add sysbuild option for 54h20
Pass --no-sysbuild for 5340 same as old behaviour

Signed-off-by: Aytürk Düzen <[email protected]>
rename hci_ipc configuration for generic multicore usage.

related change in tester:
zephyrproject-rtos/zephyr#81755

Signed-off-by: Aytürk Düzen <[email protected]>
Pass '-DSNIPPET=bt-ll-sw-split' for building zll downstream.

Signed-off-by: Aytürk Düzen <[email protected]>
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.

6 participants