Hi again,

User fenugrec has made a review of my changes (over at github) which lead
to some improvements. I've also made sure that there are no compiler
warnings and have also removed a pre-existing warning.

Also, any opinions (from anyone) on adding the new configuration keys
mentioned below?

Best regards,
Mikael

Den lör 21 jan. 2023 15:15Mikael Djurfeldt <mik...@djurfeldt.com> skrev:

> Hi,
>
> libsigrok:s rdtech-dps driver currently has support for the Riden devices
> RD6006, RD6012 and RD6018.
>
> I've recently submitted support for RD6006P and RD6012P here:
>
> https://github.com/sigrokproject/libsigrok/pull/205
>
> Also, all of these devices both have a system temperature and battery
> temperature (as measured by an external battery temperature probe during
> charging) reading. It would be convenient to be able to get those displayed
> in the RDTech GUI in Smuview. Therefore, I'm considering adding the
> configuration keys SR_CONF_SYSTEM_TEMPERATURE and
> SR_CONF_BATTERY_TEMPERATURE to libsigrok.h. Would this make sense? Any
> chance that this, together with corresponding changes in the rdtech-dps api
> would be accepted?
>
> Best regards,
> Mikael Djurfeldt
>
>
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel

Reply via email to