aw-pac icon indicating copy to clipboard operation
aw-pac copied to clipboard

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

Open duskmoon314 opened this issue 1 year ago • 9 comments

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 avatar Aug 03 '22 02:08 duskmoon314

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

  • https://github.com/daym/allwinner-register-interface-extractor/issues/4#issuecomment-1224073486

Do you have the new version of the Manual?

  • https://github.com/daym/allwinner-register-interface-extractor/issues/3#issuecomment-1214195729

kassane avatar Aug 25 '22 20:08 kassane

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

duskmoon314 avatar Aug 26 '22 01:08 duskmoon314

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.

kassane avatar Aug 26 '22 19:08 kassane

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

I don't know what to say. 😂

duskmoon314 avatar Aug 27 '22 01:08 duskmoon314

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

kassane avatar Aug 27 '22 13:08 kassane

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 avatar Aug 28 '22 22:08 kassane

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 avatar Oct 16 '22 12:10 kassane

@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

kassane avatar Oct 16 '22 13:10 kassane

@kassane Great work! I will take a look ASAP

duskmoon314 avatar Oct 16 '22 13:10 duskmoon314