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

zynq: Remove support for Zynq-7000 SoC #604

Merged
merged 1 commit into from
Oct 14, 2024
Merged

Conversation

rajimoha
Copy link
Contributor

@rajimoha rajimoha commented Aug 2, 2024

Remove openamp support for Zynq-7000 SoC, following changes are done 1) remove folder apps/machine/zynq7
2) remove folder apps/system/generic/machine/zynq7 3) remove file cmake/platforms/zynq7_generic.cmake 4) remove file cmake/platforms/zynq7_linux.cmake
5) Modify README.md files

Reasons to remove:

  1. Support for Zynq-7000 has ended
  2. Removing redundant or unmaintained code
  3. Reduce technical debt carried by OpenAMP team
  4. very few customer using openamp on Zynq-7000 SoC

For using Zynq-7000 SoC support
last working and tested version is (v2023.10)
https://github.com/OpenAMP/open-amp/tree/v2023.10

Remove openamp support for Zynq-7000 SoC, following changes are done
1) remove folder apps/machine/zynq7
2) remove folder apps/system/generic/machine/zynq7
3) remove file cmake/platforms/zynq7_generic.cmake
4) remove file cmake/platforms/zynq7_linux.cmake
5) Modify README.md files

Reasons to remove:
1) Support for Zynq-7000 has ended
2) Removing redundant or unmaintained code
3) Reduce technical debt carried by OpenAMP team
4) very few customer using openamp on Zynq-7000 SoC

For using Zynq-7000 SoC support
last working and tested version is (v2023.10)
https://github.com/OpenAMP/open-amp/tree/v2023.10

Signed-off-by: Rajiv Mohan <rajiv.mohan@amd.com>
@arnopo arnopo requested review from edmooring, arnopo and tnmysh August 2, 2024 09:09
Copy link
Collaborator

@tnmysh tnmysh left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Contributor

@edmooring edmooring left a comment

Choose a reason for hiding this comment

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

Looks good to go.

@jdolanTW
Copy link

@rajimoha
Copy link
Contributor Author

@jdolanTW the plan is to remove openamp support for this platform. zynq7000 still will be there but openamp will work only in 2023.10 release for this zynq7000 - which was the last working/tested code base

@rajimoha
Copy link
Contributor Author

@arnopo anything further in this request

@arnopo arnopo added this to the Release V2024.10 milestone Sep 17, 2024
@arnopo
Copy link
Collaborator

arnopo commented Sep 17, 2024

@arnopo anything further in this request

I 'm just waiting @tnmysh to approve on OpenAMP/libmetal#303 to merge both.
but it should be integrated for October release

@arnopo arnopo merged commit 14a37c7 into OpenAMP:main Oct 14, 2024
3 checks passed
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.

6 participants