jtag: retire tap field
jtag_add_dr/ir_scan() now takes the tap as the first argument, rather than for each of the fields passed in. The code never exercised the path where there was more than one tap being scanned, who knows if it even worked. This simplifies the implementation and reduces clutter in the calling code. use jtag_add_ir/dr_plain_scan() for more fancy situations. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
This commit is contained in:
@@ -508,14 +508,12 @@ static int etm_read_reg_w_check(struct reg *reg,
|
||||
arm_jtag_scann(etm_reg->jtag_info, 0x6);
|
||||
arm_jtag_set_instr(etm_reg->jtag_info, etm_reg->jtag_info->intest_instr, NULL);
|
||||
|
||||
fields[0].tap = etm_reg->jtag_info->tap;
|
||||
fields[0].num_bits = 32;
|
||||
fields[0].out_value = reg->value;
|
||||
fields[0].in_value = NULL;
|
||||
fields[0].check_value = NULL;
|
||||
fields[0].check_mask = NULL;
|
||||
|
||||
fields[1].tap = etm_reg->jtag_info->tap;
|
||||
fields[1].num_bits = 7;
|
||||
fields[1].out_value = malloc(1);
|
||||
buf_set_u32(fields[1].out_value, 0, 7, reg_addr);
|
||||
@@ -523,7 +521,6 @@ static int etm_read_reg_w_check(struct reg *reg,
|
||||
fields[1].check_value = NULL;
|
||||
fields[1].check_mask = NULL;
|
||||
|
||||
fields[2].tap = etm_reg->jtag_info->tap;
|
||||
fields[2].num_bits = 1;
|
||||
fields[2].out_value = malloc(1);
|
||||
buf_set_u32(fields[2].out_value, 0, 1, 0);
|
||||
@@ -531,13 +528,13 @@ static int etm_read_reg_w_check(struct reg *reg,
|
||||
fields[2].check_value = NULL;
|
||||
fields[2].check_mask = NULL;
|
||||
|
||||
jtag_add_dr_scan(3, fields, jtag_get_end_state());
|
||||
jtag_add_dr_scan(etm_reg->jtag_info->tap, 3, fields, jtag_get_end_state());
|
||||
|
||||
fields[0].in_value = reg->value;
|
||||
fields[0].check_value = check_value;
|
||||
fields[0].check_mask = check_mask;
|
||||
|
||||
jtag_add_dr_scan_check(3, fields, jtag_get_end_state());
|
||||
jtag_add_dr_scan_check(etm_reg->jtag_info->tap, 3, fields, jtag_get_end_state());
|
||||
|
||||
free(fields[1].out_value);
|
||||
free(fields[2].out_value);
|
||||
@@ -594,28 +591,25 @@ static int etm_write_reg(struct reg *reg, uint32_t value)
|
||||
arm_jtag_scann(etm_reg->jtag_info, 0x6);
|
||||
arm_jtag_set_instr(etm_reg->jtag_info, etm_reg->jtag_info->intest_instr, NULL);
|
||||
|
||||
fields[0].tap = etm_reg->jtag_info->tap;
|
||||
fields[0].num_bits = 32;
|
||||
uint8_t tmp1[4];
|
||||
fields[0].out_value = tmp1;
|
||||
buf_set_u32(fields[0].out_value, 0, 32, value);
|
||||
fields[0].in_value = NULL;
|
||||
|
||||
fields[1].tap = etm_reg->jtag_info->tap;
|
||||
fields[1].num_bits = 7;
|
||||
uint8_t tmp2;
|
||||
fields[1].out_value = &tmp2;
|
||||
buf_set_u32(fields[1].out_value, 0, 7, reg_addr);
|
||||
fields[1].in_value = NULL;
|
||||
|
||||
fields[2].tap = etm_reg->jtag_info->tap;
|
||||
fields[2].num_bits = 1;
|
||||
uint8_t tmp3;
|
||||
fields[2].out_value = &tmp3;
|
||||
buf_set_u32(fields[2].out_value, 0, 1, 1);
|
||||
fields[2].in_value = NULL;
|
||||
|
||||
jtag_add_dr_scan(3, fields, jtag_get_end_state());
|
||||
jtag_add_dr_scan(etm_reg->jtag_info->tap, 3, fields, jtag_get_end_state());
|
||||
|
||||
return ERROR_OK;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user