Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

PSoC64 Release Branch to Master #3175

Merged
merged 53 commits into from
May 27, 2021
Merged

PSoC64 Release Branch to Master #3175

merged 53 commits into from
May 27, 2021

Conversation

ifyall
Copy link
Contributor

@ifyall ifyall commented May 25, 2021

CY8CKIT-064S0S2-4343W Production Updates

Description

Update CY8CKIT-064S0S2-4343W to cysecuretools v3.1 and newest versions of kit firmware. Update Cypress configuration files to align to new Amazon FreeRTOS standards. Updated OTA support to new version of mcuboot and added support for the SWAP feature.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@LienNguyen2910 LienNguyen2910 marked this pull request as ready for review May 26, 2021 01:23
@LienNguyen2910 LienNguyen2910 requested a review from a team as a code owner May 26, 2021 01:23
ifyall and others added 28 commits May 26, 2021 00:53
…defaults for 062 boards from Amazon master. Regenerate GeneratedSource.
…ed iot_pkcs11_pal.c to align with Amazon master updated to PKCS11.
…tions for configCOMMAND_INT_MAX_OUTPUT_SIZE and configINCLUDE_DEMO_DEBUG_STATS to FreeRTOSConfig.h as on master
… common mtb_global_settings.mk. Aligned aws_tests main.c to use #if BLE_SUPPORTED as seen in the aws_demos main.c
aws_demo is no longer connecting to AP. It looks like a merge error
resulted in the removal of the needed function call.

Signed-off-by: Raymond Ngun <raymond.ngun@infineon.com>
Signed-off-by: Raymond Ngun <raymond.ngun@infineon.com>
stde and others added 15 commits May 26, 2021 01:11
Create a thread to start and kick WDT before prvWifiConnect()
Return WiFI status from prvWifiConnect()
Stop after prvWifiConnect() returns.
If we fail to connect in prvWifiConnect(), reboot
Update AFR OTA docs
- 4k block size speeds up HTTP
- OTA agent can run at lowest priority

Signed-off-by: Raymond Ngun <raymond.ngun@infineon.com>
Signed-off-by: Raymond Ngun <raymond.ngun@infineon.com>
This reverts commit 5c7c72a.
This changed caused FullWiFi and FullSecureSockets IDT tests
to fail.

Signed-off-by: Raymond Ngun <raymond.ngun@infineon.com>
@mingyue86010
Copy link
Contributor

rebased to top of main branch.

@mingyue86010
Copy link
Contributor

/bot run checks

mingyue86010
mingyue86010 previously approved these changes May 26, 2021
@mingyue86010 mingyue86010 self-requested a review May 26, 2021 18:39
@mingyue86010
Copy link
Contributor

This PR contains some changes to FreeRTOS_CLI and Espressif demo code which shouldn't be. Those changes need to be remove from this PR.

Copy link
Member

@paulbartell paulbartell left a comment

Choose a reason for hiding this comment

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

A few questions, but looks good overall.

@@ -66,7 +66,7 @@ extern void IotBleMqtt_CleanupSerialize( void );
#define IOT_HTTPS_DISPATCH_USE_STATIC_MEMORY 1
#define IOT_HTTPS_DISPATCH_TASK_STACK_SIZE ( configMINIMAL_STACK_SIZE * 4 )

#define BLE_SUPPORTED ( 1 )
//#define BLE_SUPPORTED ( 1 )
Copy link
Member

Choose a reason for hiding this comment

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

Should this be commented out?

@@ -60,7 +58,7 @@
* @brief The OTA agent task priority. Normally it runs at a low priority.
* lowest: 0, max 7 (configMAX_PRIORITIES) (tskIDLE_PRIORITY == 0)
*/
#define otaconfigAGENT_PRIORITY (2)
#define otaconfigAGENT_PRIORITY (0)
Copy link
Member

Choose a reason for hiding this comment

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

Is this intentional?

@@ -132,14 +132,20 @@ static struct flash_area secondary_1 =
.fa_id = FLASH_AREA_IMAGE_SECONDARY(0),
Copy link
Member

Choose a reason for hiding this comment

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

Could mcuboot be integrated as a submodule instead?

@@ -155,7 +155,7 @@ function(cy_add_link_libraries)
TOOLCHAIN "${AFR_TOOLCHAIN}"
Copy link
Member

@paulbartell paulbartell May 26, 2021

Choose a reason for hiding this comment

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

Is there a reason we have both cmake and make build files for this port?

@@ -1,5 +1,5 @@
/*
Copy link
Member

Choose a reason for hiding this comment

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

Was this file removed on purpose?

@@ -60,7 +58,7 @@
* @brief The OTA agent task priority. Normally it runs at a low priority.
* lowest: 0, max 7 (configMAX_PRIORITIES) (tskIDLE_PRIORITY == 0)
*/
#define otaconfigAGENT_PRIORITY (2)
#define otaconfigAGENT_PRIORITY (0)
Copy link
Member

Choose a reason for hiding this comment

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

Is this change intentional?

@mingyue86010
Copy link
Contributor

/bot run checks

@mingyue86010
Copy link
Contributor

/bot run checks

@cobusve cobusve merged commit 3403475 into main May 27, 2021
@aggarw13 aggarw13 deleted the PsoC64_Release branch May 27, 2021 20:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants