search

beagleboard / am335x_pru_package

332 stars 181 forks source link

连接am335x_pru_package #46

Closed abc838 closed 8 years ago

abc838 commented 8 years ago

连接am335x_pru_package

jadonk commented 8 years ago

Too many changes at once and it needs to be rebased as well. Please rebase, break it down into smaller chunks and resubmit.

jadonk commented 8 years ago

@modmaker @abc838 Can you rebase these commits? These changes look valuable.

cdsteinkuehler commented 8 years ago

@jadonk sadly, @modmaker is no longer with us (he passed away), but I would like to see these changes included in the pasm shipped with the 'Bone. They are required for the code I wrote for Machinekit.

Besides rebasing, how small of chunks are you looking for?

cdsteinkuehler commented 8 years ago

This PR can be closed. I have reviewed modmaker's changes and support for the new instructions is already in the master branch of this repo. My initial tests with this version of pasm failed because the structure name comparisons were case insensitive in modmaker's version but are case-sensitive in this version. Once I fixed up my case issues, I was able to generate bit-identical output from both versions.