BL602 DiagnosticDataProviderImpl::GetWiFiVersion and other functions look pretty broken

project-chip / connectedhomeip

Matter (formerly Project CHIP) creates more connections between more objects, simplifying development for manufacturers and increasing compatibility for consumers, guided by the Connectivity Standards Alliance.

https://buildwithmatter.com

Apache License 2.0

7.55k stars 2.04k forks source link

BL602 DiagnosticDataProviderImpl::GetWiFiVersion and other functions look pretty broken #25546

Open bzbarsky-apple opened 1 year ago

bzbarsky-apple commented 1 year ago

My understanding is that this should return the wifi version being used right now to connect to the AP. It just always reports "a" (aka 0).

And DiagnosticDataProviderImpl::GetWiFiChannelNumber always reports a channel number of 0.

And DiagnosticDataProviderImpl::GetWiFiRssi always returns success while leaving the outparam with whatever random value it had.

These functions should return failure if not implemented, not just return garbage to callers.

@wy-hh

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.