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

feat(Examples): Change Project.md feature for MAX32655, MAX32665, and MAX32690 #1055

Closed
wants to merge 13 commits into from
Closed

Conversation

JC-282-AD
Copy link
Contributor

Description

  1. BLE Examples: add feature to set advertising name through project.mk
  2. BLE Examples: add feature to configure security though project.mk
  3. BLE Examples: Allow user to select ME17 and ME14 chip version from project.mk and select PHY properly.

@github-actions github-actions bot added MAX32655 Related to the MAX32655 (ME17) MAX32665 Related to the MAX32665 (ME14) MAX32690 Related to the MAX32690 (ME18) BLE Related to Bluetooth labels Jun 24, 2024
@JC-282-AD JC-282-AD marked this pull request as draft June 24, 2024 16:23
@JC-282-AD JC-282-AD marked this pull request as ready for review June 24, 2024 16:23
@JC-282-AD JC-282-AD marked this pull request as draft June 24, 2024 16:23
@JC-282-AD JC-282-AD marked this pull request as ready for review June 24, 2024 16:23
Delete commented previous code
Delete commented previous code
Delete commented previous code
@EricB-ADI
Copy link
Contributor

Can you add the advertising names update to otas and otac as well?

@JC-282-AD
Copy link
Contributor Author

Can you add the advertising names update to otas and otac as well?

Done! I also synced these updates to ME17. Since I don't have the board for ME17, I didn't test it, but I think it should be good. Just copy and paste for these same updates.

@JC-282-AD JC-282-AD changed the title feat(Examples): Change Project.md feature for MAX32655 and MAX32690 feat(Examples): Change Project.md feature for MAX32655, MAX32665, and MAX32690 Jun 24, 2024
@EricB-ADI
Copy link
Contributor

/clang-format-run



### CONFIGURE security
# /*! Authentication and bonding flags */
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete these commented out configs

@@ -570,11 +570,14 @@ static void datcScanReport(dmEvt_t *pMsg)
if (!connect && ((pData = DmFindAdType(DM_ADV_TYPE_LOCAL_NAME, pMsg->scanReport.len,
pMsg->scanReport.pData)) != NULL)) {
/* check length and device name */
if (pData[DM_AD_LEN_IDX] >= 4 && (pData[DM_AD_DATA_IDX] == 'D') &&
char advName[] = ADV_NAME;
/*if (pData[DM_AD_LEN_IDX] >= 4 && (pData[DM_AD_DATA_IDX] == 'D') &&
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete commented out code

Copy link
Contributor

Choose a reason for hiding this comment

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

Can you update the variable to be overwritten via the command line when make is called?

Believe this will work.

ADV_NAME?=DATS
PROJ_CFLAGS += -DADV_NAME=\"$(ADV_NAME)\"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done! My branch diverges somehow. I fix it and hopefully it is clean.

@JC-282-AD JC-282-AD closed this Jun 27, 2024
@JC-282-AD
Copy link
Contributor Author

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BLE Related to Bluetooth MAX32655 Related to the MAX32655 (ME17) MAX32665 Related to the MAX32665 (ME14) MAX32690 Related to the MAX32690 (ME18)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants