Bug 402576

Summary:	bttv module is not unloaded after change using yast tv card
Product:	[openSUSE] openSUSE 11.0	Reporter:	Forgotten User nqeDWc8OMK <forgotten_nqeDWc8OMK>
Component:	YaST2	Assignee:	Ladislav Slezák <lslezak>
Status:	RESOLVED FIXED	QA Contact:	Jiri Srain <jsrain>
Severity:	Normal
Priority:	P3 - Medium
Version:	Final
Target Milestone:	---
Hardware:	i686
OS:	Other
Whiteboard:
Found By:	---	Services Priority:
Business Priority:		Blocker:	---
Marketing QA Status:	---	IT Deployment:	---
Attachments:	yast logs Proposed patch for /usr/share/YaST2/modules/Tv.ycp

Description Forgotten User nqeDWc8OMK 2008-06-22 16:07:17 UTC

I used the TV Card module to configure mine, I have a Bt878 card that is not correctly recognized, so I have to manually change the values using Advance Settings, the problem is that after set this values, yast didn't unload the previous loaded module, so the changes didn't have effect.
I had to run rmmod bttv by myself.

I'm writing about bttv because is what I have, but I'm this the same will apply to any other module.

Comment 1 Forgotten User nqeDWc8OMK 2008-06-22 16:08:19 UTC

Created attachment 223606 [details]
yast logs

Comment 2 Ladislav Slezák 2008-12-10 09:08:01 UTC

Created attachment 259120 [details]
Proposed patch for /usr/share/YaST2/modules/Tv.ycp

This is a patch for Tv.ycp file. Run

ycpc -c /usr/share/YaST2/modules/Tv.ycp

command after applying the patch to compile it.

Comment 3 Ladislav Slezák 2008-12-10 09:09:43 UTC

Segio, please test the attached patch - it executes rmmod before each modprobe command so the modules should be reloaded now.

Comment 4 Forgotten User nqeDWc8OMK 2008-12-10 14:47:17 UTC

k, I would need some time.

Comment 5 Ladislav Slezák 2008-12-10 14:50:10 UTC

No need to hurry, I just want somebody to check the fix. It looks OK to me, but you never know...

Comment 6 Forgotten User nqeDWc8OMK 2008-12-10 21:19:34 UTC

I can confirm it works.

>>>>> Dec 10 19:17:29 banshee kernel: bttv0: unloading
Dec 10 19:17:29 banshee kernel: bttv: driver version 0.9.17 loaded
Dec 10 19:17:29 banshee kernel: bttv: using 8 buffers with 2080k (520 pages) each for capture
Dec 10 19:17:29 banshee kernel: bttv: Bt8xx card found (0).

Comment 7 Ladislav Slezák 2008-12-15 15:11:41 UTC

The fix has been submitted in yast2-tv-2.17.3 (SLES/SLED-11), will be submitted in yast2-tv-2.18.0 (openSUSE-11.2).

Comment 8 Ladislav Slezák 2009-01-13 14:16:20 UTC

Submitted in yast2-tv-2.18.0. (openSUSE-11.2)