Skip to content

Commit

Permalink
Merge pull request #395 from machinekoder/fix-hal-net-signal
Browse files Browse the repository at this point in the history
hal_signal: fix overwriting OUT dir pin value
  • Loading branch information
luminize authored Jan 29, 2024
2 parents ecf488a + 3728d8e commit e5e825a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/libraries/hal/src/hal_signal.c
Original file line number Diff line number Diff line change
Expand Up @@ -274,10 +274,10 @@ int halg_link(const int use_hal_mutex,
// good runtime assertion on 'halcmd show objects'.
pin->data_ptr = SHMOFF(&sig->value);

if (( sig->readers == 0 ) && ( sig->writers == 0 ) &&
( sig->bidirs == 0 )) {
if (( pin->dir == HAL_OUT ) || ( sig->readers + sig->writers + sig->bidirs == 0 )) {

// this signal is not linked to any pins
// or the newly linked pin is the first OUT pin
// copy value from pin's "dummy" field,
// making it 'inherit' the value of the first pin
// data_addr = hal_shmem_base + sig->data_ptr;
Expand Down

0 comments on commit e5e825a

Please sign in to comment.