Stephan Trebels wrote, On 11/02/2010 11:38 AM: > Hi Brice, > > I did not find more things to be changed, and it works fine for me now. > Given, that the responsiveness on this list is a bit underwhelming, I > wonder whether we can find someone with commit privileges, to push this > change. > Hi, I've tested this patch too and it solves problems with two different cams. (powercam v4 and cryptoworks by philips) Commiting it would be great. -- Pierre > Stephan > > On Tue, 2010-11-02 at 11:15 +0100, DUBOST Brice wrote: >> On 08/08/2010 23:53, DUBOST Brice wrote: >>> On 15/07/2010 15:20, Stephan Trebels wrote: >>>> >>>> The issue was, that LIBDVBEN50221 did not allow a CAM to re-establish >>>> the session holding non-MMI resources if using the lowlevel interface. >>>> The session_number was recorded on open, but not freed on close (which >>>> IMO is an bug in the code, I attach the scaled down hg changeset). With >>>> this change, the SMIT CAM with a showtime card works fine according to >>>> tests so far. >>>> >>>> The effect was, that the CAM tried to constantly close and re-open the >>>> session and the LIBDVBEN50221 kept telling it, that the resource is >>>> already allocated to a different session. Additionally this caused the >>>> library to use the _old_ session number in communications with the CAM, >>>> which did not even exist anymore, so caused all writes of CA PMTs to >>>> fail with EINTR. >>>> >>>> Stephan >>>> >>> >>> Hello >>> >>> Just to inform that this patch solves problems with CAM PowerCAM v4.3, >>> so I think it can interest more people. >>> >>> Before gnutv -cammenu (and other applications using libdvben50221) was >>> returning ti;eout (-3) errors constantly after the display of the system >>> IDs. >>> >>> Now, the menu is working flawlessly >>> >>> I cannot test the descrambling for the moment but it improved quite a >>> lot the situation (communication with th CAM is now possible). >>> >>> One note concerning the patch itself, the last "else if (resource_id == >>> EN50221_APP_MMI_RESOURCEID)" is useless. >>> >>> Best regards >>> >>> >> >> >> Hello >> >> >> After more testing this Patches allow several CAM models to work and >> don't seem to make any regression. >> >> Is there anything to be improved/tested for having it included upstream ? >> >> Thank you >> >> Regards >> > -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
