diff options
| author | Gerhard Sittig <gerhard.sittig@gmx.net> | 2019-11-29 21:15:48 +0100 |
|---|---|---|
| committer | Uwe Hermann <uwe@hermann-uwe.de> | 2019-11-29 23:19:54 +0100 |
| commit | 5ef0a979ca0553d43077390ea54837d3161c0ad5 (patch) | |
| tree | 4266c1fc282ccdc0dee0027ca0bf9db7d46746fb | |
| parent | 048545adcdb75f29e24e021beb09248ea2cd7da5 (diff) | |
| download | libsigrokdecode-5ef0a979ca0553d43077390ea54837d3161c0ad5.tar.gz libsigrokdecode-5ef0a979ca0553d43077390ea54837d3161c0ad5.zip | |
uart: support 'ignore' parity type, remove unsupported 'check_parity' option
The previous UART decoder implementation announced a 'check_parity'
option which took no effect (support code was missing). Remove it. Add
another 'ignore' parity choice instead, which consumes the parity bit's
position yet always passes the check.
| -rw-r--r-- | decoders/uart/pd.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/decoders/uart/pd.py b/decoders/uart/pd.py index 802c593..7a43ee4 100644 --- a/decoders/uart/pd.py +++ b/decoders/uart/pd.py @@ -58,6 +58,9 @@ TX = 1 # 'none' is _not_ allowed as value for 'parity_type'. def parity_ok(parity_type, parity_bit, data, num_data_bits): + if parity_type == 'ignore': + return True + # Handle easy cases first (parity bit is always 1 or 0). if parity_type == 'zero': return parity_bit == 0 @@ -100,9 +103,7 @@ class Decoder(srd.Decoder): {'id': 'num_data_bits', 'desc': 'Data bits', 'default': 8, 'values': (5, 6, 7, 8, 9)}, {'id': 'parity_type', 'desc': 'Parity type', 'default': 'none', - 'values': ('none', 'odd', 'even', 'zero', 'one')}, - {'id': 'parity_check', 'desc': 'Check parity?', 'default': 'yes', - 'values': ('yes', 'no')}, + 'values': ('none', 'odd', 'even', 'zero', 'one', 'ignore')}, {'id': 'num_stop_bits', 'desc': 'Stop bits', 'default': 1.0, 'values': (0.0, 0.5, 1.0, 1.5)}, {'id': 'bit_order', 'desc': 'Bit order', 'default': 'lsb-first', |