I am working on a 28335 project where we occasionally erase up to 5 sectors at a time. Last year the parts list changed and the 28335 with slower flash would watchdog reset during the sector erase. It was a tense week or two before we figured out what was going on and quite a few units were "bricked". We finally figured out the part change and the solution was to refresh during the erase using the Flash_CallbackPtr. We also used a hack of calling Flash_DepRecover() prior to Flash_Erase to recover any unit that had been "bricked" by previous software.
Fast forward a year, we are in the middle of requirements tracing and the Flash_DepRecover() call is still in the code. My question is whether there is any downside to leaving this call in? (beyond the extra work of adding a supporting requirement) The only upside I can see is recovery in the unlikely case of cutting power during an erase operation. (erase only occurs during software updates)