Re: [sigrok-devel] [PATCH] libsigrokdecode: Simple decoder for ST7735 TFT controller

[Aleksander A. <af...@gm...>]

For the record (for people not on IRC channel):

* Patch was moved to GitHub and updated according to Uwe's and Gerhard's
notes: https://github.com/afiskon/libsigrokdecode/tree/st7735-decoder The
last update was done right now, with git push -f.
* A test case for sigrok-test was added and also updated according to
latest changes in the patch for libsigrokdecode. Corresponding patch can be
found here: https://github.com/afiskon/sigrok-test/tree/st7735-test
* Part of the discussion can be found on bugzilla:
https://sigrok.org/bugzilla/show_bug.cgi?id=1190

On Tue, May 8, 2018 at 8:42 PM, Aleksander Alekseev <af...@gm...>
wrote:

> Hello everyone,
>
> Proposed patch adds decoder of ST7735 TFT controller protocol. Currently
> this decoder only allows to distinguish commands and data, and also shows
> command names and their short descriptions.
>
> Here is a screenshot:
>
> https://afiskon.ru/s/43/ea50416b0f_temp.png
>
> In this context I would like also to answer the question asked in IRC:
>
> 11:33 < uwe_> afiskon: sigrok-cli cannot throw exceptions, it's C code. do
> you mean a Python exception or an error code or such? if yes, under which
> circumstances? sounds like a bug
>
> The circumstances are:
>
> $ sigrok-cli -i ~/temp/st7735-minimal.sr \
>   -P 'st7735:cs=CS:clk=SCLK:mosi=MOSI:dc=DC' \
>   -A 'st7735=description'
> ...
> st7735-1: CASET: Column address set
> st7735-1: RASET: Row address set
> st7735-1: RAMWR: Memory write
>
> Note that the description of the last command RAMWR was successfully sent.
> This is because unlike PulseView sigrok-cli makes 'self.wait()' to throw an
> exception instead of hanging. The decoder interprets the exception as an
> end of stream and decodes buffered data.
>
> --
> Best regards,
> Aleksander Alekseev
>



-- 
Best regards,
Aleksander Alekseev