Skip to content

projects: CN0561: Added readme doc #2769

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

projects: CN0561: Added readme doc #2769

wants to merge 1 commit into from

Conversation

jvelasc2
Copy link
Contributor

@jvelasc2 jvelasc2 commented Oct 6, 2025 •
edited
Loading

projects: CN0561: Added readme doc

PR Type

  • Bug fix (change that fixes an issue)
  • New feature (change that adds new functionality)
  • Breaking change (has dependencies in other repos or will cause CI to fail)

PR Checklist

  • I have followed the Coding style guidelines
  • I have complied with the Submission Checklist
  • I have performed a self-review of the changes
  • I have commented my code, at least hard-to-understand parts
  • I have build all projects affected by the changes in this PR
  • I have tested in hardware affected projects, at the relevant boards
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe etc), if applies

actorreno
actorreno reviewed Oct 10, 2025
View reviewed changes
projects/cn0561/README.rst Outdated
#Use this if you are using a Zedboard as carrier
NEW_CFLAGS=-DCN0561_ZED_CARRIER

#Use this if you are using Cora Z7 as carrier
Copy link

Choose a reason for hiding this comment

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

update comment to Cora Z7S, eliminate confusion of possible Z7-10

actorreno
actorreno reviewed Oct 10, 2025
View reviewed changes
actorreno
actorreno reviewed Oct 10, 2025
View reviewed changes
@jvelasc2 jvelasc2 requested a review from actorreno October 13, 2025 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@buha buha Awaiting requested review from buha buha is a code owner

@CiprianRegus CiprianRegus Awaiting requested review from CiprianRegus CiprianRegus is a code owner

@dbogdan dbogdan Awaiting requested review from dbogdan dbogdan is a code owner

@amiclaus amiclaus Awaiting requested review from amiclaus amiclaus is a code owner

@rbolboac rbolboac Awaiting requested review from rbolboac rbolboac is a code owner

@danmois danmois Awaiting requested review from danmois danmois is a code owner

@actorreno actorreno Awaiting requested review from actorreno

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jvelasc2 @actorreno @buha