I have 3 remarks regarding that:
- This is not mentioned in the helpfile
- The lock/unlock routines are a little bit unnecessary made public seen above.
- by letting the unlock/lock routines called from the pps_mapping routine, the user can not choose to set the "IOL1Way" configuration bit in config word 2: the OSCCON <IOLOCK> bit must be capable of changing multiple times (unless there is only one call to "pps_mapping").
So, not calling the unlock/lock routines from "pps_mapping" would make things more flexible: the use can do several PPS_mappings and then call once "Lock_IoLock" to lock all IO effectively (this is how things are described in the MicroChip documentation).
Of course, the compatibility with the past would be broken (unless an additional pps_mapping_... routine would be created that does not lock/unlock activities...)
Thanks in advance!
