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.

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

Feature request: Padauk architecture #1236

Closed
kimstik opened this issue Jan 5, 2023 · 6 comments
Closed

Feature request: Padauk architecture #1236

kimstik opened this issue Jan 5, 2023 · 6 comments
Labels
enhancement This is a feature request

Comments

@kimstik
Copy link

kimstik commented Jan 5, 2023

Just a wish.
Architecture description:
https://free-pdk.github.io/

Some reference binaries:
@uxmal
Copy link
Owner

uxmal commented Jan 5, 2023

Any particular one of the PDK13, PDK14, PDK15, or PDK16 you're interested in?

@uxmal uxmal added the enhancement This is a feature request label Jan 5, 2023
@kimstik
Copy link
Author

kimstik commented Jan 6, 2023

Particularly I'm interested in PDK15 and 14.
I'm not sure that PDK16 even exist in reality.

uxmal added a commit that referenced this issue Jan 13, 2023
@uxmal
Feature: Padauk architecture (#1236)
038acf4 
* Architecture definition for Padauk processor family.
* Disassembler for PDK13,PDK14 and PDK15 instruction sets.
* Support per-MemoryArea granularity.
@uxmal
Copy link
Owner

uxmal commented Jan 13, 2023

Commit 038acf4 has an initial implementation of a disassembler and a rewriter to IR. Code quality could improve once provided with processor model-specific details. Here I ask that you consider volunteering some time to collect information about memory address register semantics, and perhaps generating data that could be ingested by Reko when loading the Padauk architecture.

@uxmal uxmal closed this as completed Jan 13, 2023
@kimstik
Copy link
Author

kimstik commented Mar 30, 2023

CmdLine-0.11.4-cedeb234d1.zip
decompile.exe --default-to intel-hex --arch padauk ulp_blinker.ihx
Got "Error: Cannot decompile ulp_blinker.ihx."
ulp_blinker.ihx.zip

@uxmal
Copy link
Owner

uxmal commented Mar 30, 2023

Can you tell what instruction set it is? Reko currently supports PDK14 and PDK15, but you'll need to open a separate issue if you want support for PDK13 or PDK16.

@uxmal uxmal reopened this Mar 30, 2023
uxmal added a commit that referenced this issue Mar 31, 2023
@uxmal
Fix: various problems causing #1236
e5a8a2b
@uxmal
Copy link
Owner

uxmal commented Mar 31, 2023

There were some issues that have been fixed. Use commit 603817e or later. Also, be aware that the command line syntax you need to use is

reko.exe decompile --loader intel-hex --arch padauk --arch-option isa=PDK15 ulp_blinker.ihx

The command-line driver's name has been changed from decompile.exe to reko.exe.

If you find any other issue, please report it in a separate, new issue.

@uxmal uxmal closed this as completed Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement This is a feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kimstik @uxmal