duskmoon314 / aw-pac

Peripheral access API for Allwinner SoCs generated from unofficial SVD file
Apache License 2.0
25 stars 8 forks source link

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

Open duskmoon314 opened 2 years ago

duskmoon314 commented 2 years ago

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)

kassane commented 2 years ago

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 commented 2 years ago

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 commented 2 years ago

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

https://github.com/kassane/aw-pac/commit/41a48d0314362f3d001926d5c802161ed7005c70


Edit

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

duskmoon314 commented 2 years ago

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

I don't know what to say. 😂

kassane commented 2 years ago

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:

kassane commented 2 years ago

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
  • ...

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

kassane commented 2 years ago

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.

kassane commented 2 years ago

@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 commented 2 years ago

@kassane Great work! I will take a look ASAP