patacongo edited a comment on issue #740: URL: https://github.com/apache/incubator-nuttx/issues/740#issuecomment-672937870
> Is it reasonable? Absolutely NOT! wdparm_t was added to work around the kind of errors that you are tying to re-introduce. What you are proposing will work on most 32- and 64- bit machines but no where else and, hence, cannot be permitted. I will close an PR that attempts to pass watchdog paramters using FAR void * when the correct solution is already in place. What you are proposing is just wrong! If a memory storage location is allocated as a FAR void * it will be 16, 20, 24, 32, or 64 bits wide and in many cases cannot hold a uint32_t. It it is allocated as a wdparm_t it will be 32 or 64 bits in width and is guaranteed to hold a uint32_t or a pointer. You proposal will not work and will break code. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
