Nut upsdev - Dec 2024 - <Cyber Energy> <usbhid-ups/cps-hid.c>-Add HID Usage (Temperature)

Hi NUT:

We propose adding HID usage for "Temperature" to the NUT source code
file cps-hid.c (driver:
USBHID-UPS(8)<https://networkupstools.org/docs/man/usbhid-ups.html>) so
that UPS temperature information can be displayed in the UPS status list. After
conducting internal tests, we confirmed that inserting the following code:
{ "ups.temperature", 0, 0, "UPS.PowerSummary.Temperature",
NULL, "%s", 0, kelvin_celsius_conversion },
after line 236 allows UPS temperature data to be displayed in the status list.

In NUT source code file cps-hid.c , the line 236 is
{ "ups.firmware", 0, 0,
"UPS.PowerSummary.CPSFirmwareVersion", NULL, "%s",
HU_FLAG_STATIC, stringid_conversion },
Please add the following code after line 236 for our new HID usage
(Temperature).
{ "ups.temperature", 0, 0, "UPS.PowerSummary.Temperature",
NULL, "%s", 0, kelvin_celsius_conversion },

Once you have confirmed the request to add the source code, please provide a
test version of the NUT software with the added HID usage (Temperature) code for
our final testing and verification. Please contact me if you have any questions
regarding this request. Thank you very much!

Best regards
Yvonne Chen

Cyber Energy Co., Ltd.
Tel: +886-2-8792-9628 #605
Fax: +886-2-8792-9626
Email: Yvonne.Chen at cyberenergy.com<mailto:Yvonne.Chen at
cyberenergy.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://alioth-lists.debian.net/pipermail/nut-upsdev/attachments/20241204/1d003b5c/attachment.htm>

Looks good.

Posted the PR as https://github.com/networkupstools/nut/pull/2711, code
base to test would be available at
https://github.com/jimklimov/nut/tree/cps-hid-temp currently or as part of
https://github.com/networkupstools/nut/tree/master after the pull request
passes our CI and gets merged.

Thanks,
Jim Klimov


On Wed, Dec 4, 2024 at 9:11?AM Yvonne.Chen <Yvonne.Chen at
cyberenergy.com>
wrote:
> Hi NUT:
>
>
>
> We propose adding HID usage for "Temperature" to the NUT source
code file
> cps-hid.c (driver: USBHID-UPS(8)
> <https://networkupstools.org/docs/man/usbhid-ups.html>) so that UPS
> temperature information can be displayed in the UPS status list. After
> conducting internal tests, we confirmed that inserting the following code:
>
> { "ups.temperature", 0, 0,
"UPS.PowerSummary.Temperature", NULL, "%s", 0,
> kelvin_celsius_conversion },
>
> after line 236 allows UPS temperature data to be displayed in the status
> list.
>
>
>
> In NUT source code file cps-hid.c , the line 236 is
>
> { "ups.firmware", 0, 0,
"UPS.PowerSummary.CPSFirmwareVersion", NULL, "%s",
> HU_FLAG_STATIC, stringid_conversion },
>
> Please add the following code after line 236 for our new HID usage
> (Temperature).
>
> { "ups.temperature", 0, 0,
"UPS.PowerSummary.Temperature", NULL, "%s", 0,
> kelvin_celsius_conversion },
>
>
>
> Once you have confirmed the request to add the source code, please provide
> a test version of the NUT software with the added HID usage (Temperature)
> code for our final testing and verification. Please contact me if you have
> any questions regarding this request. Thank you very much!
>
>
>
> Best regards
>
> Yvonne Chen
>
>
>
> Cyber Energy Co., Ltd.
>
> Tel: +886-2-8792-9628 #605
>
> Fax: +886-2-8792-9626
>
> Email: Yvonne.Chen at cyberenergy.com
> _______________________________________________
> Nut-upsdev mailing list
> Nut-upsdev at alioth-lists.debian.net
> https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/nut-upsdev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://alioth-lists.debian.net/pipermail/nut-upsdev/attachments/20241204/a28f7789/attachment.htm>