Only propagate register errors on some targets
Without this change, connecting to ARM targets is impossible. Fixes #115. Change-Id: Ie33c7e15ac1bed8c9cbd8e6a78de92d5498c5999
This commit is contained in:
@@ -1174,7 +1174,7 @@ static int gdb_get_registers_packet(struct connection *connection,
|
||||
for (i = 0; i < reg_list_size; i++) {
|
||||
if (!reg_list[i]->valid) {
|
||||
retval = reg_list[i]->type->get(reg_list[i]);
|
||||
if (retval != ERROR_OK) {
|
||||
if (retval != ERROR_OK && target->propagate_register_errors) {
|
||||
LOG_DEBUG("Couldn't get register %s.", reg_list[i]->name);
|
||||
free(reg_packet);
|
||||
free(reg_list);
|
||||
@@ -1242,7 +1242,7 @@ static int gdb_set_registers_packet(struct connection *connection,
|
||||
gdb_target_to_reg(target, packet_p, chars, bin_buf);
|
||||
|
||||
retval = reg_list[i]->type->set(reg_list[i], bin_buf);
|
||||
if (retval != ERROR_OK) {
|
||||
if (retval != ERROR_OK && target->propagate_register_errors) {
|
||||
LOG_DEBUG("Couldn't set register %s.", reg_list[i]->name);
|
||||
free(reg_list);
|
||||
free(bin_buf);
|
||||
@@ -1289,7 +1289,7 @@ static int gdb_get_register_packet(struct connection *connection,
|
||||
|
||||
if (!reg_list[reg_num]->valid) {
|
||||
retval = reg_list[reg_num]->type->get(reg_list[reg_num]);
|
||||
if (retval != ERROR_OK) {
|
||||
if (retval != ERROR_OK && target->propagate_register_errors) {
|
||||
LOG_DEBUG("Couldn't get register %s.", reg_list[reg_num]->name);
|
||||
free(reg_list);
|
||||
return gdb_error(connection, retval);
|
||||
@@ -1349,7 +1349,7 @@ static int gdb_set_register_packet(struct connection *connection,
|
||||
gdb_target_to_reg(target, separator + 1, chars, bin_buf);
|
||||
|
||||
retval = reg_list[reg_num]->type->set(reg_list[reg_num], bin_buf);
|
||||
if (retval != ERROR_OK) {
|
||||
if (retval != ERROR_OK && target->propagate_register_errors) {
|
||||
LOG_DEBUG("Couldn't set register %s.", reg_list[reg_num]->name);
|
||||
free(bin_buf);
|
||||
free(reg_list);
|
||||
|
||||
Reference in New Issue
Block a user