[PATCH] DVB-APPS : correct some issues in libdvben50221

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear all,

Here are two patches against the latest version of libdvben50221 in the hg
repository.

Details of corrections:
-----------------------

* In "en50221_sl_handle_close_session_response", the expected data length
wasn't good, the "close_session_response" object is 3 bytes long, not 4
bytes long (see the specification)

* The "LLCI_RESPONSE_TIMEOUT_MS" has been increased from 1000ms to 10000ms
in order to wait for long/complex MMI operations. The timeout work at TPDU
2nd level and not at LPDU 1st level of communication stack. 

* In "en50221_stdcam_llci_destroy", all data from the CA device are read
before closing the device. This prevent from keeping the last poll reply in
the dvb_core module ringbuffer. The polling function used to keep contact
with the CAM is first reading data then writing data, so there is always a
reply left in the buffer.

* In "llci_lookup_callback", some tests permitting resource usage limit are
disabled as they are not working correctly. When a new session is created,
it is declared. But when a session is closed, this isn't declared so a new
session can't be opened a second time.

* In "llci_session_callback", a test was removed as it was duplicated.

* In "en50221_stdcam_llci_poll" and "llci_datetime_enquiry_callback", if the
function "en50221_stdcam_llci_dvbtime" isn't called regularly, a wrong
date/time is send to the CAM. So, if the time wasn't supplied, we send the
UTC time from the system. Also, the "time_offset" parameter of the called
function "en50221_app_datetime_send" has been set to -1 as we don't have the
"local_offset" information and as this information is optional (see the
specification).

Best regards,
Sebastien RAILLARD.

Attachment: en50221_session.patch
Description: Binary data

Attachment: en50221_stdcam_llci.patch
Description: Binary data


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux