Read before write

From OpenCircuits
Revision as of 19:02, 8 August 2009 by DavidCary (talk | contribs) (yet another solution)
Jump to navigation Jump to search

Read before Write

The issue with 'read before write' is that, unlike a PIC18, the PIC16 does not store the state of it's output pins anywhere.

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 two 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 first, 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.

The second 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.

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

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

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

#define test_port(port, no)\
	test_bit(sport[&porta-PORTA], no)

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

Further reading