There is a mix of lock-based synchronization and usage of ConcurrentDictionary for interrupt state management in the Tca955x driver. Based on feedback in PR #2427 (see review comment), please review all code in the Tca955x driver where locks are used for state management and evaluate if ConcurrentDictionary can be used instead for consistency and thread safety. If lock usage is still required, add rationale as comments in the code.
References:
Action items:
- Audit interrupt state management code in Tca955x for locks
- Replace locks with ConcurrentDictionary where possible
- Add comments justifying any remaining locks
There is a mix of lock-based synchronization and usage of ConcurrentDictionary for interrupt state management in the Tca955x driver. Based on feedback in PR #2427 (see review comment), please review all code in the Tca955x driver where locks are used for state management and evaluate if ConcurrentDictionary can be used instead for consistency and thread safety. If lock usage is still required, add rationale as comments in the code.
References:
Action items: