Skip to content

Commit

Permalink
driver: stlink: get adapter speed through adapter_get_speed_khz()
Browse files Browse the repository at this point in the history
The stlink driver, both in dapdirect and in HLA modes, pretends to
store locally the value of the adapter speed in order to use it
later-on during adapter initialization.
It doesn't work in dapdirect mode since the code to store locally
the value will not be executed until the adapter is already fully
initialized.

This cause an issue in dapdirect mode:
- due to the local value, still kept at -1, the adapter will be
  initialized to the lowest clock speed (5 KHz on stlink v2 in SWD
  mode);
- after the adapter initialization the framework will again set
  the speed with the value requested by the user.

Some target, like nRF51822, only accepts JTAG/SWD speed in a
defined range of frequencies. The initial speed of 5 KHz used by
dapdirect can be out of range, making the target debug port not
working.

The adapter framework already stores the value of speed and makes
it available through adapter_get_speed_khz().

Drop struct hl_interface_param::initial_interface_speed.
Let the code to use adapter_get_speed_khz().

Change-Id: Ie11bf0234574f2a9180d3d3a16efb78e08dfcd86
Reported-by: Andrzej Sierżęga <[email protected]>
Signed-off-by: Antonio Borneo <[email protected]>
Reviewed-on: https://review.openocd.org/c/openocd/+/8574
Reviewed-by: Andrzej Sierżęga <[email protected]>
Tested-by: jenkins
  • Loading branch information
borneoa committed Nov 23, 2024
1 parent 8c739a4 commit c582cfb
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 9 deletions.
3 changes: 1 addition & 2 deletions src/jtag/drivers/stlink_usb.c
Original file line number Diff line number Diff line change
Expand Up @@ -3781,7 +3781,7 @@ static int stlink_open(struct hl_interface_param *param, enum stlink_mode mode,
}

/* initialize the debug hardware */
err = stlink_usb_init_mode(h, param->connect_under_reset, param->initial_interface_speed);
err = stlink_usb_init_mode(h, param->connect_under_reset, adapter_get_speed_khz());

if (err != ERROR_OK) {
LOG_ERROR("init mode failed (unable to connect to the target)");
Expand Down Expand Up @@ -5174,7 +5174,6 @@ static int stlink_dap_speed(int speed)
return ERROR_JTAG_NOT_IMPLEMENTED;
}

stlink_dap_param.initial_interface_speed = speed;
stlink_speed(stlink_dap_handle, speed, false);
return ERROR_OK;
}
Expand Down
6 changes: 1 addition & 5 deletions src/jtag/hla/hla_interface.c
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ static struct hl_interface hl_if = {
.pid = { 0 },
.transport = HL_TRANSPORT_UNKNOWN,
.connect_under_reset = false,
.initial_interface_speed = -1,
.use_stlink_tcp = false,
.stlink_tcp_port = 7184,
},
Expand Down Expand Up @@ -165,11 +164,8 @@ static int hl_interface_speed(int speed)
if (!hl_if.layout->api->speed)
return ERROR_OK;

if (!hl_if.handle) {
/* pass speed as initial param as interface not open yet */
hl_if.param.initial_interface_speed = speed;
if (!hl_if.handle)
return ERROR_OK;
}

hl_if.layout->api->speed(hl_if.handle, speed, false);

Expand Down
2 changes: 0 additions & 2 deletions src/jtag/hla/hla_interface.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,6 @@ struct hl_interface_param {
enum hl_transports transport;
/** */
bool connect_under_reset;
/** Initial interface clock clock speed */
int initial_interface_speed;
/** */
bool use_stlink_tcp;
/** */
Expand Down

0 comments on commit c582cfb

Please sign in to comment.