Hi, panic: > - makes registers in the IO memory space children of > BitAccessibleRegister (e.g. GPIORegister, CPKPRRegister) > for new hierarchy, see attached PNG.
Regarding this I like to note that I also don't like this approach of having some other registers (GPIO, CLKPR, ...) being subclasses of IOReg/BitAccessibleRegister. IMO, they are peripherals (subclass of Hardware only) and should have a IOReg as a member. IOReg or even RWMemoryMember should also contain the value of the peripheral -- avoiding the current duplication of a unsigned char in the peripheral and the IOReg member (e.g. class HWPort: ddr and ddr_reg). But this is another patch... _______________________________________________ Simulavr-devel mailing list Simulavr-devel@nongnu.org https://lists.nongnu.org/mailman/listinfo/simulavr-devel