Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Reworked score/cpu/sparc so it can be safely compiled multilib. All | Joel Sherrill | 2000-07-11 | 1 | -520/+0 |
| | | | | | | routines and structures that require CPU model specific information are now in libcpu. This primarily required moving erc32 specific information from score/cpu files to libcpu/sparc and the erc32 BSP. | ||||
* | Updated copyright notice. | Joel Sherrill | 1999-11-17 | 1 | -2/+1 |
| | |||||
* | Patch from Jiri Gaisler <jgais@ws.estec.esa.nl>: | Joel Sherrill | 1999-07-09 | 1 | -7/+7 |
| | | | | | | + interrupt masking correction + FPU rev.B workaround + minor erc32 related fixes | ||||
* | Moved erc32.h back up in tree. | Joel Sherrill | 1999-02-19 | 1 | -0/+521 |
| | |||||
* | Moved to proper rtems/score | Joel Sherrill | 1999-02-19 | 1 | -521/+0 |
| | |||||
* | Bug report from Jiri Gaisler <jgais@ws.estec.esa.nl>: | Joel Sherrill | 1999-01-13 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | > > I think I have found a bug in src/exec/scor/sparc/cpu/erc32.h in: > > > > #define ERC32_Disable_interrupt( _source, _previous ) \ > > do { \ > > unsigned32 _level; \ > > unsigned32 _mask = 1 << (_source); \ > > \ > > sparc_disable_interrupts( _level ); \ > > (_previous) = ERC32_MEC.Interrupt_Mask; \ > > ERC32_MEC.Interrupt_Mask = _previous | _mask; \ > > sparc_enable_interrupts( _level ); \ > > (_previous) &= ~_mask; \ <- IS THIS CORRECT...? > > } while (0) > > > > The previous interrupt mask is returned after first clearing the > > bit to be disabled, regardless whether the bit was set before or > > not. If the bit was set (interrupt masked), subsequent call to > > ERC32_Restore_interrupt() will enable the interrupt even though > > it was supposed to be masked. This is indeed what happens in > > DEBUG_puts when polled console I/O is used. In my opinion, the > > last statement in the macro should be removed - what is your opinion? > > I think the "~" shouldn't be there. I recall that the intent of that line > is to only return the state of the interrupts you were concerned with. > Removing the line returns entire state. Given that the value returned > shuold only be used in conjunction with the map, I suppose either removing > the ~ or the entire line is correct? I can go either way. Just let me > know which you think is more correct and the source will change. :) Hmmm, just removing the '~' should be OK. DEBUG_puts() seems to be the only user of ERC32_Restore_interrupt() anyway ... | ||||
* | updated copyright to 1998 | Joel Sherrill | 1998-02-17 | 1 | -1/+1 |
| | |||||
* | Fixed typo in the pointer to the license terms. | Joel Sherrill | 1997-10-08 | 1 | -2/+2 |
| | |||||
* | headers updated to reflect new style copyright notice as part | Joel Sherrill | 1997-04-22 | 1 | -5/+5 |
| | | | | of switching to the modified GNU GPL. | ||||
* | Changes to reflect new revision of erc32 per Jiri Gaisler's suggestions. | Joel Sherrill | 1996-12-02 | 1 | -19/+17 |
| | | | | This is current as of sis 2.6. | ||||
* | patches to erc32.h to correct handling of timer mirror from Jiri Gaisler. | Joel Sherrill | 1996-10-14 | 1 | -3/+3 |
| | |||||
* | ERC32_Force_interrupt implementation replaced per Jiri Gaisler's experience | Joel Sherrill | 1996-08-08 | 1 | -0/+5 |
| | | | | on the real erc32 hardware. | ||||
* | typos fixed | Joel Sherrill | 1995-10-31 | 1 | -0/+15 |
| | |||||
* | SPARC port passes all tests | Joel Sherrill | 1995-10-30 | 1 | -0/+503 |