Skip to content

Build fix and write cycle fix#1

Draft
mengstr wants to merge 2 commits intoinnot:masterfrom
mengstr:master
Draft

Build fix and write cycle fix#1
mengstr wants to merge 2 commits intoinnot:masterfrom
mengstr:master

Conversation

@mengstr
Copy link

@mengstr mengstr commented Dec 2, 2025

I tried to use this component in a sim for a video card I'm designing for my KIM-1 and discovered that I got sim errors caused by multiple writers on the databus. The video card reads from RAM during Ph1 in order to avoid clashing with the CPU which only should use the databus during PH2. I haven't verified this fully yet, but at least the crashes stopped.

The adf86dd was so I could compile it. Or else I got this.

[ERROR] Failed to execute goal on project petcomponentsplugin: Could not collect
 dependencies for project de.innot:petcomponentsplugin:jar:1.0.1                
[ERROR] Failed to read artifact descriptor for de.innot.sim6502:Sim6502Java:jar:
0.9.3                                                                           
[ERROR]         Caused by: The following artifacts could not be resolved: de.inn
ot.sim6502:Sim6502Java:pom:0.9.3 (absent): Could not transfer artifact de.innot.
sim6502:Sim6502Java:pom:0.9.3 from/to github (https://maven.pkg.github.com/innot
/*): status code: 401, reason phrase: Unauthorized (401)

And from what I understand the .classpath is a local artefact that should not have been checked in at all.

The 3322ae0 is the actual code fix.

I'll give it some more testing and then undraft this. I'm really just a real noob on both java and even more so maven (I've mostly just written Java Card Edition for smart cards before). So you might be better suited to fix this if you are inclined so. ;-)

- Update pom.xml to use JitPack for Sim6502Java dependency (v0.9.2) to resolve auth errors.
- Remove .classpath file containing broken local paths.
- Ensure data bus is driven only when R/W is low AND PHI2 (clock) is high.
- Prevents invalid data on the bus during the low phase of the clock cycle.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant