jlaitine commented on PR #8026: URL: https://github.com/apache/nuttx/pull/8026#issuecomment-1374467833
@xiaoxiang781216 : Here is a commit which shows how the IF would look like if you added the mm_map_s as an argument to every function (the option 1. you mentioned avove). In my opinion it just complicates things for every driver using the interface, since they need to fetch & dig into the task_group structure, even if they don't need it for anything else. But if you think that this would look better, I am fine with it. Please let me know which way you insist this to be, I will then clean up the git history accordingly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
