Read before write

Read before Write
The issue with 'read before write' is that, unlike a PIC18, it is impossible for a program running on the PIC16 to read the state of its output port latch.

The classic issue occurs when individual port bits are set, or toggled, causing outputs on the pins to behave unexpectedly.

This happens because the PIC16 reads the state of the port pins to calculate the value to write to the port. When the PIC16 writes to the port, it writes the value for all the pins at the same time.

For example with trisb set to zero, if you set portb.1=1, then later set portb.2=1. If portb.1 is excessively loaded, the PIC16 will read portb.1 as a zero and write 4 (0b100) to portb, instead of the expected 6 (0b110) to portb.

There are three solutions to the above issue:

One solution is to use a processor that does allow us to read the state of its output pins -- such as the LATx register on the PIC18 processors.

The second, and by far the best is to ensure the output pins are not excessively loaded. This is the best solution, as code not necessarily under your control, such as UART and I2C code will not affect the remainder of your application. Unfortunately, this doesn't work when implementing an open-collector bus.

The third solution is to use some kind of 'shadow' register, in which the individual bits can be manipulated, then the resulting CHAR values can then be written to the port. You use this shadow register to emulate the LATx registers on other processors.

The various _port macros below, if called in place of the _bit macros, should do the job: unsigned char sport[2]; // amend to # PIC ports

set_bit(sport[&port-PORTA], no);\ port=sport[&port-PORTA];
 * 1) define set_port(port, no)\

clear_bit(sport[&port-PORTA], no);\ port=sport[&port-PORTA];
 * 1) define clear_port(port, no)\

test_bit(sport[&port-PORTA], no)
 * 1) define test_port(port, no)\

toggle_bit(sport[&port-PORTA], no);\ port=sport[&port-PORTA];
 * 1) define toggle_port(port, no)\