jamesbowman / swapforth

Swapforth is a cross-platform ANS Forth
BSD 3-Clause "New" or "Revised" License
279 stars 56 forks source link

j1a IO is broken (SEVERE) #37

Closed RGD2 closed 7 years ago

RGD2 commented 8 years ago

This is on swapforth@9157271 , cseed/arachne-pnr@52e69ed , and cliffordwolf/yosys@7a33b98

Symptoms: LED's on iceStick (and possibly other IO ports) don't set/clear properly.

Steps to reproduce (with iceStick plugged in):

cd swapforth/j1a
make clean
make 
sudo make bootstrap 
sudo make j1a connect
-1 leds

After this, one led is not lit, but -1 -1 leds drop will set all, likewise -1 0 leds drop will leave that one led on.

The affected .bin, .blif and .txt are here: broken_j1a.zip

RGD2 commented 8 years ago

Also under cliffordwolf/yosys@21e1bac as well as cliffordwolf/yosys@7a33b98 on arm.

RGD2 commented 8 years ago

the obvious kludgy workaround is : leds dup leds drop ; , but this problem hints at deeper trouble.

bmentink commented 8 years ago

..this is still broken, also reads from led port, turns random led's on ...

RGD2 commented 8 years ago

Please save and upload the j1a.blif and .bin.

This seems to happen only sometimes, and I'd like to get to the bottom of it.

On Thursday, 29 September 2016, bmentink notifications@github.com wrote:

..this is still broken, also reads from led port, turns random led's on ...

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/jamesbowman/swapforth/issues/37#issuecomment-250386756, or mute the thread https://github.com/notifications/unsubscribe-auth/AO8-GBDQskqK5R3S2AuOVtv7pmby4fqFks5qu2JwgaJpZM4Jpkde .

-- Remy

bmentink commented 8 years ago

Here is a link to the files: https://dl.dropboxusercontent.com/u/80752266/j1a.zip Usually doing -1 leds, turns on only 4 led's .. -1 -1 leds drop fixes ..

bmentink commented 8 years ago

I have just updated yosys to r2983.4eb0d6f-1 and arachne-pnr to r171.52e69ed-1 and the LED's seem to work just fine now ( This is latest builds from AUR on ArchLinux)

EDIT: Of the two packages, I think it was yosys that did the trick ..

RGD2 commented 7 years ago

Yosys: git rev-parse --short head gives 5c2c78e.

aand it's fixed here too. Closed.