<!-- Some styling for better description lists --><style type='text/css'>dt { font-weight: bold;float: left;display:inline;margin-right: 1em} dd { display:block; margin-left: 2em}</style> xdarklight: <u>crope_</u>: mchehab: did any of you have time to look into https://patchwork.linuxtv.org/patch/37230/ yet? I have some questions regarding DVBv5 API (see http://www.mail-archive.com/linux-media@vger.kernel.org/msg103002.html) crope_: <u>xdarklight</u>: I am just looking <br> <u>xdarklight</u>: why you havent used bulk register read when read multiple registers? ***: crope_ is now known as crope crope: <u>xdarklight</u>: also, whats wrong with multiplier (1 << 24) used to logarithms you have converted it to 1000= xdarklight: <u>crope</u>: I wanted to get it working first and started with the bare minimum (= single reads) - I can change it though crope: <u>xdarklight</u>: and debug logging is also wrong, no need to add __func__ as kernel logging adds itr xdarklight: <u>crope</u>: regarding logarithm multiplier: this is what the reference driver does according to dongs <br> ok, will remove those __func__ args crope: linuxdvb math uses 1 << 24. anyhow it does not effect calculation of course xdarklight: <u>crope</u>: what about my questions regarding DVBv5 statistics API? for example I'm doing c->block_error.stat[0].uvalue += errors; and I'm not sure if "+=" should be "=" dongs: wasnt this posted like months ago