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

[Tizen] Add ble peripheral functionality #9947

Merged
merged 2 commits into from
Sep 30, 2021
Merged

[Tizen] Add ble peripheral functionality #9947

merged 2 commits into from
Sep 30, 2021

Conversation

wootak-jung
Copy link
Contributor

@wootak-jung wootak-jung commented Sep 24, 2021

Problem

The ble peripheral role is not implemented.

Change overview

Add ble peripheral functionality in Tizen platform.

Testing

lighting-app example.

Co-authored-by: Anupam Roy <[email protected]>
Co-authored-by: Dohyun Pyun <[email protected]>
@todo
Copy link

todo bot commented Sep 24, 2021

Need to implement Tizen Platform Specific events: CHIPDevicePlatformEvent

// TODO: Need to implement Tizen Platform Specific events: CHIPDevicePlatformEvent
}
void BLEManagerImpl::_OnPlatformEvent(const ChipDeviceEvent * event)
{
ChipBleUUID service_uuid;
ChipBleUUID char_notification_uuid;
ChipBleUUID char_write_uuid;
switch (event->Type)
{


This comment was generated by todo based on a TODO comment in 3e68b62 in #9947. cc @wootak-jung.

Copy link
Contributor

@bzbarsky-apple bzbarsky-apple left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not do a careful review of the various bluetooth bits....

src/platform/Tizen/BLEManagerImpl.cpp Outdated Show resolved Hide resolved
@woody-apple
Copy link
Contributor

@woody-apple woody-apple merged commit 5904a92 into project-chip:master Sep 30, 2021
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.

6 participants