Hi KR, The previous PR with QMI8658 sensor that was merged is uORB, the ordinary char device sensor was removed from the final PR.
So, since most people prefer to go only with uORB drivers, I suggest you don't spend time on this driver as a char dev. It needs to be converted to the uORB model later. I think we have a long way to go to fix the concerns I raised and as you noticed the uORB driver to BMP280 is 8KB bigger than its char dev counterpart, so we need to simplify many things on the uORB/usensor to reduce its size first. BR, Alan On Tue, Dec 2, 2025 at 4:51 PM <[email protected]> wrote: > I see that the TWI patch is merged, thanks for everyone involved. > > As for the driver - PR #17405 - user jerpelea left a comment with a > request to resolve conflicts. Did a quick check, seems that another > sensor driver was merged so it probably won't be that difficult to > resolve. I can certainly do it but the question is if the driver is not > getting rejected anyway because of the old interface issue. > > On 2025-11-30 16:46, Tomek CEDRO wrote: > > On Sun, Nov 30, 2025 at 4:38 PM <[email protected]> wrote: > >> On 2025-11-29 18:19, Tomek CEDRO wrote: > >> > Some more typos caught: > >> > ... > >> > I fixed them already lets see the CI status now :-) > >> > >> thanks for fixing those typos - I ran the patches through a > >> spellchecker > >> but there was a lot of false positives (NuttX, AVR, register names > >> etc.) > >> so unfortunately some things hid among them and got missed. > > > > no worries :-) > > > >> > Raiden already reaised an issue on GH about floats in uORG and there > >> > is a proposition to use fixed point built time alternative, please > >> > take a look here: > >> > > >> > https://github.com/apache/nuttx/issues/10644 > >> > > >> > I will start a mailing list discussion in a moment this seems > important > >> > :-) > >> > >> I did two quick tests with a code that reads data from I/O port (to > >> force data to be unknown at a build time so the compiler cannot > >> optimize) and does sprintf with it. Second test added division by 10 > >> to > >> the first one. > >> (..) > > > > thank you for valuable feedback, i copy pasted to uorb mailing thread, > > please join discussion over there :-) > > > >> I see that the first (bugfix) branch is already merged - thanks for > >> the > >> help. > > > > thank you for valuable fixes and contributions! :-) > > > > -- > > CeDeROM, SQ7MHZ, http://www.tomek.cedro.info >
