1f5d196d0SStephen Warren# Copyright (c) 2016, NVIDIA CORPORATION. All rights reserved. 2f5d196d0SStephen Warren# 3f5d196d0SStephen Warren# SPDX-License-Identifier: GPL-2.0 4f5d196d0SStephen Warren 5f5d196d0SStephen Warren# Test U-Boot's "dfu" command. The test starts DFU in U-Boot, waits for USB 6f5d196d0SStephen Warren# device enumeration on the host, executes dfu-util multiple times to test 7f5d196d0SStephen Warren# various transfer sizes, many of which trigger USB driver edge cases, and 8f5d196d0SStephen Warren# finally aborts the "dfu" command in U-Boot. 9f5d196d0SStephen Warren 10f5d196d0SStephen Warrenimport os 11f5d196d0SStephen Warrenimport os.path 12f5d196d0SStephen Warrenimport pytest 13f5d196d0SStephen Warrenimport u_boot_utils 14f5d196d0SStephen Warren 15e8debf39SStephen Warren""" 16f5d196d0SStephen WarrenNote: This test relies on: 17f5d196d0SStephen Warren 18f5d196d0SStephen Warrena) boardenv_* to contain configuration values to define which USB ports are 19f5d196d0SStephen Warrenavailable for testing. Without this, this test will be automatically skipped. 20f5d196d0SStephen WarrenFor example: 21f5d196d0SStephen Warren 22f5d196d0SStephen Warrenenv__usb_dev_ports = ( 23f5d196d0SStephen Warren { 24*d20e5e97SStephen Warren "fixture_id": "micro_b", 25f5d196d0SStephen Warren "tgt_usb_ctlr": "0", 26f5d196d0SStephen Warren "host_usb_dev_node": "/dev/usbdev-p2371-2180", 27f5d196d0SStephen Warren # This parameter is optional /if/ you only have a single board 28f5d196d0SStephen Warren # attached to your host at a time. 29f5d196d0SStephen Warren "host_usb_port_path": "3-13", 30f5d196d0SStephen Warren }, 31f5d196d0SStephen Warren) 32f5d196d0SStephen Warren 33f5d196d0SStephen Warrenenv__dfu_configs = ( 34f5d196d0SStephen Warren # eMMC, partition 1 35f5d196d0SStephen Warren { 36*d20e5e97SStephen Warren "fixture_id": "emmc", 37f5d196d0SStephen Warren "alt_info": "/dfu_test.bin ext4 0 1;/dfu_dummy.bin ext4 0 1", 38f5d196d0SStephen Warren "cmd_params": "mmc 0", 39f5d196d0SStephen Warren }, 40f5d196d0SStephen Warren) 41*d20e5e97SStephen Warren 42f5d196d0SStephen Warrenb) udev rules to set permissions on devices nodes, so that sudo is not 43f5d196d0SStephen Warrenrequired. For example: 44f5d196d0SStephen Warren 45f5d196d0SStephen WarrenACTION=="add", SUBSYSTEM=="block", SUBSYSTEMS=="usb", KERNELS=="3-13", MODE:="666" 46f5d196d0SStephen Warren 47f5d196d0SStephen Warren(You may wish to change the group ID instead of setting the permissions wide 48f5d196d0SStephen Warrenopen. All that matters is that the user ID running the test can access the 49f5d196d0SStephen Warrendevice.) 50e8debf39SStephen Warren""" 51f5d196d0SStephen Warren 52f5d196d0SStephen Warren# The set of file sizes to test. These values trigger various edge-cases such 53f5d196d0SStephen Warren# as one less than, equal to, and one greater than typical USB max packet 54f5d196d0SStephen Warren# sizes, and similar boundary conditions. 55f5d196d0SStephen Warrentest_sizes = ( 56f5d196d0SStephen Warren 64 - 1, 57f5d196d0SStephen Warren 64, 58f5d196d0SStephen Warren 64 + 1, 59f5d196d0SStephen Warren 128 - 1, 60f5d196d0SStephen Warren 128, 61f5d196d0SStephen Warren 128 + 1, 62f5d196d0SStephen Warren 960 - 1, 63f5d196d0SStephen Warren 960, 64f5d196d0SStephen Warren 960 + 1, 65f5d196d0SStephen Warren 4096 - 1, 66f5d196d0SStephen Warren 4096, 67f5d196d0SStephen Warren 4096 + 1, 68f5d196d0SStephen Warren 1024 * 1024 - 1, 69f5d196d0SStephen Warren 1024 * 1024, 70f5d196d0SStephen Warren 8 * 1024 * 1024, 71f5d196d0SStephen Warren) 72f5d196d0SStephen Warren 73f5d196d0SStephen Warrenfirst_usb_dev_port = None 74f5d196d0SStephen Warren 75f5d196d0SStephen Warren@pytest.mark.buildconfigspec('cmd_dfu') 76f5d196d0SStephen Warrendef test_dfu(u_boot_console, env__usb_dev_port, env__dfu_config): 77e8debf39SStephen Warren """Test the "dfu" command; the host system must be able to enumerate a USB 78f5d196d0SStephen Warren device when "dfu" is running, various DFU transfers are tested, and the 79f5d196d0SStephen Warren USB device must disappear when "dfu" is aborted. 80f5d196d0SStephen Warren 81f5d196d0SStephen Warren Args: 82f5d196d0SStephen Warren u_boot_console: A U-Boot console connection. 83f5d196d0SStephen Warren env__usb_dev_port: The single USB device-mode port specification on 84f5d196d0SStephen Warren which to run the test. See the file-level comment above for 85f5d196d0SStephen Warren details of the format. 86f5d196d0SStephen Warren env__dfu_config: The single DFU (memory region) configuration on which 87f5d196d0SStephen Warren to run the test. See the file-level comment above for details 88f5d196d0SStephen Warren of the format. 89f5d196d0SStephen Warren 90f5d196d0SStephen Warren Returns: 91f5d196d0SStephen Warren Nothing. 92e8debf39SStephen Warren """ 93f5d196d0SStephen Warren 94f5d196d0SStephen Warren def start_dfu(): 95e8debf39SStephen Warren """Start U-Boot's dfu shell command. 96f5d196d0SStephen Warren 97f5d196d0SStephen Warren This also waits for the host-side USB enumeration process to complete. 98f5d196d0SStephen Warren 99f5d196d0SStephen Warren Args: 100f5d196d0SStephen Warren None. 101f5d196d0SStephen Warren 102f5d196d0SStephen Warren Returns: 103f5d196d0SStephen Warren Nothing. 104e8debf39SStephen Warren """ 105f5d196d0SStephen Warren 106be1df826SStephen Warren fh = u_boot_utils.attempt_to_open_file( 107be1df826SStephen Warren env__usb_dev_port['host_usb_dev_node']) 108be1df826SStephen Warren if fh: 109be1df826SStephen Warren fh.close() 110be1df826SStephen Warren raise Exception('USB device present before dfu command invoked') 111be1df826SStephen Warren 112f5d196d0SStephen Warren u_boot_console.log.action( 113f5d196d0SStephen Warren 'Starting long-running U-Boot dfu shell command') 114f5d196d0SStephen Warren 115f5d196d0SStephen Warren cmd = 'setenv dfu_alt_info "%s"' % env__dfu_config['alt_info'] 116f5d196d0SStephen Warren u_boot_console.run_command(cmd) 117f5d196d0SStephen Warren 118f5d196d0SStephen Warren cmd = 'dfu 0 ' + env__dfu_config['cmd_params'] 119f5d196d0SStephen Warren u_boot_console.run_command(cmd, wait_for_prompt=False) 120f5d196d0SStephen Warren u_boot_console.log.action('Waiting for DFU USB device to appear') 121f5d196d0SStephen Warren fh = u_boot_utils.wait_until_open_succeeds( 122f5d196d0SStephen Warren env__usb_dev_port['host_usb_dev_node']) 123f5d196d0SStephen Warren fh.close() 124f5d196d0SStephen Warren 125f5d196d0SStephen Warren def stop_dfu(ignore_errors): 126e8debf39SStephen Warren """Stop U-Boot's dfu shell command from executing. 127f5d196d0SStephen Warren 128f5d196d0SStephen Warren This also waits for the host-side USB de-enumeration process to 129f5d196d0SStephen Warren complete. 130f5d196d0SStephen Warren 131f5d196d0SStephen Warren Args: 132f5d196d0SStephen Warren ignore_errors: Ignore any errors. This is useful if an error has 133f5d196d0SStephen Warren already been detected, and the code is performing best-effort 134f5d196d0SStephen Warren cleanup. In this case, we do not want to mask the original 135f5d196d0SStephen Warren error by "honoring" any new errors. 136f5d196d0SStephen Warren 137f5d196d0SStephen Warren Returns: 138f5d196d0SStephen Warren Nothing. 139e8debf39SStephen Warren """ 140f5d196d0SStephen Warren 141f5d196d0SStephen Warren try: 142f5d196d0SStephen Warren u_boot_console.log.action( 143f5d196d0SStephen Warren 'Stopping long-running U-Boot dfu shell command') 144f5d196d0SStephen Warren u_boot_console.ctrlc() 145f5d196d0SStephen Warren u_boot_console.log.action( 146f5d196d0SStephen Warren 'Waiting for DFU USB device to disappear') 147f5d196d0SStephen Warren u_boot_utils.wait_until_file_open_fails( 148f5d196d0SStephen Warren env__usb_dev_port['host_usb_dev_node'], ignore_errors) 149f5d196d0SStephen Warren except: 150f5d196d0SStephen Warren if not ignore_errors: 151f5d196d0SStephen Warren raise 152f5d196d0SStephen Warren 153f5d196d0SStephen Warren def run_dfu_util(alt_setting, fn, up_dn_load_arg): 154e8debf39SStephen Warren """Invoke dfu-util on the host. 155f5d196d0SStephen Warren 156f5d196d0SStephen Warren Args: 157f5d196d0SStephen Warren alt_setting: The DFU "alternate setting" identifier to interact 158f5d196d0SStephen Warren with. 159f5d196d0SStephen Warren fn: The host-side file name to transfer. 160f5d196d0SStephen Warren up_dn_load_arg: '-U' or '-D' depending on whether a DFU upload or 161f5d196d0SStephen Warren download operation should be performed. 162f5d196d0SStephen Warren 163f5d196d0SStephen Warren Returns: 164f5d196d0SStephen Warren Nothing. 165e8debf39SStephen Warren """ 166f5d196d0SStephen Warren 167f5d196d0SStephen Warren cmd = ['dfu-util', '-a', str(alt_setting), up_dn_load_arg, fn] 168f5d196d0SStephen Warren if 'host_usb_port_path' in env__usb_dev_port: 169f5d196d0SStephen Warren cmd += ['-p', env__usb_dev_port['host_usb_port_path']] 170f5d196d0SStephen Warren u_boot_utils.run_and_log(u_boot_console, cmd) 171f5d196d0SStephen Warren u_boot_console.wait_for('Ctrl+C to exit ...') 172f5d196d0SStephen Warren 173f5d196d0SStephen Warren def dfu_write(alt_setting, fn): 174e8debf39SStephen Warren """Write a file to the target board using DFU. 175f5d196d0SStephen Warren 176f5d196d0SStephen Warren Args: 177f5d196d0SStephen Warren alt_setting: The DFU "alternate setting" identifier to interact 178f5d196d0SStephen Warren with. 179f5d196d0SStephen Warren fn: The host-side file name to transfer. 180f5d196d0SStephen Warren 181f5d196d0SStephen Warren Returns: 182f5d196d0SStephen Warren Nothing. 183e8debf39SStephen Warren """ 184f5d196d0SStephen Warren 185f5d196d0SStephen Warren run_dfu_util(alt_setting, fn, '-D') 186f5d196d0SStephen Warren 187f5d196d0SStephen Warren def dfu_read(alt_setting, fn): 188e8debf39SStephen Warren """Read a file from the target board using DFU. 189f5d196d0SStephen Warren 190f5d196d0SStephen Warren Args: 191f5d196d0SStephen Warren alt_setting: The DFU "alternate setting" identifier to interact 192f5d196d0SStephen Warren with. 193f5d196d0SStephen Warren fn: The host-side file name to transfer. 194f5d196d0SStephen Warren 195f5d196d0SStephen Warren Returns: 196f5d196d0SStephen Warren Nothing. 197e8debf39SStephen Warren """ 198f5d196d0SStephen Warren 199f5d196d0SStephen Warren # dfu-util fails reads/uploads if the host file already exists 200f5d196d0SStephen Warren if os.path.exists(fn): 201f5d196d0SStephen Warren os.remove(fn) 202f5d196d0SStephen Warren run_dfu_util(alt_setting, fn, '-U') 203f5d196d0SStephen Warren 204f5d196d0SStephen Warren def dfu_write_read_check(size): 205e8debf39SStephen Warren """Test DFU transfers of a specific size of data 206f5d196d0SStephen Warren 207f5d196d0SStephen Warren This function first writes data to the board then reads it back and 208f5d196d0SStephen Warren compares the written and read back data. Measures are taken to avoid 209f5d196d0SStephen Warren certain types of false positives. 210f5d196d0SStephen Warren 211f5d196d0SStephen Warren Args: 212f5d196d0SStephen Warren size: The data size to test. 213f5d196d0SStephen Warren 214f5d196d0SStephen Warren Returns: 215f5d196d0SStephen Warren Nothing. 216e8debf39SStephen Warren """ 217f5d196d0SStephen Warren 218f5d196d0SStephen Warren test_f = u_boot_utils.PersistentRandomFile(u_boot_console, 219f5d196d0SStephen Warren 'dfu_%d.bin' % size, size) 220f5d196d0SStephen Warren readback_fn = u_boot_console.config.result_dir + '/dfu_readback.bin' 221f5d196d0SStephen Warren 222f5d196d0SStephen Warren u_boot_console.log.action('Writing test data to DFU primary ' + 223f5d196d0SStephen Warren 'altsetting') 224f5d196d0SStephen Warren dfu_write(0, test_f.abs_fn) 225f5d196d0SStephen Warren 226f5d196d0SStephen Warren u_boot_console.log.action('Writing dummy data to DFU secondary ' + 227f5d196d0SStephen Warren 'altsetting to clear DFU buffers') 228f5d196d0SStephen Warren dfu_write(1, dummy_f.abs_fn) 229f5d196d0SStephen Warren 230f5d196d0SStephen Warren u_boot_console.log.action('Reading DFU primary altsetting for ' + 231f5d196d0SStephen Warren 'comparison') 232f5d196d0SStephen Warren dfu_read(0, readback_fn) 233f5d196d0SStephen Warren 234f5d196d0SStephen Warren u_boot_console.log.action('Comparing written and read data') 235f5d196d0SStephen Warren written_hash = test_f.content_hash 236f5d196d0SStephen Warren read_back_hash = u_boot_utils.md5sum_file(readback_fn, size) 237f5d196d0SStephen Warren assert(written_hash == read_back_hash) 238f5d196d0SStephen Warren 239f5d196d0SStephen Warren # This test may be executed against multiple USB ports. The test takes a 240f5d196d0SStephen Warren # long time, so we don't want to do the whole thing each time. Instead, 241f5d196d0SStephen Warren # execute the full test on the first USB port, and perform a very limited 242f5d196d0SStephen Warren # test on other ports. In the limited case, we solely validate that the 243f5d196d0SStephen Warren # host PC can enumerate the U-Boot USB device. 244f5d196d0SStephen Warren global first_usb_dev_port 245f5d196d0SStephen Warren if not first_usb_dev_port: 246f5d196d0SStephen Warren first_usb_dev_port = env__usb_dev_port 247f5d196d0SStephen Warren if env__usb_dev_port == first_usb_dev_port: 248f5d196d0SStephen Warren sizes = test_sizes 249f5d196d0SStephen Warren else: 250f5d196d0SStephen Warren sizes = [] 251f5d196d0SStephen Warren 252f5d196d0SStephen Warren dummy_f = u_boot_utils.PersistentRandomFile(u_boot_console, 253f5d196d0SStephen Warren 'dfu_dummy.bin', 1024) 254f5d196d0SStephen Warren 255f5d196d0SStephen Warren ignore_cleanup_errors = True 256f5d196d0SStephen Warren try: 257f5d196d0SStephen Warren start_dfu() 258f5d196d0SStephen Warren 259f5d196d0SStephen Warren u_boot_console.log.action( 260f5d196d0SStephen Warren 'Overwriting DFU primary altsetting with dummy data') 261f5d196d0SStephen Warren dfu_write(0, dummy_f.abs_fn) 262f5d196d0SStephen Warren 263f5d196d0SStephen Warren for size in sizes: 264a2ec5606SStephen Warren with u_boot_console.log.section('Data size %d' % size): 265f5d196d0SStephen Warren dfu_write_read_check(size) 266f5d196d0SStephen Warren # Make the status of each sub-test obvious. If the test didn't 267f5d196d0SStephen Warren # pass, an exception was thrown so this code isn't executed. 268f5d196d0SStephen Warren u_boot_console.log.status_pass('OK') 269f5d196d0SStephen Warren ignore_cleanup_errors = False 270f5d196d0SStephen Warren finally: 271f5d196d0SStephen Warren stop_dfu(ignore_cleanup_errors) 272