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

Added LoadStore Exception handlers #9

Conversation

sachin0x18
Copy link
Contributor

Handles LoadStoreErrorCause and LoadStoreAlignmentCause allowing to use
32-bit memory region (IRAM) as 8-bit or 16-bit memory region

@RichardBarry
Copy link
Contributor

Thanks for this - are you from Espressif?

@sachin0x18
Copy link
Contributor Author

Yes

Handles LoadStoreErrorCause and LoadStoreAlignmentCause allowing to use
32-bit memory region (IRAM) as 8-bit or 16-bit memory region
@yuhui-zheng
Copy link
Contributor

yuhui-zheng commented Jan 30, 2020

I will shortly close this PR and merge the exact change into our kernel upstream. The change will be included in the coming release.

Minor question for @sachin0x18 -- what was the "historical" reason that this port is under /ThirdPaty?

@yuhui-zheng
Copy link
Contributor

The changes could be reviewed here -- https://sourceforge.net/p/freertos/code/2816/. We are still in the middle of the migration, should be able to merge PR normally afterwards.

@mahavirj
Copy link

mahavirj commented Feb 1, 2020

@yuhui-zheng

I will shortly close this PR and merge the exact change into our kernel upstream. The change will be included in the coming release.

Thanks. We have dependency of this change on PR at aws/amazon-freertos#1713. Do we need to wait for next minor/major FreeRTOS kernel release or we can immediately update submodule pointer (for FreeRTOS kernel) there?

Minor question for @sachin0x18 -- what was the "historical" reason that this port is under /ThirdPaty?

No specific reason. From our (Espressif) side, when we had first submitted ESP32 port, we did not include in ThirdParty prefix. Probably we can change that now.

CC @sachin0x18

@yuhui-zheng
Copy link
Contributor

@mahavirj

We have dependency of this change on PR at aws/amazon-freertos#1713. Do we need to wait for next minor/major FreeRTOS kernel release or we can immediately update submodule pointer (for FreeRTOS kernel) there?

Kernel team has prepared a release candidate (which has included this PR), and we will soon have a kernel release. After this release, everyone should be able to update their submodule pointer.

As for the ThirdParty directory, let me check with team and see what needs to be done to move Espressif port out of ThirdParty directory.

coldnew pushed a commit to coldnew/FreeRTOS-mirror that referenced this pull request Feb 8, 2020
FreeRTOS/FreeRTOS-Kernel#9 -- Handles LoadStoreErrorCause and LoadStoreAlignmentCause allowing to use 32-bit memory region (IRAM) as 8-bit or 16-bit memory region

git-svn-id: https://svn.code.sf.net/p/freertos/code/trunk@2816 1d2547de-c912-0410-9cb9-b8ca96c0e9e2
lundinc2 pushed a commit to FreeRTOS/FreeRTOS that referenced this pull request Feb 10, 2020
FreeRTOS/FreeRTOS-Kernel#9 -- Handles LoadStoreErrorCause and LoadStoreAlignmentCause allowing to use 32-bit memory region (IRAM) as 8-bit or 16-bit memory region
yuhui-zheng added a commit to yuhui-zheng/FreeRTOS-Kernel that referenced this pull request Feb 10, 2020
FreeRTOS#9 -- Handles LoadStoreErrorCause and LoadStoreAlignmentCause allowing to use 32-bit memory region (IRAM) as 8-bit or 16-bit memory region
lundinc2 pushed a commit to lundinc2/FreeRTOS that referenced this pull request Feb 17, 2020
FreeRTOS/FreeRTOS-Kernel#9 -- Handles LoadStoreErrorCause and LoadStoreAlignmentCause allowing to use 32-bit memory region (IRAM) as 8-bit or 16-bit memory region
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants