cleanup patch

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/osmocom-sdr@lists.osmocom.org/.

Stefan Sydow st at metafly.info
Sat Jul 7 12:10:31 UTC 2012


Here are two patches. First one just removes old code and adds some
comments. It also adds a FLO check from gr-baz [1]

The second patch adds return value checks to i2c read/write operations.
I haven't done it yet for the whole e4k driver, but will fix that if you
think, this is a good idea.
By now many i2c reads/writes aren't checked - I don't like that so I
started catching it with assert. Thats not optimal but a least you
notice when it fails. A device reset would be the best solution, but I'm
not sure if we can do that form userspace.


[1]
https://github.com/balint256/gr-baz/blob/master/lib/rtl2832-tuner_e4k.cc
(l. 1036)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 1_cleanup.diff
Type: text/x-patch
Size: 4027 bytes
Desc: not available
URL: <http://lists.osmocom.org/pipermail/osmocom-sdr/attachments/20120707/ef482904/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 2_i2c-check.diff
Type: text/x-patch
Size: 3963 bytes
Desc: not available
URL: <http://lists.osmocom.org/pipermail/osmocom-sdr/attachments/20120707/ef482904/attachment-0001.bin>


More information about the osmocom-sdr mailing list