cederom commented on PR #17405: URL: https://github.com/apache/nuttx/pull/17405#issuecomment-3591803107
Response from kerogit: > the TC74Ax driver not being uORB - well, I was not sure whether to submit the driver at all exactly because it is not done the new way. But then I noticed another such driver was merged in August so I posted it. As for the reasons why the driver is not done the new way, there are two: > > First - I did not read the documentation. Over my previous submissions, I was never able to find any howto for writing drivers. As in "if you want to make driver for X, do this." (Do not take this as a criticism please, I was thinking about writing such howto but every time I was done with a feature, it seemed to me that there is actually nothing to write about because everything is simple, clear and pretty obvious.) So I got into the habit of simply looking at other code that does the same thing and adapting it. Only after the driver was almost done, I found the documents you linked. > > The other reason - and the reason why I did not decide to rewrite the driver - is that unless I misunderstood the documentation, the new way uses floats to exchange data and seems more complex overall. My intended use case is currently 8-bit AVR and I am even not able to compile code that has floats in it. (That can be probably fixed easily by upgrading to newer gcc.) Moreover, floats are expensive to handle. So to put it simple, if I did the driver this way, it would not be of much use to me. > > All in all, if the driver gets rejected because it uses deprecated interface, that would be completely understandable from my point of view. However, I have to say that I am not planning to rewrite it in any forseeable future (maybe if it proved easy to have it configurable for > both interfaces - judging from bmp180 driver it could be - but I don't think I'll find the time even in such case.) -- 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]
