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

[d1-pac] check with user manual v1.2/v1.3 #12

Open
duskmoon314 opened this issue Aug 3, 2022 · 9 comments
Open

[d1-pac] check with user manual v1.2/v1.3 #12

duskmoon314 opened this issue Aug 3, 2022 · 9 comments
Labels
help wanted Extra attention is needed

Comments

@duskmoon314
Copy link
Owner

duskmoon314 commented Aug 3, 2022

Allwinner has released the D1-H user manual v1.2/v1.3. Need to check what has changed since v0.1/1.0 (This crate is mostly developed according to v0.1 and v1.0)

@duskmoon314 duskmoon314 added the help wanted Extra attention is needed label Aug 3, 2022
@kassane
Copy link

kassane commented Aug 25, 2022

I tried once again to lift the content extraction, but new conflicts arose in the document.

Do you have the new version of the Manual?

@duskmoon314
Copy link
Owner Author

Do you have the new version of the Manual?

Currently, I have v1.2 from allwinner's site. I have sent you an email about this. 😃

@kassane
Copy link

kassane commented Aug 26, 2022

Compare - SVD generated version (1.0 -> 1.2):

kassane@41a48d0


Edit

The new version seems to have gotten less data than 1.0.

@duskmoon314
Copy link
Owner Author

The new version seems to have gotten less data than 1.0.

I don't know what to say. 😂

@kassane
Copy link

kassane commented Aug 27, 2022

I don't know what to say. 😂

Removed some registers and fields seem a bit strange to me. Just a few naming corrections or more details about the specifications were expected. 🤔

(-) removed:

  • CPU_AXI_CFG_REG
  • PSI_CLK_REG
  • OWA_RX_CLK_REG
  • MSGBOX_WR_INT_THRESHOLD_REG
  • OWA_EXP_CTL
  • ...

@kassane
Copy link

kassane commented Aug 28, 2022

I don't know what to say. 😂

Removed some registers and fields seem a bit strange to me. Just a few naming corrections or more details about the specifications were expected. 🤔

(-) removed:

  • CPU_AXI_CFG_REG
  • PSI_CLK_REG
  • OWA_RX_CLK_REG
  • MSGBOX_WR_INT_THRESHOLD_REG
  • OWA_EXP_CTL
  • ...

Reading the manual in comparison with the previous one and see that the above mentioned register tables are missing.

@kassane
Copy link

kassane commented Oct 16, 2022

I don't know what to say. joy

Removed some registers and fields seem a bit strange to me. Just a few naming corrections or more details about the specifications were expected. thinking

(-) removed:

* `CPU_AXI_CFG_REG`

* `PSI_CLK_REG`

* `OWA_RX_CLK_REG`

* `MSGBOX_WR_INT_THRESHOLD_REG`

* `OWA_EXP_CTL`

* ...

Having checked the newer version of the 1.3 manual and some registers have been removed just like the previous version, but some renames have also been made.

@duskmoon314 duskmoon314 changed the title [d1-pac] check with user manual v1.2 [d1-pac] check with user manual v1.2/v1.3 Oct 16, 2022
@kassane
Copy link

kassane commented Oct 16, 2022

@duskmoon314 , pdf2svd. Please try to match it to your custom svd.

D1H_v1.3.svd.tar.gz

svd2rust test

[INFO  svd2rust] Parsing device from SVD file
[INFO  svd2rust] Rendering device
[ERROR svd2rust] Error rendering device
    
    Caused by:
        0: Rendering error at peripheral
           Name: TCON_LCD0
           Description: TCON_LCD0
           Group: generic
        1: Could not expand register or cluster block
        2: Error expanding cluster
           Name: _N[%s]
           Description: No description
        3: Cluster _N[%s] has size 320 bits that is more then array increment 128 bits

@duskmoon314
Copy link
Owner Author

@kassane Great work! I will take a look ASAP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants