Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BL602] 1.add matter ota support, 2. use matter example DeviceInfoPro… #19877

Merged
merged 3 commits into from
Jun 23, 2022

Conversation

jczhang777
Copy link
Contributor

…viderImpl.cpp for adaption, 3.usd matter default dnssd, 4. add compile compile parameters, 5.change product id.

Problem

  • add matter ota support.
  • use matter default DeviceInfoProviderImpl.cpp for adaption
  • use matter default dnssd
  • add compile compile parameters
  • change product id

Change overview

add matter ota, add DeviceInfoProviderImpl.cpp for adaption, use matter default dnssd , add compile parameters and change product id.

Testing

How was this tested? (at least one bullet point required)

  • add matter ota support.
    use test event TC-SU to test this case.
  • add DeviceInfoProviderImpl.cpp for adaption.
    use test event TC-TFL to test this case.
  • use matter default dnssd.
    use chip tool commission with bl602.

…viderImpl.cpp for adaption, 3.usd matter default dnssd, 4. add compile compile parameters, 5.change product id.
@jczhang777
Copy link
Contributor Author

pr 19845 triggers a restyled error. Guess the reason for this error may be that I deleted the previously restyled file. I can't figure this out, so I'm reposting this pr. #19845

@woody-apple
Copy link
Contributor

Looks clean, thanks! Fast tracking platform changes.

@jczhang777 Can you please put some simple CI in place to verify this platform builds? Happy to handle that in a follow up, but please prioritize that. If platforms get out of date/broken, we'll likely have to disable them, CI is a great way to handle that.

@Damian-Nordic Damian-Nordic merged commit a2db7c3 into project-chip:master Jun 23, 2022
@jczhang777 jczhang777 deleted the bl602_update_0623 branch July 1, 2022 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants