Zhanglong Xia
Zhanglong Xia
> I don't see RCP API version check is removed in this PR. It seems this PR is to allow an older host to work with a newer RCP. The...
> One idea/suggestion is to keep the version checks: Host uses a min RCP API version that it expects and can work with up to its current version. But then...
@abtink Thanks for your great suggestions. For the platform logging API `otLogxxxPlat`, we may define a generic logging API `otLog(otLogLevel aLogLevel, const char *aRegionPrefix, const char *aFormat, ...)` in the...
The main purpose of this commit is to make the RCP to upload the crash log after it crashes. Here are my understandings (please correct me if I'm wrong): RCP...
> @zhanglongxia Thanks for the suggestion! I never thought to use the vendor hooks to implement this. I tried out your suggestion and was able to capture the crash dump...
The error `Handle transmit done failed: Parse` means that the code has reached to the function [HandleTransmitDone()](https://github.com/zhanglongxia/openthread/blob/master/src/lib/spinel/radio_spinel.cpp#L1635) and the error code `mTxError` is set to `OT_ERROR_PARSE`. The error code `OT_ERROR_PARSE`...
Here is the backtrace we got when this issue happens. ``` Stack Trace: RELADDR FUNCTION FILE:LINE (inlined) ot::LittleEndian::ReadUint16(unsigned char const*) (BuildId: 4449f9236dbeb90480f244a574482e73) external/openthread/src/core/common/encoding.hpp:307:83 (inlined) ot::Spinel::MultiFrameBuffer::GetNextSavedFrame(unsigned char*&, unsigned short&) (BuildId: 4449f9236dbeb90480f244a574482e73)...
Is it possible to add new config file `openthread-posix-platform-config.h`, which contains the `OPENTHREAD_POSIX_CONFIG_*` configurations of the file `openthread-posix-config.h`. Then the file `openthread-posix-config.h` contains `openthread-core-config.h` and `openthread-posix-platform-config.h`. The `xxx_interface.hpp` only include...
> So maybe let's the devide the options into two categories: related with core and unrelated with core. And let's naming them as `openthread-posix-core-related-config.h` and `openthread-posix-config.h`. Thoughts? Sounds good.