Open kanji2000 opened 6 years ago
+1
+1
:+1:
+1
+1
我根据Can的写了一个CanFD的。
CAN-FD is more and more common nowadays. I'd like to use DSView to debug my CAN-FD project. It would be great to have CAN-FD decoder in DSView.
@wdim0 @CZZ187827 @1101459202
I've integrated some of sigrok
s decoders into DSView from a while, I didn't get the time to test them all, the new CAN decoder supports CAN-FD.
I've just opened a PR, if you want to test the new decoder, you can clone from my fork https://github.com/abougouffa/DSView/tree/feat/decoders
@abougouffa
I've cloned your fork and tried it, but there's an error:
I've also checked the latest version of ".../libsigrokdecode4DSL/decoders/can/pd.py" on sigrok web http://sigrok.org/gitweb/?p=libsigrokdecode.git;a=tree;f=decoders/can;h=a9b52f173e2ca7df101087af1da8e8e5e117402f;hb=HEAD
but the file has the same content (except one line at start in comment with copyright) and it produces the same error. I'm afraid we have to wait until somebody with good "python + sigrok decoder architecture" skills will fix that.
Thanks anyway!
@wdim0 You can try to replace these lines:
With:
if (self.matched & (0b1 << 1)):
self.dom_edge_seen()
if (self.matched & (0b1 << 0)):
self.handle_bit(can_rx)
It is the libsigrokdecode4dsl
which is not up-to-date, compared with the upstream libsigrokdecode
. I don't understand why they made their own fork, which is constantly far behind the upstream one, instead of depending on the upstream lib directly. There is a very good proposition in #428, but the response from DreamSourceLab's developer was kind of hostile!
@abougouffa Yep! That's working!! Thank you so much abougouffa
CAN-FD frame:
Hi, I am new here. I need to decode CAN-FD data. Critical stage of the proejct. Can anyone offer any help? I downloaded the zipped folder from "https://github.com/sigrokproject/libsigrokdecode" Replaced "decoders" folder in my laptop with the 'decoders' folder from the downloaded zipped folder. Replaced the 4 lines as suggested by @wdim0 But I get "Decoder Reported an error"
Hello @RoopaPalyegar
AFAK, CAN-FD is supported in the last 1.3.0 version. This hack was proposed for an older version.
EDIT: It seems that you are using PulseView! This discussion is intended for DSView.
I really need 'CAN FD' decoder. Could you please support it.