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

common: coraz7s: Add axi_iic_ard for evb eeprom #1517

Merged
merged 3 commits into from
Dec 6, 2024
Merged

common: coraz7s: Add axi_iic_ard for evb eeprom #1517

merged 3 commits into from
Dec 6, 2024

Conversation

gastmaier
Copy link
Contributor

@gastmaier gastmaier commented Nov 11, 2024

PR Description

Adds an AXI IIC Controller to the common project to be used mostly to read the EVB EEPROM.

    IIC Changes:
    Project       Old          New          IIC Subordinate
    ad4170_asdz   ps-12, 44a4  ps-11, 4160  24AA32A
    ad57xx_ardz   PS7 IIC      ps-11, 4160  24AA32A
    pulsar_adc    none         ps-11, 4160  -
    ad719x_asdz   none         ps-11, 4160  -
    cn0579        ps-12, 44a4  ps-11, 4160  AD5696
    cn0540        ps-12, 44a4  ps-11, 4160  LTC2606
    cn0561        PS7 IIC      ps-11, 4160  M24C02

    IRQ index changes:
    ad4170_asdz
    * $hier_spi_engine 11 -> 12

    ad57xx_ardz
    * ad57xx_dma 12 -> 13
    * $hier_spi_engine 11 -> 12

    cn0579
    * cn0579_dma 13 -> 12
    * axi_iic_ard 12 -> 11

    cn0540
    * $hier_spi_engine 11 -> 12
    * axi_iic_cn0540   12 -> 11

On linux, some rework to the cora dtsi is required to load the fpga axi as an overlay, since we will defer loading the bitsream moving forward.
And existing merged dts?

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 code style guidelines
  • I have performed a self-review of changes
  • I have compiled all hdl projects and libraries affected by this PR
  • I have tested in hardware affected projects, at least on relevant boards
  • I have commented my code, at least hard-to-understand parts
  • I have signed off all commits from this PR
  • I have updated the documentation (wiki pages, ReadMe files, Copyright etc)
  • I have not introduced new Warnings/Critical Warnings on compilation
  • I have added new hdl testbenches or updated existing ones

@gastmaier gastmaier changed the title Create common bitstream project for coraz7s common: coraz7s: Add axi_iic_ard for evb eeprom Nov 21, 2024
@gastmaier gastmaier marked this pull request as ready for review November 21, 2024 19:39
@gastmaier gastmaier force-pushed the ace branch 3 times, most recently from 320df5f to a10d519 Compare November 26, 2024 13:15
acostina
acostina previously approved these changes Dec 3, 2024
Add AXI IIC Controller connected to Arduino SCL SDA ports.
Generally, to read the EVB EEPROM.

Signed-off-by: Jorge Marques <[email protected]>
The axi_iic_ard is generally used to read the EVB EEPROM with
Arduino uno form factor.

Signed-off-by: Jorge Marques <[email protected]>
The axi_iic_ard is generally used to read the EVB EEPROM with
Arduino uno form factor.

IIC Changes:
Project       Old          New          IIC Subordinate
ad4170_asdz   ps-12, 44a4  ps-11, 4160  24AA32A
ad57xx_ardz   PS7 IIC      ps-11, 4160  24AA32A
pulsar_adc    none         ps-11, 4160  -
ad719x_asdz   none         ps-11, 4160  -
cn0579        ps-12, 44a4  ps-11, 4160  AD5696
cn0540        ps-12, 44a4  ps-11, 4160  LTC2606
cn0561        PS7 IIC      ps-11, 4160  M24C02

IRQ index changes:
ad4170_asdz
* $hier_spi_engine 11 -> 12

ad57xx_ardz
* ad57xx_dma 12 -> 13
* $hier_spi_engine 11 -> 12

cn0579
* cn0579_dma 13 -> 12
* axi_iic_ard 12 -> 11

cn0540
* $hier_spi_engine 11 -> 12

Signed-off-by: Jorge Marques <[email protected]>
@gastmaier
Copy link
Contributor Author

Rebased to solve merge conflict

@gastmaier gastmaier requested a review from acostina December 3, 2024 17:46
@gastmaier gastmaier mentioned this pull request Dec 3, 2024
12 tasks
@gastmaier gastmaier merged commit c9e9fcd into main Dec 6, 2024
3 of 5 checks passed
@gastmaier gastmaier deleted the ace branch December 6, 2024 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants